[PATCH v3 02/41] iio: accel: kxsd9: Make use of regmap_clear_bits()
Jonathan Cameron
jic23 at kernel.org
Sat Jun 22 02:43:19 PDT 2024
On Mon, 17 Jun 2024 09:49:42 -0400
Trevor Gamblin <tgamblin at baylibre.com> wrote:
> Instead of using regmap_update_bits() and passing val = 0, use
> regmap_clear_bits().
>
> Suggested-by: Uwe Kleine-König <u.kleine-koenig at baylibre.com>
> Signed-off-by: Trevor Gamblin <tgamblin at baylibre.com>
> ---
> drivers/iio/accel/kxsd9.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/iio/accel/kxsd9.c b/drivers/iio/accel/kxsd9.c
> index ba99649fe195..03ce032e06ff 100644
> --- a/drivers/iio/accel/kxsd9.c
> +++ b/drivers/iio/accel/kxsd9.c
> @@ -370,10 +370,8 @@ static int kxsd9_power_down(struct kxsd9_state *st)
> * make sure we conserve power even if there are others users on the
> * regulators.
> */
> - ret = regmap_update_bits(st->map,
> - KXSD9_REG_CTRL_B,
> - KXSD9_CTRL_B_ENABLE,
> - 0);
> + ret = regmap_clear_bits(st->map, KXSD9_REG_CTRL_B,
> + KXSD9_CTRL_B_ENABLE);
Now fits neatly on one line at precisely 80 chars.
I rewrapped whilst applying.
Jonathan
> if (ret)
> return ret;
>
>
More information about the linux-amlogic
mailing list