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

[Deepti Jaggi <quic_djaggi@quicinc.com>]

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.

>