Icebox support for ice40 LM #139
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Awesome! However, a minor nitpick, the chipdb should not be included in the repo, but gitignored like the other chipdbs to avoid the repo ballooning in size (changes to routing may also cause large changes to the chipdb.) Thanks for your work! |
+1 @awygle can you please rebase this PR so that chipdb-lm4k.txt is not even part of the git history? Thanks. |
Config SPI pins weren't present in ioctrl_lm4k.sh
|
Fixed! Sorry - saw the other DBs not in git status and read that as "committed" not "gitignored" :) |
|
Thanks for fixing the db issue. Looking at the diffs now.. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is Icebox support for the ice40 LM. It's successfully generated a chipdb and abitstream with arachne-pnr. Hardware verification is in work.