[PATCH v6 5/5] arm64: dts: qcom: sc7180-trogdor: Add nodes for onboard USB hub

2021-04-05 Thread Matthias Kaehlcke
Add nodes for the onboard USB hub on trogdor devices. Remove the
'always-on' and 'boot-on' properties from the hub regulator, since
the regulator is now managed by the onboard_usb_hub driver.

Signed-off-by: Matthias Kaehlcke 
---

Changes in v7:
- rebased on qcom/arm64-for-5.13 (with the rest of the series)

Changes in v6:
- added 'companion-hub' entry to both USB devices
- added 'vdd-supply' also to hub@2

Changes in v5:
- patch added to the series

 .../boot/dts/qcom/sc7180-trogdor-lazor-r0.dts | 19 +++--
 .../boot/dts/qcom/sc7180-trogdor-lazor-r1.dts | 11 --
 .../arm64/boot/dts/qcom/sc7180-trogdor-r1.dts | 19 +++--
 arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi  | 21 ---
 4 files changed, 38 insertions(+), 32 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dts 
b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dts
index 5c997cd90069..bae85f6b2bfa 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dts
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dts
@@ -23,17 +23,6 @@ _thermal {
status = "disabled";
 };
 
-_hub {
-   /* pp3300_l7c is used to power the USB hub */
-   /delete-property/regulator-always-on;
-   /delete-property/regulator-boot-on;
-};
-
-_l7c {
-   regulator-always-on;
-   regulator-boot-on;
-};
-
 _out {
/*
 * Lane 0 was incorrectly mapped on the cable, but we've now decided
@@ -42,3 +31,11 @@ _out {
 */
lane-polarities = <1 0>;
 };
+
+_hub_2_0 {
+vdd-supply = <_l7c>;
+};
+
+_hub_3_0 {
+vdd-supply = <_l7c>;
+};
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dts 
b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dts
index d9fbcc7bc5bd..45f014c3539d 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dts
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dts
@@ -23,13 +23,10 @@ _thermal {
status = "disabled";
 };
 
-_hub {
-   /* pp3300_l7c is used to power the USB hub */
-   /delete-property/regulator-always-on;
-   /delete-property/regulator-boot-on;
+_hub_2_0 {
+vdd-supply = <_l7c>;
 };
 
-_l7c {
-   regulator-always-on;
-   regulator-boot-on;
+_hub_3_0 {
+vdd-supply = <_l7c>;
 };
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts 
b/arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts
index 2b522f9e0d8f..2f5263e3d1b9 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts
@@ -42,17 +42,6 @@  {
compatible = "auo,b116xa01";
 };
 
-_hub {
-   /* pp3300_l7c is used to power the USB hub */
-   /delete-property/regulator-always-on;
-   /delete-property/regulator-boot-on;
-};
-
-_l7c {
-   regulator-always-on;
-   regulator-boot-on;
-};
-
 _2 {
status = "okay";
 };
@@ -61,6 +50,14 @@  {
interrupts = <58 IRQ_TYPE_EDGE_FALLING>;
 };
 
+_hub_2_0 {
+vdd-supply = <_l7c>;
+};
+
+_hub_3_0 {
+vdd-supply = <_l7c>;
+};
+
 /* PINCTRL - modifications to sc7180-trogdor.dtsi */
 
 _int_1v8_odl {
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi 
b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
index 192e2e424fde..54f9da9af376 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
@@ -206,9 +206,6 @@ pp3300_hub: pp3300-hub {
pinctrl-names = "default";
pinctrl-0 = <_pp3300_hub>;
 
-   regulator-always-on;
-   regulator-boot-on;
-
vin-supply = <_a>;
};
 
@@ -848,6 +845,24 @@ _1 {
 
 _1_dwc3 {
dr_mode = "host";
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   /* 2.0 hub on port 1 */
+   usb_hub_2_0: hub@1 {
+   compatible = "usbbda,5411";
+   reg = <1>;
+   vdd-supply = <_hub>;
+   companion-hub = <_hub_3_0>;
+   };
+
+   /* 3.0 hub on port 2 */
+   usb_hub_3_0: hub@2 {
+   compatible = "usbbda,411";
+   reg = <2>;
+   vdd-supply = <_hub>;
+   companion-hub = <_hub_2_0>;
+   };
 };
 
 _1_hsphy {
-- 
2.31.0.208.g409f899ff0-goog



[PATCH v6 5/5] arm64: dts: qcom: sc7180-trogdor: Add nodes for onboard USB hub

2021-03-05 Thread Matthias Kaehlcke
Add nodes for the onboard USB hub on trogdor devices. Remove the
'always-on' and 'boot-on' properties from the hub regulator, since
the regulator is now managed by the onboard_usb_hub driver.

Signed-off-by: Matthias Kaehlcke 
---

Changes in v6:
- added 'companion-hub' entry to both USB devices
- added 'vdd-supply' also to hub@2

Changes in v5:
- patch added to the series

 .../boot/dts/qcom/sc7180-trogdor-lazor-r0.dts | 19 +++--
 .../boot/dts/qcom/sc7180-trogdor-lazor-r1.dts | 11 --
 .../arm64/boot/dts/qcom/sc7180-trogdor-r1.dts | 15 ++---
 arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi  | 21 ---
 4 files changed, 36 insertions(+), 30 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dts 
b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dts
index 30e3e769d2b4..5fb8e12af1a0 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dts
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dts
@@ -14,17 +14,6 @@ / {
compatible = "google,lazor-rev0", "qcom,sc7180";
 };
 
-_hub {
-   /* pp3300_l7c is used to power the USB hub */
-   /delete-property/regulator-always-on;
-   /delete-property/regulator-boot-on;
-};
-
-_l7c {
-   regulator-always-on;
-   regulator-boot-on;
-};
-
 _out {
/*
 * Lane 0 was incorrectly mapped on the cable, but we've now decided
@@ -33,3 +22,11 @@ _out {
 */
lane-polarities = <1 0>;
 };
+
+_hub_2_0 {
+vdd-supply = <_l7c>;
+};
+
+_hub_3_0 {
+vdd-supply = <_l7c>;
+};
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dts 
b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dts
index c2ef06367baf..a8c23dddf25b 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dts
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dts
@@ -14,13 +14,10 @@ / {
compatible = "google,lazor-rev1", "google,lazor-rev2", "qcom,sc7180";
 };
 
-_hub {
-   /* pp3300_l7c is used to power the USB hub */
-   /delete-property/regulator-always-on;
-   /delete-property/regulator-boot-on;
+_hub_2_0 {
+vdd-supply = <_l7c>;
 };
 
-_l7c {
-   regulator-always-on;
-   regulator-boot-on;
+_hub_3_0 {
+vdd-supply = <_l7c>;
 };
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts 
b/arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts
index 2cb522d6962e..689c569872f1 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts
@@ -53,19 +53,16 @@ ap_ts: touchscreen@10 {
};
 };
 
-_hub {
-   /* pp3300_l7c is used to power the USB hub */
-   /delete-property/regulator-always-on;
-   /delete-property/regulator-boot-on;
+_2 {
+   status = "okay";
 };
 
-_l7c {
-   regulator-always-on;
-   regulator-boot-on;
+_hub_2_0 {
+vdd-supply = <_l7c>;
 };
 
-_2 {
-   status = "okay";
+_hub_3_0 {
+vdd-supply = <_l7c>;
 };
 
 /* PINCTRL - board-specific pinctrl */
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi 
b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
index 07c8b2c926c0..ebff7aca4f51 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
@@ -203,9 +203,6 @@ pp3300_hub: pp3300-hub {
pinctrl-names = "default";
pinctrl-0 = <_pp3300_hub>;
 
-   regulator-always-on;
-   regulator-boot-on;
-
vin-supply = <_a>;
};
 
@@ -826,6 +823,24 @@ _1 {
 
 _1_dwc3 {
dr_mode = "host";
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   /* 2.0 hub on port 1 */
+   usb_hub_2_0: hub@1 {
+   compatible = "usbbda,5411";
+   reg = <1>;
+   vdd-supply = <_hub>;
+   companion-hub = <_hub_3_0>;
+   };
+
+   /* 3.0 hub on port 2 */
+   usb_hub_3_0: hub@2 {
+   compatible = "usbbda,411";
+   reg = <2>;
+   vdd-supply = <_hub>;
+   companion-hub = <_hub_2_0>;
+   };
 };
 
 _1_hsphy {
-- 
2.30.1.766.gb4fecdf3b7-goog