On Fri, Aug 16, 2024 at 1:21 PM Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote: > > On 16/08/2024 12:23, Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> > > > > Add "cdsp1" as the new supported label for the CDSP1 fastrpc domain. > > > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> > > --- > > 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. Bart [1] https://lore.kernel.org/all/20240805-topic-sa8775p-iot-remoteproc-v4-4-86affdc72c04@xxxxxxxxxx/
