For some reason the BLSP UART controllers have a label with a number
behind blsp (&blsp1_uartN) while I2C/SPI are named without (&blsp_i2cN).
This is confusing, especially for proper node ordering in board DTs.
Right now all board DTs are ordered as if the number behind blsp does
not exist (&blsp_i2cN comes before &blsp1_uartN). Strictly speaking
correct ordering would be the other way around ('1' comes before '_').
End this confusion by giving the UART controllers consistent labels.
There is just one BLSP on MSM8916/39 so the number is redundant.
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20230525-msm8916-labels-v1-2-bec0f5fb46fb@gerhold.net
        compatible = "qcom,apq8016-sbc", "qcom,apq8016";
 
        aliases {
-               serial0 = &blsp1_uart2;
-               serial1 = &blsp1_uart1;
+               serial0 = &blsp_uart2;
+               serial1 = &blsp_uart1;
                usid0 = &pm8916_0;
                i2c0 = &blsp_i2c2;
                i2c1 = &blsp_i2c6;
        label = "LS-SPI0";
 };
 
-&blsp1_uart1 {
+&blsp_uart1 {
        status = "okay";
        label = "LS-UART0";
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
        label = "LS-UART1";
 };
 
        aliases {
                mmc0 = &sdhc_1;
                mmc1 = &sdhc_2;
-               serial0 = &blsp1_uart1;
-               serial1 = &blsp1_uart2;
+               serial0 = &blsp_uart1;
+               serial1 = &blsp_uart2;
        };
 
        bl: backlight {
        status = "okay";
 };
 
-&blsp1_uart1 {
+&blsp_uart1 {
        status = "okay";
 };
 
-&blsp1_uart1_default {
+&blsp_uart1_default {
        pins = "gpio0", "gpio1";
 };
 
-&blsp1_uart1_sleep {
+&blsp_uart1_sleep {
        pins = "gpio0", "gpio1";
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "tablet";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "handset";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "handset";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "tablet";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "handset";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "handset";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "handset";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        compatible = "qcom,msm8916-mtp", "qcom,msm8916-mtp/1", "qcom,msm8916";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
                usid0 = &pm8916_0;
        };
 
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
 &tlmm {
 
-       blsp1_uart1_default: blsp1-uart1-default-state {
+       blsp_uart1_default: blsp-uart1-default-state {
                /* TX, RX, CTS_N, RTS_N */
                pins = "gpio0", "gpio1", "gpio2", "gpio3";
                function = "blsp_uart1";
                bias-disable;
        };
 
-       blsp1_uart1_sleep: blsp1-uart1-sleep-state {
+       blsp_uart1_sleep: blsp-uart1-sleep-state {
                pins = "gpio0", "gpio1", "gpio2", "gpio3";
                function = "gpio";
 
                bias-pull-down;
        };
 
-       blsp1_uart2_default: blsp1-uart2-default-state {
+       blsp_uart2_default: blsp-uart2-default-state {
                pins = "gpio4", "gpio5";
                function = "blsp_uart2";
 
                bias-disable;
        };
 
-       blsp1_uart2_sleep: blsp1-uart2-sleep-state {
+       blsp_uart2_sleep: blsp-uart2-sleep-state {
                pins = "gpio4", "gpio5";
                function = "gpio";
 
 
 
 / {
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
 
 / {
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
 
 / {
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "handset";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "embedded";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        status = "okay";
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
        chassis-type = "handset";
 
        aliases {
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
        };
 };
 
-&blsp1_uart2 {
+&blsp_uart2 {
        status = "okay";
 };
 
 
                        qcom,ee = <0>;
                };
 
-               blsp1_uart1: serial@78af000 {
+               blsp_uart1: serial@78af000 {
                        compatible = "qcom,msm-uartdm-v1.4", "qcom,msm-uartdm";
                        reg = <0x078af000 0x200>;
                        interrupts = <GIC_SPI 107 IRQ_TYPE_LEVEL_HIGH>;
                        dmas = <&blsp_dma 0>, <&blsp_dma 1>;
                        dma-names = "tx", "rx";
                        pinctrl-names = "default", "sleep";
-                       pinctrl-0 = <&blsp1_uart1_default>;
-                       pinctrl-1 = <&blsp1_uart1_sleep>;
+                       pinctrl-0 = <&blsp_uart1_default>;
+                       pinctrl-1 = <&blsp_uart1_sleep>;
                        status = "disabled";
                };
 
-               blsp1_uart2: serial@78b0000 {
+               blsp_uart2: serial@78b0000 {
                        compatible = "qcom,msm-uartdm-v1.4", "qcom,msm-uartdm";
                        reg = <0x078b0000 0x200>;
                        interrupts = <GIC_SPI 108 IRQ_TYPE_LEVEL_HIGH>;
                        dmas = <&blsp_dma 2>, <&blsp_dma 3>;
                        dma-names = "tx", "rx";
                        pinctrl-names = "default", "sleep";
-                       pinctrl-0 = <&blsp1_uart2_default>;
-                       pinctrl-1 = <&blsp1_uart2_sleep>;
+                       pinctrl-0 = <&blsp_uart2_default>;
+                       pinctrl-1 = <&blsp_uart2_sleep>;
                        status = "disabled";
                };
 
 
        aliases {
                mmc0 = &sdhc_1; /* SDC1 eMMC slot */
                mmc1 = &sdhc_2; /* SDC2 SD card slot */
-               serial0 = &blsp1_uart2;
+               serial0 = &blsp_uart2;
        };
 
        chosen {
 
                        interrupt-controller;
                        #interrupt-cells = <2>;
 
-                       blsp1_uart1_default: blsp1-uart1-default-state {
+                       blsp_uart1_default: blsp-uart1-default-state {
                                pins = "gpio0", "gpio1", "gpio2", "gpio3";
                                function = "blsp_uart1";
                                drive-strength = <16>;
                                bias-disable;
                        };
 
-                       blsp1_uart1_sleep: blsp1-uart1-sleep-state {
+                       blsp_uart1_sleep: blsp-uart1-sleep-state {
                                pins = "gpio0", "gpio1", "gpio2", "gpio3";
                                function = "gpio";
                                drive-strength = <2>;
                                bias-pull-down;
                        };
 
-                       blsp1_uart2_default: blsp1-uart2-default-state {
+                       blsp_uart2_default: blsp-uart2-default-state {
                                pins = "gpio4", "gpio5";
                                function = "blsp_uart2";
                                drive-strength = <16>;
                                bias-disable;
                        };
 
-                       blsp1_uart2_sleep: blsp1-uart2-sleep-state {
+                       blsp_uart2_sleep: blsp-uart2-sleep-state {
                                pins = "gpio4", "gpio5";
                                function = "gpio";
                                drive-strength = <2>;
                        qcom,ee = <0>;
                };
 
-               blsp1_uart1: serial@78af000 {
+               blsp_uart1: serial@78af000 {
                        compatible = "qcom,msm-uartdm-v1.4", "qcom,msm-uartdm";
                        reg = <0x078af000 0x200>;
                        interrupts = <GIC_SPI 107 IRQ_TYPE_LEVEL_HIGH>;
                        clock-names = "core", "iface";
                        dmas = <&blsp_dma 0>, <&blsp_dma 1>;
                        dma-names = "tx", "rx";
-                       pinctrl-0 = <&blsp1_uart1_default>;
-                       pinctrl-1 = <&blsp1_uart1_sleep>;
+                       pinctrl-0 = <&blsp_uart1_default>;
+                       pinctrl-1 = <&blsp_uart1_sleep>;
                        pinctrl-names = "default", "sleep";
                        status = "disabled";
                };
 
-               blsp1_uart2: serial@78b0000 {
+               blsp_uart2: serial@78b0000 {
                        compatible = "qcom,msm-uartdm-v1.4", "qcom,msm-uartdm";
                        reg = <0x078b0000 0x200>;
                        interrupts = <GIC_SPI 108 IRQ_TYPE_LEVEL_HIGH>;
                        clock-names = "core", "iface";
                        dmas = <&blsp_dma 2>, <&blsp_dma 3>;
                        dma-names = "tx", "rx";
-                       pinctrl-0 = <&blsp1_uart2_default>;
-                       pinctrl-1 = <&blsp1_uart2_sleep>;
+                       pinctrl-0 = <&blsp_uart2_default>;
+                       pinctrl-1 = <&blsp_uart2_sleep>;
                        pinctrl-names = "default", "sleep";
                        status = "disabled";
                };