Merge pull request #575 from qwandor/stderror

embedded-hal-bus: Implement `Display` and `std::error::Error` for `DeviceError` if possible.

Author: Dirbaio

embedded-hal-bus/README.md

@@ -30,7 +30,8 @@ provides mechanisms to obtain multiple `I2c` instances out of a single `I2c` ins
 
 ## Optional Cargo features
 
-- **`std`**: enable shared bus implementations using `std::sync::Mutex`.
+- **`std`**: enable shared bus implementations using `std::sync::Mutex`, and implement
+  `std::error::Error` for `DeviceError`.
 - **`async`**: enable `embedded-hal-async` support.
 - **`defmt-03`**: Derive `defmt::Format` from `defmt` 0.3 for enums and structs.
 

embedded-hal-bus/src/spi/mod.rs

@@ -1,6 +1,6 @@
 //! `SpiDevice` implementations.
 
-use core::fmt::Debug;
+use core::fmt::{self, Debug, Display, Formatter};
 use embedded_hal::spi::{Error, ErrorKind};
 
 mod exclusive;
@@ -29,6 +29,18 @@ pub enum DeviceError<BUS, CS> {
     Cs(CS),
 }
 
+impl<BUS: Display, CS: Display> Display for DeviceError<BUS, CS> {
+    fn fmt(&self, f: &mut Formatter) -> fmt::Result {
+        match self {
+            Self::Spi(bus) => write!(f, "SPI bus error: {}", bus),
+            Self::Cs(cs) => write!(f, "SPI CS error: {}", cs),
+        }
+    }
+}
+
+#[cfg(feature = "std")]
+impl<BUS: Debug + Display, CS: Debug + Display> std::error::Error for DeviceError<BUS, CS> {}
+
 impl<BUS, CS> Error for DeviceError<BUS, CS>
 where
     BUS: Error + Debug,