[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