[PATCH] staging: pi433: fix coding style in function rf69_set_rx_cfg()

2017-11-30 Thread Victor Carvalho
staging: pi433: fix coding style in function
 - rf69_set_rx_cfg(struct pi433_device *dev, struct pi433_rx_cfg *rx_cfg)


Signed-off-by: Victor Carvalho 
---
 drivers/staging/pi433/pi433_if.c | 60 
 1 file changed, 30 insertions(+), 30 deletions(-)

diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c
index 763104760650..97ab5fa27059 100644
--- a/drivers/staging/pi433/pi433_if.c
+++ b/drivers/staging/pi433/pi433_if.c
@@ -183,15 +183,20 @@ rf69_set_rx_cfg(struct pi433_device *dev, struct 
pi433_rx_cfg *rx_cfg)
int payload_length;
 
/* receiver config */
-   SET_CHECKED(rf69_set_frequency  (dev->spi, rx_cfg->frequency));
-   SET_CHECKED(rf69_set_bit_rate   (dev->spi, rx_cfg->bit_rate));
-   SET_CHECKED(rf69_set_modulation (dev->spi, rx_cfg->modulation));
-   SET_CHECKED(rf69_set_antenna_impedance   (dev->spi, 
rx_cfg->antenna_impedance));
-   SET_CHECKED(rf69_set_rssi_threshold  (dev->spi, 
rx_cfg->rssi_threshold));
-   SET_CHECKED(rf69_set_ook_threshold_dec   (dev->spi, 
rx_cfg->thresholdDecrement));
-   SET_CHECKED(rf69_set_bandwidth   (dev->spi, 
rx_cfg->bw_mantisse, rx_cfg->bw_exponent));
-   SET_CHECKED(rf69_set_bandwidth_during_afc(dev->spi, 
rx_cfg->bw_mantisse, rx_cfg->bw_exponent));
-   SET_CHECKED(rf69_set_dagc(dev->spi, rx_cfg->dagc));
+   SET_CHECKED(rf69_set_frequency(dev->spi, rx_cfg->frequency));
+   SET_CHECKED(rf69_set_bit_rate(dev->spi, rx_cfg->bit_rate));
+   SET_CHECKED(rf69_set_modulation(dev->spi, rx_cfg->modulation));
+   SET_CHECKED(rf69_set_antenna_impedance(dev->spi,
+  rx_cfg->antenna_impedance));
+   SET_CHECKED(rf69_set_rssi_threshold(dev->spi, rx_cfg->rssi_threshold));
+   SET_CHECKED(rf69_set_ook_threshold_dec(dev->spi,
+  rx_cfg->thresholdDecrement));
+   SET_CHECKED(rf69_set_bandwidth(dev->spi, rx_cfg->bw_mantisse,
+  rx_cfg->bw_exponent));
+   SET_CHECKED(rf69_set_bandwidth_during_afc(dev->spi,
+ rx_cfg->bw_mantisse,
+ rx_cfg->bw_exponent));
+   SET_CHECKED(rf69_set_dagc(dev->spi, rx_cfg->dagc));
 
dev->rx_bytes_to_drop = rx_cfg->bytes_to_drop;
 
@@ -199,13 +204,11 @@ rf69_set_rx_cfg(struct pi433_device *dev, struct 
pi433_rx_cfg *rx_cfg)
/* enable */
SET_CHECKED(rf69_set_sync_enable(dev->spi, rx_cfg->enable_sync));
if (rx_cfg->enable_sync == optionOn)
-   {
-   SET_CHECKED(rf69_set_fifo_fill_condition(dev->spi, 
afterSyncInterrupt));
-   }
+   SET_CHECKED(rf69_set_fifo_fill_condition(dev->spi,
+afterSyncInterrupt));
else
-   {
SET_CHECKED(rf69_set_fifo_fill_condition(dev->spi, always));
-   }
+
if (rx_cfg->enable_length_byte == optionOn) {
ret = rf69_set_packet_format(dev->spi, packetLengthVar);
if (ret < 0)
@@ -215,34 +218,31 @@ rf69_set_rx_cfg(struct pi433_device *dev, struct 
pi433_rx_cfg *rx_cfg)
if (ret < 0)
return ret;
}
-   SET_CHECKED(rf69_set_adressFiltering(dev->spi, 
rx_cfg->enable_address_filtering));
-   SET_CHECKED(rf69_set_crc_enable (dev->spi, rx_cfg->enable_crc));
+
+   SET_CHECKED(rf69_set_adressFiltering
+   (dev->spi, rx_cfg->enable_address_filtering));
+   SET_CHECKED(rf69_set_crc_enable(dev->spi, rx_cfg->enable_crc));
 
/* lengths */
SET_CHECKED(rf69_set_sync_size(dev->spi, rx_cfg->sync_length));
-   if (rx_cfg->enable_length_byte == optionOn)
-   {
+   if (rx_cfg->enable_length_byte == optionOn) {
SET_CHECKED(rf69_set_payload_length(dev->spi, 0xff));
-   }
-   else if (rx_cfg->fixed_message_length != 0)
-   {
+   } else if (rx_cfg->fixed_message_length != 0) {
payload_length = rx_cfg->fixed_message_length;
-   if (rx_cfg->enable_length_byte  == optionOn) payload_length++;
-   if (rx_cfg->enable_address_filtering != filteringOff) 
payload_length++;
+   if (rx_cfg->enable_length_byte  == optionOn)
+   payload_length++;
+   if (rx_cfg->enable_address_filtering != filteringOff)
+   payload_length++;
SET_CHECKED(rf69_set_payload_length(dev->spi, payload_length));
-   }
-   else
-   {
+   } else {
SET_CHECKED(rf69_set_payload_length(dev->spi, 0));

[PATCH] [FIX] staging: pi433: fix last commit indentation

2017-12-03 Thread Victor Carvalho
fix indentation in commit:

 - 928150e1dbb8638f309493b7dc72d56c20896704 staging: pi433: fix coding style in 
function rf69_set_rx_cfg()

Signed-off-by: Victor Carvalho 
---
 drivers/staging/pi433/pi433_if.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c
index 97ab5fa27059..a9fd7fc209d6 100644
--- a/drivers/staging/pi433/pi433_if.c
+++ b/drivers/staging/pi433/pi433_if.c
@@ -203,11 +203,12 @@ rf69_set_rx_cfg(struct pi433_device *dev, struct 
pi433_rx_cfg *rx_cfg)
/* packet config */
/* enable */
SET_CHECKED(rf69_set_sync_enable(dev->spi, rx_cfg->enable_sync));
-   if (rx_cfg->enable_sync == optionOn)
+   if (rx_cfg->enable_sync == optionOn) {
SET_CHECKED(rf69_set_fifo_fill_condition(dev->spi,
 afterSyncInterrupt));
-   else
+   } else {
SET_CHECKED(rf69_set_fifo_fill_condition(dev->spi, always));
+   }
 
if (rx_cfg->enable_length_byte == optionOn) {
ret = rf69_set_packet_format(dev->spi, packetLengthVar);
-- 
2.14.3



[PATCH] staging: pi433: fix include asm/compat.h to linux/compat.h

2017-11-29 Thread Victor Carvalho
staging: pi433: fix include asm/compat.h to linux/compat.h

Signed-off-by: Victor Carvalho 
---
 drivers/staging/pi433/pi433_if.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c
index 3404cb9722c9..763104760650 100644
--- a/drivers/staging/pi433/pi433_if.c
+++ b/drivers/staging/pi433/pi433_if.c
@@ -48,7 +48,7 @@
 #include 
 #include 
 #ifdef CONFIG_COMPAT
-#include 
+#include 
 #endif
 
 #include "pi433_if.h"
-- 
2.14.3