[PATCH v6 4/5] dt-bindings: arm: add support for SCMI Regulators

Cristian Marussi cristian.marussi at arm.com
Mon Nov 23 13:52:08 EST 2020


On Mon, Nov 23, 2020 at 05:30:08PM +0000, Mark Brown wrote:
> On Thu, Nov 19, 2020 at 07:10:50PM +0000, Cristian Marussi wrote:
> 
> > +This binding uses the common regulator binding[6] but, due to SCMI abstractions,
> > +supports only a subset of its properties as specified below amongst Optional
> > +properties.
> 
> > +Required properties:
> > + - reg : shall identify an existent SCMI Voltage Domain.
> 
> > +Optional properties:
> > + - regulator-name
> > + - regulator-min-microvolt / regulator-max-microvolt
> > + - regulator-always-on / regulator-boot-on
> > + - regulator-max-step-microvolt
> > + - regulator-coupled-with / regulator-coupled-max-spread
> 
> Please send a followup patch removing this stuff about only specific
> properties being supported, that's just asking for bitrot and is equally
> true for most regulator drivers - people shouldn't have to do do an
> audit of every single regulator driver to add a generic feature.

Ok I'll remove any reference to partial SCMI regulator support.

Thanks

Cristian





More information about the linux-arm-kernel mailing list