Re: [PATCH] arm: dts: am335x-pocketbeagle: add gpio-line-names

2020-06-18 Thread Drew Fustini
On Thu, Jun 18, 2020 at 10:03:45AM -0700, Tony Lindgren wrote:
> * Drew Fustini  [200617 17:10]:
> > Tony - does this look ok for 5.9?
> 
> Yes looks OK to me.
> 
> Just wondering, are the line with "NA" not used internally either?
> If the "NA" lines are used internally, we should probably use
> "Reserved" or "Internal" or something like that to avoid later
> on having to patch them with internal device names..

There are many more 'no connects' as the PocketBeagle is much simpler.

There are 12 SYSBOOT pins which just go to fixed pull-up and pull-down
resistors.  I'll change those from "[NC]" to "[SYSBOOT]".

Also, after going through all the enteries again, I noticed 4 lines that
I mislabeled.

I will post a v2.

> 
> > If so, I might start making other variants like BeagleBone Blue and
> > BeagleBone {Green,Black} Wireless and submit those when ready.
> 
> OK yeah makes sense.
> 

thanks,
drew


Re: [PATCH] arm: dts: am335x-pocketbeagle: add gpio-line-names

2020-06-18 Thread Tony Lindgren
* Drew Fustini  [200617 17:10]:
> Tony - does this look ok for 5.9?

Yes looks OK to me.

Just wondering, are the line with "NA" not used internally either?
If the "NA" lines are used internally, we should probably use
"Reserved" or "Internal" or something like that to avoid later
on having to patch them with internal device names..

> If so, I might start making other variants like BeagleBone Blue and
> BeagleBone {Green,Black} Wireless and submit those when ready.

OK yeah makes sense.

Regards,

Tony


Re: [PATCH] arm: dts: am335x-pocketbeagle: add gpio-line-names

2020-06-17 Thread Drew Fustini
On Tue, Jun 09, 2020 at 04:25:04PM +0200, Drew Fustini wrote:
> The BeagleBoard.org PocketBeagle has P1 and P2 headers [0] which expose
> many of the TI AM3358 SoC balls to stacking expansion boards called 
> "capes", or to other external connections like jumper wires connected
> to a breadboard.
> 
> Note: the AM3358 die is actually embedded inside of the OSD335x-SM
> System-in-Package (SiP) [1] but that is irrelevant to the gpio driver.
> 
> Many of the P1 and P2 header pins can muxed to a GPIO line.  The
> gpio-line-names describe which P1 or P2 pin that line goes to and the
> default mux for that P1 or P2 pin if it is not GPIO.
> 
> Some GPIO lines are named "[NC]" as the corresponding balls are not
> routed to anything on the PCB.
> 
> The goal for these names is to make it easier for a user viewing the
> output of gpioinfo to determine which P1 or P2 pin is connected to a
> GPIO line.  The output of gpioinfo on a PocketBeagle would be:
> 
> gpiochip0 - 32 lines:
>   line   0:   "[NC]"   unused   input  active-high 
>   line   1:   "[NC]"   unused   input  active-high 
>   line   2: "P1.08 [SPI0_CLK]" unused input active-high 
>   line   3: "P1.10 [SPI0_MISO]" unused input active-high 
>   line   4: "P1.12 [SPI0_MOSI]" unused input active-high 
>   line   5: "P1.06 [SPI0_CS]" unused input active-high 
>   line   6:  "[MMC0_CD]" "cd"   input   active-low [used]
>   line   7: "P2.29 [SPI1_CLK]" unused input active-high 
>   line   8:   "[NC]"   unused   input  active-high 
>   line   9:   "[NC]"   unused   input  active-high 
>   line  10:   "[NC]"   unused   input  active-high 
>   line  11:   "[NC]"   unused   input  active-high 
>   line  12: "P1.26 [I2C2_SDA]" unused input active-high 
>   line  13: "P1.28 [I2C2_SCL]" unused input active-high 
>   line  14: "P2.11 [I2C1_SDA]" unused input active-high 
>   line  15: "P2.09 [I2C1_SCL]" unused input active-high 
>   line  16:   "[NC]"   unused   input  active-high 
>   line  17:   "[NC]"   unused   input  active-high 
>   line  18:   "[NC]"   unused   input  active-high 
>   line  19: "P2.31 [SPI1_CS]" unused input active-high 
>   line  20: "P1.20 [PRU0.16]" unused input active-high 
>   line  21:   "[NC]"   unused   input  active-high 
>   line  22:   "[NC]"   unused   input  active-high 
>   line  23:  "P2.03"   unused   input  active-high 
>   line  24:   "[NC]"   unused   input  active-high 
>   line  25:   "[NC]"   unused   input  active-high 
>   line  26:  "P1.34"   unused   input  active-high 
>   line  27:  "P2.19"   unused   input  active-high 
>   line  28:   "[NC]"   unused   input  active-high 
>   line  29:   "[NC]"   unused   input  active-high 
>   line  30: "P2.05 [UART4_RX]" unused input active-high 
>   line  31: "P2.07 [UART4_TX]" unused input active-high 
> gpiochip1 - 32 lines:
>   line   0:   "[NC]"   unused   input  active-high 
>   line   1:   "[NC]"   unused   input  active-high 
>   line   2:   "[NC]"   unused   input  active-high 
>   line   3:   "[NC]"   unused   input  active-high 
>   line   4:   "[NC]"   unused   input  active-high 
>   line   5:   "[NC]"   unused   input  active-high 
>   line   6: "P1.06 [SPI0_CS]" unused input active-high 
>   line   7: "P1.06 [SPI0_CS]" unused input active-high 
>   line   8: "P2.27 [SPI1_MISO]" unused input active-high 
>   line   9: "P2.25 [SPI1_MOSI]" unused input active-high 
>   line  10: "P1.32 [UART0_RX]" unused input active-high 
>   line  11: "P1.30 [UART0_TX]" unused input active-high 
>   line  12:  "P2.24"   unused   input  active-high 
>   line  13:  "P2.33"   unused   input  active-high 
>   line  14:  "P2.22"   unused   input  active-high 
>   line  15:  "P2.18"   unused   input  active-high 
>   line  16:   "[NC]"   unused   input  active-high 
>   line  17:   "[NC]"   unused   input  active-high 
>   line  18: "P2.01 [PWM1A]" unused input active-high 
>   line  19:   "[NC]"   unused   input  active-high 
>   line  20:  "P2.10"   unused   input  active-high 
>   line  21: "[user led 0]" "beaglebone:green:usr0" output active-high 
> [used]
>   line  22: "[user led 1]" "beaglebone:green:usr1" output active-high 
> [used]
>   line  23: "[user led 2]" "beaglebone:green:usr2" output active-high 
> [used]
>   line  24: "[user led 3]" "beaglebone:green:usr3" output active-high 
> [used]
>   line  25:  "P2.06"   unused   input  active-high 
>   line  26:  "P2.04"   unused   input  active-high 
>   line  27:  "P2.02"   unused   input  active-high 
> 

[PATCH] arm: dts: am335x-pocketbeagle: add gpio-line-names

2020-06-09 Thread Drew Fustini
The BeagleBoard.org PocketBeagle has P1 and P2 headers [0] which expose
many of the TI AM3358 SoC balls to stacking expansion boards called 
"capes", or to other external connections like jumper wires connected
to a breadboard.

Note: the AM3358 die is actually embedded inside of the OSD335x-SM
System-in-Package (SiP) [1] but that is irrelevant to the gpio driver.

Many of the P1 and P2 header pins can muxed to a GPIO line.  The
gpio-line-names describe which P1 or P2 pin that line goes to and the
default mux for that P1 or P2 pin if it is not GPIO.

Some GPIO lines are named "[NC]" as the corresponding balls are not
routed to anything on the PCB.

The goal for these names is to make it easier for a user viewing the
output of gpioinfo to determine which P1 or P2 pin is connected to a
GPIO line.  The output of gpioinfo on a PocketBeagle would be:

gpiochip0 - 32 lines:
line   0:   "[NC]"   unused   input  active-high 
line   1:   "[NC]"   unused   input  active-high 
line   2: "P1.08 [SPI0_CLK]" unused input active-high 
line   3: "P1.10 [SPI0_MISO]" unused input active-high 
line   4: "P1.12 [SPI0_MOSI]" unused input active-high 
line   5: "P1.06 [SPI0_CS]" unused input active-high 
line   6:  "[MMC0_CD]" "cd"   input   active-low [used]
line   7: "P2.29 [SPI1_CLK]" unused input active-high 
line   8:   "[NC]"   unused   input  active-high 
line   9:   "[NC]"   unused   input  active-high 
line  10:   "[NC]"   unused   input  active-high 
line  11:   "[NC]"   unused   input  active-high 
line  12: "P1.26 [I2C2_SDA]" unused input active-high 
line  13: "P1.28 [I2C2_SCL]" unused input active-high 
line  14: "P2.11 [I2C1_SDA]" unused input active-high 
line  15: "P2.09 [I2C1_SCL]" unused input active-high 
line  16:   "[NC]"   unused   input  active-high 
line  17:   "[NC]"   unused   input  active-high 
line  18:   "[NC]"   unused   input  active-high 
line  19: "P2.31 [SPI1_CS]" unused input active-high 
line  20: "P1.20 [PRU0.16]" unused input active-high 
line  21:   "[NC]"   unused   input  active-high 
line  22:   "[NC]"   unused   input  active-high 
line  23:  "P2.03"   unused   input  active-high 
line  24:   "[NC]"   unused   input  active-high 
line  25:   "[NC]"   unused   input  active-high 
line  26:  "P1.34"   unused   input  active-high 
line  27:  "P2.19"   unused   input  active-high 
line  28:   "[NC]"   unused   input  active-high 
line  29:   "[NC]"   unused   input  active-high 
line  30: "P2.05 [UART4_RX]" unused input active-high 
line  31: "P2.07 [UART4_TX]" unused input active-high 
gpiochip1 - 32 lines:
line   0:   "[NC]"   unused   input  active-high 
line   1:   "[NC]"   unused   input  active-high 
line   2:   "[NC]"   unused   input  active-high 
line   3:   "[NC]"   unused   input  active-high 
line   4:   "[NC]"   unused   input  active-high 
line   5:   "[NC]"   unused   input  active-high 
line   6: "P1.06 [SPI0_CS]" unused input active-high 
line   7: "P1.06 [SPI0_CS]" unused input active-high 
line   8: "P2.27 [SPI1_MISO]" unused input active-high 
line   9: "P2.25 [SPI1_MOSI]" unused input active-high 
line  10: "P1.32 [UART0_RX]" unused input active-high 
line  11: "P1.30 [UART0_TX]" unused input active-high 
line  12:  "P2.24"   unused   input  active-high 
line  13:  "P2.33"   unused   input  active-high 
line  14:  "P2.22"   unused   input  active-high 
line  15:  "P2.18"   unused   input  active-high 
line  16:   "[NC]"   unused   input  active-high 
line  17:   "[NC]"   unused   input  active-high 
line  18: "P2.01 [PWM1A]" unused input active-high 
line  19:   "[NC]"   unused   input  active-high 
line  20:  "P2.10"   unused   input  active-high 
line  21: "[user led 0]" "beaglebone:green:usr0" output active-high 
[used]
line  22: "[user led 1]" "beaglebone:green:usr1" output active-high 
[used]
line  23: "[user led 2]" "beaglebone:green:usr2" output active-high 
[used]
line  24: "[user led 3]" "beaglebone:green:usr3" output active-high 
[used]
line  25:  "P2.06"   unused   input  active-high 
line  26:  "P2.04"   unused   input  active-high 
line  27:  "P2.02"   unused   input  active-high 
line  28:  "P2.08"   unused   input  active-high 
line  29:   "[NC]"   unused   input