Documentation/devicetree/bindings/firmware/arm,scmi.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
From: Nikunj Kela <quic_nkela@quicinc.com>
Allow multiple SCMI instances by extending the scmi node name to include
an instance number suffix.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Cristian Marussi <cristian.marussi@arm.com>
Signed-off-by: Nikunj Kela <quic_nkela@quicinc.com>
Signed-off-by: Deepti Jaggi <quic_djaggi@quicinc.com>
---
Changes in v6:
- Dropped 'this change' from description
- Link to v5: https://lore.kernel.org/all/20250423005824.3993256-1-quic_djaggi@quicinc.com
Changes in v5:
- Added Reviewed-by tag
- Link to v4: https://lore.kernel.org/all/20240910163456.2383372-1-quic_nkela@quicinc.com
Changes in v4:
- Dropped 'virtual' from subject and description
- Link to v3: https://lore.kernel.org/all/20240905201217.3815113-1-quic_nkela@quicinc.com
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 abbd62f1fed0993ab98fa44bdb9a0575f8e1c78e..be817fd9cc34b14009a3b1d69e78b802215571b6 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:
---
base-commit: 0b90c3b6d76ea512dc3dac8fb30215e175b0019a
change-id: 20250728-8255-scmi-ed963ef8f155
Best regards,
--
Deepti Jaggi <quic_djaggi@quicinc.com>On Wed, 30 Jul 2025 14:30:00 -0700, Deepti Jaggi wrote:
> Allow multiple SCMI instances by extending the scmi node name to include
> an instance number suffix.
>
Applied to sudeep.holla/linux (for-next/scmi/updates), thanks!
[1/1] dt-bindings: firmware: arm,scmi: allow multiple instances
https://git.kernel.org/sudeep.holla/c/a4d5f63d4384
--
Regards,
Sudeep
On 7/30/25 14:30, Deepti Jaggi wrote: > From: Nikunj Kela <quic_nkela@quicinc.com> > > Allow multiple SCMI instances by extending the scmi node name to include > an instance number suffix. > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> > Signed-off-by: Nikunj Kela <quic_nkela@quicinc.com> > Signed-off-by: Deepti Jaggi <quic_djaggi@quicinc.com> > --- > Changes in v6: > - Dropped 'this change' from description > - Link to v5: https://lore.kernel.org/all/20250423005824.3993256-1-quic_djaggi@quicinc.com > > Changes in v5: > - Added Reviewed-by tag > - Link to v4: https://lore.kernel.org/all/20240910163456.2383372-1-quic_nkela@quicinc.com > > Changes in v4: > - Dropped 'virtual' from subject and description > - Link to v3: https://lore.kernel.org/all/20240905201217.3815113-1-quic_nkela@quicinc.com > > 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 abbd62f1fed0993ab98fa44bdb9a0575f8e1c78e..be817fd9cc34b14009a3b1d69e78b802215571b6 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: > > --- > base-commit: 0b90c3b6d76ea512dc3dac8fb30215e175b0019a > change-id: 20250728-8255-scmi-ed963ef8f155 Gentle ping!! Could you please review the patch? > Best regards,
On Wed, Aug 20, 2025 at 11:44:26AM -0700, Deepti Jaggi wrote: > > On 7/30/25 14:30, Deepti Jaggi wrote: > > From: Nikunj Kela <quic_nkela@quicinc.com> > > Hi, > > Allow multiple SCMI instances by extending the scmi node name to include > > an instance number suffix. > > > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> > > Signed-off-by: Nikunj Kela <quic_nkela@quicinc.com> > > Signed-off-by: Deepti Jaggi <quic_djaggi@quicinc.com> > > --- > > Changes in v6: > > - Dropped 'this change' from description > > - Link to v5: https://lore.kernel.org/all/20250423005824.3993256-1-quic_djaggi@quicinc.com > > > > Changes in v5: > > - Added Reviewed-by tag > > - Link to v4: https://lore.kernel.org/all/20240910163456.2383372-1-quic_nkela@quicinc.com > > > > Changes in v4: > > - Dropped 'virtual' from subject and description > > - Link to v3: https://lore.kernel.org/all/20240905201217.3815113-1-quic_nkela@quicinc.com > > > > 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 abbd62f1fed0993ab98fa44bdb9a0575f8e1c78e..be817fd9cc34b14009a3b1d69e78b802215571b6 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: > > > > --- > > base-commit: 0b90c3b6d76ea512dc3dac8fb30215e175b0019a > > change-id: 20250728-8255-scmi-ed963ef8f155 > > Gentle ping!! > Could you please review the patch? > I thought this was already reviewed/acked and merged since ages.... ...also the wording-change in the commit message LGTM. Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> Thanks, Cristian
On 21/08/2025 10:04, Cristian Marussi wrote: > On Wed, Aug 20, 2025 at 11:44:26AM -0700, Deepti Jaggi wrote: >> >> On 7/30/25 14:30, Deepti Jaggi wrote: >>> From: Nikunj Kela <quic_nkela@quicinc.com> >>> > > Hi, > >>> Allow multiple SCMI instances by extending the scmi node name to include >>> an instance number suffix. >>> >>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>> Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> >>> Signed-off-by: Nikunj Kela <quic_nkela@quicinc.com> >>> Signed-off-by: Deepti Jaggi <quic_djaggi@quicinc.com> >>> --- >>> Changes in v6: >>> - Dropped 'this change' from description >>> - Link to v5: https://lore.kernel.org/all/20250423005824.3993256-1-quic_djaggi@quicinc.com >>> >>> Changes in v5: >>> - Added Reviewed-by tag >>> - Link to v4: https://lore.kernel.org/all/20240910163456.2383372-1-quic_nkela@quicinc.com >>> >>> Changes in v4: >>> - Dropped 'virtual' from subject and description >>> - Link to v3: https://lore.kernel.org/all/20240905201217.3815113-1-quic_nkela@quicinc.com >>> >>> 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 abbd62f1fed0993ab98fa44bdb9a0575f8e1c78e..be817fd9cc34b14009a3b1d69e78b802215571b6 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: >>> >>> --- >>> base-commit: 0b90c3b6d76ea512dc3dac8fb30215e175b0019a >>> change-id: 20250728-8255-scmi-ed963ef8f155 >> >> Gentle ping!! You cannot ping gently while screaming!!! (you see how gentle this feels?) >> Could you please review the patch? >> > > I thought this was already reviewed/acked and merged since ages.... > ...also the wording-change in the commit message LGTM. > > Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> It was. Twice! This is just unjustified ping. Best regards, Krzysztof
On Thu, Aug 21, 2025 at 10:29:00AM +0200, Krzysztof Kozlowski wrote: > On 21/08/2025 10:04, Cristian Marussi wrote: > > On Wed, Aug 20, 2025 at 11:44:26AM -0700, Deepti Jaggi wrote: > >> > >> On 7/30/25 14:30, Deepti Jaggi wrote: > >>> From: Nikunj Kela <quic_nkela@quicinc.com> > >>> > > > > Hi, > > > >>> Allow multiple SCMI instances by extending the scmi node name to include > >>> an instance number suffix. > >>> > >>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > >>> Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> > >>> Signed-off-by: Nikunj Kela <quic_nkela@quicinc.com> > >>> Signed-off-by: Deepti Jaggi <quic_djaggi@quicinc.com> > >>> --- > >>> Changes in v6: > >>> - Dropped 'this change' from description > >>> - Link to v5: https://lore.kernel.org/all/20250423005824.3993256-1-quic_djaggi@quicinc.com > >>> > >>> Changes in v5: > >>> - Added Reviewed-by tag > >>> - Link to v4: https://lore.kernel.org/all/20240910163456.2383372-1-quic_nkela@quicinc.com > >>> > >>> Changes in v4: > >>> - Dropped 'virtual' from subject and description > >>> - Link to v3: https://lore.kernel.org/all/20240905201217.3815113-1-quic_nkela@quicinc.com > >>> > >>> 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 abbd62f1fed0993ab98fa44bdb9a0575f8e1c78e..be817fd9cc34b14009a3b1d69e78b802215571b6 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: > >>> > >>> --- > >>> base-commit: 0b90c3b6d76ea512dc3dac8fb30215e175b0019a > >>> change-id: 20250728-8255-scmi-ed963ef8f155 > >> > >> Gentle ping!! > > You cannot ping gently while screaming!!! (you see how gentle this feels?) > > >> Could you please review the patch? > >> > > > > I thought this was already reviewed/acked and merged since ages.... > > ...also the wording-change in the commit message LGTM. > > > > Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> > > It was. Twice! This is just unjustified ping. I'd like to add a new tag at this point. Reviewed-!!-by-!!: Cristian Marussi <cristian.marussi@arm.com> :P Cheers, Cristian
On Thu, Aug 21, 2025 at 09:52:21AM +0100, Cristian Marussi wrote: > On Thu, Aug 21, 2025 at 10:29:00AM +0200, Krzysztof Kozlowski wrote: > > On 21/08/2025 10:04, Cristian Marussi wrote: > > > On Wed, Aug 20, 2025 at 11:44:26AM -0700, Deepti Jaggi wrote: > > >> > > >> On 7/30/25 14:30, Deepti Jaggi wrote: > > >>> From: Nikunj Kela <quic_nkela@quicinc.com> > > >>> > > > > > > Hi, > > > > > >>> Allow multiple SCMI instances by extending the scmi node name to include > > >>> an instance number suffix. > > >>> > > >>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > >>> Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> > > >>> Signed-off-by: Nikunj Kela <quic_nkela@quicinc.com> > > >>> Signed-off-by: Deepti Jaggi <quic_djaggi@quicinc.com> > > >>> --- > > >>> Changes in v6: > > >>> - Dropped 'this change' from description > > >>> - Link to v5: https://lore.kernel.org/all/20250423005824.3993256-1-quic_djaggi@quicinc.com > > >>> > > >>> Changes in v5: > > >>> - Added Reviewed-by tag > > >>> - Link to v4: https://lore.kernel.org/all/20240910163456.2383372-1-quic_nkela@quicinc.com > > >>> > > >>> Changes in v4: > > >>> - Dropped 'virtual' from subject and description > > >>> - Link to v3: https://lore.kernel.org/all/20240905201217.3815113-1-quic_nkela@quicinc.com > > >>> > > >>> 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 abbd62f1fed0993ab98fa44bdb9a0575f8e1c78e..be817fd9cc34b14009a3b1d69e78b802215571b6 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: > > >>> > > >>> --- > > >>> base-commit: 0b90c3b6d76ea512dc3dac8fb30215e175b0019a > > >>> change-id: 20250728-8255-scmi-ed963ef8f155 > > >> > > >> Gentle ping!! > > > > You cannot ping gently while screaming!!! (you see how gentle this feels?) > > > > >> Could you please review the patch? > > >> > > > > > > I thought this was already reviewed/acked and merged since ages.... > > > ...also the wording-change in the commit message LGTM. > > > > > > Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> > > > > It was. Twice! This is just unjustified ping. > > I'd like to add a new tag at this point. > > Reviewed-!!-by-!!: Cristian Marussi <cristian.marussi@arm.com> > I wish I could apply something fancy like this 😁. I will apply it soon. -- Regards, Sudeep
On 8/21/25 03:09, Sudeep Holla wrote: > On Thu, Aug 21, 2025 at 09:52:21AM +0100, Cristian Marussi wrote: >> On Thu, Aug 21, 2025 at 10:29:00AM +0200, Krzysztof Kozlowski wrote: >>> On 21/08/2025 10:04, Cristian Marussi wrote: >>>> On Wed, Aug 20, 2025 at 11:44:26AM -0700, Deepti Jaggi wrote: >>>>> On 7/30/25 14:30, Deepti Jaggi wrote: >>>>>> From: Nikunj Kela <quic_nkela@quicinc.com> >>>>>> >>>> Hi, >>>> >>>>>> Allow multiple SCMI instances by extending the scmi node name to include >>>>>> an instance number suffix. >>>>>> >>>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>>>>> Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> >>>>>> Signed-off-by: Nikunj Kela <quic_nkela@quicinc.com> >>>>>> Signed-off-by: Deepti Jaggi <quic_djaggi@quicinc.com> >>>>>> --- >>>>>> Changes in v6: >>>>>> - Dropped 'this change' from description >>>>>> - Link to v5: https://lore.kernel.org/all/20250423005824.3993256-1-quic_djaggi@quicinc.com >>>>>> >>>>>> Changes in v5: >>>>>> - Added Reviewed-by tag >>>>>> - Link to v4: https://lore.kernel.org/all/20240910163456.2383372-1-quic_nkela@quicinc.com >>>>>> >>>>>> Changes in v4: >>>>>> - Dropped 'virtual' from subject and description >>>>>> - Link to v3: https://lore.kernel.org/all/20240905201217.3815113-1-quic_nkela@quicinc.com >>>>>> >>>>>> 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 abbd62f1fed0993ab98fa44bdb9a0575f8e1c78e..be817fd9cc34b14009a3b1d69e78b802215571b6 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: >>>>>> >>>>>> --- >>>>>> base-commit: 0b90c3b6d76ea512dc3dac8fb30215e175b0019a >>>>>> change-id: 20250728-8255-scmi-ed963ef8f155 >>>>> Gentle ping!! >>> You cannot ping gently while screaming!!! (you see how gentle this feels?) >>> >>>>> Could you please review the patch? >>>>> >>>> I thought this was already reviewed/acked and merged since ages.... >>>> ...also the wording-change in the commit message LGTM. >>>> >>>> Reviewed-by: Cristian Marussi <cristian.marussi@arm.com> >>> It was. Twice! This is just unjustified ping. >> I'd like to add a new tag at this point. >> >> Reviewed-!!-by-!!: Cristian Marussi <cristian.marussi@arm.com> >> > I wish I could apply something fancy like this 😁. I will apply it soon. I guess my “Gentle Ping!!” wasn’t so gentle after all :) .Sorry about that. I sent the reminder because I had updated the commit message and the patch hadn’t been merged yet. I wanted to make sure the changes are good to go and that nothing else is needed from my end. Thank you for reviewing the patch—I appreciate everyone’s time and feedback. I’ll wait for the patch to be merged. >
© 2016 - 2025 Red Hat, Inc.