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

Fix minor issues with Gowin vendor #1

Open
wants to merge 12 commits into
base: support-gw1n-9c
Choose a base branch
from

Conversation

psentee
Copy link

@psentee psentee commented Jun 1, 2023

Small PR on top of amaranth-lang#734 – I can confirm it works with my Tang Nano 1K (and will send a separate PR on top of amaranth-lang/amaranth-boards#208 once I clean it up). Thank you!

  • add missing osc_freq property (used in default_clk_constraint when sys_clk0 is used)
  • compute --freq value for nextpnr-gowin instead of hardcoding 27 MHz
  • remove -v gowin_pack flag (gowin_pack doesn't have that flag)

bl0x and others added 12 commits March 29, 2023 21:00
… oscillators. Fix comment style in generated cst and sdc files.
…n_command_templates to be a list instead of dict.
…Introduced subseries property to be able to specify in the board platform.
- add missing osc_freq property (used in default_clk_constraint when
  sys_clk0 is used)
- compute --freq value for nextpnr-gowin instead of hardcoding 27 MHz
- remove -v gowin_pack flag (gowin_pack doesn't have that flag)
@bl0x bl0x force-pushed the support-gw1n-9c branch from 9c613d6 to f63c120 Compare August 3, 2023 19:52
@whitequark whitequark force-pushed the support-gw1n-9c branch 2 times, most recently from 512d8bf to efc0860 Compare August 8, 2023 12:32
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.

2 participants