[PATCH 04/19] gpio: wcove: use new GPIO line value setter callbacks
Andy Shevchenko
andriy.shevchenko at intel.com
Wed Jul 9 01:41:57 PDT 2025
On Wed, Jul 09, 2025 at 11:32:40AM +0300, Andy Shevchenko wrote:
> On Wed, Jul 9, 2025 at 9:42 AM Bartosz Golaszewski <brgl at bgdev.pl> wrote:
...
> > if (value)
> > - regmap_set_bits(wg->regmap, reg, 1);
> > - else
> > - regmap_clear_bits(wg->regmap, reg, 1);
> > + return regmap_set_bits(wg->regmap, reg, 1);
> > +
> > + return regmap_clear_bits(wg->regmap, reg, 1);
>
> return regmap_assign_bits(...);
Btw, since you are doing sometimes two patches, it might worth to have
a prerequisite that changes set/clear to assign and then your patch will
be cleaner.
> Otherwise LGTM,
> Reviewed-by: Andy Shevchenko <andy at kernel.org>
--
With Best Regards,
Andy Shevchenko
More information about the linux-arm-kernel
mailing list