-
Notifications
You must be signed in to change notification settings - Fork 201
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
Added support for Digilent Genesys 2 #1961
base: master
Are you sure you want to change the base?
Conversation
Will you keep maintaining this, including porting the clocking changes #1939 / m-labs/misoc#129 ? |
I believe we can support it both in TechnoSystem and on WUT (both sites have boards). We can wait with this merge until both #1939 and m-labs/misoc#129 are approved and merged so I can port changes. |
@kaolpr Sync RTIO is merged now. Is this PR ready? |
I'll update to latest master and will be ready tomorrow. |
Great. Corresponding MiSoC PR is also OK? |
@sbourdeauducq Sorry, I didn't manage to test it on hardware before my winter vacation. I'll verify operation for this PR and m-labs/misoc#130 just after coming back on Feb 21st. |
@sbourdeauducq I've made updates to both to this PR and MiSoC. Please take a look at And sorry that it took so long. |
We don't have this anymore since synchronous RTIO clocking. Your change breaks Kasli and KC705. |
Can I use |
Probably has_si5324. And please test on Kasli and KC705. |
Modified to use |
ARTIQ Pull Request
Description of Changes
Adds support for Digilent Genesys 2 target.
Related PRs:
Related Issue
Type of Changes
Steps (Choose relevant, delete irrelevant before submitting)
All Pull Requests
git commit --signoff
, see copyright).Code Changes
flake8
to check code style (follow PEP-8 style).flake8
has issues with parsing Migen/gateware code, ignore as necessary.TestVariant
to blinkuser_led
from ARTIQ experiment.Documentation Changes
Git Logistics
git rebase --interactive
). Merge/squash/fixup commits that just fix or amend previous commits. Remove unintended changes & cleanup. See tutorial.git show
). Format:Licensing
See copyright & licensing for more info.
ARTIQ files that do not contain a license header are copyrighted by M-Labs Limited and are licensed under LGPLv3+.