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

esp32c6: svd: add patch for SLC peripheral #322

Merged
merged 2 commits into from
Feb 24, 2025

Conversation

rmsyn
Copy link
Contributor

@rmsyn rmsyn commented Feb 19, 2025

Adds a SVD patch entry for the SLC peripheral to configure SDIO.

Adjusts the base address of the SLCHOST peripheral to its correct offset.

TODO

  • define register fields
  • simplify SVD using clusters (?)

Resolves: #320

@rmsyn rmsyn marked this pull request as draft February 19, 2025 06:56
@rmsyn rmsyn marked this pull request as ready for review February 19, 2025 12:45
Adds a SVD patch entry for the SLC register set to configure the SDIO
peripheral.

Adjusts the base address of the SLCHOST register set to its correct
offset.
Generates the `slc` peripheral using the definitions added to the SVD
patches file.
Copy link
Member

@jessebraham jessebraham left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! These are always a bit difficult to review, not going to go through each register and field to verify, so let's just roll with it and we can fix anything as it comes up, if needed. Addresses look good, at least 😁

@jessebraham jessebraham added this pull request to the merge queue Feb 24, 2025
Merged via the queue into esp-rs:main with commit 7530070 Feb 24, 2025
11 checks passed
@rmsyn
Copy link
Contributor Author

rmsyn commented Feb 24, 2025

These are always a bit difficult to review, not going to go through each register and field to verify

I was very careful to ensure the fields were correct when transferring from the spec, but having more eyes on it is always good.

The generated code is definitely a lot, but the patch additions are relatively more contained (940 LoC of YAML). A lot of that is repeated, with some opportunity to further reduce by using derivedFrom and/or dim definitions.

Thank you for reviewing and merging the changes. Please feel free to ping me on any issues related to these changes, or the esp32c6 code in general :)

@rmsyn rmsyn deleted the esp32c6/svd-slc branch February 25, 2025 00:02
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.

svd: esp32c6 is missing the slc peripheral
2 participants