* [PATCH 0/2] regulator: scmi: add support for registering SCMI regulators by name
@ 2021-12-10 1:54 David Collins
2021-12-10 1:54 ` [PATCH 1/2] dt-bindings: firmware: arm,scmi: define support for name based regulators David Collins
2022-01-24 23:20 ` [RESEND PATCH 0/2] regulator: scmi: add support for registering SCMI regulators by name David Collins
0 siblings, 2 replies; 5+ messages in thread
From: David Collins @ 2021-12-10 1:54 UTC (permalink / raw)
To: Rob Herring, Sudeep Holla, Mark Brown, Liam Girdwood, devicetree
Cc: David Collins, Cristian Marussi, linux-arm-kernel, linux-kernel,
linux-arm-msm, Subbaraman Narayanamurthy
Add support to register SCMI regulator subnodes based on an SCMI
Voltage Domain name specified via the 'regulator-name' device tree
property. In doing so, make the 'reg' property optional with the
constraint that at least one of 'reg' or 'regulator-name' must be
specified. If both are specified, then both must match the
Voltage Domain data exposed by the SCMI platform.
Name based SCMI regulator registration helps ensure that an SCMI
agent doesn't need to be aware of the numbering scheme used for
Voltage Domains by the SCMI platform. It also ensures that the
correct Voltage Domain is selected for a given physical regulator.
This cannot be guaranteed with numeric Voltage Domain IDs alone.
David Collins (2):
dt-bindings: firmware: arm,scmi: define support for name based
regulators
regulator: scmi: add support for registering SCMI regulators by name
.../bindings/firmware/arm,scmi.yaml | 11 +++-
drivers/regulator/scmi-regulator.c | 57 ++++++++++++++++++-
2 files changed, 62 insertions(+), 6 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/2] dt-bindings: firmware: arm,scmi: define support for name based regulators
2021-12-10 1:54 [PATCH 0/2] regulator: scmi: add support for registering SCMI regulators by name David Collins
@ 2021-12-10 1:54 ` David Collins
2021-12-15 19:51 ` Rob Herring
2022-01-24 23:20 ` [RESEND PATCH " David Collins
2022-01-24 23:20 ` [RESEND PATCH 0/2] regulator: scmi: add support for registering SCMI regulators by name David Collins
1 sibling, 2 replies; 5+ messages in thread
From: David Collins @ 2021-12-10 1:54 UTC (permalink / raw)
To: Rob Herring, Sudeep Holla, devicetree
Cc: David Collins, Mark Brown, Liam Girdwood, Cristian Marussi,
linux-arm-kernel, linux-kernel, linux-arm-msm,
Subbaraman Narayanamurthy
Allow SCMI regulator subnodes to be specified either by ID using
the "reg" property or by name using the "regulator-name" property.
Name based SCMI regulator specification helps ensure that an SCMI
agent doesn't need to be aware of the numbering scheme used for
Voltage Domains by the SCMI platform. It also ensures that the
correct Voltage Domain is selected for a given physical regulator.
This cannot be guaranteed with numeric Voltage Domain IDs alone.
Signed-off-by: David Collins <quic_collinsd@quicinc.com>
---
.../devicetree/bindings/firmware/arm,scmi.yaml | 11 ++++++++---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
index 5c4c6782e052..bc4a84fe25d2 100644
--- a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
+++ b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
@@ -155,7 +155,7 @@ properties:
The list of all regulators provided by this SCMI controller.
patternProperties:
- '^regulators@[0-9a-f]+$':
+ '^regulator.+$':
type: object
$ref: "../regulator/regulator.yaml#"
@@ -164,8 +164,13 @@ properties:
maxItems: 1
description: Identifier for the voltage regulator.
- required:
- - reg
+ regulator-name: true
+
+ anyOf:
+ - required:
+ - reg
+ - required:
+ - regulator-name
additionalProperties: false
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] dt-bindings: firmware: arm,scmi: define support for name based regulators
2021-12-10 1:54 ` [PATCH 1/2] dt-bindings: firmware: arm,scmi: define support for name based regulators David Collins
@ 2021-12-15 19:51 ` Rob Herring
2022-01-24 23:20 ` [RESEND PATCH " David Collins
1 sibling, 0 replies; 5+ messages in thread
From: Rob Herring @ 2021-12-15 19:51 UTC (permalink / raw)
To: David Collins
Cc: Rob Herring, linux-arm-kernel, linux-arm-msm, Cristian Marussi,
Mark Brown, Liam Girdwood, Sudeep Holla, devicetree,
linux-kernel, Subbaraman Narayanamurthy
On Thu, 09 Dec 2021 17:54:41 -0800, David Collins wrote:
> Allow SCMI regulator subnodes to be specified either by ID using
> the "reg" property or by name using the "regulator-name" property.
>
> Name based SCMI regulator specification helps ensure that an SCMI
> agent doesn't need to be aware of the numbering scheme used for
> Voltage Domains by the SCMI platform. It also ensures that the
> correct Voltage Domain is selected for a given physical regulator.
> This cannot be guaranteed with numeric Voltage Domain IDs alone.
>
> Signed-off-by: David Collins <quic_collinsd@quicinc.com>
> ---
> .../devicetree/bindings/firmware/arm,scmi.yaml | 11 ++++++++---
> 1 file changed, 8 insertions(+), 3 deletions(-)
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [RESEND PATCH 0/2] regulator: scmi: add support for registering SCMI regulators by name
2021-12-10 1:54 [PATCH 0/2] regulator: scmi: add support for registering SCMI regulators by name David Collins
2021-12-10 1:54 ` [PATCH 1/2] dt-bindings: firmware: arm,scmi: define support for name based regulators David Collins
@ 2022-01-24 23:20 ` David Collins
1 sibling, 0 replies; 5+ messages in thread
From: David Collins @ 2022-01-24 23:20 UTC (permalink / raw)
To: Rob Herring, Sudeep Holla, Mark Brown, Liam Girdwood, devicetree
Cc: David Collins, Cristian Marussi, linux-arm-kernel, linux-kernel,
linux-arm-msm, Subbaraman Narayanamurthy
Add support to register SCMI regulator subnodes based on an SCMI
Voltage Domain name specified via the 'regulator-name' device tree
property. In doing so, make the 'reg' property optional with the
constraint that at least one of 'reg' or 'regulator-name' must be
specified. If both are specified, then both must match the
Voltage Domain data exposed by the SCMI platform.
Name based SCMI regulator registration helps ensure that an SCMI
agent doesn't need to be aware of the numbering scheme used for
Voltage Domains by the SCMI platform. It also ensures that the
correct Voltage Domain is selected for a given physical regulator.
This cannot be guaranteed with numeric Voltage Domain IDs alone.
David Collins (2):
dt-bindings: firmware: arm,scmi: define support for name based
regulators
regulator: scmi: add support for registering SCMI regulators by name
.../bindings/firmware/arm,scmi.yaml | 11 +++-
drivers/regulator/scmi-regulator.c | 57 ++++++++++++++++++-
2 files changed, 62 insertions(+), 6 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [RESEND PATCH 1/2] dt-bindings: firmware: arm,scmi: define support for name based regulators
2021-12-10 1:54 ` [PATCH 1/2] dt-bindings: firmware: arm,scmi: define support for name based regulators David Collins
2021-12-15 19:51 ` Rob Herring
@ 2022-01-24 23:20 ` David Collins
1 sibling, 0 replies; 5+ messages in thread
From: David Collins @ 2022-01-24 23:20 UTC (permalink / raw)
To: Rob Herring, Sudeep Holla, devicetree
Cc: David Collins, Mark Brown, Liam Girdwood, Cristian Marussi,
linux-arm-kernel, linux-kernel, linux-arm-msm,
Subbaraman Narayanamurthy
Allow SCMI regulator subnodes to be specified either by ID using
the "reg" property or by name using the "regulator-name" property.
Name based SCMI regulator specification helps ensure that an SCMI
agent doesn't need to be aware of the numbering scheme used for
Voltage Domains by the SCMI platform. It also ensures that the
correct Voltage Domain is selected for a given physical regulator.
This cannot be guaranteed with numeric Voltage Domain IDs alone.
Signed-off-by: David Collins <quic_collinsd@quicinc.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
.../devicetree/bindings/firmware/arm,scmi.yaml | 11 ++++++++---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
index 5c4c6782e052..bc4a84fe25d2 100644
--- a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
+++ b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
@@ -155,7 +155,7 @@ properties:
The list of all regulators provided by this SCMI controller.
patternProperties:
- '^regulators@[0-9a-f]+$':
+ '^regulator.+$':
type: object
$ref: "../regulator/regulator.yaml#"
@@ -164,8 +164,13 @@ properties:
maxItems: 1
description: Identifier for the voltage regulator.
- required:
- - reg
+ regulator-name: true
+
+ anyOf:
+ - required:
+ - reg
+ - required:
+ - regulator-name
additionalProperties: false
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-01-25 0:32 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-10 1:54 [PATCH 0/2] regulator: scmi: add support for registering SCMI regulators by name David Collins
2021-12-10 1:54 ` [PATCH 1/2] dt-bindings: firmware: arm,scmi: define support for name based regulators David Collins
2021-12-15 19:51 ` Rob Herring
2022-01-24 23:20 ` [RESEND PATCH " David Collins
2022-01-24 23:20 ` [RESEND PATCH 0/2] regulator: scmi: add support for registering SCMI regulators by name David Collins
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).