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

ice40: simulate SPRAM poweron delay in cells_sim #2471

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sapphire-arches
Copy link

Makes the SB_SPRAM simulation model slightly more accurate, including a coarse simulation of how the block behaves when modifying the various power/sleep state control lines.

I'd also like to write up some tests for this, but the actual modifications to the cell simulation should be ready for review.

@smunaut
Copy link
Contributor

smunaut commented Dec 3, 2020

Mmm, I don't think that logic is correct. If you release power poweroff and then release sleep slightly after, you wouldn't see the 41 ns delay, because state would go POWEROFF -> SLEEP -> NORMAL.

Makes the SB_SPRAM simulation model slightly more accurate, including a coarse
simulation of how the block behaves when modifying the various power/sleep
state control lines.
@sapphire-arches
Copy link
Author

I've reworked this to use a handful of counters instead of a state machine, which was getting extremely unwieldy when trying to account for the various ways the power control lines can interact.

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

Successfully merging this pull request may close these issues.

None yet

2 participants