diff mbox series

[v4] dt-bindings: firmware: arm,scmi: allow multiple instances

Message ID 20240910163456.2383372-1-quic_nkela@quicinc.com (mailing list archive)
State New, archived
Headers show
Series [v4] dt-bindings: firmware: arm,scmi: allow multiple instances | expand

Commit Message

Nikunj Kela Sept. 10, 2024, 4:34 p.m. UTC 
This change extends scmi node name so as to allow multiple
SCMI instances.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Nikunj Kela <quic_nkela@quicinc.com>
---

Changes in v4:
	- Dropped 'virtual' from subject and description

Changes in v3:
        - Added Reviewed-by tag
        - Removed the patch from original series[1]

Changes in v2:
        - Fixed scmi nodename pattern

[1]: https://lore.kernel.org/all/20240903220240.2594102-1-quic_nkela@quicinc.com/
---
 Documentation/devicetree/bindings/firmware/arm,scmi.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Cristian Marussi Sept. 10, 2024, 5:35 p.m. UTC | #1 
On Tue, Sep 10, 2024 at 09:34:56AM -0700, Nikunj Kela wrote:
> This change extends scmi node name so as to allow multiple
> SCMI instances.
> 

Hi Nikunj,

LGTM.
Reviewed-by: Cristian Marussi <cristian.marussi@arm.com>

Thanks,
Cristian
Nikunj Kela Sept. 10, 2024, 6:23 p.m. UTC | #2 
On 9/10/2024 10:35 AM, Cristian Marussi wrote:
> On Tue, Sep 10, 2024 at 09:34:56AM -0700, Nikunj Kela wrote:
>> This change extends scmi node name so as to allow multiple
>> SCMI instances.
>>
> Hi Nikunj,
>
> LGTM.
> Reviewed-by: Cristian Marussi <cristian.marussi@arm.com>
>
> Thanks,
> Cristian

Thanks Cristian for reviewing the patch!
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
index 54d7d11bfed4..5d79b15a1610 100644
--- a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
+++ b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
@@ -27,7 +27,7 @@  anyOf:
 
 properties:
   $nodename:
-    const: scmi
+    pattern: '^scmi(-[0-9]+)?$'
 
   compatible:
     oneOf: