Re: [PATCH] PM / Domains: Do not print PM domain add error message if EPROBE_DEFER

[Rafael J. Wysocki]

On Wednesday, November 30, 2016 01:24:56 PM Geert Uytterhoeven wrote:
> EPROBE_DEFER is not an error, hence printing an error message like
> 
>     renesas_irqc e61c0000.interrupt-controller: failed to add to PM domain 
> always-on: -517
> 
> may confuse the user.
> 
> Suppress the error message in case of EPROBE_DEFER to fix this.
> 
> Reported-by: Yoshihiro Shimoda <yoshihiro.shimoda...@renesas.com>
> Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be>
> ---
>  drivers/base/power/domain.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c
> index 64164aac6ae3990a..f22137b52dcb641b 100644
> --- a/drivers/base/power/domain.c
> +++ b/drivers/base/power/domain.c
> @@ -2032,8 +2032,9 @@ int genpd_dev_pm_attach(struct device *dev)
>       mutex_unlock(&gpd_list_lock);
>  
>       if (ret < 0) {
> -             dev_err(dev, "failed to add to PM domain %s: %d",
> -                     pd->name, ret);
> +             if (ret != -EPROBE_DEFER)
> +                     dev_err(dev, "failed to add to PM domain %s: %d",
> +                             pd->name, ret);
>               goto out;
>       }
>  
> 

Applied (with ACKs etc).

Thanks,
Rafael