| Message ID | cover.1679479634.git.quic_varada@quicinc.com |
|---|---|
| Headers | show |
| Series | Enable IPQ9754 USB | expand |
On 22/03/2023 11:44, Varadarajan Narayanan wrote: > Add dt-bindings for USB3 PHY found on Qualcomm IPQ9574 > > Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com> > > --- > Changes in v2: > - Updated sections missed in previous patch > --- > .../bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml > index e81a382..beae44c 100644 > --- a/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml > +++ b/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml > @@ -21,6 +21,7 @@ properties: > enum: > - qcom,ipq6018-qmp-usb3-phy > - qcom,ipq8074-qmp-usb3-phy > + - qcom,ipq9574-qmp-usb3-phy > - qcom,msm8996-qmp-usb3-phy > - qcom,msm8998-qmp-usb3-phy > - qcom,qcm2290-qmp-usb3-phy > @@ -204,6 +205,27 @@ allOf: > compatible: > contains: > enum: > + - qcom,ipq9574-qmp-usb3-phy > + then: > + properties: > + clocks: > + maxItems: 2 toplevel defines minItems as 3, so are you sure this works? Did you test it? Best regards, Krzysztof
On Wed, Mar 22, 2023 at 10:52:44PM +0100, Krzysztof Kozlowski wrote: > On 22/03/2023 11:44, Varadarajan Narayanan wrote: > > Add dt-bindings for USB3 PHY found on Qualcomm IPQ9574 > > > > Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com> > > > > --- > > Changes in v2: > > - Updated sections missed in previous patch > > --- > > .../bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml | 22 ++++++++++++++++++++++ > > 1 file changed, 22 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml > > index e81a382..beae44c 100644 > > --- a/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml > > +++ b/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml > > @@ -21,6 +21,7 @@ properties: > > enum: > > - qcom,ipq6018-qmp-usb3-phy > > - qcom,ipq8074-qmp-usb3-phy > > + - qcom,ipq9574-qmp-usb3-phy > > - qcom,msm8996-qmp-usb3-phy > > - qcom,msm8998-qmp-usb3-phy > > - qcom,qcm2290-qmp-usb3-phy > > @@ -204,6 +205,27 @@ allOf: > > compatible: > > contains: > > enum: > > + - qcom,ipq9574-qmp-usb3-phy > > + then: > > + properties: > > + clocks: > > + maxItems: 2 > > toplevel defines minItems as 3, so are you sure this works? Did you test it? Yes, this is tested. Able to do I/O. We only have 2 items. Is it ok change the minItems to 2? Thanks Varada > > Best regards, > Krzysztof >
On 23/03/2023 07:37, Varadarajan Narayanan wrote: > On Wed, Mar 22, 2023 at 10:52:44PM +0100, Krzysztof Kozlowski wrote: >> On 22/03/2023 11:44, Varadarajan Narayanan wrote: >>> Add dt-bindings for USB3 PHY found on Qualcomm IPQ9574 >>> >>> Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com> >>> >>> --- >>> Changes in v2: >>> - Updated sections missed in previous patch >>> --- >>> .../bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml | 22 ++++++++++++++++++++++ >>> 1 file changed, 22 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml >>> index e81a382..beae44c 100644 >>> --- a/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml >>> +++ b/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml >>> @@ -21,6 +21,7 @@ properties: >>> enum: >>> - qcom,ipq6018-qmp-usb3-phy >>> - qcom,ipq8074-qmp-usb3-phy >>> + - qcom,ipq9574-qmp-usb3-phy >>> - qcom,msm8996-qmp-usb3-phy >>> - qcom,msm8998-qmp-usb3-phy >>> - qcom,qcm2290-qmp-usb3-phy >>> @@ -204,6 +205,27 @@ allOf: >>> compatible: >>> contains: >>> enum: >>> + - qcom,ipq9574-qmp-usb3-phy >>> + then: >>> + properties: >>> + clocks: >>> + maxItems: 2 >> >> toplevel defines minItems as 3, so are you sure this works? Did you test it? > > Yes, this is tested. Able to do I/O. Bindings do not impact on whether you can or can not do IO, so I meant tested as DTS is compliant with bindings. I assume it was not, so please test bindings and DTS before sending new version. see Documentation/devicetree/bindings/writing-schema.rst for instructions > We only have 2 items. Is it ok change the minItems to 2? Yes, because you must update top level (and maybe other) constraints to correct ones. Just test the code... Best regards, Krzysztof
On Thu, Mar 23, 2023 at 07:42:37AM +0100, Krzysztof Kozlowski wrote: > On 23/03/2023 07:37, Varadarajan Narayanan wrote: > > On Wed, Mar 22, 2023 at 10:52:44PM +0100, Krzysztof Kozlowski wrote: > >> On 22/03/2023 11:44, Varadarajan Narayanan wrote: > >>> Add dt-bindings for USB3 PHY found on Qualcomm IPQ9574 > >>> > >>> Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com> > >>> > >>> --- > >>> Changes in v2: > >>> - Updated sections missed in previous patch > >>> --- > >>> .../bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml | 22 ++++++++++++++++++++++ > >>> 1 file changed, 22 insertions(+) > >>> > >>> diff --git a/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml > >>> index e81a382..beae44c 100644 > >>> --- a/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml > >>> +++ b/Documentation/devicetree/bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml > >>> @@ -21,6 +21,7 @@ properties: > >>> enum: > >>> - qcom,ipq6018-qmp-usb3-phy > >>> - qcom,ipq8074-qmp-usb3-phy > >>> + - qcom,ipq9574-qmp-usb3-phy > >>> - qcom,msm8996-qmp-usb3-phy > >>> - qcom,msm8998-qmp-usb3-phy > >>> - qcom,qcm2290-qmp-usb3-phy > >>> @@ -204,6 +205,27 @@ allOf: > >>> compatible: > >>> contains: > >>> enum: > >>> + - qcom,ipq9574-qmp-usb3-phy > >>> + then: > >>> + properties: > >>> + clocks: > >>> + maxItems: 2 > >> > >> toplevel defines minItems as 3, so are you sure this works? Did you test it? > > > > Yes, this is tested. Able to do I/O. > > Bindings do not impact on whether you can or can not do IO, so I meant > tested as DTS is compliant with bindings. I assume it was not, so please > test bindings and DTS before sending new version. > see Documentation/devicetree/bindings/writing-schema.rst for instructions Sorry. Misunderstood the question. Yes, I ran make dtbs_check and got these messages. But couldn't relate them with the minItems/maxItems mismatch... phy@7d000: clocks: [[8, 93], [8, 102]] is too short phy@7d000: clock-names: ['aux', 'cfg_ahb'] is too short Will fix it now. Thanks for the clarification. > > We only have 2 items. Is it ok change the minItems to 2? > > Yes, because you must update top level (and maybe other) constraints to > correct ones. Just test the code... Ok. Thanks Varada > > > Best regards, > Krzysztof >
This patch series adds the relevant phy and controller configurations for enabling USB on IPQ9754 Depends on: https://lore.kernel.org/all/20230217142030.16012-1-quic_devipriy@quicinc.com/ [v3]: - Incorporated review comments regarding coding style [v2]: - Incorporated review comments regarding coding style, maintaining sorted order of entries and unused phy register offsets - Removed NOC clock entries from DT node (will be implemented later with interconnect support) - Fixed 'make dtbs_check' errors/warnings [v1]: https://lore.kernel.org/linux-arm-msm/5dac3aa4-8dc7-f9eb-5cf3-b361efdc9494@linaro.org/T/ Varadarajan Narayanan (8): dt-bindings: phy: qcom,qusb2: Document IPQ9574 compatible dt-bindings: phy: qcom,qmp-usb: Add IPQ9574 USB3 PHY dt-bindings: usb: dwc3: Add IPQ9574 compatible clk: qcom: gcc-ipq9574: Add USB related clocks phy: qcom-qusb2: add QUSB2 support for IPQ9574 phy: qcom: qmp: Update IPQ9574 USB Phy initialization Sequence arm64: dts: qcom: ipq9574: Add USB related nodes arm64: dts: qcom: ipq9574: Enable USB .../bindings/phy/qcom,msm8996-qmp-usb3-phy.yaml | 22 ++++ .../devicetree/bindings/phy/qcom,qusb2-phy.yaml | 3 +- .../devicetree/bindings/usb/qcom,dwc3.yaml | 1 + arch/arm64/boot/dts/qcom/ipq9574-al02-c7.dts | 12 +++ arch/arm64/boot/dts/qcom/ipq9574.dtsi | 86 +++++++++++++++ drivers/clk/qcom/gcc-ipq9574.c | 37 +++++++ drivers/phy/qualcomm/phy-qcom-qmp-usb.c | 119 +++++++++++++++++++++ drivers/phy/qualcomm/phy-qcom-qusb2.c | 3 + include/dt-bindings/clock/qcom,ipq9574-gcc.h | 2 + 9 files changed, 284 insertions(+), 1 deletion(-)