[PATCH V2] pmdomain: rockchip: quiet regulator error on -EPROBE_DEFER
Anand Moon
linux.amoon at gmail.com
Sat Jan 31 08:36:20 PST 2026
Hi Chris,
On Fri, 30 Jan 2026 at 22:03, Chris Morgan <macroalpha82 at gmail.com> wrote:
>
> From: Chris Morgan <macromorgan at hotmail.com>
>
> Change the dev_err() to dev_err_probe() under rockchip_pd_power_on()
> to prevent errors early in the boot process when the requested
> regulator is not yet available. This converts errors like the following
> to debug messages:
>
> rockchip-pm-domain fd8d8000.power-management:power-controller: Failed to enable supply: -517
>
> Signed-off-by: Chris Morgan <macromorgan at hotmail.com>
> Tested-by: Diederik de Haas <diederik at cknow-tech.com>
Reviewed-by: Anand Moon <linux.amoon at gmail.com>
Thanks
-Anand
> ---
> Changes since v1:
> - Simplified return to single line
> ---
> drivers/pmdomain/rockchip/pm-domains.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/pmdomain/rockchip/pm-domains.c b/drivers/pmdomain/rockchip/pm-domains.c
> index 997e93c12951..6d3fb18f4ab3 100644
> --- a/drivers/pmdomain/rockchip/pm-domains.c
> +++ b/drivers/pmdomain/rockchip/pm-domains.c
> @@ -705,10 +705,9 @@ static int rockchip_pd_power_on(struct generic_pm_domain *domain)
> int ret;
>
> ret = rockchip_pd_regulator_enable(pd);
> - if (ret) {
> - dev_err(pd->pmu->dev, "Failed to enable supply: %d\n", ret);
> - return ret;
> - }
> + if (ret)
> + return dev_err_probe(pd->pmu->dev, ret,
> + "Failed to enable supply: %d\n", ret);
>
> ret = rockchip_pd_power(pd, true);
> if (ret)
> --
> 2.43.0
>
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip
More information about the Linux-rockchip
mailing list