[PATCH v2 3/3] soc: rockchip: io-domain: do more thing about regulator notify

jay.xu at rock-chips.com jay.xu at rock-chips.com
Tue Aug 17 18:16:15 PDT 2021


Hi Mark
--------------
jay.xu at rock-chips.com
>On Tue, Aug 17, 2021 at 11:38:48AM +0800, Jianqun Xu wrote:
>
>> +	} else if (event & REGULATOR_EVENT_ENABLE) {
>> +	uV = regulator_get_voltage(supply->reg);
>>  } else {
>
>I am very surprised this doesn't cause locking issues given that we
>might call notifiers with the regulator API's locks held.  Have you
>tested this with lockdep on? 
Thanks for your reply, there really has locking issue, our test team fail to find out it
but get a pass result.

So, if the voltage cannot get here by driver, can the notify  pass it, like pre-version patch?


More information about the Linux-rockchip mailing list