Linux USB
 help / color / mirror / Atom feed
* [PATCH 0/5] Enable ipq5210 USB support
@ 2026-05-15 10:47 Varadarajan Narayanan
  2026-05-15 10:47 ` [PATCH 1/5] dt-bindings: phy: qcom,qusb2: Document ipq5210 compatible Varadarajan Narayanan
                   ` (4 more replies)
  0 siblings, 5 replies; 8+ messages in thread
From: Varadarajan Narayanan @ 2026-05-15 10:47 UTC (permalink / raw)
  To: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Wesley Cheng, Greg Kroah-Hartman
  Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel, linux-usb,
	Varadarajan Narayanan

The ipq5210 SoC has both USB2.0 and USB3.0 controllers. The USB3.0
can connect to either of USB2.0 or USB3.0 phy and operate in the
respective mode.

Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>
---
Varadarajan Narayanan (5):
      dt-bindings: phy: qcom,qusb2: Document ipq5210 compatible
      dt-bindings: phy: qcom,qmp-usb: Add ipq5210 USB3 PHY
      dt-bindings: usb: qcom,dwc3: Add ipq5210 to USB DWC3 bindings
      phy: qcom: qmp: Enable ipq5210 support
      phy: qcom-qusb2: add QUSB2 support for ipq5210

 Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml              | 1 +
 .../devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml        | 1 +
 Documentation/devicetree/bindings/usb/qcom,dwc3.yaml                   | 1 +
 drivers/phy/qualcomm/phy-qcom-qmp-usb.c                                | 3 +++
 drivers/phy/qualcomm/phy-qcom-qusb2.c                                  | 3 +++
 5 files changed, 9 insertions(+)
---
base-commit: e98d21c170b01ddef366f023bbfcf6b31509fa83
change-id: 20260515-usb2phy-bd850e1fb3de

Best regards,
-- 
Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 1/5] dt-bindings: phy: qcom,qusb2: Document ipq5210 compatible
  2026-05-15 10:47 [PATCH 0/5] Enable ipq5210 USB support Varadarajan Narayanan
@ 2026-05-15 10:47 ` Varadarajan Narayanan
  2026-05-15 10:47 ` [PATCH 2/5] dt-bindings: phy: qcom,qmp-usb: Add ipq5210 USB3 PHY Varadarajan Narayanan
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 8+ messages in thread
From: Varadarajan Narayanan @ 2026-05-15 10:47 UTC (permalink / raw)
  To: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Wesley Cheng, Greg Kroah-Hartman
  Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel, linux-usb,
	Varadarajan Narayanan

Document the compatible string used for the qusb2 phy in ipq5210.

Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>
---
 Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml
index 39851ba9de43..509cadd31adf 100644
--- a/Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml
@@ -18,6 +18,7 @@ properties:
     oneOf:
       - items:
           - enum:
+              - qcom,ipq5210-qusb2-phy
               - qcom,ipq5424-qusb2-phy
               - qcom,ipq6018-qusb2-phy
               - qcom,ipq8074-qusb2-phy

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 2/5] dt-bindings: phy: qcom,qmp-usb: Add ipq5210 USB3 PHY
  2026-05-15 10:47 [PATCH 0/5] Enable ipq5210 USB support Varadarajan Narayanan
  2026-05-15 10:47 ` [PATCH 1/5] dt-bindings: phy: qcom,qusb2: Document ipq5210 compatible Varadarajan Narayanan
@ 2026-05-15 10:47 ` Varadarajan Narayanan
  2026-05-15 10:47 ` [PATCH 3/5] dt-bindings: usb: qcom,dwc3: Add ipq5210 to USB DWC3 bindings Varadarajan Narayanan
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 8+ messages in thread
From: Varadarajan Narayanan @ 2026-05-15 10:47 UTC (permalink / raw)
  To: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Wesley Cheng, Greg Kroah-Hartman
  Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel, linux-usb,
	Varadarajan Narayanan

Add dt-bindings for USB3 PHY found on Qualcomm ipq5210

Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>
---
 .../devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml          | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml
index 623c2f8c7d22..bdb80371ea68 100644
--- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-usb3-uni-phy.yaml
@@ -17,6 +17,7 @@ properties:
   compatible:
     enum:
       - qcom,glymur-qmp-usb3-uni-phy
+      - qcom,ipq5210-qmp-usb3-phy
       - qcom,ipq5424-qmp-usb3-phy
       - qcom,ipq6018-qmp-usb3-phy
       - qcom,ipq8074-qmp-usb3-phy

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 3/5] dt-bindings: usb: qcom,dwc3: Add ipq5210 to USB DWC3 bindings
  2026-05-15 10:47 [PATCH 0/5] Enable ipq5210 USB support Varadarajan Narayanan
  2026-05-15 10:47 ` [PATCH 1/5] dt-bindings: phy: qcom,qusb2: Document ipq5210 compatible Varadarajan Narayanan
  2026-05-15 10:47 ` [PATCH 2/5] dt-bindings: phy: qcom,qmp-usb: Add ipq5210 USB3 PHY Varadarajan Narayanan
@ 2026-05-15 10:47 ` Varadarajan Narayanan
  2026-05-15 11:05   ` Krzysztof Kozlowski
  2026-05-15 10:47 ` [PATCH 4/5] phy: qcom: qmp: Enable ipq5210 support Varadarajan Narayanan
  2026-05-15 10:47 ` [PATCH 5/5] phy: qcom-qusb2: add QUSB2 support for ipq5210 Varadarajan Narayanan
  4 siblings, 1 reply; 8+ messages in thread
From: Varadarajan Narayanan @ 2026-05-15 10:47 UTC (permalink / raw)
  To: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Wesley Cheng, Greg Kroah-Hartman
  Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel, linux-usb,
	Varadarajan Narayanan

Update dt-bindings to add ipq5210 to USB DWC3 controller list.

Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>
---
 Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml b/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml
index a7f58114c02e..f8e0819be894 100644
--- a/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml
+++ b/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml
@@ -26,6 +26,7 @@ properties:
       - enum:
           - qcom,ipq4019-dwc3
           - qcom,ipq5018-dwc3
+          - qcom,ipq5210-dwc3
           - qcom,ipq5332-dwc3
           - qcom,ipq5424-dwc3
           - qcom,ipq6018-dwc3

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 4/5] phy: qcom: qmp: Enable ipq5210 support
  2026-05-15 10:47 [PATCH 0/5] Enable ipq5210 USB support Varadarajan Narayanan
                   ` (2 preceding siblings ...)
  2026-05-15 10:47 ` [PATCH 3/5] dt-bindings: usb: qcom,dwc3: Add ipq5210 to USB DWC3 bindings Varadarajan Narayanan
@ 2026-05-15 10:47 ` Varadarajan Narayanan
  2026-05-15 11:29   ` Konrad Dybcio
  2026-05-15 10:47 ` [PATCH 5/5] phy: qcom-qusb2: add QUSB2 support for ipq5210 Varadarajan Narayanan
  4 siblings, 1 reply; 8+ messages in thread
From: Varadarajan Narayanan @ 2026-05-15 10:47 UTC (permalink / raw)
  To: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Wesley Cheng, Greg Kroah-Hartman
  Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel, linux-usb,
	Varadarajan Narayanan

Enable QMP USB3 phy support for ipq5210 SoC.

Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>
---
 drivers/phy/qualcomm/phy-qcom-qmp-usb.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-usb.c b/drivers/phy/qualcomm/phy-qcom-qmp-usb.c
index b0ecd5ba2464..8d3d7e39697e 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp-usb.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp-usb.c
@@ -2449,6 +2449,9 @@ static const struct of_device_id qmp_usb_of_match_table[] = {
 	{
 		.compatible = "qcom,glymur-qmp-usb3-uni-phy",
 		.data = &glymur_usb3_uniphy_cfg,
+	}, {
+		.compatible = "qcom,ipq5210-qmp-usb3-phy",
+		.data = &ipq9574_usb3phy_cfg,
 	}, {
 		.compatible = "qcom,ipq5424-qmp-usb3-phy",
 		.data = &ipq9574_usb3phy_cfg,

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 5/5] phy: qcom-qusb2: add QUSB2 support for ipq5210
  2026-05-15 10:47 [PATCH 0/5] Enable ipq5210 USB support Varadarajan Narayanan
                   ` (3 preceding siblings ...)
  2026-05-15 10:47 ` [PATCH 4/5] phy: qcom: qmp: Enable ipq5210 support Varadarajan Narayanan
@ 2026-05-15 10:47 ` Varadarajan Narayanan
  4 siblings, 0 replies; 8+ messages in thread
From: Varadarajan Narayanan @ 2026-05-15 10:47 UTC (permalink / raw)
  To: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Wesley Cheng, Greg Kroah-Hartman
  Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel, linux-usb,
	Varadarajan Narayanan

Add the phy init sequence for the Super Speed ports found
on ipq5210.

Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>
---
 drivers/phy/qualcomm/phy-qcom-qusb2.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/phy/qualcomm/phy-qcom-qusb2.c b/drivers/phy/qualcomm/phy-qcom-qusb2.c
index eb93015be841..749106175742 100644
--- a/drivers/phy/qualcomm/phy-qcom-qusb2.c
+++ b/drivers/phy/qualcomm/phy-qcom-qusb2.c
@@ -929,6 +929,9 @@ static const struct phy_ops qusb2_phy_gen_ops = {
 
 static const struct of_device_id qusb2_phy_of_match_table[] = {
 	{
+		.compatible	= "qcom,ipq5210-qusb2-phy",
+		.data		= &ipq6018_phy_cfg,
+	}, {
 		.compatible	= "qcom,ipq5424-qusb2-phy",
 		.data		= &ipq6018_phy_cfg,
 	}, {

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 3/5] dt-bindings: usb: qcom,dwc3: Add ipq5210 to USB DWC3 bindings
  2026-05-15 10:47 ` [PATCH 3/5] dt-bindings: usb: qcom,dwc3: Add ipq5210 to USB DWC3 bindings Varadarajan Narayanan
@ 2026-05-15 11:05   ` Krzysztof Kozlowski
  0 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2026-05-15 11:05 UTC (permalink / raw)
  To: Varadarajan Narayanan, Vinod Koul, Neil Armstrong, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Wesley Cheng,
	Greg Kroah-Hartman
  Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel, linux-usb

On 15/05/2026 12:47, Varadarajan Narayanan wrote:
> Update dt-bindings to add ipq5210 to USB DWC3 controller list.
> 
> Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>
> ---
>  Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 1 +
>  1 file changed, 1 insertion(+)

Same feedback as usually. What USB patch is doing here?

Also, other typical comment:
A nit, subject: drop second/last, redundant "bindings". The
"dt-bindings" prefix is already stating that these are bindings.
See also:
https://elixir.bootlin.com/linux/v6.17-rc3/source/Documentation/devicetree/bindings/submitting-patches.rst#L18

Considering that did not pass through internal review toolset and it has
trivial issues, which this toolset and internal review is suppose to
catch, I drop all patches from you from the DT Patchwork.

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 4/5] phy: qcom: qmp: Enable ipq5210 support
  2026-05-15 10:47 ` [PATCH 4/5] phy: qcom: qmp: Enable ipq5210 support Varadarajan Narayanan
@ 2026-05-15 11:29   ` Konrad Dybcio
  0 siblings, 0 replies; 8+ messages in thread
From: Konrad Dybcio @ 2026-05-15 11:29 UTC (permalink / raw)
  To: Varadarajan Narayanan, Vinod Koul, Neil Armstrong, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Wesley Cheng,
	Greg Kroah-Hartman
  Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel, linux-usb

On 5/15/26 12:47 PM, Varadarajan Narayanan wrote:
> Enable QMP USB3 phy support for ipq5210 SoC.
> 
> Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>
> ---

Since they seem fully compatible (also in patch 5), could we simply
use a fallback compatible?

Konrad

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2026-05-15 11:29 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-15 10:47 [PATCH 0/5] Enable ipq5210 USB support Varadarajan Narayanan
2026-05-15 10:47 ` [PATCH 1/5] dt-bindings: phy: qcom,qusb2: Document ipq5210 compatible Varadarajan Narayanan
2026-05-15 10:47 ` [PATCH 2/5] dt-bindings: phy: qcom,qmp-usb: Add ipq5210 USB3 PHY Varadarajan Narayanan
2026-05-15 10:47 ` [PATCH 3/5] dt-bindings: usb: qcom,dwc3: Add ipq5210 to USB DWC3 bindings Varadarajan Narayanan
2026-05-15 11:05   ` Krzysztof Kozlowski
2026-05-15 10:47 ` [PATCH 4/5] phy: qcom: qmp: Enable ipq5210 support Varadarajan Narayanan
2026-05-15 11:29   ` Konrad Dybcio
2026-05-15 10:47 ` [PATCH 5/5] phy: qcom-qusb2: add QUSB2 support for ipq5210 Varadarajan Narayanan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox