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

siwx917.svd has clusters with missing %s placeholders. #21

Open
i509VCB opened this issue Dec 25, 2024 · 0 comments
Open

siwx917.svd has clusters with missing %s placeholders. #21

i509VCB opened this issue Dec 25, 2024 · 0 comments

Comments

@i509VCB
Copy link

i509VCB commented Dec 25, 2024

Per https://arm-software.github.io/CMSIS_5/SVD/html/elem_registers.html#elem_cluster, a cluster with a dim set should have a %s placeholder in the name. However the SVD file has multiple occurrences of clusters, such as PWM_DUTYCYCLE_REG_WR_VALUEn, which have a dim value set to act as an array but do not contain a %s placeholder in place of the n seen in the register names.

This seems to cause issues with svdtools and chiptool which expect clusters to follow the SVD specification for naming for code generation. I have attached a file showing that the changes I needed to make to allow svdtools and chiptool to successfully read the file (ignore the CRLF and end of document new lines).
siwx917_svd_fix.txt

I would appreciate if the svd file could be fixed.

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

1 participant