linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [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 11: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).