arrow_deca: add Arrow DECA platform #158
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
|
Also related: nmigen_boards/de10_lite.py seems to contain "F484" as part of the I do have the DE10 Lite on the way to confirm whether the current board definition works or not. |
StephanvanSchaik
changed the title
StephanvanSchaik
force-pushed
the
arrow_deca
branch
from
ec116b2
to
a1ac9ef
Compare
whitequark
reviewed
Jun 7, 2021
StephanvanSchaik
force-pushed
the
arrow_deca
branch
from
a1ac9ef
to
f3b605e
Compare
whitequark
reviewed
Jun 7, 2021
nmigen_boards/arrow_deca.py
Outdated
| # Configure the voltages of the I/O banks by appending the global assignments to the | ||
| # template. However, we create our own copy of the file templates before modifying them to | ||
| # avoid modifying the original. | ||
| templates = {**super().file_templates} |
There was a problem hiding this comment.
Something like this would be more elegant:
return {
**super().file_templates,
"{{name}}.qsf": r"""
...
"""Though this can be merged as-is.
StephanvanSchaik
force-pushed
the
arrow_deca
branch
from
f3b605e
to
8f3539a
Compare
whitequark
approved these changes
Jun 7, 2021
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.
Adds rudimentary support for the Arrow DECA board (see also the user manual) including the clocks, LEDs, switches, buttons and GPIO connectors as well as setitng the correct voltages for the I/O banks. The board features the Intel Altera MAX10 (10M50DAF484C6GES) FPGA.
There are more peripherals like the ADV7513 for HDMI TX, the TI TLV320AIC3254 24-bit audio codec, an SD card reader, etc. but this commit does not add support for those due to the varying voltages required for the different pins in a lot of cases. In addition, I would like to test any further additions thoroughly before committing them :).
To set the I/O bank voltages correctly, and to make sure Quartus doesn't complain (see also Tom Verbeure's blog post), we have to specify some additional set_global_assignment directives to the QSF file by overriding the file_templates() method.
I tested the board successfully with Blinky.
This also includes the fix discussed in pull request #156 to separate the "F484" part of the SKU out to the
packagevariable. This change has been successfully tested with Blinky before opening this pull request.