Skip to content

Conversation

robamu
Copy link
Contributor

@robamu robamu commented Sep 28, 2025

No description provided.

@robamu robamu force-pushed the add-register-derives branch from ce0f203 to 7489ede Compare September 28, 2025 16:07
@robamu robamu force-pushed the add-register-derives branch from 7489ede to 6a7f9b5 Compare September 28, 2025 16:15
@jonathanpallant
Copy link
Contributor

What's the use-case for sending copies of processor registers through serde?

@robamu
Copy link
Contributor Author

robamu commented Sep 29, 2025

I commonly add those derives for most types. It might possibly be interesting for error related registers (DFSR and IFSR) or registers like CPSR. In the end, I thought, why not add them for every register then, and added it to all registers using sd. I suppose it is not a common use-case though. Cortex-M has serde support, but only for the SCB (https://github.com/rust-embedded/cortex-m/blob/master/cortex-m/src/peripheral/scb.rs). Does not cost much though (serde is not added by default) and might possibly be useful.

@jonathanpallant jonathanpallant added this pull request to the merge queue Sep 29, 2025
Merged via the queue into main with commit dac87f1 Sep 29, 2025
57 checks passed
@robamu robamu deleted the add-register-derives branch September 29, 2025 13:07
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.

2 participants