diff mbox series

[v2,2/3] arm64: dts: qcom: Cosmetic changes - arrange USB nodes together in sm8150 dts

Message ID 20210627114616.717101-3-bhupesh.sharma@linaro.org
State New
Headers show
Series arm64: dts: qcom: Fix usb entries for SA8155p-adp board | expand

Commit Message

Bhupesh Sharma June 27, 2021, 11:46 a.m. UTC
Arrange the two usb controller, hs-phy and
ss-phy nodes closer in the dts for better readability.

Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
---
 arch/arm64/boot/dts/qcom/sm8150.dtsi | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

Comments

Bjorn Andersson July 19, 2021, 4:59 p.m. UTC | #1
On Sun 27 Jun 06:46 CDT 2021, Bhupesh Sharma wrote:

> Arrange the two usb controller, hs-phy and

> ss-phy nodes closer in the dts for better readability.

> 


The nodes are supposed to be sorted by address, then alphabetically by
name and lastly when referred to using the &label by the label.

As such the two nodes should live after &usb_2_qmpphy, so I took the
liberty of adjusting your patch to resolve this.

Thanks,
Bjorn

> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>

> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>

> ---

>  arch/arm64/boot/dts/qcom/sm8150.dtsi | 28 ++++++++++++++--------------

>  1 file changed, 14 insertions(+), 14 deletions(-)

> 

> diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi

> index 9c931beeb614..163eb430eb1e 100644

> --- a/arch/arm64/boot/dts/qcom/sm8150.dtsi

> +++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi

> @@ -2205,6 +2205,20 @@ glink-edge {

>  			};

>  		};

>  

> +		dc_noc: interconnect@9160000 {

> +			compatible = "qcom,sm8150-dc-noc";

> +			reg = <0 0x09160000 0 0x3200>;

> +			#interconnect-cells = <1>;

> +			qcom,bcm-voters = <&apps_bcm_voter>;

> +		};

> +

> +		gem_noc: interconnect@9680000 {

> +			compatible = "qcom,sm8150-gem-noc";

> +			reg = <0 0x09680000 0 0x3e200>;

> +			#interconnect-cells = <1>;

> +			qcom,bcm-voters = <&apps_bcm_voter>;

> +		};

> +

>  		usb_1_hsphy: phy@88e2000 {

>  			compatible = "qcom,sm8150-usb-hs-phy",

>  				     "qcom,usb-snps-hs-7nm-phy";

> @@ -2266,20 +2280,6 @@ usb_1_ssphy: lanes@88e9200 {

>  			};

>  		};

>  

> -		dc_noc: interconnect@9160000 {

> -			compatible = "qcom,sm8150-dc-noc";

> -			reg = <0 0x09160000 0 0x3200>;

> -			#interconnect-cells = <1>;

> -			qcom,bcm-voters = <&apps_bcm_voter>;

> -		};

> -

> -		gem_noc: interconnect@9680000 {

> -			compatible = "qcom,sm8150-gem-noc";

> -			reg = <0 0x09680000 0 0x3e200>;

> -			#interconnect-cells = <1>;

> -			qcom,bcm-voters = <&apps_bcm_voter>;

> -		};

> -

>  		usb_2_qmpphy: phy@88eb000 {

>  			compatible = "qcom,sm8150-qmp-usb3-uni-phy";

>  			reg = <0 0x088eb000 0 0x200>;

> -- 

> 2.31.1

>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi
index 9c931beeb614..163eb430eb1e 100644
--- a/arch/arm64/boot/dts/qcom/sm8150.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi
@@ -2205,6 +2205,20 @@  glink-edge {
 			};
 		};
 
+		dc_noc: interconnect@9160000 {
+			compatible = "qcom,sm8150-dc-noc";
+			reg = <0 0x09160000 0 0x3200>;
+			#interconnect-cells = <1>;
+			qcom,bcm-voters = <&apps_bcm_voter>;
+		};
+
+		gem_noc: interconnect@9680000 {
+			compatible = "qcom,sm8150-gem-noc";
+			reg = <0 0x09680000 0 0x3e200>;
+			#interconnect-cells = <1>;
+			qcom,bcm-voters = <&apps_bcm_voter>;
+		};
+
 		usb_1_hsphy: phy@88e2000 {
 			compatible = "qcom,sm8150-usb-hs-phy",
 				     "qcom,usb-snps-hs-7nm-phy";
@@ -2266,20 +2280,6 @@  usb_1_ssphy: lanes@88e9200 {
 			};
 		};
 
-		dc_noc: interconnect@9160000 {
-			compatible = "qcom,sm8150-dc-noc";
-			reg = <0 0x09160000 0 0x3200>;
-			#interconnect-cells = <1>;
-			qcom,bcm-voters = <&apps_bcm_voter>;
-		};
-
-		gem_noc: interconnect@9680000 {
-			compatible = "qcom,sm8150-gem-noc";
-			reg = <0 0x09680000 0 0x3e200>;
-			#interconnect-cells = <1>;
-			qcom,bcm-voters = <&apps_bcm_voter>;
-		};
-
 		usb_2_qmpphy: phy@88eb000 {
 			compatible = "qcom,sm8150-qmp-usb3-uni-phy";
 			reg = <0 0x088eb000 0 0x200>;