Fix compile error

dzvon · dzvon · commit b6b90546a5e1 · 2021-05-29T10:52:09.000+08:00
fix(09-clocks-and-timers): Compile failed in release mode

docs: Update putting-it-all-together.md

fix: compile error in chapter 11

fix: compile error in chapter 14

fix: compile error in chapter 15

fix: compile error in chapter 16

chore: handle `Result` with unwrap rather than ok

chore: remove useless dependencies

fix: restore main.rs
diff --git a/src/09-clocks-and-timers/auxiliary/Cargo.toml b/src/09-clocks-and-timers/auxiliary/Cargo.toml
@@ -8,7 +8,8 @@ version = "0.1.0"
 cortex-m = "0.6.3"
 cortex-m-rt = "0.6.3"
 panic-itm = "0.4.0"
+stm32f3-discovery = "0.6.0"
 
-[dependencies.f3]
-features = ["rt"]
-version = "0.6.1"
+[dependencies.stm32f3]
+version = "0.12.1"
+features = ["stm32f303", "rt"]
diff --git a/src/09-clocks-and-timers/auxiliary/src/lib.rs b/src/09-clocks-and-timers/auxiliary/src/lib.rs
@@ -7,26 +7,37 @@ extern crate panic_itm; // panic handler
 
 pub use cortex_m::asm::{bkpt, nop};
 pub use cortex_m_rt::entry;
-pub use f3::{
-    hal::stm32f30x::{rcc, tim6},
-    led::Leds,
-};
+pub use stm32f3::stm32f303::{rcc, tim6, RCC, TIM6};
+pub use stm32f3_discovery::switch_hal;
 
-use f3::hal::{
-    prelude::*,
-    stm32f30x::{self, RCC, TIM6},
+use stm32f3_discovery::{
+    leds::Leds,
+    stm32f3xx_hal::{prelude::*, stm32},
 };
 
 pub fn init() -> (
     Leds,
     &'static rcc::RegisterBlock,
     &'static tim6::RegisterBlock,
 ) {
-    let p = stm32f30x::Peripherals::take().unwrap();
+    let p = stm32::Peripherals::take().unwrap();
 
     let mut rcc = p.RCC.constrain();
 
-    let leds = Leds::new(p.GPIOE.split(&mut rcc.ahb));
+    let mut gpioe = p.GPIOE.split(&mut rcc.ahb);
+
+    let leds = Leds::new(
+        gpioe.pe8,
+        gpioe.pe9,
+        gpioe.pe10,
+        gpioe.pe11,
+        gpioe.pe12,
+        gpioe.pe13,
+        gpioe.pe14,
+        gpioe.pe15,
+        &mut gpioe.moder,
+        &mut gpioe.otyper,
+    );
 
     (leds, unsafe { &*RCC::ptr() }, unsafe { &*TIM6::ptr() })
 }
diff --git a/src/09-clocks-and-timers/putting-it-all-together.md b/src/09-clocks-and-timers/putting-it-all-together.md
@@ -4,7 +4,7 @@
 #![no_main]
 #![no_std]
 
-use aux9::{entry, tim6};
+use aux9::{entry, switch_hal::OutputSwitch, tim6};
 
 #[inline(never)]
 fn delay(tim6: &tim6::RegisterBlock, ms: u16) {
@@ -24,7 +24,8 @@ fn delay(tim6: &tim6::RegisterBlock, ms: u16) {
 
 #[entry]
 fn main() -> ! {
-    let (mut leds, rcc, tim6) = aux9::init();
+    let (leds, rcc, tim6) = aux9::init();
+    let mut leds = leds.into_array();
 
     // Power on the TIM6 timer
     rcc.apb1enr.modify(|_, w| w.tim6en().set_bit());
@@ -45,9 +46,9 @@ fn main() -> ! {
         for curr in 0..8 {
             let next = (curr + 1) % 8;
 
-            leds[next].on();
+            leds[next].on().unwrap();
             delay(tim6, ms);
-            leds[curr].off();
+            leds[curr].off().unwrap();
             delay(tim6, ms);
         }
     }
diff --git a/src/09-clocks-and-timers/src/main.rs b/src/09-clocks-and-timers/src/main.rs
@@ -1,7 +1,7 @@
 #![no_main]
 #![no_std]
 
-use aux9::{entry, tim6};
+use aux9::{entry, switch_hal::OutputSwitch, tim6};
 
 #[inline(never)]
 fn delay(tim6: &tim6::RegisterBlock, ms: u16) {
@@ -10,7 +10,8 @@ fn delay(tim6: &tim6::RegisterBlock, ms: u16) {
 
 #[entry]
 fn main() -> ! {
-    let (mut leds, rcc, tim6) = aux9::init();
+    let (leds, rcc, tim6) = aux9::init();
+    let mut leds = leds.into_array();
 
     // TODO initialize TIM6
 
@@ -19,9 +20,9 @@ fn main() -> ! {
         for curr in 0..8 {
             let next = (curr + 1) % 8;
 
-            leds[next].on();
+            leds[next].on().unwrap();
             delay(tim6, ms);
-            leds[curr].off();
+            leds[curr].off().unwrap();
             delay(tim6, ms);
         }
     }
diff --git a/src/11-usart/auxiliary/Cargo.toml b/src/11-usart/auxiliary/Cargo.toml
@@ -5,13 +5,10 @@ name = "aux11"
 version = "0.1.0"
 
 [dependencies]
-cortex-m = "=0.5.6" # 0.5.11 introduces a breaking change.  Use 0.5.6, since we know it works for this example
+cortex-m = "0.6.3"
 cortex-m-rt = "0.6.3"
 panic-itm = "0.4.0"
-
-[dependencies.f3]
-features = ["rt"]
-version = "0.6.1"
+stm32f3-discovery = "0.6.0"
 
 [features]
-adapter = []
+adapter = []
diff --git a/src/11-usart/auxiliary/src/lib.rs b/src/11-usart/auxiliary/src/lib.rs
@@ -7,16 +7,18 @@ extern crate panic_itm; // panic handler
 
 pub use cortex_m::{asm::bkpt, iprint, iprintln, peripheral::ITM};
 pub use cortex_m_rt::entry;
-pub use f3::hal::{prelude, serial::Serial, stm32f30x::usart1, time::MonoTimer};
+pub use stm32f3_discovery::stm32f3xx_hal::stm32::usart1;
 
-use f3::hal::{
+use stm32f3_discovery::stm32f3xx_hal::{
     prelude::*,
-    stm32f30x::{self, USART1},
+    serial::Serial,
+    stm32::{self, USART1},
+    time::MonoTimer,
 };
 
 pub fn init() -> (&'static mut usart1::RegisterBlock, MonoTimer, ITM) {
     let cp = cortex_m::Peripherals::take().unwrap();
-    let dp = stm32f30x::Peripherals::take().unwrap();
+    let dp = stm32::Peripherals::take().unwrap();
 
     let mut flash = dp.FLASH.constrain();
     let mut rcc = dp.RCC.constrain();
diff --git a/src/11-usart/buffer-overrun.md b/src/11-usart/buffer-overrun.md
@@ -3,7 +3,6 @@
 If you wrote your program like this:
 
 ``` rust
-#![deny(unsafe_code)]
 #![no_main]
 #![no_std]
 
@@ -16,7 +15,9 @@ fn main() -> ! {
 
     // Send a string
     for byte in b"The quick brown fox jumps over the lazy dog.".iter() {
-        usart1.tdr.write(|w| w.tdr().bits(u16::from(*byte)));
+        usart1
+            .tdr
+            .write(|w| unsafe { w.tdr().bits(u16::from(*byte)) });
     }
 
     loop {}
@@ -108,7 +109,6 @@ to write to the `TDR` register without incurring in data loss.
 Let's use that to slowdown the processor.
 
 ``` rust
-#![deny(unsafe_code)]
 #![no_main]
 #![no_std]
 
@@ -125,7 +125,9 @@ fn main() -> ! {
         // wait until it's safe to write to TDR
         while usart1.isr.read().txe().bit_is_clear() {} // <- NEW!
 
-        usart1.tdr.write(|w| w.tdr().bits(u16::from(*byte)));
+        usart1
+            .tdr
+            .write(|w| unsafe { w.tdr().bits(u16::from(*byte)) });
     }
     let elapsed = instant.elapsed(); // in ticks
 
diff --git a/src/11-usart/my-solution.md b/src/11-usart/my-solution.md
@@ -1,7 +1,6 @@
 # My solution
 
 ```rust
-#![deny(unsafe_code)]
 #![no_main]
 #![no_std]
 
@@ -27,7 +26,9 @@ fn main() -> ! {
                 // buffer full
                 for byte in b"error: buffer full\n\r" {
                     while usart1.isr.read().txe().bit_is_clear() {}
-                    usart1.tdr.write(|w| w.tdr().bits(u16::from(*byte)));
+                    usart1
+                        .tdr
+                        .write(|w| unsafe { w.tdr().bits(u16::from(*byte)) });
                 }
 
                 break;
@@ -38,7 +39,9 @@ fn main() -> ! {
                 // Respond
                 for byte in buffer.iter().rev().chain(&[b'\n', b'\r']) {
                     while usart1.isr.read().txe().bit_is_clear() {}
-                    usart1.tdr.write(|w| w.tdr().bits(u16::from(*byte)));
+                    usart1
+                        .tdr
+                        .write(|w| unsafe { w.tdr().bits(u16::from(*byte)) });
                 }
 
                 break;
diff --git a/src/11-usart/src/main.rs b/src/11-usart/src/main.rs
@@ -1,4 +1,3 @@
-#![deny(unsafe_code)]
 #![no_main]
 #![no_std]
 
@@ -10,7 +9,9 @@ fn main() -> ! {
     let (usart1, mono_timer, itm) = aux11::init();
 
     // Send a single character
-    usart1.tdr.write(|w| w.tdr().bits(u16::from(b'X')));
+    usart1
+        .tdr
+        .write(|w| unsafe { w.tdr().bits(u16::from(b'X')) });
 
     loop {}
 }
diff --git a/src/14-i2c/auxiliary/Cargo.toml b/src/14-i2c/auxiliary/Cargo.toml
@@ -5,10 +5,7 @@ name = "aux14"
 version = "0.1.0"
 
 [dependencies]
-cortex-m = "=0.5.6"
+cortex-m = "0.6.3"
 cortex-m-rt = "0.6.3"
 panic-itm = "0.4.0"
-
-[dependencies.f3]
-features = ["rt"]
-version = "0.6.1"
+stm32f3-discovery = "0.6.0"
diff --git a/src/14-i2c/auxiliary/src/lib.rs b/src/14-i2c/auxiliary/src/lib.rs
@@ -7,21 +7,21 @@ extern crate panic_itm; // panic handler
 
 pub use cortex_m::{asm::bkpt, iprint, iprintln};
 pub use cortex_m_rt::entry;
-pub use f3::hal::{delay::Delay, prelude, stm32f30x::i2c1};
+pub use stm32f3_discovery::stm32f3xx_hal::{delay::Delay, prelude, stm32::i2c1};
 
 use cortex_m::peripheral::ITM;
-use f3::{
-    hal::{
+use stm32f3_discovery::{
+    lsm303dlhc::Lsm303dlhc,
+    stm32f3xx_hal::{
         i2c::I2c,
         prelude::*,
-        stm32f30x::{self, I2C1},
+        stm32::{self, I2C1},
     },
-    Lsm303dlhc,
 };
 
 pub fn init() -> (&'static i2c1::RegisterBlock, Delay, ITM) {
     let cp = cortex_m::Peripherals::take().unwrap();
-    let dp = stm32f30x::Peripherals::take().unwrap();
+    let dp = stm32::Peripherals::take().unwrap();
 
     let mut flash = dp.FLASH.constrain();
     let mut rcc = dp.RCC.constrain();
@@ -32,7 +32,7 @@ pub fn init() -> (&'static i2c1::RegisterBlock, Delay, ITM) {
     let scl = gpiob.pb6.into_af4(&mut gpiob.moder, &mut gpiob.afrl);
     let sda = gpiob.pb7.into_af4(&mut gpiob.moder, &mut gpiob.afrl);
 
-    let i2c = I2c::i2c1(dp.I2C1, (scl, sda), 400.khz(), clocks, &mut rcc.apb1);
+    let i2c = I2c::new(dp.I2C1, (scl, sda), 400.khz(), clocks, &mut rcc.apb1);
 
     Lsm303dlhc::new(i2c).unwrap();
 
diff --git a/src/14-i2c/read-several-registers.md b/src/14-i2c/read-several-registers.md
@@ -53,7 +53,7 @@ Putting it all together inside a loop alongside a delay to reduce the data throu
 use aux14::{entry, iprint, iprintln, prelude::*};
 
 // Slave address
-const MAGNETOMETER: u8 = 0b001_1110;
+const MAGNETOMETER: u16 = 0b0011_1100;
 
 // Addresses of the magnetometer's registers
 const OUT_X_H_M: u8 = 0x03;
@@ -68,7 +68,7 @@ fn main() -> ! {
         // Broadcast the MAGNETOMETER address with the R/W bit set to Write
         i2c1.cr2.write(|w| {
             w.start().set_bit();
-            w.sadd1().bits(MAGNETOMETER);
+            w.sadd().bits(MAGNETOMETER);
             w.rd_wrn().clear_bit();
             w.nbytes().bits(1);
             w.autoend().clear_bit()
diff --git a/src/14-i2c/src/main.rs b/src/14-i2c/src/main.rs
@@ -6,7 +6,7 @@
 use aux14::{entry, iprint, iprintln, prelude::*};
 
 // Slave address
-const MAGNETOMETER: u8 = 0b001_1110;
+const MAGNETOMETER: u16 = 0b0011_1100;
 
 // Addresses of the magnetometer's registers
 const OUT_X_H_M: u8 = 0x03;
diff --git a/src/14-i2c/the-solution.md b/src/14-i2c/the-solution.md
@@ -9,7 +9,7 @@
 use aux14::{entry, iprint, iprintln, prelude::*};
 
 // Slave address
-const MAGNETOMETER: u8 = 0b001_1110;
+const MAGNETOMETER: u16 = 0b0011_1100;
 
 // Addresses of the magnetometer's registers
 const OUT_X_H_M: u8 = 0x03;
@@ -26,7 +26,7 @@ fn main() -> ! {
         // Broadcast the MAGNETOMETER address with the R/W bit set to Write
         i2c1.cr2.write(|w| {
             w.start().set_bit();
-            w.sadd1().bits(MAGNETOMETER);
+            w.sadd().bits(MAGNETOMETER);
             w.rd_wrn().clear_bit();
             w.nbytes().bits(1);
             w.autoend().clear_bit()
diff --git a/src/15-led-compass/auxiliary/Cargo.toml b/src/15-led-compass/auxiliary/Cargo.toml
@@ -5,10 +5,7 @@ name = "aux15"
 version = "0.1.0"
 
 [dependencies]
-cortex-m = "=0.5.6"
+cortex-m = "0.6.3"
 cortex-m-rt = "0.6.3"
 panic-itm = "0.4.0"
-
-[dependencies.f3]
-features = ["rt"]
-version = "0.6.1"
+stm32f3-discovery = "0.6.0"
diff --git a/src/15-led-compass/auxiliary/src/lib.rs b/src/15-led-compass/auxiliary/src/lib.rs
diff --git a/src/15-led-compass/solution-1.md b/src/15-led-compass/solution-1.md
diff --git a/src/15-led-compass/solution-2.md b/src/15-led-compass/solution-2.md
diff --git a/src/15-led-compass/take-1.md b/src/15-led-compass/take-1.md
diff --git a/src/15-led-compass/take-2.md b/src/15-led-compass/take-2.md
diff --git a/src/16-punch-o-meter/auxiliary/Cargo.toml b/src/16-punch-o-meter/auxiliary/Cargo.toml
diff --git a/src/16-punch-o-meter/auxiliary/src/lib.rs b/src/16-punch-o-meter/auxiliary/src/lib.rs
