alex/rp-hal-boards
1
0
Fork 0
mirror of https://github.com/italicsjenga/rp-hal-boards.git synced 2024-12-24 05:01:31 +11:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #197 from chinoto/pio_sound_send

Browse source 
pio: Fix unsound Send clippy warning
This commit is contained in:
Jonathan 'theJPster' Pallant 2022-01-06 12:57:26 +00:00 committed by GitHub
parent 78176deffc c7c5fb00d8
commit f23ababb1a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
rp2040-hal/src/pio.rs
View file

@ -11,7 +11,7 @@ const PIO_INSTRUCTION_COUNT: usize = 32;
/// PIO Instance /// PIO Instance
pub trait PIOExt: pub trait PIOExt:
core::ops::Deref<Target = rp2040_pac::pio0::RegisterBlock> + SubsystemReset + Sized core::ops::Deref<Target = rp2040_pac::pio0::RegisterBlock> + SubsystemReset + Sized + Send
{ {
/// Create a new PIO wrapper and split the state machines into individual objects. /// Create a new PIO wrapper and split the state machines into individual objects.
#[allow(clippy::type_complexity)] // Required for symmetry with PIO::free(). #[allow(clippy::type_complexity)] // Required for symmetry with PIO::free().
@ -94,7 +94,7 @@ impl<P: PIOExt> core::fmt::Debug for PIO<P> {
// Safety: `PIO` only provides access to those registers which are not directly used by // Safety: `PIO` only provides access to those registers which are not directly used by
// `StateMachine`. // `StateMachine`.
unsafe impl<P: PIOExt + Send> Send for PIO<P> {} unsafe impl<P: PIOExt> Send for PIO<P> {}
// Safety: `PIO` is marked Send so ensure all accesses remain atomic and no new concurrent accesses // Safety: `PIO` is marked Send so ensure all accesses remain atomic and no new concurrent accesses
// are added. // are added.
@ -315,7 +315,7 @@ impl<P: PIOExt> InstalledProgram<P> {
} }
/// State machine identifier (without a specified PIO block). /// State machine identifier (without a specified PIO block).
pub trait StateMachineIndex { pub trait StateMachineIndex: Send {
/// Numerical index of the state machine (0 to 3). /// Numerical index of the state machine (0 to 3).
fn id() -> usize; fn id() -> usize;
} }
@ -902,7 +902,7 @@ pub struct Interrupt<P: PIOExt> {
} }
// Safety: `Interrupt` provides exclusive access to interrupt registers. // Safety: `Interrupt` provides exclusive access to interrupt registers.
unsafe impl<P: PIOExt + Send> Send for Interrupt<P> {} unsafe impl<P: PIOExt> Send for Interrupt<P> {}
// Safety: `Interrupt` is marked Send so ensure all accesses remain atomic and no new concurrent // Safety: `Interrupt` is marked Send so ensure all accesses remain atomic and no new concurrent
// accesses are added. // accesses are added.