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

RFNoC "addsub" Block YAML Parameter #809

Closed
sumdudelr opened this issue Nov 11, 2024 · 2 comments
Closed

RFNoC "addsub" Block YAML Parameter #809

sumdudelr opened this issue Nov 11, 2024 · 2 comments
Labels

Comments

@sumdudelr
Copy link

Issue Description

The included "addsub" block's YAML definition contains a string parameter. The quotation marks do not carry forward to the generated rfnoc_image_core.sv. Building FPGA image will fail.

Setup Details

Expected Behavior

FPGA image builds successfully

Actual Behaviour

Synthesis fails with:

ERROR: [Synth 8-1031] Verilog is not declared [repo/icores/build-x310_rfnoc_image_core/rfnoc_image_core.sv:1591]

Steps to reproduce the problem

Just add an "addsub" block to your RFNoC image.

Additional Information

Can be fixed by adding a few lines to addsub.yml to use hdl_parameters as is done in OOT example gain.yml.

@mbr0wn mbr0wn added the bug label Dec 13, 2024
@mbr0wn
Copy link
Contributor

mbr0wn commented Dec 13, 2024

Thanks for reporting. Can confirm.

@joergho joergho closed this as completed Dec 13, 2024
@joergho
Copy link
Contributor

joergho commented Dec 13, 2024

We fixed the issue and will update master shortly.

marino-gnss pushed a commit to marino-gnss/uhd that referenced this issue Feb 17, 2025
When instantiating an addsub block, the quotes would get munged,
rendering the parameter as such:

   .USE_IMPL(Verilog)

The correct rendering should be:

   .USE_IMPL("Verilog")

This leads to Vivado complaining about how the expression `Verilog` is
not known (it should be a string constant).

We fix this by using the recently introduced 'q' filter.

Discussed in EttusResearch#809.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants