Merge pull request #317 from bobmcwhirter/peri_irq

Peri irq
This commit is contained in:
Bob McWhirter 2021-07-28 15:16:51 -04:00 committed by GitHub
commit 6e4f4b63a8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 35 additions and 66 deletions

View File

@ -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

View File

@ -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.

View File

@ -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()

View File

@ -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()

@ -1 +1 @@
Subproject commit 5412725bf5401c6d49c4932487d80463fdb02286
Subproject commit 07eae071915f8ffbf311bd5dfc6391dcca92ee10

View File

@ -50,6 +50,8 @@ pub struct Peripheral {
pub pins: Vec<Pin>,
#[serde(default)]
pub dma_channels: HashMap<String, Vec<PeripheralDmaChannel>>,
#[serde(default)]
pub interrupts: HashMap<String, String>
}
#[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()]);
}