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

[enh] use Misc() properties in trellis' PCF #221

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chmousset
Copy link

Currently Misc() properties are ignored when generating the pcf constraint file for trellis.

To keep compatibility with existing platforms, Lattice's Diamond properties are translated to nextpnr's.
Only PULLUP has been successfully tested (on icestick), but PULLUP_RESISTOR should be supported aswell, based on document found here (p91)

the IOStandard() properties are not supported by nextpnr, therefore nothing is done with them.

@sbourdeauducq
Copy link
Member

Hmm, the idea behind Misc() is to allow toolchain-specific attributes that are just passed as-is.
How about adding Pullup() instead (and migrating existing code)?

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.

None yet

2 participants