[PATCH 2/4] regulator: core: notify regulator enable with the voltage value

jay.xu at rock-chips.com jay.xu at rock-chips.com
Mon Aug 16 20:23:04 PDT 2021


--------------
jay.xu at rock-chips.com
>Hi, Mark
>--------------
>jay.xu at rock-chips.com
>>On Mon, Aug 16, 2021 at 11:20:01AM +0800, Jianqun Xu wrote:
>>
>>> +	ret = _regulator_get_voltage(rdev);
>>> +	if (ret < 0)
>>> +	ret = 0;
>>> +
>>>  _notifier_call_chain(rdev, REGULATOR_EVENT_ENABLE,
>>> -	     NULL);
>>> +	     &ret);
>>
>>This is going to be really expensive for devices without a cache and
>>it's going to be *very* rarely used if ever.
>
>How about to only to notify the EVENT_ENABLE without get voltage, then driver who care about this event
>could get the voltage itself ? 

ignore my reply here about EVENT_ENABLE, sorry

How about the next patch about PRE_ENABLE ? At least I need a notify without voltage



More information about the Linux-rockchip mailing list