max/embassy
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge

Browse Source 
1205: stm32/rng Fix rng generation lock-up r=Dirbaio a=lucasgranberg

This PR fixes a problem where the device gets locked in case of rng errors.

The PR also includes a hack for stm32wl based devices where the more complicated RNG peripheral can get stuck on seed errors.

Co-authored-by: Lucas Granberg <lukkeg@gmail.com>
This commit is contained in:
bors[bot] 2023-02-09 11:39:52 +00:00 committed by GitHub
commit e1a0df7d46
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 3 deletions
embassy-stm32/src
rng.rs
stm32-data

11
embassy-stm32/src/rng.rs

@ -32,6 +32,11 @@ impl<'d, T: Instance> Rng<'d, T> {
} }
pub fn reset(&mut self) { pub fn reset(&mut self) {
// rng_v2 locks up on seed error, needs reset
#[cfg(rng_v2)]
if unsafe { T::regs().sr().read().seis() } {
T::reset();
}
unsafe { unsafe {
T::regs().cr().modify(|reg| { T::regs().cr().modify(|reg| {
reg.set_rngen(true); reg.set_rngen(true);
@ -90,8 +95,10 @@ impl<'d, T: Instance> Rng<'d, T> {
impl<'d, T: Instance> RngCore for Rng<'d, T> { impl<'d, T: Instance> RngCore for Rng<'d, T> {
fn next_u32(&mut self) -> u32 { fn next_u32(&mut self) -> u32 {
loop { loop {
let bits = unsafe { T::regs().sr().read() }; let sr = unsafe { T::regs().sr().read() };
if bits.drdy() { if sr.seis() | sr.ceis() {
self.reset();
} else if sr.drdy() {
return unsafe { T::regs().dr().read() }; return unsafe { T::regs().dr().read() };
} }
} }

2
stm32-data

@ -1 +1 @@
Subproject commit cc93f9d10395077770bebefb6b9488e06b0e5811 Subproject commit 66252982939014e94fc4a1b7423c30c3d108ae0b