ARM: dts: am335x-pocketbeagle: unique gpio-line-names
authorDrew Fustini <drew@beagleboard.org>
Wed, 27 Jan 2021 00:03:03 +0000 (16:03 -0800)
committerTony Lindgren <tony@atomide.com>
Wed, 24 Mar 2021 16:31:08 +0000 (18:31 +0200)
Based on linux-gpio discussion [1], it is best practice to make the
gpio-line-names unique. Generic names like "[ethernet]" are replaced
with the name of the unique signal on the AM3358 SoC ball corresponding
to the gpio line. "[NC]" is also renamed to the standard "NC" name to
represent "not connected".

[1] https://lore.kernel.org/linux-gpio/20201216195357.GA2583366@x1/

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Reviewed-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Signed-off-by: Drew Fustini <drew@beagleboard.org>
Signed-off-by: Tony Lindgren <tony@atomide.com>
arch/arm/boot/dts/am335x-pocketbeagle.dts

index d526c59..209cdd1 100644 (file)
 
 &gpio0 {
        gpio-line-names =
-               "[NC]",
-               "[NC]",
+               "NC",
+               "NC",
                "P1.08 [SPI0_CLK]",
                "P1.10 [SPI0_MISO]",
                "P1.12 [SPI0_MOSI]",
                "P1.06 [SPI0_CS]",
                "[MMC0_CD]",
                "P2.29 [SPI1_CLK]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
+               "[SYSBOOT 12]",
+               "[SYSBOOT 13]",
+               "[SYSBOOT 14]",
+               "[SYSBOOT 15]",
                "P1.26 [I2C2_SDA]",
                "P1.28 [I2C2_SCL]",
                "P2.11 [I2C1_SDA]",
                "P2.09 [I2C1_SCL]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
+               "NC",
+               "NC",
+               "NC",
                "P2.31 [SPI1_CS]",
                "P1.20 [PRU0.16]",
-               "[NC]",
-               "[NC]",
+               "NC",
+               "NC",
                "P2.03",
-               "[NC]",
-               "[NC]",
+               "NC",
+               "NC",
                "P1.34",
                "P2.19",
-               "[NC]",
-               "[NC]",
+               "NC",
+               "NC",
                "P2.05 [UART4_RX]",
                "P2.07 [UART4_TX]";
 };
 
 &gpio1 {
        gpio-line-names =
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
                "P2.25 [SPI1_MOSI]",
                "P1.32 [UART0_RX]",
                "P1.30 [UART0_TX]",
                "P2.33",
                "P2.22",
                "P2.18",
-               "[NC]",
-               "[NC]",
+               "NC",
+               "NC",
                "P2.01 [PWM1A]",
-               "[NC]",
+               "NC",
                "P2.10",
                "[USR LED 0]",
                "[USR LED 1]",
                "P2.04",
                "P2.02",
                "P2.08",
-               "[NC]",
-               "[NC]",
-               "[NC]";
+               "NC",
+               "NC",
+               "NC";
 };
 
 &gpio2 {
        gpio-line-names =
                "P2.20",
                "P2.17",
-               "[NC]",
-               "[NC]",
-               "[NC]",
+               "NC",
+               "NC",
+               "NC",
                "[EEPROM_WP]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[SYSBOOT]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
+               "[SYSBOOT 0]",
+               "[SYSBOOT 1]",
+               "[SYSBOOT 2]",
+               "[SYSBOOT 3]",
+               "[SYSBOOT 4]",
+               "[SYSBOOT 5]",
+               "[SYSBOOT 6]",
+               "[SYSBOOT 7]",
+               "[SYSBOOT 8]",
+               "[SYSBOOT 9]",
+               "[SYSBOOT 10]",
+               "[SYSBOOT 11]",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
                "P2.35 [AIN5]",
                "P1.02 [AIN6]",
                "P1.35 [PRU1.10]",
 
 &gpio3 {
        gpio-line-names =
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
                "[I2C0_SDA]",
                "[I2C0_SCL]",
-               "[JTAG]",
-               "[JTAG]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
+               "[JTAG EMU0]",
+               "[JTAG EMU1]",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
                "P1.03 [USB1]",
                "P1.36 [PWM0A]",
                "P1.33 [PRU0.1]",
                "P2.34 [PRU0.5]",
                "P2.28 [PRU0.6]",
                "P1.29 [PRU0.7]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]",
-               "[NC]";
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC",
+               "NC";
 };
 
 &am33xx_pinmux {