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:

quoted

+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.

quoted

+Required properties:
+ - reg : shall identify an existent SCMI Voltage Domain.

quoted

+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