arm64: dts: qcom: msm8916/39: Clean up MDSS labels
authorStephan Gerhold <stephan@gerhold.net>
Mon, 29 May 2023 12:47:01 +0000 (14:47 +0200)
committerBjorn Andersson <andersson@kernel.org>
Mon, 29 May 2023 21:37:01 +0000 (14:37 -0700)
Right now MDSS related definitions cannot be properly grouped together
in board DTs because the labels do not use consistent prefixes. The DSI
PHY label is particularly weird because the DSI number is at the end
(&dsi_phy0) while DSI itself is called &dsi0.

Follow the example of more recent SoCs and give all the MDSS related
nodes a consistent label that allows proper grouping.

Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20230525-msm8916-labels-v1-4-bec0f5fb46fb@gerhold.net
arch/arm64/boot/dts/qcom/apq8016-sbc.dts
arch/arm64/boot/dts/qcom/msm8916-pm8916.dtsi
arch/arm64/boot/dts/qcom/msm8916-samsung-a2015-common.dtsi
arch/arm64/boot/dts/qcom/msm8916-samsung-a3u-eur.dts
arch/arm64/boot/dts/qcom/msm8916-ufi.dtsi
arch/arm64/boot/dts/qcom/msm8916.dtsi
arch/arm64/boot/dts/qcom/msm8939-pm8916.dtsi
arch/arm64/boot/dts/qcom/msm8939.dtsi

index 4f3d837578fd0fd07502693041b59b0801608458..9081825c95746d5614e8ff7fa7308017bc6ecbb8 100644 (file)
                        port@0 {
                                reg = <0>;
                                adv7533_in: endpoint {
-                                       remote-endpoint = <&dsi0_out>;
+                                       remote-endpoint = <&mdss_dsi0_out>;
                                };
                        };
 
        };
 };
 
-&dsi0_out {
-       data-lanes = <0 1 2 3>;
-       remote-endpoint = <&adv7533_in>;
-};
-
 &lpass {
        status = "okay";
 };
        status = "okay";
 };
 
+&mdss_dsi0_out {
+       data-lanes = <0 1 2 3>;
+       remote-endpoint = <&adv7533_in>;
+};
+
 &mpss {
        status = "okay";
 
index 64d7228bee07d01d7afa74147bd2a19669cecb80..4e7289e6d22da5668e69994c34fcf129e92b751d 100644 (file)
        vdda-supply = <&pm8916_l2>;
 };
 
-&dsi0 {
+&mdss_dsi0 {
        vdda-supply = <&pm8916_l2>;
        vddio-supply = <&pm8916_l6>;
 };
 
-&dsi_phy0 {
+&mdss_dsi0_phy {
        vddio-supply = <&pm8916_l6>;
 };
 
index 497fc83803ef04dbb457df0c695918ce03e1ac2b..b1af9c5d07b25898f339939aca37bb5096df324b 100644 (file)
        status = "okay";
 };
 
-&dsi0 {
+&mdss {
+       status = "okay";
+};
+
+&mdss_dsi0 {
        pinctrl-names = "default", "sleep";
        pinctrl-0 = <&mdss_default>;
        pinctrl-1 = <&mdss_sleep>;
 };
 
-&mdss {
-       status = "okay";
-};
-
 &pm8916_resin {
        status = "okay";
        linux,code = <KEY_VOLUMEDOWN>;
index 9068aa6f7b293cbce4a1c1187b129a61b6082b0f..e5a569698c4f222209802767968471067d70034b 100644 (file)
@@ -85,7 +85,7 @@
        status = "okay";
 };
 
-&dsi0 {
+&mdss_dsi0 {
        panel@0 {
                reg = <0>;
 
 
                port {
                        panel_in: endpoint {
-                               remote-endpoint = <&dsi0_out>;
+                               remote-endpoint = <&mdss_dsi0_out>;
                        };
                };
        };
 };
 
-&dsi0_out {
+&mdss_dsi0_out {
        data-lanes = <0 1>;
        remote-endpoint = <&panel_in>;
 };
index 898722ee86a51642fa03e1d115df4b69370c0456..ac39ac6a21c1663bda348b5f763cbfdb7284a935 100644 (file)
@@ -86,7 +86,7 @@
        status = "okay";
 };
 
-/* Remove &dsi_phy0 from clocks to make sure that gcc probes with display disabled */
+/* Remove &mdss_dsi0_phy from clocks to make sure that gcc probes with display disabled */
 &gcc {
        clocks = <&xo_board>, <&sleep_clk>, <0>, <0>, <0>, <0>, <0>;
 };
index b7736a07bca37d34e11ebeb089c6135697710300..ff1834a624eb022bc8ea5c9ab7b2322e574c823e 100644 (file)
                        reg = <0x01800000 0x80000>;
                        clocks = <&xo_board>,
                                 <&sleep_clk>,
-                                <&dsi_phy0 1>,
-                                <&dsi_phy0 0>,
+                                <&mdss_dsi0_phy 1>,
+                                <&mdss_dsi0_phy 0>,
                                 <0>,
                                 <0>,
                                 <0>;
                        #size-cells = <1>;
                        ranges;
 
-                       mdp: display-controller@1a01000 {
+                       mdss_mdp: display-controller@1a01000 {
                                compatible = "qcom,msm8916-mdp5", "qcom,mdp5";
                                reg = <0x01a01000 0x89000>;
                                reg-names = "mdp_phys";
 
                                        port@0 {
                                                reg = <0>;
-                                               mdp5_intf1_out: endpoint {
-                                                       remote-endpoint = <&dsi0_in>;
+                                               mdss_mdp_intf1_out: endpoint {
+                                                       remote-endpoint = <&mdss_dsi0_in>;
                                                };
                                        };
                                };
                        };
 
-                       dsi0: dsi@1a98000 {
+                       mdss_dsi0: dsi@1a98000 {
                                compatible = "qcom,msm8916-dsi-ctrl",
                                             "qcom,mdss-dsi-ctrl";
                                reg = <0x01a98000 0x25c>;
 
                                assigned-clocks = <&gcc BYTE0_CLK_SRC>,
                                                  <&gcc PCLK0_CLK_SRC>;
-                               assigned-clock-parents = <&dsi_phy0 0>,
-                                                        <&dsi_phy0 1>;
+                               assigned-clock-parents = <&mdss_dsi0_phy 0>,
+                                                        <&mdss_dsi0_phy 1>;
 
                                clocks = <&gcc GCC_MDSS_MDP_CLK>,
                                         <&gcc GCC_MDSS_AHB_CLK>,
                                              "byte",
                                              "pixel",
                                              "core";
-                               phys = <&dsi_phy0>;
+                               phys = <&mdss_dsi0_phy>;
 
                                #address-cells = <1>;
                                #size-cells = <0>;
 
                                        port@0 {
                                                reg = <0>;
-                                               dsi0_in: endpoint {
-                                                       remote-endpoint = <&mdp5_intf1_out>;
+                                               mdss_dsi0_in: endpoint {
+                                                       remote-endpoint = <&mdss_mdp_intf1_out>;
                                                };
                                        };
 
                                        port@1 {
                                                reg = <1>;
-                                               dsi0_out: endpoint {
+                                               mdss_dsi0_out: endpoint {
                                                };
                                        };
                                };
                        };
 
-                       dsi_phy0: phy@1a98300 {
+                       mdss_dsi0_phy: phy@1a98300 {
                                compatible = "qcom,dsi-phy-28nm-lp";
                                reg = <0x01a98300 0xd4>,
                                      <0x01a98500 0x280>,
index 24393a159058035859da308318f2501172a8bc2f..33e02f42f5e4e7031ec583c2dfb66df38c8710fe 100644 (file)
@@ -3,21 +3,21 @@
 #include "msm8939.dtsi"
 #include "pm8916.dtsi"
 
-&dsi0 {
+&mdss_dsi0 {
        vdda-supply = <&pm8916_l2>;
        vddio-supply = <&pm8916_l6>;
 };
 
-&dsi1 {
-       vdda-supply = <&pm8916_l2>;
+&mdss_dsi0_phy {
        vddio-supply = <&pm8916_l6>;
 };
 
-&dsi_phy0 {
+&mdss_dsi1 {
+       vdda-supply = <&pm8916_l2>;
        vddio-supply = <&pm8916_l6>;
 };
 
-&dsi_phy1 {
+&mdss_dsi1_phy {
        vddio-supply = <&pm8916_l6>;
 };
 
index 1196a962897f50ea18d350ae33b575b9aeae83e1..0d9f8b951b66e4ed06ed2ff97fff6829e94179fc 100644 (file)
                        reg = <0x01800000 0x80000>;
                        clocks = <&rpmcc RPM_SMD_XO_CLK_SRC>,
                                 <&sleep_clk>,
-                                <&dsi_phy0 1>,
-                                <&dsi_phy0 0>,
+                                <&mdss_dsi0_phy 1>,
+                                <&mdss_dsi0_phy 0>,
                                 <0>,
                                 <0>,
                                 <0>;
 
                        status = "disabled";
 
-                       mdp: display-controller@1a01000 {
+                       mdss_mdp: display-controller@1a01000 {
                                compatible = "qcom,mdp5";
                                reg = <0x01a01000 0x89000>;
                                reg-names = "mdp_phys";
 
                                        port@0 {
                                                reg = <0>;
-                                               mdp5_intf1_out: endpoint {
-                                                       remote-endpoint = <&dsi0_in>;
+                                               mdss_mdp_intf1_out: endpoint {
+                                                       remote-endpoint = <&mdss_dsi0_in>;
                                                };
                                        };
 
                                        port@1 {
                                                reg = <1>;
-                                               mdp5_intf2_out: endpoint {
-                                                       remote-endpoint = <&dsi1_in>;
+                                               mdss_mdp_intf2_out: endpoint {
+                                                       remote-endpoint = <&mdss_dsi1_in>;
                                                };
                                        };
                                };
                        };
 
-                       dsi0: dsi@1a98000 {
+                       mdss_dsi0: dsi@1a98000 {
                                compatible = "qcom,msm8916-dsi-ctrl",
                                             "qcom,mdss-dsi-ctrl";
                                reg = <0x01a98000 0x25c>;
                                              "core";
                                assigned-clocks = <&gcc BYTE0_CLK_SRC>,
                                                  <&gcc PCLK0_CLK_SRC>;
-                               assigned-clock-parents = <&dsi_phy0 0>,
-                                                        <&dsi_phy0 1>;
+                               assigned-clock-parents = <&mdss_dsi0_phy 0>,
+                                                        <&mdss_dsi0_phy 1>;
 
-                               phys = <&dsi_phy0>;
+                               phys = <&mdss_dsi0_phy>;
                                status = "disabled";
 
                                #address-cells = <1>;
 
                                        port@0 {
                                                reg = <0>;
-                                               dsi0_in: endpoint {
-                                                       remote-endpoint = <&mdp5_intf1_out>;
+                                               mdss_dsi0_in: endpoint {
+                                                       remote-endpoint = <&mdss_mdp_intf1_out>;
                                                };
                                        };
 
                                        port@1 {
                                                reg = <1>;
-                                               dsi0_out: endpoint {
+                                               mdss_dsi0_out: endpoint {
                                                };
                                        };
                                };
                        };
 
-                       dsi_phy0: phy@1a98300 {
+                       mdss_dsi0_phy: phy@1a98300 {
                                compatible = "qcom,dsi-phy-28nm-lp";
                                reg = <0x01a98300 0xd4>,
                                      <0x01a98500 0x280>,
                                status = "disabled";
                        };
 
-                       dsi1: dsi@1aa0000 {
+                       mdss_dsi1: dsi@1aa0000 {
                                compatible = "qcom,msm8916-dsi-ctrl",
                                             "qcom,mdss-dsi-ctrl";
                                reg = <0x01aa0000 0x25c>;
                                              "core";
                                assigned-clocks = <&gcc BYTE1_CLK_SRC>,
                                                  <&gcc PCLK1_CLK_SRC>;
-                               assigned-clock-parents = <&dsi_phy0 0>,
-                                                        <&dsi_phy0 1>;
-                               phys = <&dsi_phy1>;
+                               assigned-clock-parents = <&mdss_dsi0_phy 0>,
+                                                        <&mdss_dsi0_phy 1>;
+                               phys = <&mdss_dsi1_phy>;
                                status = "disabled";
 
                                ports {
 
                                        port@0 {
                                                reg = <0>;
-                                               dsi1_in: endpoint {
-                                                       remote-endpoint = <&mdp5_intf2_out>;
+                                               mdss_dsi1_in: endpoint {
+                                                       remote-endpoint = <&mdss_mdp_intf2_out>;
                                                };
                                        };
 
                                        port@1 {
                                                reg = <1>;
-                                               dsi1_out: endpoint {
+                                               mdss_dsi1_out: endpoint {
                                                };
                                        };
                                };
                        };
 
-                       dsi_phy1: phy@1aa0300 {
+                       mdss_dsi1_phy: phy@1aa0300 {
                                compatible = "qcom,dsi-phy-28nm-lp";
                                reg = <0x01aa0300 0xd4>,
                                      <0x01aa0500 0x280>,