On Thu, 30 Mar 2023 at 11:42, Varadarajan Narayanan <quic_varada@xxxxxxxxxxx> wrote: > > Document the IPQ9574 dwc3 compatible. > > Signed-off-by: Varadarajan Narayanan <quic_varada@xxxxxxxxxxx> > --- > Changes in v5: > - Restore removed constraints > > Changes in v4: > - Update other relevant sections > - Remove constraints not applicable to IPQ9574 > --- > Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml b/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml > index a2aabda..8def6a3 100644 > --- a/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml > +++ b/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml > @@ -17,6 +17,7 @@ properties: > - qcom,ipq6018-dwc3 > - qcom,ipq8064-dwc3 > - qcom,ipq8074-dwc3 > + - qcom,ipq9574-dwc3 > - qcom,msm8953-dwc3 > - qcom,msm8994-dwc3 > - qcom,msm8996-dwc3 > @@ -242,6 +243,24 @@ allOf: > compatible: > contains: > enum: > + - qcom,ipq9574-dwc3 > + then: > + properties: > + clocks: > + maxItems: 5 > + clock-names: > + items: > + - const: sys_noc_axi > + - const: anoc_axi > + - const: master > + - const: sleep > + - const: mock_utmi Is there any chance of using clock names that are close to the names used by existing platforms? I think this fits into the third 'if' bucket: cfg_noc = sys_noc_axi core = master iface = anoc_axi sleep = sleep mock_utmi = mock_utmi > + > + - if: > + properties: > + compatible: > + contains: > + enum: > - qcom,msm8994-dwc3 > - qcom,qcs404-dwc3 > then: > -- > 2.7.4 > -- With best wishes Dmitry
