Re: [PATCH v1 10/11] regulator: core: Use ww_mutex for regulators locking

From: Dmitry Osipenko
Date: Thu Nov 08 2018 - 11:54:36 EST


On 08.11.2018 16:07, Mark Brown wrote:
> On Fri, Oct 05, 2018 at 06:36:37PM +0300, Dmitry Osipenko wrote:
>
>> Wait/wound mutex shall be used in order to avoid lockups on locking of
>> coupled regulators.
>
> This breaks the build due to a few of the drivers (wm8350 and da9210 at
> least) taking rdev locks in their interrupt handlers. I'd suggest
> adding a function for those drivers to use - exposing the ww context
> seems like the wrong thing to do there.

I totally missed those drivers, will fix them in v2.

> Otherwise the series looks good, I've applied everything up to here.
> Thanks for taking the time to pick this up, and especially for figuring
> out the ww_mutex conversion.
>

Thank you.