Re: [PATCH v1 4/4] gpio: pca953x: disable regmap locking for automatic address incrementing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 15, 2020 at 04:18:04PM +0200, Uwe Kleine-König wrote:
> On Mon, Jun 15, 2020 at 04:38:23PM +0300, Andy Shevchenko wrote:
> > On Mon, Jun 15, 2020 at 4:23 PM Uwe Kleine-König
> > <u.kleine-koenig@xxxxxxxxxxxxxx> wrote:
> > > On Mon, Jun 15, 2020 at 03:53:49PM +0300, Andy Shevchenko wrote:
> > > > On Mon, Jun 15, 2020 at 02:20:44PM +0200, Uwe Kleine-König wrote:
> > > > > On Fri, Jun 05, 2020 at 04:40:36PM +0300, Andy Shevchenko wrote:
> > > > > > It's a repetition of the commit aa58a21ae378
> > > > > >   ("gpio: pca953x: disable regmap locking")
> > > > > > which states the following:
> > > > > >
> > > > > >   This driver uses its own locking but regmap silently uses
> > > > > >   a mutex for all operations too. Add the option to disable
> > > > > >   locking to the regmap config struct.
> > > > > >
> > > > > > Fixes: bcf41dc480b1 ("gpio: pca953x: fix handling of automatic address incrementing")
> > > > > > Cc: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>
> > > > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > > > >
> > > > > Ah, good catch. I assume that I didn't have aa58a21ae378 in my tree when
> > > > > I created the patch that then became bcf41dc480b1.
> > > > >
> > > > > Looks right
> > > > >
> > > > > Reviewed-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>
> > > >
> > > > Thanks!
> > > >
> > > > Linus, Bart, just to clarify that this is material for one of the next v5.8-rcX
> > > > (this cycle!).
> > >
> > > I didn't test but I wonder if this patch is really urgent.
> > 
> > I'm talking about this entire fix-series.
> 
> Ah, I didn't notice this is a series as I only got patch 4.

In case you are curious / want to try:
https://lore.kernel.org/linux-gpio/20200605134036.9013-1-andriy.shevchenko@xxxxxxxxxxxxxxx/T/#m5e01e6462f2f72fbc4bdd3f71c330b7a2f75d5ba

-- 
With Best Regards,
Andy Shevchenko





[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux