Staging iio: adc: Match parenthesis alignment
Subsystems (4)
PATCH v2 1/1 Staging iio: adc: Match parenthesis alignment
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.
--- drivers/staging/iio/adc/ad7606.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/iio/adc/ad7606.c +1 -1
@@ -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); }
Signature · 1 line
2.17.1
Jonathan Cameron
maintainerQuoted 4 lines · On Wed, 17 Oct 2018 22:31:47 -0300 Marcelo Schmitt…
Change close parenthesis alignment to match respective open parenthesis at
iio/drivers/staging/iio/adc/ad7606.cline 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
Quoted 19 lines
Signed-of-by Marcelo Schmitt <[email protected]>--- drivers/staging/iio/adc/ad7606.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)drivers/staging/iio/adc/ad7606.c +1
-1@@ -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); }