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

[Tanmay Kathpalia]

Hi,


On 1/13/2026 3:05 PM, Neil Armstrong wrote:
> 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>
> 

Reviewed-by: Tanmay Kathpalia <tanmay.kathpalia at altera.com>

Regards,
Tanmay