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

schowdhu
2021-10-05T12:57:35Z

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@codeaurora.org>
>> ---
>> 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
>>