Skip to contentSkip to conversation

usb: chipidea: add CI_HDRC_TURN_VBUS_EARLY_ON for imx23

PATCH New Stefan Wahren 3 participants 5 messages Dec 29, 2015 – Jan 18, 2016

Subsystems (4)
Device tree bindings USB Arm i.MX and Layerscape Chipidea USB high speed dual role controller
lore
Share & tools
lore

Stefan Wahren

author
#0 lore raw

PATCH 1/2 usb: chipidea: add CI_HDRC_TURN_VBUS_EARLY_ON for imx23

Until now the imx23 uses the imx27 platform flag. But the imx23 needs the flag CI_HDRC_TURN_VBUS_EARLY_ON, too. So fix this by adding a separate platform flag.

Suggested-by Peter Chen <[email protected]>
Signed-off-by Stefan Wahren <[email protected]>
---
 drivers/usb/chipidea/ci_hdrc_imx.c |    6 ++++++
 1 file changed, 6 insertions(+)
drivers/usb/chipidea/ci_hdrc_imx.c +6 -0
index f14f4ab..b4605dd 100644
@@ -28,6 +28,11 @@ struct ci_hdrc_imx_platform_flag {
 	bool runtime_pm;
 };
 
+static const struct ci_hdrc_imx_platform_flag imx23_usb_data = {
+	.flags = CI_HDRC_TURN_VBUS_EARLY_ON |
+		CI_HDRC_DISABLE_STREAMING,
+};
+
 static const struct ci_hdrc_imx_platform_flag imx27_usb_data = {
 		CI_HDRC_DISABLE_STREAMING,
 };

@@ -66,6 +71,7 @@ static const struct ci_hdrc_imx_platform_flag imx7d_usb_data = {
 };
 
 static const struct of_device_id ci_hdrc_imx_dt_ids[] = {
+	{ .compatible = "fsl,imx23-usb", .data = &imx23_usb_data},
 	{ .compatible = "fsl,imx28-usb", .data = &imx28_usb_data},
 	{ .compatible = "fsl,imx27-usb", .data = &imx27_usb_data},
 	{ .compatible = "fsl,imx6q-usb", .data = &imx6q_usb_data},
Signature · 1 line
1.7.9.5
author
#1 lore raw

PATCH 1/2 usb: chipidea: add CI_HDRC_TURN_VBUS_EARLY_ON for imx23

Hi Peter,

Quoted 8 lines

Stefan Wahren <[email protected]> hat am 29. Dezember 2015 um 17:41 geschrieben:

Until now the imx23 uses the imx27 platform flag. But the imx23 needs the flag CI_HDRC_TURN_VBUS_EARLY_ON, too. So fix this by adding a separate platform flag.

Quoted 6 lines
Suggested-by Peter Chen <[email protected]>
Signed-off-by Stefan Wahren <[email protected]>

---
drivers/usb/chipidea/ci_hdrc_imx.c | 6 ++++++

1 file changed, 6 insertions(+)

is the series okay?

Regards
Stefan

#2 lore raw

PATCH 1/2 usb: chipidea: add CI_HDRC_TURN_VBUS_EARLY_ON for imx23

Quoted 10 lines · Stefan Wahren

Hi Peter,

Quoted 8 lines

Stefan Wahren <[email protected]> hat am 29. Dezember 2015 um 17:41 geschrieben:

Until now the imx23 uses the imx27 platform flag. But the imx23 needs the flag CI_HDRC_TURN_VBUS_EARLY_ON, too. So fix this by adding a separate platform flag.

Quoted 8 lines
Quoted 6 lines
Suggested-by Peter Chen <[email protected]>
Signed-off-by Stefan Wahren <[email protected]>

---
drivers/usb/chipidea/ci_hdrc_imx.c | 6 ++++++

1 file changed, 6 insertions(+)

is the series okay?

Yes, it is ok, already at my patch queue

Signature · 2 lines
Best Regards,
Peter Chen
author
#3 lore raw

PATCH 2/2 dt-bindings: ci-hdrc-usb2: add missing compatibles

In reply to Stefan Wahren

This patch adds the missing compatible strings from ci_hdrc_imx.

Signed-off-by Stefan Wahren <[email protected]>
---
 .../devicetree/bindings/usb/ci-hdrc-usb2.txt       |    7 +++++++
 1 file changed, 7 insertions(+)
Documentation/devicetree/bindings/usb/ci-hdrc-usb2.txt +7 -0
index 781296b..1e83509 100644
@@ -2,7 +2,14 @@
 
 Required properties:
 - compatible: should be one of:
+	"fsl,imx23-usb"
 	"fsl,imx27-usb"
+	"fsl,imx28-usb"
+	"fsl,imx6q-usb"
+	"fsl,imx6sl-usb"
+	"fsl,imx6sx-usb"
+	"fsl,imx6ul-usb"
+	"fsl,imx7d-usb"
 	"lsi,zevio-usb"
 	"qcom,ci-hdrc"
 	"chipidea,usb2"
Signature · 1 line
1.7.9.5