Skip to content

Conversation

Sh3Rm4n
Copy link
Member

@Sh3Rm4n Sh3Rm4n commented Nov 21, 2023

No description provided.

@adamgreig
Copy link
Member

It looks like this changes the access of the data register from Word-sized (which I think might be u8 or u16?) to always be a 32-bit access. Annoyingly the data register is access-size-sensitive, and doing a 32-bit read can cause up to 4 8-bit words to be popped from the peripheral's FIFO. This is the same problem as stm32-rs/stm32f7xx-hal#207

@Sh3Rm4n
Copy link
Member Author

Sh3Rm4n commented Nov 22, 2023

Thanks for the info. This means, we currently can not read 8 or 16 bits from the data register through the svd2rust API, but casting is not the correct way either?

@Sh3Rm4n Sh3Rm4n merged commit a8db75e into stm32-rs:master Nov 22, 2023
@Sh3Rm4n Sh3Rm4n deleted the fix/compiler-error branch November 22, 2023 21:06
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