> -----Original Message-----
> From: Patrice Chotard <[email protected]>
> Sent: Tuesday, December 3, 2024 7:06 PM
> To: [email protected]
> Cc: Patrice CHOTARD <[email protected]>; Patrick DELAUNAY 
> <[email protected]>; U-
> Boot STM32 <[email protected]>; Jaehoon Chung 
> <[email protected]>; Jonas
> Karlman <[email protected]>; Marek Vasut <[email protected]>; Quentin Schulz 
> <[email protected]>;
> Simon Glass <[email protected]>; Tom Rini <[email protected]>
> Subject: [PATCH v2 2/2] power: regulator: replace dev_dbg() by dev_err() in 
> regulator_post_bind()
>
> To ease debugging, use dev_err() instead of dev_dbg() for
> alerting when regulator has nonunique value.
>
> Signed-off-by: Patrice Chotard <[email protected]>

Reviewed-by: Jaehoon Chung <[email protected]>

Best Regards,
Jaehoon Chung

>
> ---
>
> Changes in v2:
>   - split initial patch into 2 commits to separate dev_dbg() and
>     dev_err() migration.
>
>  drivers/power/regulator/regulator-uclass.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/power/regulator/regulator-uclass.c 
> b/drivers/power/regulator/regulator-uclass.c
> index 80ea5e65d48..09567eb9dbb 100644
> --- a/drivers/power/regulator/regulator-uclass.c
> +++ b/drivers/power/regulator/regulator-uclass.c
> @@ -446,7 +446,7 @@ static int regulator_post_bind(struct udevice *dev)
>       }
>
>       if (!regulator_name_is_unique(dev, uc_pdata->name)) {
> -             dev_dbg(dev, "'%s' has nonunique value: '%s\n",
> +             dev_err(dev, "'%s' has nonunique value: '%s\n",
>                       property, uc_pdata->name);
>               return -EINVAL;
>       }
> --
> 2.25.1



Reply via email to