[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <9113fd98-f58c-485b-8d7a-1ac0c541c04a@kernel.org>
Date: Fri, 16 Aug 2024 16:05:34 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Bartosz Golaszewski <brgl@...ev.pl>
Cc: Srinivas Kandagatla <srinivas.kandagatla@...aro.org>,
Amol Maheshwari <amahesh@....qualcomm.com>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>,
Tengfei Fan <quic_tengfan@...cinc.com>, Ling Xu <quic_lxu5@...cinc.com>,
linux-arm-msm@...r.kernel.org, dri-devel@...ts.freedesktop.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Bartosz Golaszewski <bartosz.golaszewski@...aro.org>
Subject: Re: [PATCH 1/2] dt-bindings: misc: qcom,fastrpc: document new domain
ID
On 16/08/2024 14:29, Bartosz Golaszewski wrote:
> On Fri, Aug 16, 2024 at 1:21 PM Krzysztof Kozlowski <krzk@...nel.org> wrote:
>>
>> On 16/08/2024 12:23, Bartosz Golaszewski wrote:
>>> From: Bartosz Golaszewski <bartosz.golaszewski@...aro.org>
>>>
>>> Add "cdsp1" as the new supported label for the CDSP1 fastrpc domain.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@...aro.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
Powered by blists - more mailing lists