Skip to content

Commit 2f7ccb0

Browse files
fix some code smells
1 parent 3f575c2 commit 2f7ccb0

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

include/sx127x.h

+4-4
Original file line numberDiff line numberDiff line change
@@ -685,7 +685,7 @@ int sx127x_lora_tx_set_for_transmission(const uint8_t *data, uint8_t data_length
685685
* - SX127X_ERR_INVALID_ARG if parameter is invalid
686686
* - SX127X_OK on success
687687
*/
688-
int sx127x_fsk_ook_tx_set_for_transmission(uint8_t *data, uint16_t data_length, sx127x *device);
688+
int sx127x_fsk_ook_tx_set_for_transmission(const uint8_t *data, uint16_t data_length, sx127x *device);
689689

690690
/**
691691
* @brief Write packet into sx127x's FIFO for transmittion. Once packet is written, set opmod to TX.
@@ -698,7 +698,7 @@ int sx127x_fsk_ook_tx_set_for_transmission(uint8_t *data, uint16_t data_length,
698698
* - SX127X_ERR_INVALID_ARG if parameter is invalid
699699
* - SX127X_OK on success
700700
*/
701-
int sx127x_fsk_ook_tx_set_for_transmission_with_address(uint8_t *data, uint16_t data_length, uint8_t address_to, sx127x *device);
701+
int sx127x_fsk_ook_tx_set_for_transmission_with_address(const uint8_t *data, uint16_t data_length, uint8_t address_to, sx127x *device);
702702

703703
/**
704704
* @brief Start transmitting periodic beacon using FSK/OOK modulation. Packet format must be configured as SX127X_FIXED.
@@ -712,7 +712,7 @@ int sx127x_fsk_ook_tx_set_for_transmission_with_address(uint8_t *data, uint16_t
712712
* - SX127X_ERR_INVALID_STATE if configured packet format is not SX127X_FIXED or selected modem is not FSK/OOK
713713
* - SX127X_OK on success
714714
*/
715-
int sx127x_fsk_ook_tx_start_beacon(uint8_t *data, uint8_t data_length, uint32_t interval_ms, sx127x *device);
715+
int sx127x_fsk_ook_tx_start_beacon(const uint8_t *data, uint8_t data_length, uint32_t interval_ms, sx127x *device);
716716

717717
/**
718718
* @brief Stop transmitting periodic beacon.
@@ -764,7 +764,7 @@ int sx127x_fsk_set_fdev(float frequency_deviation, sx127x *device);
764764
* - SX127X_ERR_INVALID_ARG if parameter is invalid
765765
* - SX127X_OK on success
766766
*/
767-
int sx127x_fsk_ook_set_syncword(uint8_t *syncword, uint8_t syncword_length, sx127x *device);
767+
int sx127x_fsk_ook_set_syncword(const uint8_t *syncword, uint8_t syncword_length, sx127x *device);
768768

769769
/**
770770
* @brief Set data whitening or scrambling is widely used for randomizing the user data before radio transmission. Scrambling can improve bit synchronizer accuracy.

src/sx127x.c

+4-4
Original file line numberDiff line numberDiff line change
@@ -922,7 +922,7 @@ int sx127x_fsk_ook_tx_set_for_transmission_with_remaining(uint16_t data_length,
922922
return sx127x_shadow_spi_write_buffer(REGFIFO, device->packet, to_send, &device->spi_device);
923923
}
924924

925-
int sx127x_fsk_ook_tx_set_for_transmission(uint8_t *data, uint16_t data_length, sx127x *device) {
925+
int sx127x_fsk_ook_tx_set_for_transmission(const uint8_t *data, uint16_t data_length, sx127x *device) {
926926
CHECK_FSK_OOK_MODULATION(device);
927927
if (device->fsk_ook_format == SX127X_VARIABLE && data_length > MAX_PACKET_SIZE) {
928928
return SX127X_ERR_INVALID_ARG;
@@ -941,7 +941,7 @@ int sx127x_fsk_ook_tx_set_for_transmission(uint8_t *data, uint16_t data_length,
941941
return sx127x_fsk_ook_tx_set_for_transmission_with_remaining(data_length, device);
942942
}
943943

944-
int sx127x_fsk_ook_tx_set_for_transmission_with_address(uint8_t *data, uint16_t data_length, uint8_t address_to, sx127x *device) {
944+
int sx127x_fsk_ook_tx_set_for_transmission_with_address(const uint8_t *data, uint16_t data_length, uint8_t address_to, sx127x *device) {
945945
CHECK_FSK_OOK_MODULATION(device);
946946
if (device->fsk_ook_format == SX127X_VARIABLE && data_length > (MAX_PACKET_SIZE - 1)) {
947947
return SX127X_ERR_INVALID_ARG;
@@ -962,7 +962,7 @@ int sx127x_fsk_ook_tx_set_for_transmission_with_address(uint8_t *data, uint16_t
962962
return sx127x_fsk_ook_tx_set_for_transmission_with_remaining(packet_length, device);
963963
}
964964

965-
int sx127x_fsk_ook_tx_start_beacon(uint8_t *data, uint8_t data_length, uint32_t interval_ms, sx127x *device) {
965+
int sx127x_fsk_ook_tx_start_beacon(const uint8_t *data, uint8_t data_length, uint32_t interval_ms, sx127x *device) {
966966
CHECK_FSK_OOK_MODULATION(device);
967967
if (device->fsk_ook_format != SX127X_FIXED) {
968968
return SX127X_ERR_INVALID_STATE;
@@ -1161,7 +1161,7 @@ int sx127x_fsk_ook_rx_set_trigger(sx127x_rx_trigger_t trigger, sx127x *device) {
11611161
return sx127x_append_register(REGRXCONFIG, trigger, 0b11111000, &device->spi_device);
11621162
}
11631163

1164-
int sx127x_fsk_ook_set_syncword(uint8_t *syncword, uint8_t syncword_length, sx127x *device) {
1164+
int sx127x_fsk_ook_set_syncword(const uint8_t *syncword, uint8_t syncword_length, sx127x *device) {
11651165
CHECK_FSK_OOK_MODULATION(device);
11661166
if (syncword_length == 0 || syncword_length > 8) {
11671167
return SX127X_ERR_INVALID_ARG;

0 commit comments

Comments
 (0)