[U-Boot] [PATCH V2 2/2] core: device: use dev_power_domain_on

[Lokesh Vutla]

Hi Peng,

On 25/09/19 6:56 AM, Peng Fan wrote:
> Hi Lokesh,
> 
>> Subject: Re: [PATCH V2 2/2] core: device: use dev_power_domain_on
>>
>>
>>
>> On 17/09/19 2:59 PM, Peng Fan wrote:
>>> When multiple power domains attached to a device, need power on them
>>> all, so use dev_power_domain_on to do that.
>>>
>>> Signed-off-by: Peng Fan <peng.fan at nxp.com>
>>> ---
>>>
>>> V2:
>>>  use dev_power_domain_on in patch 1/2
>>>
>>>  drivers/core/device.c | 6 +++---
>>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/drivers/core/device.c b/drivers/core/device.c index
>>> 05dadf98f9..f4d7140698 100644
>>> --- a/drivers/core/device.c
>>> +++ b/drivers/core/device.c
>>> @@ -307,7 +307,6 @@ static void *alloc_priv(int size, uint flags)
>>>
>>>  int device_probe(struct udevice *dev)  {
>>> -	struct power_domain pd;
>>>  	const struct driver *drv;
>>>  	int size = 0;
>>>  	int ret;
>>> @@ -390,8 +389,9 @@ int device_probe(struct udevice *dev)
>>>
>>>  	if (CONFIG_IS_ENABLED(POWER_DOMAIN) && dev->parent &&
>>>  	    device_get_uclass_id(dev) != UCLASS_POWER_DOMAIN) {
>>> -		if (!power_domain_get(dev, &pd))
>>> -			power_domain_on(&pd);
>>> +		ret = dev_power_domain_on(dev);
>>> +		if (ret)
>>> +			goto fail;
>>
>> Also can you not return here in case of failure? It might be the case that
>> power_domain driver might not be available yet
> 
> Only when POWER_DOMAIN is enabled, dev_power_domain_on will be called.
> I not follow you not available yet.

Never mind, it is an issue with enabling power domain on my end. Ill take back
my comment, Sorry for the noise.

Thanks and regards,
Lokesh

> 
> Thanks,
> Peng.
> 
>>
>> Thanks and regards,
>> Lokesh
>>
>>>  	}
>>>
>>>  	ret = uclass_pre_probe_device(dev);
>>>