arm64: dts: qcom: msm8916/39: Use consistent name for I2C/SPI pinctrl

Make the labels for the BLSP I2C/SPI pinctrl consistent with the one
used for UART by adding the missing blsp_ prefix. This allows having
them properly grouped together.

The nodes are only reordered in msm8939.dtsi for now since the pinctrl
definitions in msm8916-pins.dtsi are currently still unordered anyway.
(I will try fixing this in a future patch.)
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-3-bec0f5fb46fb@gerhold.net

arch/arm64/boot/dts/qcom/apq8016-sbc.dts

@@ -479,15 +479,15 @@ &wcnss_iris {
  * 2mA drive strength is not enough when connecting multiple
  * I2C devices with different pull up resistors.
  */
-&i2c2_default {
+&blsp_i2c2_default {
 	drive-strength = <16>;
 };
 
-&i2c4_default {
+&blsp_i2c4_default {
 	drive-strength = <16>;
 };
 
-&i2c6_default {
+&blsp_i2c6_default {
 	drive-strength = <16>;
 };
 

arch/arm64/boot/dts/qcom/msm8916-pins.dtsi

@@ -38,7 +38,7 @@ blsp_uart2_sleep: blsp-uart2-sleep-state {
 		bias-pull-down;
 	};
 
-	spi1_default: spi1-default-state {
+	blsp_spi1_default: blsp-spi1-default-state {
 		spi-pins {
 			pins = "gpio0", "gpio1", "gpio3";
 			function = "blsp_spi1";
@@ -56,7 +56,7 @@ cs-pins {
 		};
 	};
 
-	spi1_sleep: spi1-sleep-state {
+	blsp_spi1_sleep: blsp-spi1-sleep-state {
 		pins = "gpio0", "gpio1", "gpio2", "gpio3";
 		function = "gpio";
 
@@ -64,7 +64,7 @@ spi1_sleep: spi1-sleep-state {
 		bias-pull-down;
 	};
 
-	spi2_default: spi2-default-state {
+	blsp_spi2_default: blsp-spi2-default-state {
 		spi-pins {
 			pins = "gpio4", "gpio5", "gpio7";
 			function = "blsp_spi2";
@@ -82,7 +82,7 @@ cs-pins {
 		};
 	};
 
-	spi2_sleep: spi2-sleep-state {
+	blsp_spi2_sleep: blsp-spi2-sleep-state {
 		pins = "gpio4", "gpio5", "gpio6", "gpio7";
 		function = "gpio";
 
@@ -90,7 +90,7 @@ spi2_sleep: spi2-sleep-state {
 		bias-pull-down;
 	};
 
-	spi3_default: spi3-default-state {
+	blsp_spi3_default: blsp-spi3-default-state {
 		spi-pins {
 			pins = "gpio8", "gpio9", "gpio11";
 			function = "blsp_spi3";
@@ -108,7 +108,7 @@ cs-pins {
 		};
 	};
 
-	spi3_sleep: spi3-sleep-state {
+	blsp_spi3_sleep: blsp-spi3-sleep-state {
 		pins = "gpio8", "gpio9", "gpio10", "gpio11";
 		function = "gpio";
 
@@ -116,7 +116,7 @@ spi3_sleep: spi3-sleep-state {
 		bias-pull-down;
 	};
 
-	spi4_default: spi4-default-state {
+	blsp_spi4_default: blsp-spi4-default-state {
 		spi-pins {
 			pins = "gpio12", "gpio13", "gpio15";
 			function = "blsp_spi4";
@@ -134,7 +134,7 @@ cs-pins {
 		};
 	};
 
-	spi4_sleep: spi4-sleep-state {
+	blsp_spi4_sleep: blsp-spi4-sleep-state {
 		pins = "gpio12", "gpio13", "gpio14", "gpio15";
 		function = "gpio";
 
@@ -142,7 +142,7 @@ spi4_sleep: spi4-sleep-state {
 		bias-pull-down;
 	};
 
-	spi5_default: spi5-default-state {
+	blsp_spi5_default: blsp-spi5-default-state {
 		spi-pins {
 			pins = "gpio16", "gpio17", "gpio19";
 			function = "blsp_spi5";
@@ -160,7 +160,7 @@ cs-pins {
 		};
 	};
 
-	spi5_sleep: spi5-sleep-state {
+	blsp_spi5_sleep: blsp-spi5-sleep-state {
 		pins = "gpio16", "gpio17", "gpio18", "gpio19";
 		function = "gpio";
 
@@ -168,7 +168,7 @@ spi5_sleep: spi5-sleep-state {
 		bias-pull-down;
 	};
 
-	spi6_default: spi6-default-state {
+	blsp_spi6_default: blsp-spi6-default-state {
 		spi-pins {
 			pins = "gpio20", "gpio21", "gpio23";
 			function = "blsp_spi6";
@@ -186,7 +186,7 @@ cs-pins {
 		};
 	};
 
-	spi6_sleep: spi6-sleep-state {
+	blsp_spi6_sleep: blsp-spi6-sleep-state {
 		pins = "gpio20", "gpio21", "gpio22", "gpio23";
 		function = "gpio";
 
@@ -194,7 +194,7 @@ spi6_sleep: spi6-sleep-state {
 		bias-pull-down;
 	};
 
-	i2c1_default: i2c1-default-state {
+	blsp_i2c1_default: blsp-i2c1-default-state {
 		pins = "gpio2", "gpio3";
 		function = "blsp_i2c1";
 
@@ -202,7 +202,7 @@ i2c1_default: i2c1-default-state {
 		bias-disable;
 	};
 
-	i2c1_sleep: i2c1-sleep-state {
+	blsp_i2c1_sleep: blsp-i2c1-sleep-state {
 		pins = "gpio2", "gpio3";
 		function = "gpio";
 
@@ -210,7 +210,7 @@ i2c1_sleep: i2c1-sleep-state {
 		bias-disable;
 	};
 
-	i2c2_default: i2c2-default-state {
+	blsp_i2c2_default: blsp-i2c2-default-state {
 		pins = "gpio6", "gpio7";
 		function = "blsp_i2c2";
 
@@ -218,7 +218,7 @@ i2c2_default: i2c2-default-state {
 		bias-disable;
 	};
 
-	i2c2_sleep: i2c2-sleep-state {
+	blsp_i2c2_sleep: blsp-i2c2-sleep-state {
 		pins = "gpio6", "gpio7";
 		function = "gpio";
 
@@ -226,7 +226,7 @@ i2c2_sleep: i2c2-sleep-state {
 		bias-disable;
 	};
 
-	i2c3_default: i2c3-default-state {
+	blsp_i2c3_default: blsp-i2c3-default-state {
 		pins = "gpio10", "gpio11";
 		function = "blsp_i2c3";
 
@@ -234,7 +234,7 @@ i2c3_default: i2c3-default-state {
 		bias-disable;
 	};
 
-	i2c3_sleep: i2c3-sleep-state {
+	blsp_i2c3_sleep: blsp-i2c3-sleep-state {
 		pins = "gpio10", "gpio11";
 		function = "gpio";
 
@@ -242,7 +242,7 @@ i2c3_sleep: i2c3-sleep-state {
 		bias-disable;
 	};
 
-	i2c4_default: i2c4-default-state {
+	blsp_i2c4_default: blsp-i2c4-default-state {
 		pins = "gpio14", "gpio15";
 		function = "blsp_i2c4";
 
@@ -250,7 +250,7 @@ i2c4_default: i2c4-default-state {
 		bias-disable;
 	};
 
-	i2c4_sleep: i2c4-sleep-state {
+	blsp_i2c4_sleep: blsp-i2c4-sleep-state {
 		pins = "gpio14", "gpio15";
 		function = "gpio";
 
@@ -258,7 +258,7 @@ i2c4_sleep: i2c4-sleep-state {
 		bias-disable;
 	};
 
-	i2c5_default: i2c5-default-state {
+	blsp_i2c5_default: blsp-i2c5-default-state {
 		pins = "gpio18", "gpio19";
 		function = "blsp_i2c5";
 
@@ -266,7 +266,7 @@ i2c5_default: i2c5-default-state {
 		bias-disable;
 	};
 
-	i2c5_sleep: i2c5-sleep-state {
+	blsp_i2c5_sleep: blsp-i2c5-sleep-state {
 		pins = "gpio18", "gpio19";
 		function = "gpio";
 
@@ -274,7 +274,7 @@ i2c5_sleep: i2c5-sleep-state {
 		bias-disable;
 	};
 
-	i2c6_default: i2c6-default-state {
+	blsp_i2c6_default: blsp-i2c6-default-state {
 		pins = "gpio22", "gpio23";
 		function = "blsp_i2c6";
 
@@ -282,7 +282,7 @@ i2c6_default: i2c6-default-state {
 		bias-disable;
 	};
 
-	i2c6_sleep: i2c6-sleep-state {
+	blsp_i2c6_sleep: blsp-i2c6-sleep-state {
 		pins = "gpio22", "gpio23";
 		function = "gpio";
 

arch/arm64/boot/dts/qcom/msm8916.dtsi

@@ -1636,8 +1636,8 @@ blsp_i2c1: i2c@78b5000 {
 			dmas = <&blsp_dma 4>, <&blsp_dma 5>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&i2c1_default>;
-			pinctrl-1 = <&i2c1_sleep>;
+			pinctrl-0 = <&blsp_i2c1_default>;
+			pinctrl-1 = <&blsp_i2c1_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1653,8 +1653,8 @@ blsp_spi1: spi@78b5000 {
 			dmas = <&blsp_dma 4>, <&blsp_dma 5>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&spi1_default>;
-			pinctrl-1 = <&spi1_sleep>;
+			pinctrl-0 = <&blsp_spi1_default>;
+			pinctrl-1 = <&blsp_spi1_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1670,8 +1670,8 @@ blsp_i2c2: i2c@78b6000 {
 			dmas = <&blsp_dma 6>, <&blsp_dma 7>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&i2c2_default>;
-			pinctrl-1 = <&i2c2_sleep>;
+			pinctrl-0 = <&blsp_i2c2_default>;
+			pinctrl-1 = <&blsp_i2c2_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1687,8 +1687,8 @@ blsp_spi2: spi@78b6000 {
 			dmas = <&blsp_dma 6>, <&blsp_dma 7>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&spi2_default>;
-			pinctrl-1 = <&spi2_sleep>;
+			pinctrl-0 = <&blsp_spi2_default>;
+			pinctrl-1 = <&blsp_spi2_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1704,8 +1704,8 @@ blsp_i2c3: i2c@78b7000 {
 			dmas = <&blsp_dma 8>, <&blsp_dma 9>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&i2c3_default>;
-			pinctrl-1 = <&i2c3_sleep>;
+			pinctrl-0 = <&blsp_i2c3_default>;
+			pinctrl-1 = <&blsp_i2c3_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1721,8 +1721,8 @@ blsp_spi3: spi@78b7000 {
 			dmas = <&blsp_dma 8>, <&blsp_dma 9>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&spi3_default>;
-			pinctrl-1 = <&spi3_sleep>;
+			pinctrl-0 = <&blsp_spi3_default>;
+			pinctrl-1 = <&blsp_spi3_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1738,8 +1738,8 @@ blsp_i2c4: i2c@78b8000 {
 			dmas = <&blsp_dma 10>, <&blsp_dma 11>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&i2c4_default>;
-			pinctrl-1 = <&i2c4_sleep>;
+			pinctrl-0 = <&blsp_i2c4_default>;
+			pinctrl-1 = <&blsp_i2c4_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1755,8 +1755,8 @@ blsp_spi4: spi@78b8000 {
 			dmas = <&blsp_dma 10>, <&blsp_dma 11>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&spi4_default>;
-			pinctrl-1 = <&spi4_sleep>;
+			pinctrl-0 = <&blsp_spi4_default>;
+			pinctrl-1 = <&blsp_spi4_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1772,8 +1772,8 @@ blsp_i2c5: i2c@78b9000 {
 			dmas = <&blsp_dma 12>, <&blsp_dma 13>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&i2c5_default>;
-			pinctrl-1 = <&i2c5_sleep>;
+			pinctrl-0 = <&blsp_i2c5_default>;
+			pinctrl-1 = <&blsp_i2c5_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1789,8 +1789,8 @@ blsp_spi5: spi@78b9000 {
 			dmas = <&blsp_dma 12>, <&blsp_dma 13>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&spi5_default>;
-			pinctrl-1 = <&spi5_sleep>;
+			pinctrl-0 = <&blsp_spi5_default>;
+			pinctrl-1 = <&blsp_spi5_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1806,8 +1806,8 @@ blsp_i2c6: i2c@78ba000 {
 			dmas = <&blsp_dma 14>, <&blsp_dma 15>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&i2c6_default>;
-			pinctrl-1 = <&i2c6_sleep>;
+			pinctrl-0 = <&blsp_i2c6_default>;
+			pinctrl-1 = <&blsp_i2c6_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";
@@ -1823,8 +1823,8 @@ blsp_spi6: spi@78ba000 {
 			dmas = <&blsp_dma 14>, <&blsp_dma 15>;
 			dma-names = "tx", "rx";
 			pinctrl-names = "default", "sleep";
-			pinctrl-0 = <&spi6_default>;
-			pinctrl-1 = <&spi6_sleep>;
+			pinctrl-0 = <&blsp_spi6_default>;
+			pinctrl-1 = <&blsp_spi6_sleep>;
 			#address-cells = <1>;
 			#size-cells = <0>;
 			status = "disabled";