Re: [PATCH] regulator: Fix deadlock during regulator registration

From: Javier Martinez Canillas
Date: Wed Mar 30 2016 - 19:38:42 EST


Hello Mark,

On 03/30/2016 07:34 PM, Mark Brown wrote:
> On Wed, Mar 30, 2016 at 07:25:49PM -0400, Javier Martinez Canillas wrote:
>
>> You are doing some cleanup of the clean and scrub error paths by removing
>> rdev and returning ERR_PTR(ret) directly. I believe that should be in a
>> separate patch since is not related to the fix.
>
> There needs to be some reorganization due to the movement with the mutex
> handling.
>

Yes, I know. My point was that besides the reorganization that is needed,
he is removing some variables and that seemed like a separate change to me.

But of course you are the maintainer so ignore my comment if you don't mind
about that.

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America