arm64: dts: qcom: msm8998: align TLMM pin configuration with DT schema
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Thu, 13 Oct 2022 21:06:11 +0000 (17:06 -0400)
committerBjorn Andersson <andersson@kernel.org>
Tue, 18 Oct 2022 03:01:46 +0000 (22:01 -0500)
DT schema expects TLMM pin configuration nodes to be named with
'-state' suffix and their optional children with '-pins' suffix.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Konrad Dybcio <konrad.dybcio@somainline.org>
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20221013210612.95994-3-krzysztof.kozlowski@linaro.org
arch/arm64/boot/dts/qcom/msm8998-clamshell.dtsi
arch/arm64/boot/dts/qcom/msm8998-fxtec-pro1.dts
arch/arm64/boot/dts/qcom/msm8998-mtp.dts
arch/arm64/boot/dts/qcom/msm8998-oneplus-cheeseburger.dts
arch/arm64/boot/dts/qcom/msm8998-oneplus-common.dtsi
arch/arm64/boot/dts/qcom/msm8998-sony-xperia-yoshino.dtsi
arch/arm64/boot/dts/qcom/msm8998.dtsi

index 63413e3..3b7172a 100644 (file)
@@ -35,7 +35,7 @@
 };
 
 &blsp1_uart3_on {
-       rx {
+       rx-pins {
                /delete-property/ bias-disable;
                /*
                 * Configure a pull-up on 45 (RX). This is needed to
@@ -46,7 +46,7 @@
                bias-pull-up;
        };
 
-       cts {
+       cts-pins {
                /delete-property/ bias-disable;
                /*
                 * Configure a pull-down on 47 (CTS) to match the pull
 &tlmm {
        gpio-reserved-ranges = <0 4>, <81 4>;
 
-       touchpad: touchpad-pin {
+       touchpad: touchpad-pin-state {
                pins = "gpio123";
+               function = "gpio";
                bias-pull-up;
        };
 };
index a7a79dd..2aee2fd 100644 (file)
 };
 
 &blsp1_uart3_on {
-       rx {
+       rx-pins {
                /delete-property/ bias-disable;
                /*
                 * Configure a pull-up on 45 (RX). This is needed to
                bias-pull-up;
        };
 
-       cts {
+       cts-pins {
                /delete-property/ bias-disable;
                /*
                 * Configure a pull-down on 47 (CTS) to match the pull
 &tlmm {
        gpio-reserved-ranges = <0 4>;
 
-       mdp_vsync_n: mdp-vsync-n {
+       mdp_vsync_n: mdp-vsync-n-state {
                pins = "gpio10";
                function = "mdp_vsync_a";
                bias-pull-down;
                drive-strength = <2>;
        };
 
-       gpio_kb_pins_extra: gpio-kb-pins-extra {
+       gpio_kb_pins_extra: gpio-kb-pins-extra-state {
                pins = "gpio21", "gpio32", "gpio33", "gpio114",
                       "gpio128", "gpio129";
                function = "gpio";
                bias-pull-up;
        };
 
-       ts_vio_default: ts-vio-def {
+       ts_vio_default: ts-vio-def-state {
                pins = "gpio81";
                function = "gpio";
                bias-disable;
                drive-strength = <2>;
        };
 
-       ts_rst_n: ts-rst-n {
+       ts_rst_n: ts-rst-n-state {
                pins = "gpio89";
                function = "gpio";
                bias-pull-up;
                drive-strength = <8>;
        };
 
-       hall_sensor1_default: hall-sensor1-def {
+       hall_sensor1_default: hall-sensor1-def-state {
                pins = "gpio124";
                function = "gpio";
                bias-disable;
                input-enable;
        };
 
-       ts_int_n: ts-int-n {
+       ts_int_n: ts-int-n-state {
                pins = "gpio125";
                function = "gpio";
                bias-disable;
index abea3ff..00032ed 100644 (file)
@@ -46,7 +46,7 @@
 };
 
 &blsp1_uart3_on {
-       rx {
+       rx-pins {
                /delete-property/ bias-disable;
                /*
                 * Configure a pull-up on 45 (RX). This is needed to
@@ -57,7 +57,7 @@
                bias-pull-up;
        };
 
-       cts {
+       cts-pins {
                /delete-property/ bias-disable;
                /*
                 * Configure a pull-down on 47 (CTS) to match the pull
index 122f6c2..b951f98 100644 (file)
 
 &pmi8998_gpio {
        button_backlight_default: button-backlight-state {
-               pinconf {
-                       pins = "gpio5";
-                       function = "gpio";
-                       bias-pull-down;
-                       qcom,drive-strength = <PMIC_GPIO_STRENGTH_NO>;
-               };
+               pins = "gpio5";
+               function = "gpio";
+               bias-pull-down;
+               qcom,drive-strength = <PMIC_GPIO_STRENGTH_NO>;
        };
 };
index 3af6dee..5b058c7 100644 (file)
 };
 
 &blsp1_uart3_on {
-       rx {
+       rx-pins {
                /delete-property/ bias-disable;
                /*
                 * Configure a pull-up on 46 (RX). This is needed to
                bias-pull-up;
        };
 
-       cts {
+       cts-pins {
                /delete-property/ bias-disable;
                /*
                 * Configure a pull-down on 47 (CTS) to match the pull
 &tlmm {
        gpio-reserved-ranges = <0 4>, <81 4>;
 
-       hall_sensor_default: hall-sensor-default {
+       hall_sensor_default: hall-sensor-default-state {
                pins = "gpio124";
                function = "gpio";
                drive-strength = <2>;
                input-enable;
        };
 
-       ts_int_active: ts-int-active {
+       ts_int_active: ts-int-active-state {
                pins = "gpio125";
                function = "gpio";
                drive-strength = <8>;
                bias-pull-up;
        };
 
-       ts_reset_active: ts-reset-active {
+       ts_reset_active: ts-reset-active-state {
                pins = "gpio89";
                function = "gpio";
                drive-strength = <8>;
                bias-pull-up;
        };
 
-       nfc_int_active: nfc-int-active {
+       nfc_int_active: nfc-int-active-state {
                pins = "gpio92";
                function = "gpio";
                drive-strength = <6>;
                bias-pull-up;
        };
 
-       nfc_enable_active: nfc-enable-active {
+       nfc_enable_active: nfc-enable-active-state {
                pins = "gpio12", "gpio116";
                function = "gpio";
                drive-strength = <6>;
index 332eda4..5da87ba 100644 (file)
 &tlmm {
        gpio-reserved-ranges = <0 4>, <81 4>;
 
-       mdp_vsync_n: mdp-vsync-n {
+       mdp_vsync_n: mdp-vsync-n-state {
                pins = "gpio10";
                function = "mdp_vsync_a";
                drive-strength = <2>;
                bias-pull-down;
        };
 
-       nfc_ven: nfc-ven {
+       nfc_ven: nfc-ven-state {
                pins = "gpio12";
                function = "gpio";
                bias-disable;
                output-low;
        };
 
-       msm_mclk0_default: msm-mclk0-active {
+       msm_mclk0_default: msm-mclk0-active-state {
                pins = "gpio13";
                function = "cam_mclk";
                drive-strength = <2>;
                bias-disable;
        };
 
-       msm_mclk1_default: msm-mclk1-active {
+       msm_mclk1_default: msm-mclk1-active-state {
                pins = "gpio14";
                function = "cam_mclk";
                drive-strength = <2>;
                bias-disable;
        };
 
-       cci0_default: cci0-default {
+       cci0_default: cci0-default-state {
                pins = "gpio18", "gpio19";
                function = "cci_i2c";
                bias-disable;
                drive-strength = <2>;
        };
 
-       cci1_default: cci1-default {
+       cci1_default: cci1-default-state {
                pins = "gpio19", "gpio20";
                function = "cci_i2c";
                bias-disable;
                drive-strength = <2>;
        };
 
-       cam0_vdig_default: cam0-vdig-default {
+       cam0_vdig_default: cam0-vdig-default-state {
                pins = "gpio21";
                function = "gpio";
                bias-disable;
                drive-strength = <2>;
        };
 
-       tof_int: tof-int {
+       tof_int: tof-int-state {
                pins = "gpio22";
                function = "gpio";
                bias-pull-up;
                input-enable;
        };
 
-       cam1_vdig_default: cam1-vdig-default {
+       cam1_vdig_default: cam1-vdig-default-state {
                pins = "gpio25";
                function = "gpio";
                bias-disable;
                drive-strength = <2>;
        };
 
-       usb_extcon_active: usb-extcon-active {
+       usb_extcon_active: usb-extcon-active-state {
                pins = "gpio38";
                function = "gpio";
                bias-disable;
                drive-strength = <16>;
        };
 
-       tof_reset: tof-reset {
+       tof_reset: tof-reset-state {
                pins = "gpio27";
                function = "gpio";
                bias-disable;
                drive-strength = <2>;
        };
 
-       hall_sensor0_default: acc-cover-open {
+       hall_sensor0_default: acc-cover-open-state {
                pins = "gpio124";
                function = "gpio";
                bias-disable;
                input-enable;
        };
 
-       ts_int_n: ts-int-n {
+       ts_int_n: ts-int-n-state {
                pins = "gpio125";
                function = "gpio";
                drive-strength = <8>;
                bias-pull-up;
        };
 
-       usb_vbus_active: usb-vbus-active {
+       usb_vbus_active: usb-vbus-active-state {
                pins = "gpio128";
                function = "gpio";
                bias-disable;
                output-low;
        };
 
-       ts_vddio_en: ts-vddio-en-default {
+       ts_vddio_en: ts-vddio-en-default-state {
                pins = "gpio133";
                function = "gpio";
                bias-disable;
index bc3c285..da2dd87 100644 (file)
                        interrupt-controller;
                        #interrupt-cells = <2>;
 
-                       sdc2_on: sdc2-on {
-                               clk {
+                       sdc2_on: sdc2-on-state {
+                               clk-pins {
                                        pins = "sdc2_clk";
                                        drive-strength = <16>;
                                        bias-disable;
                                };
 
-                               cmd {
+                               cmd-pins {
                                        pins = "sdc2_cmd";
                                        drive-strength = <10>;
                                        bias-pull-up;
                                };
 
-                               data {
+                               data-pins {
                                        pins = "sdc2_data";
                                        drive-strength = <10>;
                                        bias-pull-up;
                                };
                        };
 
-                       sdc2_off: sdc2-off {
-                               clk {
+                       sdc2_off: sdc2-off-state {
+                               clk-pins {
                                        pins = "sdc2_clk";
                                        drive-strength = <2>;
                                        bias-disable;
                                };
 
-                               cmd {
+                               cmd-pins {
                                        pins = "sdc2_cmd";
                                        drive-strength = <2>;
                                        bias-pull-up;
                                };
 
-                               data {
+                               data-pins {
                                        pins = "sdc2_data";
                                        drive-strength = <2>;
                                        bias-pull-up;
                                };
                        };
 
-                       sdc2_cd: sdc2-cd {
+                       sdc2_cd: sdc2-cd-state {
                                pins = "gpio95";
                                function = "gpio";
                                bias-pull-up;
                                drive-strength = <2>;
                        };
 
-                       blsp1_uart3_on: blsp1-uart3-on {
-                               tx {
+                       blsp1_uart3_on: blsp1-uart3-on-state {
+                               tx-pins {
                                        pins = "gpio45";
                                        function = "blsp_uart3_a";
                                        drive-strength = <2>;
                                        bias-disable;
                                };
 
-                               rx {
+                               rx-pins {
                                        pins = "gpio46";
                                        function = "blsp_uart3_a";
                                        drive-strength = <2>;
                                        bias-disable;
                                };
 
-                               cts {
+                               cts-pins {
                                        pins = "gpio47";
                                        function = "blsp_uart3_a";
                                        drive-strength = <2>;
                                        bias-disable;
                                };
 
-                               rfr {
+                               rfr-pins {
                                        pins = "gpio48";
                                        function = "blsp_uart3_a";
                                        drive-strength = <2>;
                                };
                        };
 
-                       blsp1_i2c1_default: blsp1-i2c1-default {
+                       blsp1_i2c1_default: blsp1-i2c1-default-state {
                                pins = "gpio2", "gpio3";
                                function = "blsp_i2c1";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp1_i2c1_sleep: blsp1-i2c1-sleep {
+                       blsp1_i2c1_sleep: blsp1-i2c1-sleep-state-state {
                                pins = "gpio2", "gpio3";
                                function = "blsp_i2c1";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp1_i2c2_default: blsp1-i2c2-default {
+                       blsp1_i2c2_default: blsp1-i2c2-default-state {
                                pins = "gpio32", "gpio33";
                                function = "blsp_i2c2";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp1_i2c2_sleep: blsp1-i2c2-sleep {
+                       blsp1_i2c2_sleep: blsp1-i2c2-sleep-state-state {
                                pins = "gpio32", "gpio33";
                                function = "blsp_i2c2";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp1_i2c3_default: blsp1-i2c3-default {
+                       blsp1_i2c3_default: blsp1-i2c3-default-state {
                                pins = "gpio47", "gpio48";
                                function = "blsp_i2c3";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp1_i2c3_sleep: blsp1-i2c3-sleep {
+                       blsp1_i2c3_sleep: blsp1-i2c3-sleep-state {
                                pins = "gpio47", "gpio48";
                                function = "blsp_i2c3";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp1_i2c4_default: blsp1-i2c4-default {
+                       blsp1_i2c4_default: blsp1-i2c4-default-state {
                                pins = "gpio10", "gpio11";
                                function = "blsp_i2c4";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp1_i2c4_sleep: blsp1-i2c4-sleep {
+                       blsp1_i2c4_sleep: blsp1-i2c4-sleep-state {
                                pins = "gpio10", "gpio11";
                                function = "blsp_i2c4";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp1_i2c5_default: blsp1-i2c5-default {
+                       blsp1_i2c5_default: blsp1-i2c5-default-state {
                                pins = "gpio87", "gpio88";
                                function = "blsp_i2c5";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp1_i2c5_sleep: blsp1-i2c5-sleep {
+                       blsp1_i2c5_sleep: blsp1-i2c5-sleep-state {
                                pins = "gpio87", "gpio88";
                                function = "blsp_i2c5";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp1_i2c6_default: blsp1-i2c6-default {
+                       blsp1_i2c6_default: blsp1-i2c6-default-state {
                                pins = "gpio43", "gpio44";
                                function = "blsp_i2c6";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp1_i2c6_sleep: blsp1-i2c6-sleep {
+                       blsp1_i2c6_sleep: blsp1-i2c6-sleep-state {
                                pins = "gpio43", "gpio44";
                                function = "blsp_i2c6";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
                        /* 6 interfaces per QUP, BLSP2 indexes are numbered (n)+6 */
-                       blsp2_i2c1_default: blsp2-i2c1-default {
+                       blsp2_i2c1_default: blsp2-i2c1-default-state {
                                pins = "gpio55", "gpio56";
                                function = "blsp_i2c7";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp2_i2c1_sleep: blsp2-i2c1-sleep {
+                       blsp2_i2c1_sleep: blsp2-i2c1-sleep-state {
                                pins = "gpio55", "gpio56";
                                function = "blsp_i2c7";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp2_i2c2_default: blsp2-i2c2-default {
+                       blsp2_i2c2_default: blsp2-i2c2-default-state {
                                pins = "gpio6", "gpio7";
                                function = "blsp_i2c8";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp2_i2c2_sleep: blsp2-i2c2-sleep {
+                       blsp2_i2c2_sleep: blsp2-i2c2-sleep-state {
                                pins = "gpio6", "gpio7";
                                function = "blsp_i2c8";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp2_i2c3_default: blsp2-i2c3-default {
+                       blsp2_i2c3_default: blsp2-i2c3-default-state {
                                pins = "gpio51", "gpio52";
                                function = "blsp_i2c9";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp2_i2c3_sleep: blsp2-i2c3-sleep {
+                       blsp2_i2c3_sleep: blsp2-i2c3-sleep-state {
                                pins = "gpio51", "gpio52";
                                function = "blsp_i2c9";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp2_i2c4_default: blsp2-i2c4-default {
+                       blsp2_i2c4_default: blsp2-i2c4-default-state {
                                pins = "gpio67", "gpio68";
                                function = "blsp_i2c10";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp2_i2c4_sleep: blsp2-i2c4-sleep {
+                       blsp2_i2c4_sleep: blsp2-i2c4-sleep-state {
                                pins = "gpio67", "gpio68";
                                function = "blsp_i2c10";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp2_i2c5_default: blsp2-i2c5-default {
+                       blsp2_i2c5_default: blsp2-i2c5-default-state {
                                pins = "gpio60", "gpio61";
                                function = "blsp_i2c11";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp2_i2c5_sleep: blsp2-i2c5-sleep {
+                       blsp2_i2c5_sleep: blsp2-i2c5-sleep-state {
                                pins = "gpio60", "gpio61";
                                function = "blsp_i2c11";
                                drive-strength = <2>;
                                bias-pull-up;
                        };
 
-                       blsp2_i2c6_default: blsp2-i2c6-default {
+                       blsp2_i2c6_default: blsp2-i2c6-default-state {
                                pins = "gpio83", "gpio84";
                                function = "blsp_i2c12";
                                drive-strength = <2>;
                                bias-disable;
                        };
 
-                       blsp2_i2c6_sleep: blsp2-i2c6-sleep {
+                       blsp2_i2c6_sleep: blsp2-i2c6-sleep-state {
                                pins = "gpio83", "gpio84";
                                function = "blsp_i2c12";
                                drive-strength = <2>;