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

Update altera build #2006

Merged
merged 2 commits into from
Jul 4, 2024

Conversation

trabucayre
Copy link
Collaborator

This PR contains 2 commits:

With (at least) Quartus Prime Pro Edition 24.1 quartus_map becomes obsolete and must be replaced by quartus_syn, otherwise build fails with:

+ quartus_map --read_settings_files=on --write_settings_files=off xxx -c xxx
Internal Error: Sub-system: QSYN, File: /quartus/synth/qsyn/qsyn_cmd.cpp, Line: 869
Command quartus_map is no longer supported. Please use quartus_syn.

The first commit address this issue by updating build/altera/platform and build/altera/quartus to adds fills_args, get_argdict to allows users selecting between these two tools with --synth-tool (default behavior is to keep quartus_map for backward compatibility with previous Quartus releases).

For some peripherals a QSYS script is required, and sometime this one refers to .ip files. Currently Add IPs section (using platform.ips is hardcoded to QSYS_FILE). The second commit address this issue by using file extension to create the XXX_FILE.

@enjoy-digital enjoy-digital merged commit d838a9c into enjoy-digital:master Jul 4, 2024
1 check passed
@enjoy-digital
Copy link
Owner

Thanks @trabucayre.

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