Skip to content

Commit

Permalink
Fix incorrect mapping of DMA channel to DMAMUX channel
Browse files Browse the repository at this point in the history
This is quite an ugly hack but it does try to keep the API as unchanged as possible.
This fix only fixes the problem on cat 3 and 4 devices for now.

According to RM0440 under "DMAMUX mapping":

For category 3 and category 4 devices:
* DMAMUX channels 0 to 7 are connected to DMA1 channels 1 to 8
* DMAMUX channels 8 to 15 are connected to DMA2 channels 1 to 8

For category 2 devices:
* DMAMUX channels 0 to 5 are connected to DMA1 channels 1 to 6
* DMAMUX channels 6 to 11 are connected to DMA2 channels 1 to 6
  • Loading branch information
usbalbin committed Dec 18, 2023
1 parent c34b89b commit 09f2a88
Showing 1 changed file with 32 additions and 11 deletions.
43 changes: 32 additions & 11 deletions src/dma/stream.rs
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@ pub type DMAMUXRegisterBlock = stm32::dmamux::RegisterBlock;

/// Trait that represents an instance of a DMA peripheral
pub trait Instance: Deref<Target = DMARegisterBlock> + Sealed {
const IS_DMA1: bool;

/// Gives a pointer to the RegisterBlock.
fn ptr() -> *const DMARegisterBlock;

Expand All @@ -33,6 +35,8 @@ pub trait Instance: Deref<Target = DMARegisterBlock> + Sealed {
}

impl Instance for DMA1 {
const IS_DMA1: bool = true;

#[inline(always)]
fn ptr() -> *const DMARegisterBlock {
DMA1::ptr()
Expand All @@ -45,6 +49,8 @@ impl Instance for DMA1 {
}

impl Instance for DMA2 {
const IS_DMA1: bool = false;

#[inline(always)]
fn ptr() -> *const DMARegisterBlock {
DMA2::ptr()
Expand Down Expand Up @@ -178,7 +184,7 @@ macro_rules! dma_stream {
($(($name:ident, $number:expr,
regs => $ccr:ident, $cparX:ident, $cmarX:ident, $cndtrX:ident,
fields => $tcif:ident, $htif:ident, $teif:ident, $gif:ident, $tcisr:ident, $htisr:ident, $teisr:ident, $gisr:ident,
dmamux => $cXcr:ident,)
dmamux => $dma1_cXcr:ident, $dma2_cXcr:ident, )
),+$(,)*) => {
$(
impl<I: Instance> Stream for $name<I> {
Expand Down Expand Up @@ -287,8 +293,13 @@ macro_rules! dma_stream {
//NOTE(unsafe) We only access the registers that belongs to the StreamX
let dmamux = unsafe { &*I::mux_ptr() };
unsafe {
dmamux.$cXcr
.modify(|_, w| w.dmareq_id().bits(request_line));
if I::IS_DMA1 {
dmamux.$dma1_cXcr
.modify(|_, w| w.dmareq_id().bits(request_line));
} else {
dmamux.$dma2_cXcr
.modify(|_, w| w.dmareq_id().bits(request_line));
};
}
}

Expand Down Expand Up @@ -497,55 +508,65 @@ macro_rules! dma_stream {
};
}

// Cat 3 and 4 devices
#[cfg(any(
feature = "stm32g471",
feature = "stm32g473",
feature = "stm32g474",
feature = "stm32g483",
feature = "stm32g484",
feature = "stm32g491",
feature = "stm32g49a",
))]
dma_stream!(
// Note: the field names start from one, unlike the RM where they start from
// zero. May need updating if it gets fixed upstream.
(
Stream0, 0,
regs => ccr1, cpar1, cmar1, cndtr1,
fields => tcif1, htif1, teif1, gif1, tcif1, htif1, teif1, gif1,
dmamux => c0cr,
dmamux => c0cr, c8cr,
),
(
Stream1, 1,
regs => ccr2, cpar2, cmar2, cndtr2,
fields => tcif2, htif2, teif2, gif2, tcif2, htif2, teif2, gif2,
dmamux => c1cr,
dmamux => c1cr, c9cr,
),
(
Stream2, 2,
regs => ccr3, cpar3, cmar3, cndtr3,
fields => tcif3, htif3, teif3, gif3, tcif3, htif3, teif3, gif3,
dmamux => c2cr,
dmamux => c2cr, c10cr,
),
(
Stream3, 3,
regs => ccr4, cpar4, cmar4, cndtr4,
fields => tcif4, htif4, teif4, gif4, tcif4, htif4, teif4, gif4,
dmamux => c3cr,
dmamux => c3cr, c11cr,
),
(
Stream4, 4,
regs => ccr5, cpar5, cmar5, cndtr5,
fields => tcif5, htif5, teif5, gif5, tcif5, htif5, teif5, gif5,
dmamux => c4cr,
dmamux => c4cr, c12cr,
),
(
Stream5, 5,
regs => ccr6, cpar6, cmar6, cndtr6,
fields => tcif6, htif6, teif6, gif6, tcif6, htif6, teif6, gif6,
dmamux => c5cr,
dmamux => c5cr, c13cr,
),
(
Stream6, 6,
regs => ccr7, cpar7, cmar7, cndtr7,
fields => tcif7, htif7, teif7, gif7, tcif7, htif7, teif7, gif7,
dmamux => c6cr,
dmamux => c6cr, c14cr,
),
(
Stream7, 7,
regs => ccr8, cpar8, cmar8, cndtr8,
fields => tcif8, htif8, teif8, gif8, tcif8, htif8, teif8, gif8,
dmamux => c7cr,
dmamux => c7cr, c15cr,
),
);

0 comments on commit 09f2a88

Please sign in to comment.