This patch fixes the following checkpatch.pl error:

ERROR: that open brace { should be on the previous line

in pi433_if.c as requested by TODO file

Signed-off-by: Oliver Graute <oliver.gra...@gmail.com>
---
 drivers/staging/pi433/pi433_if.c | 38 +++++++++++++-------------------------
 1 file changed, 13 insertions(+), 25 deletions(-)

diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c
index 0d18412..1515198 100644
--- a/drivers/staging/pi433/pi433_if.c
+++ b/drivers/staging/pi433/pi433_if.c
@@ -133,19 +133,14 @@ static irqreturn_t DIO0_irq_handler(int irq, void *dev_id)
 {
        struct pi433_device *device = dev_id;
 
-       if      (device->irq_state[DIO0] == DIO_PacketSent)
-       {
+       if (device->irq_state[DIO0] == DIO_PacketSent) {
                device->free_in_fifo = FIFO_SIZE;
                dev_dbg(device->dev, "DIO0 irq: Packet sent\n");
                wake_up_interruptible(&device->fifo_wait_queue);
-       }
-       else if (device->irq_state[DIO0] == DIO_Rssi_DIO0)
-       {
+       } else if (device->irq_state[DIO0] == DIO_Rssi_DIO0) {
                dev_dbg(device->dev, "DIO0 irq: RSSI level over threshold\n");
                wake_up_interruptible(&device->rx_wait_queue);
-       }
-       else if (device->irq_state[DIO0] == DIO_PayloadReady)
-       {
+       } else if (device->irq_state[DIO0] == DIO_PayloadReady) {
                dev_dbg(device->dev, "DIO0 irq: PayloadReady\n");
                device->free_in_fifo = 0;
                wake_up_interruptible(&device->fifo_wait_queue);
@@ -158,14 +153,13 @@ static irqreturn_t DIO1_irq_handler(int irq, void *dev_id)
 {
        struct pi433_device *device = dev_id;
 
-       if      (device->irq_state[DIO1] == DIO_FifoNotEmpty_DIO1)
-       {
+       if (device->irq_state[DIO1] == DIO_FifoNotEmpty_DIO1) {
                device->free_in_fifo = FIFO_SIZE;
-       }
-       else if (device->irq_state[DIO1] == DIO_FifoLevel)
-       {
-               if (device->rx_active)  device->free_in_fifo = FIFO_THRESHOLD - 
1;
-               else                    device->free_in_fifo = FIFO_SIZE - 
FIFO_THRESHOLD - 1;
+       } else if (device->irq_state[DIO1] == DIO_FifoLevel) {
+               if (device->rx_active)
+                       device->free_in_fifo = FIFO_THRESHOLD - 1;
+               else
+                       device->free_in_fifo = FIFO_SIZE - FIFO_THRESHOLD - 1;
        }
        dev_dbg(device->dev,
                "DIO1 irq: %d bytes free in fifo\n", device->free_in_fifo);
@@ -198,12 +192,9 @@ 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)
-       {
+       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) {
@@ -220,12 +211,9 @@ static irqreturn_t DIO1_irq_handler(int irq, void *dev_id)
 
        /* 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++;
-- 
1.9.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to