> 
> In dt-bindings for ufs, "<name>-max-microamp" property indicates
> current limit and is mandatory if "<name>-fixed-regulator" is not
> defined on a specified regulator.
> 
> However, in some platforms, regulators without "<name>-fixed-regulator"
> property may not need to define their current limit because they may
> want to define voltage range only for proper voltage switching in
> different power modes, especially for vcc, vccq or vccq2.
> 
> Currently missing "<name>-max-microamp" property in device tree will
> lead to initialization fail currently, thus such limitation shall be
lead initialization to fail, thus ...

> resolved to tolerate this kind of regulators.
> 
> After resolving this, regulators without "<name>-max-microamp"
> property will have undefined "max current" value, i.e., zero value
> in "max_uA" field in struct ufs_vreg. Because we do bypass current
> switching operation (by regulator_set_load) in case of undefined
> current limit, this patch shall be safe.
> 
> Signed-off-by: Stanley Chu <stanley....@mediatek.com>
Reviewed-by: Avri Altman <avri.alt...@wdc.com>

You might want to cc some platform guys, e.g. Evan Green and/or Codeaurora guys,
To take a look.

Thanks,
Avri

Reply via email to