Skip to contentSkip to conversation

Fixed brace positions in a series of if statements.

PATCH In review Seth Kushniryk 2 participants 2 messages Aug 10, 2017 – Aug 17, 2017 Staging

lore
Share & tools
lore

Seth Kushniryk

author
#0 lore raw

PATCH 1/1 Fixed brace positions in a series of if statements.

My first try at this. I hope it's alright.

Signed-off-by Seth Kushniryk <[email protected]>
---
 drivers/staging/pi433/pi433_if.c | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)
drivers/staging/pi433/pi433_if.c +3 -8
index ed737f4b1e77..55737741e83a 100644
@@ -133,19 +133,14 @@ DIO0_irq_handler(unsigned int irq, void *dev_id, struct pt_regs *regs)
 {
 	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;
 		printk("DIO0 irq: Packet sent\n"); // TODO: printk() should include KERN_ facility level
 		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) {
 		printk("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) {
 		printk("DIO0 irq: PayloadReady\n");
 		device->free_in_fifo = 0;
 		wake_up_interruptible(&device->fifo_wait_queue);
Signature · 1 line
2.11.0
maintainer
#1 lore raw
Quoted 1 line · Seth Kushniryk

My first try at this. I hope it's alright.

That's a very odd changelog text :(

Also please make your subject match other patches for this subsystem and try again.

thanks,

greg k-h