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

Adding hw-1.0 build test and FMC size fix #1940

Merged

Conversation

nquarton
Copy link
Contributor

@nquarton nquarton commented Feb 4, 2025

Adding check that runs during build and test for FMC and RT build for hw-1.0
Adjusting FMC logging to fix size issue on hw-1.0
Adding compile-time check to limit runtime (FMC+FW) size to 118kB (128-10kB for the manifest)

@nquarton nquarton changed the title Adding hw-1.0 build test for PR CI Adding hw-1.0 build test and FMC size fix Feb 4, 2025
@nquarton nquarton force-pushed the hw-1.0_fmc_build_fix_and_ci_check branch from db03b08 to 1adf6c7 Compare February 4, 2025 18:12
@rusty1968
Copy link
Contributor

We should add a compile time assertion to constrain the combined size of FMC and RT to 118kb as per Jordan's comment.

@nquarton
Copy link
Contributor Author

nquarton commented Feb 4, 2025

We should add a compile time assertion to constrain the combined size of FMC and RT to 118kb as per Jordan's comment.

That is probably a good idea. I'm not quite sure how to go about doing that though. Do you have an idea?

@nquarton nquarton force-pushed the hw-1.0_fmc_build_fix_and_ci_check branch from 1adf6c7 to f97396d Compare February 4, 2025 21:35
@nquarton nquarton force-pushed the hw-1.0_fmc_build_fix_and_ci_check branch from f97396d to 64dd25e Compare February 4, 2025 23:46
@nquarton
Copy link
Contributor Author

nquarton commented Feb 4, 2025

We should add a compile time assertion to constrain the combined size of FMC and RT to 118kb as per Jordan's comment.

That is probably a good idea. I'm not quite sure how to go about doing that though. Do you have an idea?

This has been addressed now

@nquarton nquarton marked this pull request as ready for review February 5, 2025 00:24
@jhand2 jhand2 merged commit a476f51 into chipsalliance:main Feb 5, 2025
11 checks passed
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.

4 participants