Skip to contentSkip to conversation

arm64: dts: Unify pinctrl-single pin group nodes for altera

PATCH Applied Tony Lindgren 2 participants 2 messages May 23, 2023 – May 24, 2023

Subsystems (2)
Device tree bindings Arm/socfpga architecture
lore
Share & tools
lore

Tony Lindgren

author
#0 lore raw

PATCH 1/1 arm64: dts: Unify pinctrl-single pin group nodes for altera

We want to unify the pinctrl-single pin group nodes to use naming "pins". Otherwise non-standad pin group names will add make dtbs checks errors when the pinctrl-single yaml binding gets merged.

Cc Conor Dooley <[email protected]>
Cc Krzysztof Kozlowski <[email protected]>
Cc Rob Herring <[email protected]>
Signed-off-by Tony Lindgren <[email protected]>
---
 arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts +2 -2
@@ -66,14 +66,14 @@ sdmmca-ecc@ff8c8c00 {
 };
 
 &pinctrl0 {
-	i2c1_pmx_func: i2c1-pmx-func {
+	i2c1_pmx_func: i2c1-pins {
 		pinctrl-single,pins = <
 			0x78   0x4   /* I2C1_SDA (IO6-B) PIN30SEL) */
 			0x7c   0x4   /* I2C1_SCL (IO7-B) PIN31SEL */
 		>;
 	};
 
-	i2c1_pmx_func_gpio: i2c1-pmx-func-gpio {
+	i2c1_pmx_func_gpio: i2c1-gpio-pins {
 		pinctrl-single,pins = <
 			0x78   0x8   /* I2C1_SDA (IO6-B) PIN30SEL) */
 			0x7c   0x8   /* I2C1_SCL (IO7-B) PIN31SEL */
Signature · 1 line
2.40.1
maintainer Applied
#1 lore raw
Quoted 22 lines · Tony Lindgren

We want to unify the pinctrl-single pin group nodes to use naming "pins". Otherwise non-standad pin group names will add make dtbs checks errors when the pinctrl-single yaml binding gets merged.

Cc Conor Dooley <[email protected]>
Cc Krzysztof Kozlowski <[email protected]>
Cc Rob Herring <[email protected]>
Signed-off-by Tony Lindgren <[email protected]>
---
  arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts +1 -1
@@ -66,14 +66,14 @@ sdmmca-ecc@ff8c8c00 {
  };
 
  &pinctrl0 {
-	i2c1_pmx_func: i2c1-pmx-func {
+	i2c1_pmx_func: i2c1-pins {
  		pinctrl-single,pins = <

arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts sdmmca-ecc@ff8c8c00 { · line 79

Quoted 10 lines
arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts +1 -1
  			0x78   0x4   /* I2C1_SDA (IO6-B) PIN30SEL) */
  			0x7c   0x4   /* I2C1_SCL (IO7-B) PIN31SEL */
  		>;
  	};
 
-	i2c1_pmx_func_gpio: i2c1-pmx-func-gpio {
+	i2c1_pmx_func_gpio: i2c1-gpio-pins {
  		pinctrl-single,pins = <
  			0x78   0x8   /* I2C1_SDA (IO6-B) PIN30SEL) */
  			0x7c   0x8   /* I2C1_SCL (IO7-B) PIN31SEL */

Applied!

Thanks,
Dinh