Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

spi_* signals in Pin Description table not appearing in Pinout table #70

Open
Piyush3dB opened this issue Aug 5, 2021 · 1 comment
Open

Comments

@Piyush3dB
Copy link

I am referring to the documentation page at Caravel Pinout Description [ https://caravel-harness.readthedocs.io/en/latest/pinout.html ]

The attached diagram shows the 4 signals in the pin description table

spi_sdo
spi_sck
spi_csb
spi_sdi

do not appear in the pinout table. Is this an error in the pinout table?

Caravel_Pinout

@RTimothyEdwards
Copy link
Contributor

That appears to be a naming redundancy in the pin description list. If you look at the left-hand column, the signals are duplicated with slightly different names; e.g., pin E8 is listed as both CSB and spi_csb. The first refers to the housekeeping SPI and the second refers to the SPI master. For convenience, they share the same pins.

Since the pinout table is supposed to list all signal names associated with each pin, that table should be updated to add the spi_ names to those four pins.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants