[PATCH v4 2/2] ufs: qcom: add ufs host regulator support

[Neha Malcom Francis]

On 2026-05-25 21:41:00+02:00, Federico Amedeo Izzo wrote:
> Il 2026-05-25 14:12 Neha Malcom Francis ha scritto:
> 
> > On Sat, 23 May 2026 15:04:25 +0200, Federico Amedeo Izzo 
> > <federico at izzo.pro> wrote:
> > 
> > 
> > vccq2 is optional right? Is it verified that the optional code route 
> > will never
> > return ENOSYS and there is handling in regulator_set_enable for NULL 
> > regulators?
> 
> Hi,
> vccq2 is indeed optional, if it's not present, regulator_set_enable() 
> will handle the NULL regulator and return -ENOSYS.
> The warning is skipped for -ENOSYS as we don't want to print warnings 
> for non existing regulators,
> 
> Below is a section of regulator_set_enable() from regulator_uclass.c
> 
> int regulator_set_enable(struct udevice *dev, bool enable)
> {
>          const struct dm_regulator_ops *ops = dev_get_driver_ops(dev);
>          struct dm_regulator_uclass_plat *uc_pdata;
>          int ret, old_enable = 0;
> 
>          if (!ops || !ops->set_enable)
>                  return -ENOSYS;
> [...]
>

Reviewed-by: Neha Malcom Francis <n-francis at ti.com>

> Greetings,
> Federico Amedeo Izzo