[PATCH] staging: pi433: pi433_if.c codestyle brace on previous line
Oliver Graute
oliver.graute at gmail.com
Tue Dec 5 20:12:26 UTC 2017
This patch fixes the following checkpatch.pl error:
WARNING: line over 80 characters SET_CHECKED(rf69_set_fifo_fill_condition(dev->spi, afterSyncInterrupt));
ERROR: that open brace { should be on the previous line
+ else
+ {
ERROR: else should follow close brace '}'
+ }
+ else
Signed-off-by: Oliver Graute <oliver.graute at gmail.com>
---
drivers/staging/pi433/pi433_if.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c
index 3404cb9..17be4b6 100644
--- a/drivers/staging/pi433/pi433_if.c
+++ b/drivers/staging/pi433/pi433_if.c
@@ -198,12 +198,10 @@ static irqreturn_t DIO1_irq_handler(int irq, void *dev_id)
/* packet config */
/* 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));
- }
- else
- {
+ if (rx_cfg->enable_sync == optionOn) {
+ 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) {
--
1.9.1
More information about the devel
mailing list