[PATCH v2 2/3] power: regulator: common: use dm_gpio_is_valid helper

[Neil Armstrong]

On 1/8/26 15:42, Julien Stephan wrote:
> Use dm_gpio_is_valid() helper function instead of manually checking the
> gpio.
> 
> Reviewed-by: Peng Fan <peng.fan at nxp.com>
> Signed-off-by: Julien Stephan <jstephan at baylibre.com>
> ---
>   drivers/power/regulator/regulator_common.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/power/regulator/regulator_common.c b/drivers/power/regulator/regulator_common.c
> index ce3d80670de045dde7e263c5301bccd744abd2f5..cf98998579aa8a3cb7f09dccd2124207a58d8b00 100644
> --- a/drivers/power/regulator/regulator_common.c
> +++ b/drivers/power/regulator/regulator_common.c
> @@ -51,7 +51,7 @@ int regulator_common_get_enable(const struct udevice *dev,
>   	struct regulator_common_plat *plat)
>   {
>   	/* Enable GPIO is optional */
> -	if (!plat->gpio.dev)
> +	if (!dm_gpio_is_valid(&plat->gpio))
>   		return true;
>   
>   	return dm_gpio_get_value(&plat->gpio);
> 

Reviewed-by: Neil Armstrong <neil.armstrong at linaro.org>

Thanks,
Neil