Issue with RFDC device tree generator parameter names
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 1.2k
- Forks
- 1.1k
- PR merge metrics
- No merged PRs in 30d
Description
Configuration
Vivado/PetaLinux/device-tree-xlnx at 2023.2 on Ubuntu 20.04
Expected behavior
Device tree writer correctly outputs NCOFreq parameters into device tree param-list.
Actual behavior
Device tree writer outputs NCOFreq as "0.0" to device tree param-list.
Issue
Between 2022.2 and 2023.2 the device tree writer for the RFDC driver added the NCOFreq parameter (see commit). However that commit reads from a parameter named C_DAC_NCO_FREQ00 (and C_DAC_NCO_FREQ01, etc). However Vivado appears to believe this parameter is named C_DAC_NCO_Freq00 (and C_DAC_NCO_Freq01, etc).
When the device tree generator is run on a design that contains an usp_rf_data_converter block, the device tree writer complains about not finding the relevant configuration:
WARNING: [Common 17-673] Cannot get value of property 'CONFIG.C_DAC_NCO_FREQ00' because this property is not valid in conjunction with other property setting on this object.
When checking the output device tree param-list, the output has all the NCO frequencies at 0.0 instead of the correct value.
As an experiment, I changed rfdc.tcl to use C_DAC_NCO_Freq00 instead of C_DAC_NCO_FREQ00, and this appears to resolve this issue. I suspect rfdc.tcl needs to updated to use the names used in Vivado?
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in rfdc.tcl and compare the parameter names it reads with the Vivado names described in the issue, especially C_DAC_NCO_FREQ00 versus C_DAC_NCO_Freq00. Run the device-tree generator on a design containing an usp_rf_data_converter block and verify that the generated param-list contains the correct NCO frequencies instead of 0.0.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ubuntu
- Domain
- embedded-iot
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 62/100