[PATCH] power-domain: Fix use of uninitialized value in dev_power_domain_ctrl

Jaehoon Chung jh80.chung at samsung.com
Tue Mar 29 01:21:51 CEST 2022


On 3/24/22 03:26, Sean Anderson wrote:
> If dev_count_phandle_with_args returns 0 or another error, then pd will never
> have been initialized by power_domain_get_by_index. Avoid comparing against
> pd.dev in this situation.
> 
> Fixes: 3e4fcfa4bc ("power-domain: fix hang in endless loop on i.MX8")
> Signed-off-by: Sean Anderson <seanga2 at gmail.com>

Reviewed-by: Jaehoon Chung <jh80.chung at samsung.com>

Best Regards,
Jaehoon Chung

> ---
> 
>  drivers/power/domain/power-domain-uclass.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/power/domain/power-domain-uclass.c b/drivers/power/domain/power-domain-uclass.c
> index 33f9206bd0..1a317a00aa 100644
> --- a/drivers/power/domain/power-domain-uclass.c
> +++ b/drivers/power/domain/power-domain-uclass.c
> @@ -137,7 +137,7 @@ static int dev_power_domain_ctrl(struct udevice *dev, bool on)
>  	 * off their power-domain parent. So we will get here again and
>  	 * again and will be stuck in an endless loop.
>  	 */
> -	if (!on && dev_get_parent(dev) == pd.dev &&
> +	if (count > 0 && !on && dev_get_parent(dev) == pd.dev &&
>  	    device_get_uclass_id(dev) == UCLASS_POWER_DOMAIN)
>  		return ret;
>  



More information about the U-Boot mailing list