[PATCH v4 1/5] rockchip: io-domain: Add debug logging for regulators during probe

[Kever Yang]

On 2025/5/24 00:53, Justin Klaassen wrote:
> Log the value of the regulators during initialization of the IO-domain
> driver to aid in debugging GPIO voltage configuration problems.
>
> Signed-off-by: Justin Klaassen <justin at tidylabs.net>
> Reviewed-by: Quentin Schulz <quentin.schulz at cherry.de>
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
>
> (no changes since v3)
>
> Changes in v3:
> - Use dev_dbg instead of log_debug
>
> Changes in v2:
> - Made added IO-domain driver logs verbose and consistent
>
>   drivers/misc/rockchip-io-domain.c | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/misc/rockchip-io-domain.c b/drivers/misc/rockchip-io-domain.c
> index 025b6049a9f..a0573c52193 100644
> --- a/drivers/misc/rockchip-io-domain.c
> +++ b/drivers/misc/rockchip-io-domain.c
> @@ -344,8 +344,10 @@ static int rockchip_iodomain_probe(struct udevice *dev)
>   			continue;
>   
>   		ret = device_get_supply_regulator(dev, supply_name, &reg);
> -		if (ret)
> +		if (ret) {
> +			dev_dbg(dev, "%s: Regulator not found\n", supply_name);
>   			continue;
> +		}
>   
>   		ret = regulator_autoset(reg);
>   		if (ret && ret != -EALREADY && ret != -EMEDIUMTYPE &&
> @@ -353,6 +355,7 @@ static int rockchip_iodomain_probe(struct udevice *dev)
>   			continue;
>   
>   		uV = regulator_get_value(reg);
> +		dev_dbg(dev, "%s: Regulator %s at %d uV\n", supply_name, reg->name, uV);
>   		if (uV <= 0)
>   			continue;
>