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

Possible spi_fifo.sv bug #1139

Open
rosethompson opened this issue Nov 25, 2024 · 0 comments
Open

Possible spi_fifo.sv bug #1139

rosethompson opened this issue Nov 25, 2024 · 0 comments

Comments

@rosethompson
Copy link
Contributor

I believe there is a bug in the fifo by code inspection and FPGA debugging. It appears possible to stall a load of the RXDATA register and potentially read out the head of the fifo and then immediately read out the next value. The fifo works by combinationally reading out the head and sampling this data on the cycle before PSEL and PENABLE are high. However if the the load is stalled the head pointer is still incremented so the fifo changes the output data while the instruction is stalled. Several times while debugging the FPGA zsbl I found what appeared to be reading the wrong value from the FIFO, but after fixing the zsbl issues I haven't been able to reproduce this error.

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

1 participant