On Thu, Mar 14, 2019 at 3:46 AM Marc Gonzalez <marc.w.gonzalez@xxxxxxx> wrote: > > On 13/03/2019 20:48, Rob Herring wrote: > > Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt mentions: > - <name>-fixed-regulator : boolean property specifying that <name>-supply is a fixed regulator. > > > There are no users of that property in tree and doesn't look like > > adding it was ever reviewed. We have standard ways to handle this and > > don't need a custom property. > > FWIW, it seems to come from downstream: The question would be is downstream compliant with the upstream binding. If so, then we shouldn't really break things as that's just out of tree which is fine. Or we just need a better explanation of why it is needed. MMC has some properties related to card voltages for example. Maybe the need is similar. Rob