Pocket Beagle SPI1 ChipSelect on silkscreen is not CS0 but CS1

Hi,
Little heads-up: I just spent an hour or so to realize that the simple ‘CS’ (ChipSelect) printed on the silkscreen for SPI1 bus on the PocketBeagle is not CS0 as I expected, but CS1. So this is actually bus /dev/spidev2.1

Not sure if this was intended, but if it is, it should probably be marked on the silk-screen as CS1 to avoid future confusion; I can send a pull request for the board (in general, even CS0 should be marked as such)

If it is not intended, the am335x-pocketbeagle.dts would need to be changed before capes start assuming this is CS1 (around line 389 Jason).

Cheers,
Henner.

HI Henner,

I've noticed this to while doing the overlays:

SPI1 CS1 is used as CS

https://github.com/beagleboard/bb.org-overlays/blob/master/src/arm/PB-SPI1-ETH-CLICK.dts#L83

SPI0 CS0 is used as CS

https://github.com/beagleboard/bb.org-overlays/blob/master/src/arm/PB-SPI0-ETH-CLICK.dts#L83

I haven't looked to closely, (just let my automated script setup the
pinmux's).. But if SPI1 (CS0) and SPI0 (CS1) is not on the header, we
should remove them from the base tree:

Regards,

Yes, this would certainly less confusing to leave out the
non-available chip-selects in the /dev/spidev* devices.

Is there a reason why SPI0 shows up as /dev/spidev1.* and SPI1 as
/dev/spidev2.* ? This does look like an off-by-one. Is this
intentional ?

Thanks,
  Henner.

Robert:

Are the default device tree pinmux connections for the PocketBeagle the functions
named in the silkscreen labels on the bottom of the PocketBeagle?

So far this appears to be the case. Any exceptions?

— Graham

Robert:

Are the default device tree pinmux connections for the PocketBeagle the
functions
named in the silkscreen labels on the bottom of the PocketBeagle?

Yes,

So far this appears to be the case. Any exceptions?

If not, let me know. :wink: So we can fix it..

Regards,

Robert:

Are the default device tree pinmux connections for the PocketBeagle the
functions
named in the silkscreen labels on the bottom of the PocketBeagle?

Yes,

So far this appears to be the case. Any exceptions?

If not, let me know. :wink: So we can fix it..

48 should be labeled 44.

SPI1 is CS1, not CS0 (silk screen doesn�ft say).

Actually, my pull request to the board fixed the chip-select printing:

https://github.com/beagleboard/pocketbeagle/pull/9
https://github.com/beagleboard/pocketbeagle/commit/1d96a30c13ed9ab13bd1372aa87de865339e5499

-h