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); > } >
