Re: [PATCH 01/11] dt-bindings: remoteproc: qcom: Add support for multipd model

[Manikanta Mylavarapu]

On 5/9/2023 6:31 PM, Krzysztof Kozlowski wrote:
> On 09/05/2023 12:34, Manikanta Mylavarapu wrote:
> > > > Sure, will add.
> > > > > > +    description:
> > > > > > +      Qualcomm G-Link subnode which represents communication edge, channels
> > > > > > +      and devices related to the Modem.
> > > > > > +
> > > > > > +patternProperties:
> > > > > > +  "^remoteproc_pd1|remoteproc_pd2|remoteproc_pd3":
> > > > >
> > > > > No, underscores are not allowed. Also, what is pd?
> > > > Sure, will remove underscores.
> > >
> > > Shouldn't this be just pd-1?
> > I think 'pd-1' not enough. Because child's i.e userpd's also considered
> > as remote processor's, so name should be like "remoteproc-pd1".
>
> "pd-1" is not enough for what? Why the node name has to be more specific?
>
>
> Best regards,
> Krzysztof

Sure. "pd-1" also seems fine. I will change child node name to "pd-1".

Thanks & Regards,
Manikanta.