Re: [PATCH 1/2] dt-bindings: misc: qcom,fastrpc: document new domain ID

[Krzysztof Kozlowski <krzk@kernel.org>]

On 16/08/2024 14:29, Bartosz Golaszewski wrote:
> On Fri, Aug 16, 2024 at 1:21 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>
>> On 16/08/2024 12:23, Bartosz Golaszewski wrote:
>>> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
>>>
>>> Add "cdsp1" as the new supported label for the CDSP1 fastrpc domain.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
>>> ---
>>>  Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml b/Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml
>>> index c27a8f33d8d7..2a5b18982804 100644
>>> --- a/Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml
>>> +++ b/Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml
>>> @@ -26,6 +26,7 @@ properties:
>>>        - mdsp
>>>        - sdsp
>>>        - cdsp
>>> +      - cdsp1
>>
>> Are there more than one cdsp domains? Why adding suffixes? Driver source
>> code does not have "cdsp1" domain, so this is confusing.
>>
>> Best regards,
>> Krzysztof
>>
> 
> It does, Srini picked up this patch earlier today. I'm not an expert
> in fast RPC but it looks like the domain ID number matters here.

Ah, ok, I did not see driver change here so it was a bit confusing.

Best regards,
Krzysztof