Re: [PATCH v2] Staging iio: adc: Match parenthesis alignment

From: Jonathan Cameron
Date: Sun Oct 21 2018 - 10:42:01 EST


On Wed, 17 Oct 2018 22:31:47 -0300
Marcelo Schmitt <marcelo.schmitt1@xxxxxxxxx> wrote:

> Change close parenthesis alignment to match respective open parenthesis at
> iio/drivers/staging/iio/adc/ad7606.c line 379.
> This makes the file more compliant with the preferred coding style for the
> linux kernel.
Hi Marcelo,

Nice cleanup and applied to the togreg branch of iio.git and pushed
out as testing for the autobuilders to play with it.
(basically this lets 0-day run a whole bunch of build tests on
it before I push it out in a form that others can rely on not changing)

Patch naming is one of those things that is very inconsistent
in different areas of the kernel. I take the view that if I'm going
to be fussy I'll fix it up myself when applying if that is the only
thing I'm otherwise going to comment on.

However, definite preference for the name of the driver being in there
somewhere! When I saw this description I thought we were looking
at a cleanup of the whole directory rather than the single driver
you have here.

Thanks,

Jonathan


>
> Signed-of-by: Marcelo Schmitt <marcelo.schmitt1@xxxxxxxxx>
> ---
> drivers/staging/iio/adc/ad7606.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/iio/adc/ad7606.c b/drivers/staging/iio/adc/ad7606.c
> index 0b728b6ea135..230faae38c12 100644
> --- a/drivers/staging/iio/adc/ad7606.c
> +++ b/drivers/staging/iio/adc/ad7606.c
> @@ -376,7 +376,7 @@ static int ad7606_request_gpios(struct ad7606_state *st)
> return 0;
>
> st->gpio_os = devm_gpiod_get_array_optional(dev, "oversampling-ratio",
> - GPIOD_OUT_LOW);
> + GPIOD_OUT_LOW);
> return PTR_ERR_OR_ZERO(st->gpio_os);
> }
>