Re: [PATCH gpio v1 2/2] gpio: ks8695: fix 'else should follow close brace '}''

[Alexandre Courbot <gnurou@xxxxxxxxx>]

On Sun, Sep 21, 2014 at 12:14 AM, Varka Bhadram <varkabhadram@xxxxxxxxx> wrote:
>
> Signed-off-by: Varka Bhadram <varkab@xxxxxxx>
> ---
>  drivers/gpio/gpio-ks8695.c |    9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpio/gpio-ks8695.c b/drivers/gpio/gpio-ks8695.c
> index ba2d2f1..10c7377 100644
> --- a/drivers/gpio/gpio-ks8695.c
> +++ b/drivers/gpio/gpio-ks8695.c
> @@ -276,17 +276,14 @@ static int ks8695_gpio_show(struct seq_file *s, void *unused)
>                                 case IOPC_TM_EDGE:
>                                         seq_printf(s, "(Edges)");       break;
>                                 }
> -                       }
> -                       else
> +                       } else
>                                 seq_printf(s, "GPIO\t");

According to the kernel code formatting guidelines, this should actually be

        } else {
                seq_printf(s, "GPIO\t");
        }
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html