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
Add Alchitry Au board definition #48
Conversation
Looks good to me.
That's fine--we have a bunch of untested peripherals on most boards. For now I think it doesn't make sense to be super strict about it, especially given that the effort to fully functionally verify the pinouts can be quite large.
We have a few missing peripherals on a few boards, too. It's not great, but it's not a blocker, especially given that there are quite a few boards with an absolutely massive number of peripherals and pins used. |
nmigen_boards/alchitry_au.py
Outdated
] | ||
|
||
connectors = [ | ||
Connector("bank", 0, "T8 T7 T5 R5 R8 P8 L2 L3 J1 K1 H1 H2 G1 G2 K5 E6" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is unfortunately wrong because you ended up defining a pin "E6T10" (there's no space at the end). I'll fix this in this PR, and I'll try to fix this properly in nmigen.build (probably allow using multiple string arguments).
@@ -0,0 +1,85 @@ | |||
import os | |||
import subprocess | |||
import shutil |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've moved the shutil import here since nMigen already depends on it, anyway.
"B1 C1 C2 C3 D3 E3 C4 D4 G4 G5 E5 F5 D5 D6 C6 C7 "), | ||
Connector("bank", 2, "T13 R13 T12 R12 R11 R10 N2 N3 P3 P4 M4 L4 N4 M5 L5 P5 " | ||
"P11 P10 N12 N11 P13 N13 M1 M2 P1 N1 R1 R2 T2 R3 T3 T4 "), | ||
Connector("bank", 3, "L14 L13 M12 N16 R16 R15 P14 M15 P16 P15 - - - - - - " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You did define non-IO pins appropriately here; the -
marker exists exactly for this case.
Thanks! |
Copy-Paste from m-labs/nmigen-boards#44 follows:
Notes:
nmigen uses to locate Yosys/Vivado but I'm not sure if that's the right approach
LED's. I was debating adding a resource for that, similar to how the icebreaker pmod's are defined, but ultimately decided I was gambling whether I'd finish that pin definition before running out of motivation. Let me know if you'd like that pin definition in this PR and I'll try to add it.