diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml index 69cd9ff1..118143da 100644 --- a/.github/workflows/rust.yml +++ b/.github/workflows/rust.yml @@ -95,24 +95,21 @@ jobs: - uses: actions/checkout@v2 with: submodules: true - - uses: actions/cache@v2 + - uses: actions-rs/toolchain@v1 with: - path: | - ~/.cargo/bin/ - ~/.cargo/registry/index/ - ~/.cargo/registry/cache/ - ~/.cargo/git/db/ - target/ - key: ${{ runner.os }}-cargo-${{ matrix.target }}-${{ hashFiles('**/Cargo.toml') }} - restore-keys: | - ${{ runner.os }}-cargo-${{ matrix.target }}- + toolchain: stable + - name: cache + id: cache-target + uses: actions/cache@v2 + with: + path: target + key: ${{ runner.os }}-${{ matrix.target }} # We have to append the "-D warnings" flag to .cargo/config rather than # using the RUSTFLAGS environment variable because if we set RUSTFLAGS # cargo will ignore the rustflags config in .cargo/config. - name: Check run: | - export CARGO_TARGET_DIR=$PWD/target mkdir -p .cargo echo -e '[target."cfg(all())"]\nrustflags = ["-D", "warnings"]' >> .cargo/config cd ${{ matrix.package }} && RUSTFLAGS=-Dwarnings cargo check --features=${{ matrix.features }} --target=${{ matrix.target }} @@ -121,6 +118,9 @@ jobs: runs-on: ubuntu-latest steps: - uses: actions/checkout@v2 + - uses: actions-rs/toolchain@v1 + with: + toolchain: stable - name: Check fmt run: for i in embassy-*; do (cd $i; cargo fmt -- --check); done @@ -128,22 +128,11 @@ jobs: runs-on: ubuntu-latest steps: - uses: actions/checkout@v2 - - uses: actions/cache@v2 + - uses: actions-rs/toolchain@v1 with: - path: | - ~/.cargo/bin/ - ~/.cargo/registry/index/ - ~/.cargo/registry/cache/ - ~/.cargo/git/db/ - target/ - key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.toml') }} - restore-keys: | - ${{ runner.os }}-cargo- + toolchain: stable - name: Test - run: | - export CARGO_TARGET_DIR=$PWD/target - cd embassy - cargo test + run: cd embassy && cargo test metapac_gen: runs-on: ubuntu-latest @@ -151,19 +140,8 @@ jobs: - uses: actions/checkout@v2 with: submodules: true - - uses: actions/cache@v2 + - uses: actions-rs/toolchain@v1 with: - path: | - ~/.cargo/bin/ - ~/.cargo/registry/index/ - ~/.cargo/registry/cache/ - ~/.cargo/git/db/ - target/ - key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.toml') }} - restore-keys: | - ${{ runner.os }}-cargo- + toolchain: stable - name: Generate pregenerated metapac - run: | - export CARGO_TARGET_DIR=$PWD/target - cd stm32-metapac-gen - cargo run --release + run: cd stm32-metapac-gen; cargo run --release diff --git a/README.md b/README.md index 8d5fddd0..13bcf6e5 100644 --- a/README.md +++ b/README.md @@ -87,3 +87,4 @@ This work is licensed under either of - MIT license ([LICENSE-MIT](LICENSE-MIT) or http://opensource.org/licenses/MIT) at your option. + diff --git a/embassy-stm32/src/dma/bdma.rs b/embassy-stm32/src/dma/bdma.rs index c52188c3..2aa00ebe 100644 --- a/embassy-stm32/src/dma/bdma.rs +++ b/embassy-stm32/src/dma/bdma.rs @@ -143,7 +143,7 @@ unsafe fn on_irq() { /// safety: must be called only once pub(crate) unsafe fn init() { pac::interrupts! { - (BDMA, $irq:ident) => { + ($peri:ident, bdma, $block:ident, $signal_name:ident, $irq:ident) => { crate::interrupt::$irq::steal().enable(); }; } @@ -242,7 +242,7 @@ pac::dma_channels! { } pac::interrupts! { - (BDMA, $irq:ident) => { + ($peri:ident, bdma, $block:ident, $signal_name:ident, $irq:ident) => { #[crate::interrupt] unsafe fn $irq () { on_irq() diff --git a/embassy-stm32/src/dma/dma.rs b/embassy-stm32/src/dma/dma.rs index 606c9d3d..47120c39 100644 --- a/embassy-stm32/src/dma/dma.rs +++ b/embassy-stm32/src/dma/dma.rs @@ -149,7 +149,7 @@ unsafe fn on_irq() { /// safety: must be called only once pub(crate) unsafe fn init() { pac::interrupts! { - (DMA, $irq:ident) => { + ($peri:ident, dma, $block:ident, $signal_name:ident, $irq:ident) => { interrupt::$irq::steal().enable(); }; } @@ -248,7 +248,7 @@ pac::dma_channels! { } pac::interrupts! { - (DMA, $irq:ident) => { + ($peri:ident, dma, $block:ident, $signal_name:ident, $irq:ident) => { #[crate::interrupt] unsafe fn $irq () { on_irq() diff --git a/stm32-data b/stm32-data index 5412725b..07eae071 160000 --- a/stm32-data +++ b/stm32-data @@ -1 +1 @@ -Subproject commit 5412725bf5401c6d49c4932487d80463fdb02286 +Subproject commit 07eae071915f8ffbf311bd5dfc6391dcca92ee10 diff --git a/stm32-metapac-gen/src/lib.rs b/stm32-metapac-gen/src/lib.rs index 203d943d..ce720c67 100644 --- a/stm32-metapac-gen/src/lib.rs +++ b/stm32-metapac-gen/src/lib.rs @@ -50,6 +50,8 @@ pub struct Peripheral { pub pins: Vec, #[serde(default)] pub dma_channels: HashMap>, + #[serde(default)] + pub interrupts: HashMap } #[derive(Debug, Eq, PartialEq, Clone, Deserialize)] @@ -306,9 +308,6 @@ pub fn gen(options: Options) { } } - let mut has_bdma = false; - let mut has_dma = false; - for (name, p) in &core.peripherals { let captures = number_suffix_re.captures(&name).unwrap(); let root_peri_name = captures.get(1).unwrap().as_str().to_string(); @@ -328,12 +327,6 @@ pub fn gen(options: Options) { if let Some(block) = &p.block { let bi = BlockInfo::parse(block); - if bi.module == "bdma" { - has_bdma = true - } else if bi.module == "dma" { - has_dma = true - } - peripheral_counts.insert( bi.module.clone(), peripheral_counts.get(&bi.module).map_or(1, |v| v + 1), @@ -352,6 +345,16 @@ pub fn gen(options: Options) { peripheral_pins_table.push(row); } + for (signal, irq_name) in &p.interrupts { + let mut row = Vec::new(); + row.push(name.clone()); + row.push(bi.module.clone()); + row.push(bi.block.clone()); + row.push( signal.clone() ); + row.push( irq_name.to_ascii_uppercase() ); + interrupt_table.push(row) + } + for (request, dma_channels) in &p.dma_channels { for channel in dma_channels.iter() { let mut row = Vec::new(); @@ -553,19 +556,6 @@ pub fn gen(options: Options) { interrupt_table.push(vec![name.clone()]); - if name.starts_with("DMA1_") || name.starts_with("DMA2_") || name.contains("_DMA") { - if has_dma { - interrupt_table.push(vec!["DMA".to_string(), name.clone()]); - } else if has_bdma { - interrupt_table.push(vec!["BDMA".to_string(), name.clone()]); - } - } - - if name.starts_with("BDMA_") || name.starts_with("BDMA1_") || name.starts_with("BDMA2_") - { - interrupt_table.push(vec!["BDMA".to_string(), name.clone()]); - } - if name.contains("EXTI") { interrupt_table.push(vec!["EXTI".to_string(), name.clone()]); }