Re: [PATCH V0 2/7] dt-bindings: usb: dwc3: Update dwc3 properties for EUD connector

[schowdhu]

On 2021-10-04 21:20, Bjorn Andersson wrote:
> On Mon 04 Oct 04:16 PDT 2021, Souradeep Chowdhury wrote:
>
> > Adding the address size,cell size and ranges property for EUD 
> > connector.
> > Adding the connector property for EUD which is child of dwc3 node.
>
> When we have a Type-C controller involved, the connector is described
> using of_graph, so will we not then have two different connectors
> described, in two different ways?
>
> Regards,
> Bjorn

Ack. Will have this updated with the of_graph in the next version.

> > Signed-off-by: Souradeep Chowdhury <schowdhu@xxxxxxxxxxxxxx>
> > ---
> >  Documentation/devicetree/bindings/usb/snps,dwc3.yaml | 15 
> > +++++++++++++++
> >  1 file changed, 15 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml 
> > b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml
> > index 078fb78..3e71205 100644
> > --- a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml
> > +++ b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml
> > @@ -36,6 +36,14 @@ properties:
> >          - const: synopsys,dwc3
> >            deprecated: true
> >
> > +  "#address-cells":
> > +    enum: [ 1, 2 ]
> > +
> > +  "#size-cells":
> > +    enum: [ 1, 2 ]
> > +
> > +  ranges: true
> > +
> >    interrupts:
> >      description:
> >        It's either a single common DWC3 interrupt (dwc_usb3) or 
> > individual
> > @@ -318,6 +326,13 @@ properties:
> >      items:
> >        enum: [1, 4, 8, 16, 32, 64, 128, 256]
> >
> > +  connector:
> > +    type: object
> > +    $ref: /connector/usb-connector.yaml#
> > +    description:
> > +      Connector for dual role switch, especially for 
> > "eud-usb-c-connector"
> > +
> > +
> >  unevaluatedProperties: false
> >
> >  required:
> > --
> > QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a 
> > member
> > of Code Aurora Forum, hosted by The Linux Foundation