Re: [PATCH 1/6] staging: pi433: pi433_if.c: fix opening curly brace coding style issues

From: Greg KH
Date: Tue Nov 28 2017 - 08:08:45 EST


On Wed, Nov 15, 2017 at 04:35:48PM -0800, Chase Metzger wrote:
> Put opening curly braces for if/else/else if statements and for/while loops
> on the same line as the statements or loops to comply with the kernel
> coding style.
>
> Signed-off-by: Chase Metzger <chasemetzger15@xxxxxxxxx>
> ---
> drivers/staging/pi433/pi433_if.c | 110 ++++++++++++++-------------------------
> 1 file changed, 38 insertions(+), 72 deletions(-)
>
> diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c
> index ff86db1c6cbd..76d70345229a 100644
> --- a/drivers/staging/pi433/pi433_if.c
> +++ b/drivers/staging/pi433/pi433_if.c
> @@ -153,12 +153,10 @@ 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) {

Again, fix up the space issue here at the same time please.

thanks,

greg k-h