[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20240622190706.670224da@jic23-huawei>
Date: Sat, 22 Jun 2024 19:07:06 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: Uwe Kleine-König <u.kleine-koenig@...libre.com>
Cc: Trevor Gamblin <tgamblin@...libre.com>, 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
Subject: Re: [PATCH v3 00/41] iio: simplify with regmap_set_bits(),
regmap_clear_bits()
On Mon, 17 Jun 2024 17:17:27 +0200
Uwe Kleine-König <u.kleine-koenig@...libre.com> wrote:
> Hello Trevor,
>
> On Mon, Jun 17, 2024 at 09:49:40AM -0400, Trevor Gamblin wrote:
> > Simplify the way regmap is accessed in iio drivers.
> >
> > 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().
> >
> > The series is marked as v3, but the previous two revisions were single
> > patches. There was also a resend of v1 due to not properly CCing the
> > mailing lists on the first attempt. Trailers were pulled in from those
> > where relevant.
> >
> > Link to v2: https://lore.kernel.org/linux-iio/20240613133927.3851344-1-tgamblin@baylibre.com/
> > Link to v1: https://lore.kernel.org/linux-iio/20240611165214.4091591-1-tgamblin@baylibre.com/
> >
> > Suggested-by: Uwe Kleine-König <u.kleine-koenig@...libre.com>
> > Reviewed-by: Linus Walleij <linus.walleij@...aro.org>
> > Signed-off-by: Trevor Gamblin <tgamblin@...libre.com>
>
> Acked-by: Uwe Kleine-König <u.kleine-koenig@...libre.com>
>
Series applied with a few wrapping tweaks.
Applied to the togreg branch of iio.git and pushed out as testing for 0-day
to take a look + because I might rebase that tree after some fixes
make it upstream on which I have other dependencies and I don't like doing
that once it is in the public version of the togreg branch.
Thanks
Jonathan
> Thanks
> Uwe
Powered by blists - more mailing lists