On Tue, Nov 14, 2017 at 12:03:51PM -0800, Chase Metzger wrote: > Fix if/else if and braces in function DIO0_irq_handler to comply with > the kernel coding style. > > Signed-off-by: Chase Metzger <chasemetzger15@xxxxxxxxx> > --- > drivers/staging/pi433/pi433_if.c | 11 +++-------- > 1 file changed, 3 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c > index d946838450d4..dee70c47629c 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) { When you are touching the line, be sure to fix the obvious space issue here as well. thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel