linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v4] dt-bindings: firmware: arm,scmi: allow multiple instances
@ 2024-09-10 16:34 Nikunj Kela
  2024-09-10 17:35 ` Cristian Marussi
  0 siblings, 1 reply; 3+ messages in thread
From: Nikunj Kela @ 2024-09-10 16:34 UTC (permalink / raw)
  To: sudeep.holla, robh, krzk+dt, conor+dt
  Cc: cristian.marussi, arm-scmi, linux-arm-kernel, devicetree,
	linux-kernel, kernel, quic_psodagud, Nikunj Kela,
	Krzysztof Kozlowski

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



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v4] dt-bindings: firmware: arm,scmi: allow multiple instances
  2024-09-10 16:34 [PATCH v4] dt-bindings: firmware: arm,scmi: allow multiple instances Nikunj Kela
@ 2024-09-10 17:35 ` Cristian Marussi
  2024-09-10 18:23   ` Nikunj Kela
  0 siblings, 1 reply; 3+ messages in thread
From: Cristian Marussi @ 2024-09-10 17:35 UTC (permalink / raw)
  To: Nikunj Kela
  Cc: sudeep.holla, robh, krzk+dt, conor+dt, cristian.marussi, arm-scmi,
	linux-arm-kernel, devicetree, linux-kernel, kernel, quic_psodagud,
	Krzysztof Kozlowski

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



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v4] dt-bindings: firmware: arm,scmi: allow multiple instances
  2024-09-10 17:35 ` Cristian Marussi
@ 2024-09-10 18:23   ` Nikunj Kela
  0 siblings, 0 replies; 3+ messages in thread
From: Nikunj Kela @ 2024-09-10 18:23 UTC (permalink / raw)
  To: Cristian Marussi
  Cc: sudeep.holla, robh, krzk+dt, conor+dt, arm-scmi, linux-arm-kernel,
	devicetree, linux-kernel, kernel, quic_psodagud,
	Krzysztof Kozlowski


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!




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-09-10 18:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-10 16:34 [PATCH v4] dt-bindings: firmware: arm,scmi: allow multiple instances Nikunj Kela
2024-09-10 17:35 ` Cristian Marussi
2024-09-10 18:23   ` Nikunj Kela

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