Bring in defmt derive for error types

hal/Cargo.toml

@@ -62,6 +62,10 @@ version = "0.3"
 default-features = false
 version = "1.0"
 
+[dependencies.defmt]
+version = "0.3"
+optional = true
+
 [dependencies.embedded-hal-async]
 version = "0.1.0-alpha.3"
 optional = true
@@ -251,6 +255,7 @@ unproven = ["embedded-hal/unproven"]
 use_rtt = ["jlink_rtt"]
 usb = ["usb-device"]
 dma = ["unproven"]
+defmt = ["dep:defmt"]
 max-channels = ["dma"]
 sdmmc = ["embedded-sdmmc"]
 rtic = ["rtic-monotonic", "fugit"]

hal/src/dmac/mod.rs

@@ -258,6 +258,7 @@ pub use dma_controller::*;
 pub use transfer::*;
 
 #[derive(Debug, Clone, Copy)]
+#[cfg_attr(feature = "defmt", derive(defmt::Format))]
 /// Runtime errors that may occur when dealing with DMA transfers.
 pub enum Error {
     /// Supplied buffers both have lengths > 1 beat, but not equal to each other

hal/src/sercom/i2c/flags.rs

@@ -74,6 +74,7 @@ impl Status {
 
 /// Errors available for I2C transactions
 #[derive(Debug, Clone, Copy)]
+#[cfg_attr(feature = "defmt", derive(defmt::Format))]
 pub enum Error {
     BusError,
     ArbitrationLost,

hal/src/sercom/spi.rs

@@ -461,6 +461,7 @@ impl TryFrom<Status> for () {
 /// The SPI peripheral only has two error types, buffer overflow and transaction
 /// length error.
 #[derive(Debug)]
+#[cfg_attr(feature = "defmt", derive(defmt::Format))]
 pub enum Error {
     Overflow,
     LengthError,

hal/src/sercom/uart/flags.rs

@@ -85,6 +85,7 @@ bitflags! {
 
 /// Errors available for UART transactions
 #[derive(Debug, Clone, Copy)]
+#[cfg_attr(feature = "defmt", derive(defmt::Format))]
 pub enum Error {
     /// Detected a parity error
     ParityError,