Skip to content

Conversation

@NikBel3476
Copy link
Contributor

In timer/atmega128rfa1.yaml "_replace_enum" from timer/dev/*.yaml applies only for first COM* definition (svdtools 0.1.27) so I added it manually. I don't know what the reason but patch works correctly without manual definitions with svdtools 0.3.20 installed from cargo.

@tones111
Copy link
Contributor

tones111 commented Jan 5, 2025

Maybe we can update to the rust version of svdtools (#174)?

@Rahix
Copy link
Owner

Rahix commented Jan 6, 2025

Just for clarification: Do these changes depend on #174 or are they independent in their current form?

@NikBel3476
Copy link
Contributor Author

NikBel3476 commented Jan 6, 2025

I just checked it out and it doesn't depend on changes in #174 and also works with svdtools 0.4.0.

Copy link
Owner

@Rahix Rahix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, thanks a lot for your contribution!

@Rahix Rahix merged commit 6f6cbd4 into Rahix:main Jan 6, 2025
2 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.

3 participants