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

Minimal spi master #203

Closed
wants to merge 48 commits into from
Closed

Minimal spi master #203

wants to merge 48 commits into from

Conversation

secworks
Copy link
Contributor

@secworks secworks commented Apr 9, 2024

This PR adds a minimal SPI master to the application_fpga. This allows applications to access the Flash memory on the Tkey. The SPI master basically supports generating a clock, control the chip select, and send and receive a byte. SW much implement the functionality required to send commands and receive responses. Updates to tkey_libs will add this functionality.

Currently the SPI master is enabled in FPGA builds by including the flag symbol INCLUDE_SPI_MASTER to Yosys in the Makefile. The Makefile does not yet change pin constraint file. Instead the constraint file now always includes pins for the SPI interface. We would rather have a different target for builds with the SPI master, and have a specific constraint file for builds with our without the SPI master.

The commits should probably be squashed. And cleaned up.

secworks and others added 30 commits April 3, 2023 09:57
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Daniel Lublin <daniel@lublin.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Daniel Lublin <daniel@lublin.se>
Signed-off-by: Daniel Lublin <daniel@lublin.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
debug build of SPI master simulation
Winbond Flash model

Signed-off-by: Joachim Strömbergson <joachim@assured.se>
and protocol, the memory type, and where to find the
datasheet form the memory.

Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
with the memory model attached to the DUT. We can read
out device info and dump memory

Signed-off-by: Joachim Strömbergson <joachim@assured.se>
in README on how to retrieve the Winbond memory model

Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Winbond flash memory model to be able to build and run the
SPI-master simulation

Signed-off-by: Joachim Strömbergson <joachim@assured.se>
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
@secworks secworks added enhancement New feature or request fpga Related to the FPGA design labels Apr 9, 2024
@secworks secworks self-assigned this Apr 9, 2024
Signed-off-by: Joachim Strömbergson <joachim@assured.se>
@dehanj dehanj mentioned this pull request Apr 18, 2024
@dehanj
Copy link
Member

dehanj commented Apr 19, 2024

See #207 instead

@dehanj dehanj closed this Apr 19, 2024
@dehanj dehanj deleted the minimal_spi_master branch June 12, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fpga Related to the FPGA design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants