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

icetime doesn't recognize new pcf options #207

Open
Forty-Bot opened this issue Mar 21, 2019 · 2 comments
Open

icetime doesn't recognize new pcf options #207

Forty-Bot opened this issue Mar 21, 2019 · 2 comments

Comments

@Forty-Bot
Copy link

Forty-Bot commented Mar 21, 2019

icetime can't parse new pcf options such as -nowarn, -pullup, and -pullup-resistor. It fails with the message:

// Reading input .pcf file..
icetime: icetime.cc:230: void read_pcf(const char*): Assertion `args.size() == 2' failed.

This occurs on Arch Linux with icestorm r684.fa1c932-1

@Arvind-Srinivasan
Copy link

Arvind-Srinivasan commented Jul 14, 2020

From comparing the code that reads the pcf file in icetime.cc to its counterpart in nextpnr, there is some sort of mismatch in what they expect?

@Forty-Bot
Copy link
Author

Yes. Both arachne-pnr and nextpnr accept the options -nowarn, --warn-no-port, and -pullup. nextpnr also supports -pullup-resistor. icetime only accepts --warn-no-port.

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

No branches or pull requests

2 participants