Merge pull request #164 from Piroro-hs/i2c-fix-clock-i2c3

Fix I2C clock and support I2C3

Author: Sh3Rm4n

CHANGELOG.md

@@ -22,6 +22,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
   devices. ([#99](https://github.com/stm32-rs/stm32f3xx-hal/pull/99))
 - Support for I2C transfer of more than 255 bytes, and 0 byte write ([#154](https://github.com/stm32-rs/stm32f3xx-hal/pull/154))
 - Support for HSE bypass and CSS ([#156](https://github.com/stm32-rs/stm32f3xx-hal/pull/156))
+- Impls for missing I2C pin definitions ([#164](https://github.com/stm32-rs/stm32f3xx-hal/pull/164))
+- Support I2C3 ([#164](https://github.com/stm32-rs/stm32f3xx-hal/pull/164))
 
 ### Changed
 
@@ -33,6 +35,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 - Fixed [#151][] not being able to generate 72 MHz HCLK for stm32f303xc devices
   ([#152](https://github.com/stm32-rs/stm32f3xx-hal/pull/152))
+- Wrong I2C clock source ([#164](https://github.com/stm32-rs/stm32f3xx-hal/pull/164))
 
 [#151]: https://github.com/stm32-rs/stm32f3xx-hal/issues/151
 
@@ -47,6 +50,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
   not mean, that we guarantee any MSRV policy. It is rather for documentation
   purposes and if a new useful feature arises, we will increase the MSRV.
   ([#170](https://github.com/stm32-rs/stm32f3xx-hal/pull/170))
+- Removed I2C2 support for `stm32f303x6`, `stm32f303x8` and `stm32f328` targets. ([#164](https://github.com/stm32-rs/stm32f3xx-hal/pull/164))
+- `I2c::i2c1` and `I2c::i2c2` functions are renamed to `I2c::new`. ([#164](https://github.com/stm32-rs/stm32f3xx-hal/pull/164))
 
 ## [v0.5.0] - 2020-07-21
 

Cargo.toml

@@ -134,3 +134,7 @@ required-features = ["stm32f303"]
 [[example]]
 name = "adc"
 required-features = ["stm32f303"]
+
+[[example]]
+name = "i2c_scanner"
+required-features = ["stm32f303xc"]

examples/i2c_scanner.rs

@@ -0,0 +1,61 @@
+//! Example of using I2C.
+//! Scans available I2C devices on bus and print the result.
+//! Appropriate pull-up registers should be installed on I2C bus.
+//! Target board: STM32F3DISCOVERY
+
+#![no_std]
+#![no_main]
+
+use core::ops::Range;
+
+use panic_semihosting as _;
+
+use cortex_m::asm;
+use cortex_m_rt::entry;
+use cortex_m_semihosting::{hprint, hprintln};
+
+use stm32f3xx_hal::{self as hal, pac, prelude::*};
+
+const VALID_ADDR_RANGE: Range<u8> = 0x08..0x78;
+
+#[entry]
+fn main() -> ! {
+    let dp = pac::Peripherals::take().unwrap();
+
+    let mut flash = dp.FLASH.constrain();
+    let mut rcc = dp.RCC.constrain();
+    let clocks = rcc.cfgr.freeze(&mut flash.acr);
+
+    let mut gpiob = dp.GPIOB.split(&mut rcc.ahb);
+
+    // Configure I2C1
+    let pins = (
+        gpiob.pb6.into_af4(&mut gpiob.moder, &mut gpiob.afrl), // SCL
+        gpiob.pb7.into_af4(&mut gpiob.moder, &mut gpiob.afrl), // SDA
+    );
+    let mut i2c = hal::i2c::I2c::new(dp.I2C1, pins, 100.khz(), clocks, &mut rcc.apb1);
+
+    hprintln!("Start i2c scanning...").expect("Error using hprintln.");
+    hprintln!().unwrap();
+
+    for addr in 0x00_u8..0x80 {
+        // Write the empty array and check the slave response.
+        if VALID_ADDR_RANGE.contains(&addr) && i2c.write(addr, &[]).is_ok() {
+            hprint!("{:02x}", addr).unwrap();
+        } else {
+            hprint!("..").unwrap();
+        }
+        if addr % 0x10 == 0x0F {
+            hprintln!().unwrap();
+        } else {
+            hprint!(" ").unwrap();
+        }
+    }
+
+    hprintln!().unwrap();
+    hprintln!("Done!").unwrap();
+
+    loop {
+        asm::wfi();
+    }
+}