* [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances
@ 2025-07-30 21:30 Deepti Jaggi
2025-08-20 18:44 ` Deepti Jaggi
2025-08-26 10:39 ` Sudeep Holla
0 siblings, 2 replies; 8+ messages in thread
From: Deepti Jaggi @ 2025-07-30 21:30 UTC (permalink / raw)
To: Sudeep Holla, Cristian Marussi, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: arm-scmi, linux-arm-kernel, devicetree, linux-kernel, kernel,
Nikunj Kela, Krzysztof Kozlowski, Deepti Jaggi
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>
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances
2025-07-30 21:30 [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances Deepti Jaggi
@ 2025-08-20 18:44 ` Deepti Jaggi
2025-08-21 8:04 ` Cristian Marussi
2025-08-26 10:39 ` Sudeep Holla
1 sibling, 1 reply; 8+ messages in thread
From: Deepti Jaggi @ 2025-08-20 18:44 UTC (permalink / raw)
To: Sudeep Holla, Cristian Marussi, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: arm-scmi, linux-arm-kernel, devicetree, linux-kernel, kernel,
Nikunj Kela, Krzysztof Kozlowski
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,
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances
2025-08-20 18:44 ` Deepti Jaggi
@ 2025-08-21 8:04 ` Cristian Marussi
2025-08-21 8:29 ` Krzysztof Kozlowski
0 siblings, 1 reply; 8+ messages in thread
From: Cristian Marussi @ 2025-08-21 8:04 UTC (permalink / raw)
To: Deepti Jaggi
Cc: Sudeep Holla, Cristian Marussi, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, arm-scmi, linux-arm-kernel, devicetree,
linux-kernel, kernel, Nikunj Kela, Krzysztof Kozlowski
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances
2025-08-21 8:04 ` Cristian Marussi
@ 2025-08-21 8:29 ` Krzysztof Kozlowski
2025-08-21 8:52 ` Cristian Marussi
0 siblings, 1 reply; 8+ messages in thread
From: Krzysztof Kozlowski @ 2025-08-21 8:29 UTC (permalink / raw)
To: Cristian Marussi, Deepti Jaggi
Cc: Sudeep Holla, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
arm-scmi, linux-arm-kernel, devicetree, linux-kernel, kernel,
Nikunj Kela
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances
2025-08-21 8:29 ` Krzysztof Kozlowski
@ 2025-08-21 8:52 ` Cristian Marussi
2025-08-21 10:09 ` Sudeep Holla
0 siblings, 1 reply; 8+ messages in thread
From: Cristian Marussi @ 2025-08-21 8:52 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Cristian Marussi, Deepti Jaggi, Sudeep Holla, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, arm-scmi, linux-arm-kernel,
devicetree, linux-kernel, kernel, Nikunj Kela
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances
2025-08-21 8:52 ` Cristian Marussi
@ 2025-08-21 10:09 ` Sudeep Holla
2025-08-22 21:21 ` Deepti Jaggi
0 siblings, 1 reply; 8+ messages in thread
From: Sudeep Holla @ 2025-08-21 10:09 UTC (permalink / raw)
To: Cristian Marussi
Cc: Krzysztof Kozlowski, Deepti Jaggi, Rob Herring, Sudeep Holla,
Krzysztof Kozlowski, Conor Dooley, arm-scmi, linux-arm-kernel,
devicetree, linux-kernel, kernel, Nikunj Kela
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances
2025-08-21 10:09 ` Sudeep Holla
@ 2025-08-22 21:21 ` Deepti Jaggi
0 siblings, 0 replies; 8+ messages in thread
From: Deepti Jaggi @ 2025-08-22 21:21 UTC (permalink / raw)
To: Sudeep Holla, Cristian Marussi
Cc: Krzysztof Kozlowski, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, arm-scmi, linux-arm-kernel, devicetree,
linux-kernel, kernel, Nikunj Kela
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.
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances
2025-07-30 21:30 [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances Deepti Jaggi
2025-08-20 18:44 ` Deepti Jaggi
@ 2025-08-26 10:39 ` Sudeep Holla
1 sibling, 0 replies; 8+ messages in thread
From: Sudeep Holla @ 2025-08-26 10:39 UTC (permalink / raw)
To: Cristian Marussi, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Deepti Jaggi
Cc: Sudeep Holla, arm-scmi, linux-arm-kernel, devicetree,
linux-kernel, kernel, Nikunj Kela, Krzysztof Kozlowski
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
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2025-08-26 10:39 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-30 21:30 [PATCH v6] dt-bindings: firmware: arm,scmi: allow multiple instances Deepti Jaggi
2025-08-20 18:44 ` Deepti Jaggi
2025-08-21 8:04 ` Cristian Marussi
2025-08-21 8:29 ` Krzysztof Kozlowski
2025-08-21 8:52 ` Cristian Marussi
2025-08-21 10:09 ` Sudeep Holla
2025-08-22 21:21 ` Deepti Jaggi
2025-08-26 10:39 ` Sudeep Holla
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).