Commit c310ca82 authored by Stephan Gerhold's avatar Stephan Gerhold Committed by Bjorn Andersson

arm64: dts: qcom: msm8916/39: Rename &blsp1_uartN -> &blsp_uartN

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: default avatarStephan Gerhold <stephan@gerhold.net>
Reviewed-by: default avatarBryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: default avatarBjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20230525-msm8916-labels-v1-2-bec0f5fb46fb@gerhold.net
parent 41e22c2f
......@@ -18,8 +18,8 @@ / {
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;
......@@ -243,12 +243,12 @@ &blsp_spi5 {
label = "LS-SPI0";
};
&blsp1_uart1 {
&blsp_uart1 {
status = "okay";
label = "LS-UART0";
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
label = "LS-UART1";
};
......
......@@ -24,8 +24,8 @@ / {
aliases {
mmc0 = &sdhc_1;
mmc1 = &sdhc_2;
serial0 = &blsp1_uart1;
serial1 = &blsp1_uart2;
serial0 = &blsp_uart1;
serial1 = &blsp_uart2;
};
bl: backlight {
......@@ -115,19 +115,19 @@ &blsp_i2c5 {
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";
};
......
......@@ -22,7 +22,7 @@ / {
chassis-type = "tablet";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -105,7 +105,7 @@ touchscreen@38 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -13,7 +13,7 @@ / {
chassis-type = "handset";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -56,7 +56,7 @@ usb_id: usb-id {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -13,7 +13,7 @@ / {
chassis-type = "handset";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -124,7 +124,7 @@ touchscreen@38 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -14,7 +14,7 @@ / {
chassis-type = "tablet";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -105,7 +105,7 @@ touchscreen@38 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -26,7 +26,7 @@ / {
chassis-type = "handset";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -210,7 +210,7 @@ nfc@28 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -14,7 +14,7 @@ / {
chassis-type = "handset";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -214,7 +214,7 @@ rmi4-f12@12 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -13,7 +13,7 @@ / {
chassis-type = "handset";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -101,7 +101,7 @@ imu@68 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -12,7 +12,7 @@ / {
compatible = "qcom,msm8916-mtp", "qcom,msm8916-mtp/1", "qcom,msm8916";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
usid0 = &pm8916_0;
};
......@@ -21,6 +21,6 @@ chosen {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......@@ -5,7 +5,7 @@
&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";
......@@ -14,7 +14,7 @@ blsp1_uart1_default: blsp1-uart1-default-state {
bias-disable;
};
blsp1_uart1_sleep: blsp1-uart1-sleep-state {
blsp_uart1_sleep: blsp-uart1-sleep-state {
pins = "gpio0", "gpio1", "gpio2", "gpio3";
function = "gpio";
......@@ -22,7 +22,7 @@ blsp1_uart1_sleep: blsp1-uart1-sleep-state {
bias-pull-down;
};
blsp1_uart2_default: blsp1-uart2-default-state {
blsp_uart2_default: blsp-uart2-default-state {
pins = "gpio4", "gpio5";
function = "blsp_uart2";
......@@ -30,7 +30,7 @@ blsp1_uart2_default: blsp1-uart2-default-state {
bias-disable;
};
blsp1_uart2_sleep: blsp1-uart2-sleep-state {
blsp_uart2_sleep: blsp-uart2-sleep-state {
pins = "gpio4", "gpio5";
function = "gpio";
......
......@@ -8,7 +8,7 @@
/ {
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -233,7 +233,7 @@ battery@35 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -9,7 +9,7 @@
/ {
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -111,7 +111,7 @@ accelerometer@1d {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -7,7 +7,7 @@
/ {
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -84,7 +84,7 @@ muic: extcon@25 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -28,7 +28,7 @@ / {
chassis-type = "handset";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -259,7 +259,7 @@ touchscreen@20 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -9,7 +9,7 @@ / {
chassis-type = "embedded";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -82,7 +82,7 @@ &bam_dmux_dma {
status = "okay";
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -16,7 +16,7 @@ / {
chassis-type = "handset";
aliases {
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......@@ -140,7 +140,7 @@ led@2 {
};
};
&blsp1_uart2 {
&blsp_uart2 {
status = "okay";
};
......
......@@ -1598,7 +1598,7 @@ blsp_dma: dma-controller@7884000 {
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>;
......@@ -1607,12 +1607,12 @@ blsp1_uart1: serial@78af000 {
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>;
......@@ -1621,8 +1621,8 @@ blsp1_uart2: serial@78b0000 {
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";
};
......
......@@ -23,7 +23,7 @@ / {
aliases {
mmc0 = &sdhc_1; /* SDC1 eMMC slot */
mmc1 = &sdhc_2; /* SDC2 SD card slot */
serial0 = &blsp1_uart2;
serial0 = &blsp_uart2;
};
chosen {
......
......@@ -660,28 +660,28 @@ tlmm: pinctrl@1000000 {
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>;
......@@ -1679,7 +1679,7 @@ blsp_dma: dma-controller@7884000 {
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>;
......@@ -1687,13 +1687,13 @@ blsp1_uart1: serial@78af000 {
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>;
......@@ -1701,8 +1701,8 @@ blsp1_uart2: serial@78b0000 {
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";
};
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment