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

Nikunj Kela posted 1 patch 2 months, 2 weeks ago
Documentation/devicetree/bindings/firmware/arm,scmi.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH v4] dt-bindings: firmware: arm,scmi: allow multiple instances
Posted by Nikunj Kela 2 months, 2 weeks ago
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(-)

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:
-- 
2.34.1
Re: [PATCH v4] dt-bindings: firmware: arm,scmi: allow multiple instances
Posted by Cristian Marussi 2 months, 2 weeks ago
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
Re: [PATCH v4] dt-bindings: firmware: arm,scmi: allow multiple instances
Posted by Nikunj Kela 2 months, 2 weeks ago
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!