SDIO working on stm32f4

This commit is contained in:
chemicstry 2022-03-16 18:52:27 +02:00
parent 34b5175d2c
commit 6d547b1143
6 changed files with 269 additions and 125 deletions

View File

@ -407,17 +407,17 @@ fn main() {
(("timer", "BKIN2"), (quote!(crate::pwm::BreakInput2Pin), quote!())), (("timer", "BKIN2"), (quote!(crate::pwm::BreakInput2Pin), quote!())),
(("timer", "BKIN2_COMP1"), (quote!(crate::pwm::BreakInput2Comparator1Pin), quote!())), (("timer", "BKIN2_COMP1"), (quote!(crate::pwm::BreakInput2Comparator1Pin), quote!())),
(("timer", "BKIN2_COMP2"), (quote!(crate::pwm::BreakInput2Comparator2Pin), quote!())), (("timer", "BKIN2_COMP2"), (quote!(crate::pwm::BreakInput2Comparator2Pin), quote!())),
(("sdmmc", "CK"), (quote!(crate::sdmmc::CkPin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "CK"), (quote!(crate::sdmmc::CkPin), quote!())),
(("sdmmc", "CMD"), (quote!(crate::sdmmc::CmdPin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "CMD"), (quote!(crate::sdmmc::CmdPin), quote!())),
(("sdmmc", "D0"), (quote!(crate::sdmmc::D0Pin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "D0"), (quote!(crate::sdmmc::D0Pin), quote!())),
(("sdmmc", "D1"), (quote!(crate::sdmmc::D1Pin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "D1"), (quote!(crate::sdmmc::D1Pin), quote!())),
(("sdmmc", "D2"), (quote!(crate::sdmmc::D2Pin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "D2"), (quote!(crate::sdmmc::D2Pin), quote!())),
(("sdmmc", "D3"), (quote!(crate::sdmmc::D3Pin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "D3"), (quote!(crate::sdmmc::D3Pin), quote!())),
(("sdmmc", "D4"), (quote!(crate::sdmmc::D4Pin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "D4"), (quote!(crate::sdmmc::D4Pin), quote!())),
(("sdmmc", "D5"), (quote!(crate::sdmmc::D5Pin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "D5"), (quote!(crate::sdmmc::D5Pin), quote!())),
(("sdmmc", "D6"), (quote!(crate::sdmmc::D6Pin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "D6"), (quote!(crate::sdmmc::D6Pin), quote!())),
(("sdmmc", "D6"), (quote!(crate::sdmmc::D7Pin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "D6"), (quote!(crate::sdmmc::D7Pin), quote!())),
(("sdmmc", "D8"), (quote!(crate::sdmmc::D8Pin), quote!(#[cfg(feature="sdmmc-rs")]))), (("sdmmc", "D8"), (quote!(crate::sdmmc::D8Pin), quote!())),
].into(); ].into();
for p in METADATA.peripherals { for p in METADATA.peripherals {
@ -483,6 +483,7 @@ fn main() {
(("i2c", "TX"), quote!(crate::i2c::TxDma)), (("i2c", "TX"), quote!(crate::i2c::TxDma)),
(("dcmi", "DCMI"), quote!(crate::dcmi::FrameDma)), (("dcmi", "DCMI"), quote!(crate::dcmi::FrameDma)),
(("dcmi", "PSSI"), quote!(crate::dcmi::FrameDma)), (("dcmi", "PSSI"), quote!(crate::dcmi::FrameDma)),
(("sdmmc", "SDIO"), quote!(crate::sdmmc::SdioDma)),
] ]
.into(); .into();

View File

@ -12,12 +12,13 @@ use embassy_hal_common::unborrow;
use futures::future::poll_fn; use futures::future::poll_fn;
use sdio_host::{BusWidth, CardCapacity, CardStatus, CurrentState, SDStatus, CID, CSD, OCR, SCR}; use sdio_host::{BusWidth, CardCapacity, CardStatus, CurrentState, SDStatus, CID, CSD, OCR, SCR};
use crate::dma::NoDma; use crate::dma::{NoDma, TransferOptions};
use crate::gpio::sealed::AFType; use crate::gpio::sealed::AFType;
use crate::gpio::{Pull, Speed}; use crate::gpio::{Pull, Speed};
use crate::interrupt::Interrupt; use crate::interrupt::Interrupt;
use crate::pac::sdmmc::Sdmmc as RegBlock; use crate::pac::sdmmc::Sdmmc as RegBlock;
use crate::peripherals; use crate::peripherals;
use crate::rcc::RccPeripheral;
use crate::time::Hertz; use crate::time::Hertz;
/// The signalling scheme used on the SDMMC bus /// The signalling scheme used on the SDMMC bus
@ -92,12 +93,6 @@ impl Card {
} }
} }
/// Indicates transfer direction
enum Dir {
CardToHost,
HostToCard,
}
#[repr(u8)] #[repr(u8)]
enum PowerCtrl { enum PowerCtrl {
Off = 0b00, Off = 0b00,
@ -196,16 +191,16 @@ pub struct Sdmmc<'d, T: Instance, P: Pins<T>, Dma = NoDma> {
pins: P, pins: P,
irq: T::Interrupt, irq: T::Interrupt,
config: Config, config: Config,
dma: Dma,
/// Current clock to card /// Current clock to card
clock: Hertz, pub clock: Hertz,
/// Current signalling scheme to card /// Current signalling scheme to card
signalling: Signalling, signalling: Signalling,
/// Card /// Card
card: Option<Card>, card: Option<Card>,
dma: Dma,
} }
impl<'d, T: Instance, P: Pins<T>, Dma> Sdmmc<'d, T, P, Dma> { impl<'d, T: Instance, P: Pins<T>, Dma: SdioDma<T>> Sdmmc<'d, T, P, Dma> {
/// # Safety /// # Safety
/// ///
/// Futures that borrow this type can't be leaked /// Futures that borrow this type can't be leaked
@ -220,6 +215,9 @@ impl<'d, T: Instance, P: Pins<T>, Dma> Sdmmc<'d, T, P, Dma> {
unborrow!(irq, pins); unborrow!(irq, pins);
pins.configure(); pins.configure();
T::enable();
T::reset();
let inner = T::inner(); let inner = T::inner();
let clock = inner.new_inner(config.kernel_clk); let clock = inner.new_inner(config.kernel_clk);
@ -232,10 +230,10 @@ impl<'d, T: Instance, P: Pins<T>, Dma> Sdmmc<'d, T, P, Dma> {
pins, pins,
irq, irq,
config, config,
dma,
clock, clock,
signalling: Default::default(), signalling: Default::default(),
card: None, card: None,
dma,
} }
} }
@ -255,6 +253,7 @@ impl<'d, T: Instance, P: Pins<T>, Dma> Sdmmc<'d, T, P, Dma> {
&mut self.clock, &mut self.clock,
T::state(), T::state(),
self.config.data_transfer_timeout, self.config.data_transfer_timeout,
&mut self.dma,
) )
.await .await
} }
@ -278,6 +277,7 @@ impl<'d, T: Instance, P: Pins<T>, Dma> Sdmmc<'d, T, P, Dma> {
card_capacity, card_capacity,
state, state,
self.config.data_transfer_timeout, self.config.data_transfer_timeout,
&mut self.dma,
) )
.await .await
} }
@ -296,6 +296,7 @@ impl<'d, T: Instance, P: Pins<T>, Dma> Sdmmc<'d, T, P, Dma> {
card, card,
state, state,
self.config.data_transfer_timeout, self.config.data_transfer_timeout,
&mut self.dma,
) )
.await .await
} }
@ -349,6 +350,9 @@ impl SdmmcInner {
w.set_pwrsav(false); w.set_pwrsav(false);
w.set_negedge(false); w.set_negedge(false);
w.set_hwfc_en(true); w.set_hwfc_en(true);
#[cfg(sdmmc_v1)]
w.set_clken(true);
}); });
// Power off, writen 00: Clock to the card is stopped; // Power off, writen 00: Clock to the card is stopped;
@ -361,7 +365,7 @@ impl SdmmcInner {
/// Initializes card (if present) and sets the bus at the /// Initializes card (if present) and sets the bus at the
/// specified frequency. /// specified frequency.
#[allow(clippy::too_many_arguments)] #[allow(clippy::too_many_arguments)]
async fn init_card( async fn init_card<T: Instance, Dma: SdioDma<T>>(
&self, &self,
freq: Hertz, freq: Hertz,
bus_width: BusWidth, bus_width: BusWidth,
@ -372,6 +376,7 @@ impl SdmmcInner {
clock: &mut Hertz, clock: &mut Hertz,
waker_reg: &AtomicWaker, waker_reg: &AtomicWaker,
data_transfer_timeout: u32, data_transfer_timeout: u32,
dma: &mut Dma,
) -> Result<(), Error> { ) -> Result<(), Error> {
let regs = self.0; let regs = self.0;
@ -382,7 +387,7 @@ impl SdmmcInner {
// Check if cards supports CMD8 (with pattern) // Check if cards supports CMD8 (with pattern)
self.cmd(Cmd::hs_send_ext_csd(0x1AA), false)?; self.cmd(Cmd::hs_send_ext_csd(0x1AA), false)?;
let r1 = regs.resp(0).read().cardstatus(); let r1 = regs.respr(0).read().cardstatus();
let mut card = if r1 == 0x1AA { let mut card = if r1 == 0x1AA {
// Card echoed back the pattern. Must be at least v2 // Card echoed back the pattern. Must be at least v2
@ -406,7 +411,7 @@ impl SdmmcInner {
Err(Error::Crc) => (), Err(Error::Crc) => (),
Err(err) => return Err(err), Err(err) => return Err(err),
} }
let ocr: OCR = regs.resp(0).read().cardstatus().into(); let ocr: OCR = regs.respr(0).read().cardstatus().into();
if !ocr.is_busy() { if !ocr.is_busy() {
// Power up done // Power up done
break ocr; break ocr;
@ -422,26 +427,27 @@ impl SdmmcInner {
card.ocr = ocr; card.ocr = ocr;
self.cmd(Cmd::all_send_cid(), false)?; // CMD2 self.cmd(Cmd::all_send_cid(), false)?; // CMD2
let cid0 = regs.resp(0).read().cardstatus() as u128; let cid0 = regs.respr(0).read().cardstatus() as u128;
let cid1 = regs.resp(1).read().cardstatus() as u128; let cid1 = regs.respr(1).read().cardstatus() as u128;
let cid2 = regs.resp(2).read().cardstatus() as u128; let cid2 = regs.respr(2).read().cardstatus() as u128;
let cid3 = regs.resp(3).read().cardstatus() as u128; let cid3 = regs.respr(3).read().cardstatus() as u128;
let cid = (cid0 << 96) | (cid1 << 64) | (cid2 << 32) | (cid3); let cid = (cid0 << 96) | (cid1 << 64) | (cid2 << 32) | (cid3);
card.cid = cid.into(); card.cid = cid.into();
self.cmd(Cmd::send_rel_addr(), false)?; self.cmd(Cmd::send_rel_addr(), false)?;
card.rca = regs.resp(0).read().cardstatus() >> 16; card.rca = regs.respr(0).read().cardstatus() >> 16;
self.cmd(Cmd::send_csd(card.rca << 16), false)?; self.cmd(Cmd::send_csd(card.rca << 16), false)?;
let csd0 = regs.resp(0).read().cardstatus() as u128; let csd0 = regs.respr(0).read().cardstatus() as u128;
let csd1 = regs.resp(1).read().cardstatus() as u128; let csd1 = regs.respr(1).read().cardstatus() as u128;
let csd2 = regs.resp(2).read().cardstatus() as u128; let csd2 = regs.respr(2).read().cardstatus() as u128;
let csd3 = regs.resp(3).read().cardstatus() as u128; let csd3 = regs.respr(3).read().cardstatus() as u128;
let csd = (csd0 << 96) | (csd1 << 64) | (csd2 << 32) | (csd3); let csd = (csd0 << 96) | (csd1 << 64) | (csd2 << 32) | (csd3);
card.csd = csd.into(); card.csd = csd.into();
self.select_card(Some(&card))?; self.select_card(Some(&card))?;
self.get_scr(&mut card, waker_reg, data_transfer_timeout)
self.get_scr(&mut card, waker_reg, data_transfer_timeout, dma)
.await?; .await?;
// Set bus width // Set bus width
@ -475,13 +481,18 @@ impl SdmmcInner {
} }
// Read status // Read status
self.read_sd_status(&mut card, waker_reg, data_transfer_timeout) self.read_sd_status(&mut card, waker_reg, data_transfer_timeout, dma)
.await?; .await?;
if freq.0 > 25_000_000 { if freq.0 > 25_000_000 {
// Switch to SDR25 // Switch to SDR25
*signalling = self *signalling = self
.switch_signalling_mode(Signalling::SDR25, waker_reg, data_transfer_timeout) .switch_signalling_mode(
Signalling::SDR25,
waker_reg,
data_transfer_timeout,
dma,
)
.await?; .await?;
if *signalling == Signalling::SDR25 { if *signalling == Signalling::SDR25 {
@ -494,7 +505,7 @@ impl SdmmcInner {
} }
} }
// Read status after signalling change // Read status after signalling change
self.read_sd_status(&mut card, waker_reg, data_transfer_timeout) self.read_sd_status(&mut card, waker_reg, data_transfer_timeout, dma)
.await?; .await?;
old_card.replace(card); old_card.replace(card);
} }
@ -502,13 +513,14 @@ impl SdmmcInner {
Ok(()) Ok(())
} }
async fn read_block( async fn read_block<T: Instance, Dma: SdioDma<T>>(
&self, &self,
block_idx: u32, block_idx: u32,
buffer: &mut [u32; 128], buffer: &mut [u32; 128],
capacity: CardCapacity, capacity: CardCapacity,
waker_reg: &AtomicWaker, waker_reg: &AtomicWaker,
data_transfer_timeout: u32, data_transfer_timeout: u32,
dma: &mut Dma,
) -> Result<(), Error> { ) -> Result<(), Error> {
// Always read 1 block of 512 bytes // Always read 1 block of 512 bytes
// SDSC cards are byte addressed hence the blockaddress is in multiples of 512 bytes // SDSC cards are byte addressed hence the blockaddress is in multiples of 512 bytes
@ -521,14 +533,13 @@ impl SdmmcInner {
let regs = self.0; let regs = self.0;
let on_drop = OnDrop::new(|| unsafe { self.on_drop() }); let on_drop = OnDrop::new(|| unsafe { self.on_drop() });
let buf_addr = buffer as *mut [u32; 128] as u32;
unsafe { unsafe {
self.prepare_datapath_transfer( self.prepare_datapath_read(
buf_addr, buffer as *mut [u32; 128],
512, 512,
9, 9,
Dir::CardToHost,
data_transfer_timeout, data_transfer_timeout,
dma,
); );
self.data_interrupts(true); self.data_interrupts(true);
} }
@ -536,7 +547,7 @@ impl SdmmcInner {
let res = poll_fn(|cx| { let res = poll_fn(|cx| {
waker_reg.register(cx.waker()); waker_reg.register(cx.waker());
let status = unsafe { regs.sta().read() }; let status = unsafe { regs.star().read() };
if status.dcrcfail() { if status.dcrcfail() {
return Poll::Ready(Err(Error::Crc)); return Poll::Ready(Err(Error::Crc));
@ -552,20 +563,19 @@ impl SdmmcInner {
if res.is_ok() { if res.is_ok() {
on_drop.defuse(); on_drop.defuse();
unsafe { self.stop_datapath();
regs.idmactrlr().modify(|w| w.set_idmaen(false));
}
} }
res res
} }
async fn write_block( async fn write_block<T: Instance, Dma: SdioDma<T>>(
&self, &self,
block_idx: u32, block_idx: u32,
buffer: &[u32; 128], buffer: &[u32; 128],
card: &mut Card, card: &mut Card,
waker_reg: &AtomicWaker, waker_reg: &AtomicWaker,
data_transfer_timeout: u32, data_transfer_timeout: u32,
dma: &mut Dma,
) -> Result<(), Error> { ) -> Result<(), Error> {
// Always read 1 block of 512 bytes // Always read 1 block of 512 bytes
// SDSC cards are byte addressed hence the blockaddress is in multiples of 512 bytes // SDSC cards are byte addressed hence the blockaddress is in multiples of 512 bytes
@ -578,14 +588,13 @@ impl SdmmcInner {
let regs = self.0; let regs = self.0;
let on_drop = OnDrop::new(|| unsafe { self.on_drop() }); let on_drop = OnDrop::new(|| unsafe { self.on_drop() });
let buf_addr = buffer as *const [u32; 128] as u32;
unsafe { unsafe {
self.prepare_datapath_transfer( self.prepare_datapath_write(
buf_addr, buffer as *const [u32; 128],
512, 512,
9, 9,
Dir::HostToCard,
data_transfer_timeout, data_transfer_timeout,
dma,
); );
self.data_interrupts(true); self.data_interrupts(true);
} }
@ -593,7 +602,7 @@ impl SdmmcInner {
let res = poll_fn(|cx| { let res = poll_fn(|cx| {
waker_reg.register(cx.waker()); waker_reg.register(cx.waker());
let status = unsafe { regs.sta().read() }; let status = unsafe { regs.star().read() };
if status.dcrcfail() { if status.dcrcfail() {
return Poll::Ready(Err(Error::Crc)); return Poll::Ready(Err(Error::Crc));
@ -610,16 +619,15 @@ impl SdmmcInner {
match res { match res {
Ok(_) => { Ok(_) => {
on_drop.defuse(); on_drop.defuse();
unsafe { self.stop_datapath();
regs.idmactrlr().modify(|w| w.set_idmaen(false));
}
// TODO: Make this configurable // TODO: Make this configurable
let mut timeout: u32 = 0x00FF_FFFF; let mut timeout: u32 = 0x00FF_FFFF;
// Try to read card status (ACMD13) // Try to read card status (ACMD13)
while timeout > 0 { while timeout > 0 {
match self match self
.read_sd_status(card, waker_reg, data_transfer_timeout) .read_sd_status(card, waker_reg, data_transfer_timeout, dma)
.await .await
{ {
Ok(_) => return Ok(()), Ok(_) => return Ok(()),
@ -646,7 +654,7 @@ impl SdmmcInner {
// NOTE(unsafe) Atomic read with no side-effects // NOTE(unsafe) Atomic read with no side-effects
unsafe { unsafe {
let status = regs.sta().read(); let status = regs.star().read();
cfg_if::cfg_if! { cfg_if::cfg_if! {
if #[cfg(sdmmc_v1)] { if #[cfg(sdmmc_v1)] {
status.rxact() || status.txact() status.rxact() || status.txact()
@ -664,7 +672,7 @@ impl SdmmcInner {
// NOTE(unsafe) Atomic read with no side-effects // NOTE(unsafe) Atomic read with no side-effects
unsafe { unsafe {
let status = regs.sta().read(); let status = regs.star().read();
cfg_if::cfg_if! { cfg_if::cfg_if! {
if #[cfg(sdmmc_v1)] { if #[cfg(sdmmc_v1)] {
status.cmdact() status.cmdact()
@ -683,23 +691,18 @@ impl SdmmcInner {
/// # Safety /// # Safety
/// ///
/// `buffer_addr` must be valid for the whole transfer and word aligned /// `buffer` must be valid for the whole transfer and word aligned
unsafe fn prepare_datapath_transfer( unsafe fn prepare_datapath_read<T: Instance, Dma: SdioDma<T>>(
&self, &self,
buffer_addr: u32, buffer: *mut [u32],
length_bytes: u32, length_bytes: u32,
block_size: u8, block_size: u8,
direction: Dir,
data_transfer_timeout: u32, data_transfer_timeout: u32,
dma: &mut Dma,
) { ) {
assert!(block_size <= 14, "Block size up to 2^14 bytes"); assert!(block_size <= 14, "Block size up to 2^14 bytes");
let regs = self.0; let regs = self.0;
let dtdir = match direction {
Dir::CardToHost => true,
Dir::HostToCard => false,
};
// Command AND Data state machines must be idle // Command AND Data state machines must be idle
self.wait_idle(); self.wait_idle();
self.clear_interrupt_flags(); self.clear_interrupt_flags();
@ -708,16 +711,94 @@ impl SdmmcInner {
regs.dtimer() regs.dtimer()
.write(|w| w.set_datatime(data_transfer_timeout)); .write(|w| w.set_datatime(data_transfer_timeout));
regs.dlen().write(|w| w.set_datalength(length_bytes)); regs.dlenr().write(|w| w.set_datalength(length_bytes));
cfg_if::cfg_if! {
if #[cfg(sdmmc_v1)] {
let request = dma.request();
dma.start_read(request, regs.fifor().ptr() as *const u32, buffer, TransferOptions {
pburst: crate::dma::Burst::Incr4,
flow_ctrl: crate::dma::FlowControl::Peripheral,
..Default::default()
});
} else if #[cfg(sdmmc_v2)] {
regs.idmabase0r().write(|w| w.set_idmabase0(buffer_addr)); regs.idmabase0r().write(|w| w.set_idmabase0(buffer_addr));
regs.idmactrlr().modify(|w| w.set_idmaen(true)); regs.idmactrlr().modify(|w| w.set_idmaen(true));
}
}
regs.dctrl().modify(|w| { regs.dctrl().modify(|w| {
w.set_dblocksize(block_size); w.set_dblocksize(block_size);
w.set_dtdir(dtdir); w.set_dtdir(true);
#[cfg(sdmmc_v1)]
{
w.set_dmaen(true);
w.set_dten(true);
}
}); });
} }
/// # Safety
///
/// `buffer` must be valid for the whole transfer and word aligned
unsafe fn prepare_datapath_write<T: Instance, Dma: SdioDma<T>>(
&self,
buffer: *const [u32],
length_bytes: u32,
block_size: u8,
data_transfer_timeout: u32,
dma: &mut Dma,
) {
assert!(block_size <= 14, "Block size up to 2^14 bytes");
let regs = self.0;
// Command AND Data state machines must be idle
self.wait_idle();
self.clear_interrupt_flags();
// NOTE(unsafe) We have exclusive access to the regisers
regs.dtimer()
.write(|w| w.set_datatime(data_transfer_timeout));
regs.dlenr().write(|w| w.set_datalength(length_bytes));
cfg_if::cfg_if! {
if #[cfg(sdmmc_v1)] {
let request = dma.request();
dma.start_write(request, buffer, regs.fifor().ptr() as *mut u32, TransferOptions {
pburst: crate::dma::Burst::Incr4,
flow_ctrl: crate::dma::FlowControl::Peripheral,
..Default::default()
});
} else if #[cfg(sdmmc_v2)] {
regs.idmabase0r().write(|w| w.set_idmabase0(buffer_addr));
regs.idmactrlr().modify(|w| w.set_idmaen(true));
}
}
regs.dctrl().modify(|w| {
w.set_dblocksize(block_size);
w.set_dtdir(false);
});
}
fn stop_datapath(&self) {
let regs = self.0;
unsafe {
cfg_if::cfg_if! {
if #[cfg(sdmmc_v1)] {
regs.dctrl().modify(|w| {
w.set_dmaen(false);
w.set_dten(false);
});
} else if #[cfg(sdmmc_v2)] {
regs.idmactrlr().modify(|w| w.set_idmaen(false));
}
}
}
}
/// Sets the CLKDIV field in CLKCR. Updates clock field in self /// Sets the CLKDIV field in CLKCR. Updates clock field in self
fn clkcr_set_clkdiv( fn clkcr_set_clkdiv(
&self, &self,
@ -751,11 +832,12 @@ impl SdmmcInner {
/// Attempt to set a new signalling mode. The selected /// Attempt to set a new signalling mode. The selected
/// signalling mode is returned. Expects the current clock /// signalling mode is returned. Expects the current clock
/// frequency to be > 12.5MHz. /// frequency to be > 12.5MHz.
async fn switch_signalling_mode( async fn switch_signalling_mode<T: Instance, Dma: SdioDma<T>>(
&self, &self,
signalling: Signalling, signalling: Signalling,
waker_reg: &AtomicWaker, waker_reg: &AtomicWaker,
data_transfer_timeout: u32, data_transfer_timeout: u32,
dma: &mut Dma,
) -> Result<Signalling, Error> { ) -> Result<Signalling, Error> {
// NB PLSS v7_10 4.3.10.4: "the use of SET_BLK_LEN command is not // NB PLSS v7_10 4.3.10.4: "the use of SET_BLK_LEN command is not
// necessary" // necessary"
@ -771,19 +853,18 @@ impl SdmmcInner {
}; };
let mut status = [0u32; 16]; let mut status = [0u32; 16];
let status_addr = &mut status as *mut [u32; 16] as u32;
// Arm `OnDrop` after the buffer, so it will be dropped first // Arm `OnDrop` after the buffer, so it will be dropped first
let regs = self.0; let regs = self.0;
let on_drop = OnDrop::new(|| unsafe { self.on_drop() }); let on_drop = OnDrop::new(|| unsafe { self.on_drop() });
unsafe { unsafe {
self.prepare_datapath_transfer( self.prepare_datapath_read(
status_addr, &mut status as *mut [u32; 16],
64, 64,
6, 6,
Dir::CardToHost,
data_transfer_timeout, data_transfer_timeout,
dma,
); );
self.data_interrupts(true); self.data_interrupts(true);
} }
@ -791,7 +872,7 @@ impl SdmmcInner {
let res = poll_fn(|cx| { let res = poll_fn(|cx| {
waker_reg.register(cx.waker()); waker_reg.register(cx.waker());
let status = unsafe { regs.sta().read() }; let status = unsafe { regs.star().read() };
if status.dcrcfail() { if status.dcrcfail() {
return Poll::Ready(Err(Error::Crc)); return Poll::Ready(Err(Error::Crc));
@ -816,9 +897,8 @@ impl SdmmcInner {
match res { match res {
Ok(_) => { Ok(_) => {
on_drop.defuse(); on_drop.defuse();
unsafe { self.stop_datapath();
regs.idmactrlr().modify(|w| w.set_idmaen(false));
}
// Function Selection of Function Group 1 // Function Selection of Function Group 1
let selection = (u32::from_be(status[4]) >> 24) & 0xF; let selection = (u32::from_be(status[4]) >> 24) & 0xF;
@ -844,35 +924,35 @@ impl SdmmcInner {
self.cmd(Cmd::card_status(rca << 16), false)?; // CMD13 self.cmd(Cmd::card_status(rca << 16), false)?; // CMD13
// NOTE(unsafe) Atomic read with no side-effects // NOTE(unsafe) Atomic read with no side-effects
let r1 = unsafe { regs.resp(0).read().cardstatus() }; let r1 = unsafe { regs.respr(0).read().cardstatus() };
Ok(r1.into()) Ok(r1.into())
} }
/// Reads the SD Status (ACMD13) /// Reads the SD Status (ACMD13)
async fn read_sd_status( async fn read_sd_status<T: Instance, Dma: SdioDma<T>>(
&self, &self,
card: &mut Card, card: &mut Card,
waker_reg: &AtomicWaker, waker_reg: &AtomicWaker,
data_transfer_timeout: u32, data_transfer_timeout: u32,
dma: &mut Dma,
) -> Result<(), Error> { ) -> Result<(), Error> {
let rca = card.rca; let rca = card.rca;
self.cmd(Cmd::set_block_length(64), false)?; // CMD16 self.cmd(Cmd::set_block_length(64), false)?; // CMD16
self.cmd(Cmd::app_cmd(rca << 16), false)?; // APP self.cmd(Cmd::app_cmd(rca << 16), false)?; // APP
let mut status = [0u32; 16]; let mut status = [0u32; 16];
let status_addr = &mut status as *mut [u32; 16] as u32;
// Arm `OnDrop` after the buffer, so it will be dropped first // Arm `OnDrop` after the buffer, so it will be dropped first
let regs = self.0; let regs = self.0;
let on_drop = OnDrop::new(|| unsafe { self.on_drop() }); let on_drop = OnDrop::new(|| unsafe { self.on_drop() });
unsafe { unsafe {
self.prepare_datapath_transfer( self.prepare_datapath_read(
status_addr, &mut status as *mut [u32; 16],
64, 64,
6, 6,
Dir::CardToHost,
data_transfer_timeout, data_transfer_timeout,
dma,
); );
self.data_interrupts(true); self.data_interrupts(true);
} }
@ -880,7 +960,7 @@ impl SdmmcInner {
let res = poll_fn(|cx| { let res = poll_fn(|cx| {
waker_reg.register(cx.waker()); waker_reg.register(cx.waker());
let status = unsafe { regs.sta().read() }; let status = unsafe { regs.star().read() };
if status.dcrcfail() { if status.dcrcfail() {
return Poll::Ready(Err(Error::Crc)); return Poll::Ready(Err(Error::Crc));
@ -896,9 +976,8 @@ impl SdmmcInner {
if res.is_ok() { if res.is_ok() {
on_drop.defuse(); on_drop.defuse();
unsafe { self.stop_datapath();
regs.idmactrlr().modify(|w| w.set_idmaen(false));
}
for byte in status.iter_mut() { for byte in status.iter_mut() {
*byte = u32::from_be(*byte); *byte = u32::from_be(*byte);
} }
@ -941,11 +1020,8 @@ impl SdmmcInner {
w.set_dbckendc(true); w.set_dbckendc(true);
w.set_sdioitc(true); w.set_sdioitc(true);
cfg_if::cfg_if! { #[cfg(sdmmc_v2)]
if #[cfg(sdmmc_v1)] { {
w.set_stbiterrc(true);
w.set_ceataendc(true);
} else if #[cfg(sdmmc_v2)] {
w.set_dholdc(true); w.set_dholdc(true);
w.set_dabortc(true); w.set_dabortc(true);
w.set_busyd0endc(true); w.set_busyd0endc(true);
@ -956,7 +1032,6 @@ impl SdmmcInner {
w.set_idmatec(true); w.set_idmatec(true);
w.set_idmabtcc(true); w.set_idmabtcc(true);
} }
}
}); });
} }
} }
@ -967,7 +1042,7 @@ impl SdmmcInner {
let regs = self.0; let regs = self.0;
// NOTE(unsafe) Atomic write // NOTE(unsafe) Atomic write
unsafe { unsafe {
regs.mask().write(|w| { regs.maskr().write(|w| {
w.set_dcrcfailie(enable); w.set_dcrcfailie(enable);
w.set_dtimeoutie(enable); w.set_dtimeoutie(enable);
w.set_dataendie(enable); w.set_dataendie(enable);
@ -978,32 +1053,32 @@ impl SdmmcInner {
} }
} }
async fn get_scr( async fn get_scr<T: Instance, Dma: SdioDma<T>>(
&self, &self,
card: &mut Card, card: &mut Card,
waker_reg: &AtomicWaker, waker_reg: &AtomicWaker,
data_transfer_timeout: u32, data_transfer_timeout: u32,
dma: &mut Dma,
) -> Result<(), Error> { ) -> Result<(), Error> {
// Read the the 64-bit SCR register // Read the the 64-bit SCR register
self.cmd(Cmd::set_block_length(8), false)?; // CMD16 self.cmd(Cmd::set_block_length(8), false)?; // CMD16
self.cmd(Cmd::app_cmd(card.rca << 16), false)?; self.cmd(Cmd::app_cmd(card.rca << 16), false)?;
let mut scr = [0u32; 2]; let mut scr = [0u32; 2];
let scr_addr = &mut scr as *mut u32 as u32;
// Arm `OnDrop` after the buffer, so it will be dropped first // Arm `OnDrop` after the buffer, so it will be dropped first
let regs = self.0; let regs = self.0;
let on_drop = OnDrop::new(move || unsafe { self.on_drop() }); let on_drop = OnDrop::new(move || unsafe { self.on_drop() });
unsafe { unsafe {
self.prepare_datapath_transfer(scr_addr, 8, 3, Dir::CardToHost, data_transfer_timeout); self.prepare_datapath_read(&mut scr as *mut [u32], 8, 3, data_transfer_timeout, dma);
self.data_interrupts(true); self.data_interrupts(true);
} }
self.cmd(Cmd::cmd51(), true)?; self.cmd(Cmd::cmd51(), true)?;
let res = poll_fn(|cx| { let res = poll_fn(|cx| {
waker_reg.register(cx.waker()); waker_reg.register(cx.waker());
let status = unsafe { regs.sta().read() }; let status = unsafe { regs.star().read() };
if status.dcrcfail() { if status.dcrcfail() {
return Poll::Ready(Err(Error::Crc)); return Poll::Ready(Err(Error::Crc));
@ -1019,9 +1094,9 @@ impl SdmmcInner {
if res.is_ok() { if res.is_ok() {
on_drop.defuse(); on_drop.defuse();
self.stop_datapath();
unsafe { unsafe {
regs.idmactrlr().modify(|w| w.set_idmaen(false));
let scr_bytes = &*(&scr as *const [u32; 2] as *const [u8; 8]); let scr_bytes = &*(&scr as *const [u32; 2] as *const [u8; 8]);
card.scr = SCR(u64::from_be_bytes(*scr_bytes)); card.scr = SCR(u64::from_be_bytes(*scr_bytes));
} }
@ -1030,6 +1105,7 @@ impl SdmmcInner {
} }
/// Send command to card /// Send command to card
#[allow(unused_variables)]
fn cmd(&self, cmd: Cmd, data: bool) -> Result<(), Error> { fn cmd(&self, cmd: Cmd, data: bool) -> Result<(), Error> {
let regs = self.0; let regs = self.0;
@ -1040,10 +1116,10 @@ impl SdmmcInner {
while self.cmd_active() {} while self.cmd_active() {}
// Command arg // Command arg
regs.arg().write(|w| w.set_cmdarg(cmd.arg)); regs.argr().write(|w| w.set_cmdargr(cmd.arg));
// Command index and start CP State Machine // Command index and start CP State Machine
regs.cmd().write(|w| { regs.cmdr().write(|w| {
w.set_waitint(false); w.set_waitint(false);
w.set_waitresp(cmd.resp as u8); w.set_waitresp(cmd.resp as u8);
w.set_cmdindex(cmd.cmd); w.set_cmdindex(cmd.cmd);
@ -1064,13 +1140,13 @@ impl SdmmcInner {
if cmd.resp == Response::None { if cmd.resp == Response::None {
// Wait for CMDSENT or a timeout // Wait for CMDSENT or a timeout
while { while {
status = regs.sta().read(); status = regs.star().read();
!(status.ctimeout() || status.cmdsent()) !(status.ctimeout() || status.cmdsent())
} {} } {}
} else { } else {
// Wait for CMDREND or CCRCFAIL or a timeout // Wait for CMDREND or CCRCFAIL or a timeout
while { while {
status = regs.sta().read(); status = regs.star().read();
!(status.ctimeout() || status.cmdrend() || status.ccrcfail()) !(status.ctimeout() || status.cmdrend() || status.ccrcfail())
} {} } {}
} }
@ -1096,10 +1172,10 @@ impl SdmmcInner {
while self.cmd_active() {} while self.cmd_active() {}
// Command arg // Command arg
regs.arg().write(|w| w.set_cmdarg(0)); regs.argr().write(|w| w.set_cmdargr(0));
// Command index and start CP State Machine // Command index and start CP State Machine
regs.cmd().write(|w| { regs.cmdr().write(|w| {
w.set_waitint(false); w.set_waitint(false);
w.set_waitresp(Response::Short as u8); w.set_waitresp(Response::Short as u8);
w.set_cmdindex(12); w.set_cmdindex(12);
@ -1118,7 +1194,7 @@ impl SdmmcInner {
} }
self.data_interrupts(false); self.data_interrupts(false);
self.clear_interrupt_flags(); self.clear_interrupt_flags();
regs.idmactrlr().modify(|w| w.set_idmaen(false)); self.stop_datapath();
} }
} }
@ -1224,7 +1300,7 @@ pub(crate) mod sealed {
pub trait Pins<T: Instance> {} pub trait Pins<T: Instance> {}
} }
pub trait Instance: sealed::Instance + 'static {} pub trait Instance: sealed::Instance + RccPeripheral + 'static {}
pin_trait!(CkPin, Instance); pin_trait!(CkPin, Instance);
pin_trait!(CmdPin, Instance); pin_trait!(CmdPin, Instance);
pin_trait!(D0Pin, Instance); pin_trait!(D0Pin, Instance);

View File

@ -424,7 +424,10 @@ impl<'d, T: Instance, Tx, Rx> Spi<'d, T, Tx, Rx> {
let tx_request = self.txdma.request(); let tx_request = self.txdma.request();
let tx_dst = T::REGS.tx_ptr(); let tx_dst = T::REGS.tx_ptr();
unsafe { self.txdma.start_write(tx_request, data, tx_dst) } unsafe {
self.txdma
.start_write(tx_request, data, tx_dst, Default::default())
}
let tx_f = Transfer::new(&mut self.txdma); let tx_f = Transfer::new(&mut self.txdma);
unsafe { unsafe {
@ -470,7 +473,10 @@ impl<'d, T: Instance, Tx, Rx> Spi<'d, T, Tx, Rx> {
let rx_request = self.rxdma.request(); let rx_request = self.rxdma.request();
let rx_src = T::REGS.rx_ptr(); let rx_src = T::REGS.rx_ptr();
unsafe { self.rxdma.start_read(rx_request, rx_src, data) }; unsafe {
self.rxdma
.start_read(rx_request, rx_src, data, Default::default())
};
let rx_f = Transfer::new(&mut self.rxdma); let rx_f = Transfer::new(&mut self.rxdma);
let tx_request = self.txdma.request(); let tx_request = self.txdma.request();
@ -532,12 +538,18 @@ impl<'d, T: Instance, Tx, Rx> Spi<'d, T, Tx, Rx> {
let rx_request = self.rxdma.request(); let rx_request = self.rxdma.request();
let rx_src = T::REGS.rx_ptr(); let rx_src = T::REGS.rx_ptr();
unsafe { self.rxdma.start_read(rx_request, rx_src, read) }; unsafe {
self.rxdma
.start_read(rx_request, rx_src, read, Default::default())
};
let rx_f = Transfer::new(&mut self.rxdma); let rx_f = Transfer::new(&mut self.rxdma);
let tx_request = self.txdma.request(); let tx_request = self.txdma.request();
let tx_dst = T::REGS.tx_ptr(); let tx_dst = T::REGS.tx_ptr();
unsafe { self.txdma.start_write(tx_request, write, tx_dst) } unsafe {
self.txdma
.start_write(tx_request, write, tx_dst, Default::default())
}
let tx_f = Transfer::new(&mut self.txdma); let tx_f = Transfer::new(&mut self.txdma);
unsafe { unsafe {

View File

@ -23,3 +23,6 @@ nb = "1.0.0"
usb-device = "0.2" usb-device = "0.2"
usbd-serial = "0.1.1" usbd-serial = "0.1.1"
[profile.release]
debug = 2

View File

@ -0,0 +1,52 @@
#![no_std]
#![no_main]
#![feature(type_alias_impl_trait)]
#[path = "../example_common.rs"]
mod example_common;
use embassy::executor::Spawner;
use embassy_stm32::sdmmc::{self, Sdmmc};
use embassy_stm32::time::U32Ext;
use embassy_stm32::{interrupt, Config, Peripherals};
use example_common::*;
fn config() -> Config {
let mut config = Config::default();
config.rcc.hse = Some(8.mhz().into());
config.rcc.hclk = Some(48.mhz().into());
config.rcc.pclk2 = Some(48.mhz().into());
config.rcc.pll48 = true;
config
}
#[embassy::main(config = "config()")]
async fn main(_spawner: Spawner, p: Peripherals) -> ! {
info!("Hello World, dude!");
let irq = interrupt::take!(SDIO);
let mut config = sdmmc::Config::default();
config.hclk = 48.mhz().into();
config.kernel_clk = 48.mhz().into();
let mut sdmmc = unsafe {
Sdmmc::new(
p.SDIO,
(p.PC12, p.PD2, p.PC8, p.PC9, p.PC10, p.PC11),
irq,
config,
p.DMA2_CH3,
)
};
info!("Configured clock: {}", sdmmc.clock.0);
unwrap!(sdmmc.init_card(25.mhz()).await);
let card = unwrap!(sdmmc.card());
info!("Card: {:#?}", Debug2Format(card));
loop {}
}

@ -1 +1 @@
Subproject commit 9a23a90cc038ce657611770d919c26e89719d1f3 Subproject commit 2b8eb83c7aa01200f8215248793da2489209116f