[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240617171353.00006b1f@Huawei.com>
Date: Mon, 17 Jun 2024 17:13:53 +0100
From: Jonathan Cameron <Jonathan.Cameron@...wei.com>
To: Trevor Gamblin <tgamblin@...libre.com>
CC: Jonathan Cameron <jic23@...nel.org>, Lars-Peter Clausen <lars@...afoo.de>,
Dmitry Rokosov <ddrokosov@...rdevices.ru>, "Michael Hennerich"
<Michael.Hennerich@...log.com>, Cosmin Tanislav <cosmin.tanislav@...log.com>,
Chen-Yu Tsai <wens@...e.org>, Hans de Goede <hdegoede@...hat.com>, Ray Jui
<rjui@...adcom.com>, Scott Branden <sbranden@...adcom.com>, Broadcom internal
kernel review list <bcm-kernel-feedback-list@...adcom.com>, Shawn Guo
<shawnguo@...nel.org>, Sascha Hauer <s.hauer@...gutronix.de>, Pengutronix
Kernel Team <kernel@...gutronix.de>, Fabio Estevam <festevam@...il.com>, Neil
Armstrong <neil.armstrong@...aro.org>, Kevin Hilman <khilman@...libre.com>,
"Jerome Brunet" <jbrunet@...libre.com>, Martin Blumenstingl
<martin.blumenstingl@...glemail.com>, Saravanan Sekar <sravanhome@...il.com>,
Orson Zhai <orsonzhai@...il.com>, Baolin Wang
<baolin.wang@...ux.alibaba.com>, Chunyan Zhang <zhang.lyra@...il.com>, Maxime
Coquelin <mcoquelin.stm32@...il.com>, Alexandre Torgue
<alexandre.torgue@...s.st.com>, Nuno Sá
<nuno.sa@...log.com>, Linus Walleij <linus.walleij@...aro.org>, Jean-Baptiste
Maneyrol <jmaneyrol@...ensense.com>, Crt Mori <cmo@...exis.com>,
<linux-iio@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>, <imx@...ts.linux.dev>,
<linux-amlogic@...ts.infradead.org>, <linux-arm-msm@...r.kernel.org>,
<linux-stm32@...md-mailman.stormreply.com>, Uwe Kleine-König <u.kleine-koenig@...libre.com>
Subject: Re: [PATCH v3 22/41] iio: gyro: mpu3050-core: make use of
regmap_clear_bits(), regmap_set_bits()
On Mon, 17 Jun 2024 09:50:02 -0400
Trevor Gamblin <tgamblin@...libre.com> wrote:
> Instead of using regmap_update_bits() and passing the mask twice, use
> regmap_set_bits().
>
> Instead of using regmap_update_bits() and passing val = 0, use
> regmap_clear_bits().
>
> Suggested-by: Uwe Kleine-König <u.kleine-koenig@...libre.com>
> Signed-off-by: Trevor Gamblin <tgamblin@...libre.com>
> ---
> drivers/iio/gyro/mpu3050-core.c | 33 ++++++++++++---------------------
> 1 file changed, 12 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/iio/gyro/mpu3050-core.c b/drivers/iio/gyro/mpu3050-core.c
> index a791ba3a693a..ff1c81553045 100644
> --- a/drivers/iio/gyro/mpu3050-core.c
> +++ b/drivers/iio/gyro/mpu3050-core.c
> @@ -197,8 +197,8 @@ static int mpu3050_start_sampling(struct mpu3050 *mpu3050)
> int i;
>
> /* Reset */
> - ret = regmap_update_bits(mpu3050->map, MPU3050_PWR_MGM,
> - MPU3050_PWR_MGM_RESET, MPU3050_PWR_MGM_RESET);
> + ret = regmap_set_bits(mpu3050->map, MPU3050_PWR_MGM,
> + MPU3050_PWR_MGM_RESET);
> if (ret)
> return ret;
>
> @@ -513,12 +513,8 @@ static irqreturn_t mpu3050_trigger_handler(int irq, void *p)
> "FIFO overflow! Emptying and resetting FIFO\n");
> fifo_overflow = true;
> /* Reset and enable the FIFO */
> - ret = regmap_update_bits(mpu3050->map,
> - MPU3050_USR_CTRL,
> - MPU3050_USR_CTRL_FIFO_EN |
> - MPU3050_USR_CTRL_FIFO_RST,
> - MPU3050_USR_CTRL_FIFO_EN |
> - MPU3050_USR_CTRL_FIFO_RST);
> + ret = regmap_set_bits(mpu3050->map, MPU3050_USR_CTRL,
> + MPU3050_USR_CTRL_FIFO_EN | MPU3050_USR_CTRL_FIFO_RST);
I'll probably break this line up whilst applying.
> if (ret) {
> dev_info(mpu3050->dev, "error resetting FIFO\n");
> goto out_trigger_unlock;
> @@ -799,10 +795,8 @@ static int mpu3050_hw_init(struct mpu3050 *mpu3050)
> u64 otp;
>
> /* Reset */
> - ret = regmap_update_bits(mpu3050->map,
> - MPU3050_PWR_MGM,
> - MPU3050_PWR_MGM_RESET,
> - MPU3050_PWR_MGM_RESET);
> + ret = regmap_set_bits(mpu3050->map, MPU3050_PWR_MGM,
> + MPU3050_PWR_MGM_RESET);
> if (ret)
> return ret;
>
> @@ -872,8 +866,8 @@ static int mpu3050_power_up(struct mpu3050 *mpu3050)
> msleep(200);
>
> /* Take device out of sleep mode */
> - ret = regmap_update_bits(mpu3050->map, MPU3050_PWR_MGM,
> - MPU3050_PWR_MGM_SLEEP, 0);
> + ret = regmap_clear_bits(mpu3050->map, MPU3050_PWR_MGM,
> + MPU3050_PWR_MGM_SLEEP);
> if (ret) {
> regulator_bulk_disable(ARRAY_SIZE(mpu3050->regs), mpu3050->regs);
> dev_err(mpu3050->dev, "error setting power mode\n");
> @@ -895,8 +889,8 @@ static int mpu3050_power_down(struct mpu3050 *mpu3050)
> * then we would be wasting power unless we go to sleep mode
> * first.
> */
> - ret = regmap_update_bits(mpu3050->map, MPU3050_PWR_MGM,
> - MPU3050_PWR_MGM_SLEEP, MPU3050_PWR_MGM_SLEEP);
> + ret = regmap_set_bits(mpu3050->map, MPU3050_PWR_MGM,
> + MPU3050_PWR_MGM_SLEEP);
> if (ret)
> dev_err(mpu3050->dev, "error putting to sleep\n");
>
> @@ -997,11 +991,8 @@ static int mpu3050_drdy_trigger_set_state(struct iio_trigger *trig,
> return ret;
>
> /* Reset and enable the FIFO */
> - ret = regmap_update_bits(mpu3050->map, MPU3050_USR_CTRL,
> - MPU3050_USR_CTRL_FIFO_EN |
> - MPU3050_USR_CTRL_FIFO_RST,
> - MPU3050_USR_CTRL_FIFO_EN |
> - MPU3050_USR_CTRL_FIFO_RST);
> + ret = regmap_set_bits(mpu3050->map, MPU3050_USR_CTRL,
> + MPU3050_USR_CTRL_FIFO_EN | MPU3050_USR_CTRL_FIFO_RST);
and this one.
Assuming we don't need a v4 for some other reason,
Jonathan
> if (ret)
> return ret;
>
>
Powered by blists - more mailing lists