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
