Re: [PATCH v5 3/8] iio: light: opt3001: ensure correct parenthesis alignment

From: Andy Shevchenko

Date: Thu Jun 04 2026 - 02:49:14 EST


On Wed, Jun 03, 2026 at 07:12:54PM +0200, Joshua Crofts wrote:
> Ensure correct parenthesis alignment per checkpatch.pl report.
>
> No functional change.

Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>

...

> switch (dir) {
> case IIO_EV_DIR_RISING:
> opt3001_to_iio_ret(opt, opt->high_thresh_exp,
> - opt->high_thresh_mantissa, val, val2);
> + opt->high_thresh_mantissa,
> + val, val2);
> break;
> case IIO_EV_DIR_FALLING:
> opt3001_to_iio_ret(opt, opt->low_thresh_exp,
> - opt->low_thresh_mantissa, val, val2);
> + opt->low_thresh_mantissa,
> + val, val2);
> break;

Not sure why you have split the last line, as in any case there is no logic
split on the previous line.

--
With Best Regards,
Andy Shevchenko