Merge #75

75: Add linux-i2cdev to e-h::i2c::ErrorKind mapping r=eldruin a=ryankurte



Co-authored-by: ryan kurte <ryankurte@gmail.com>

Author: bors[bot]

Cargo.toml

@@ -28,6 +28,7 @@ nb = "1"
 serial-core = "0.4.0"
 serial-unix = "0.4.0"
 spidev = "0.5.1"
+nix = "0.23.1"
 
 [dev-dependencies]
 openpty = "0.2.0"

src/i2c.rs

@@ -5,6 +5,8 @@
 use std::ops;
 use std::path::{Path, PathBuf};
 
+use embedded_hal::i2c::NoAcknowledgeSource;
+
 /// Newtype around [`i2cdev::linux::LinuxI2CDevice`] that implements the `embedded-hal` traits
 ///
 /// [`i2cdev::linux::LinuxI2CDevice`]: https://docs.rs/i2cdev/0.5.0/i2cdev/linux/struct.LinuxI2CDevice.html
@@ -130,6 +132,13 @@ pub struct I2CError {
     err: i2cdev::linux::LinuxI2CError,
 }
 
+impl I2CError {
+    /// Fetch inner (concrete) [`LinuxI2CError`]
+    pub fn inner(&self) -> &i2cdev::linux::LinuxI2CError {
+        &self.err
+    }
+}
+
 impl From<i2cdev::linux::LinuxI2CError> for I2CError {
     fn from(err: i2cdev::linux::LinuxI2CError) -> Self {
         Self { err }
@@ -138,11 +147,24 @@ impl From<i2cdev::linux::LinuxI2CError> for I2CError {
 
 impl embedded_hal::i2c::Error for I2CError {
     fn kind(&self) -> embedded_hal::i2c::ErrorKind {
-        use embedded_hal::i2c::ErrorKind::*;
-        match &self.err {
-            // i2cdev::linux::LinuxI2CError::Nix(_) => todo!(),
-            // i2cdev::linux::LinuxI2CError::Io(_) => todo!(),
-            _ => Other,
+        use embedded_hal::i2c::ErrorKind;
+        use nix::errno::Errno::*;
+
+        let errno = match &self.err {
+            i2cdev::linux::LinuxI2CError::Nix(e) => *e,
+            i2cdev::linux::LinuxI2CError::Io(e) => match e.raw_os_error() {
+                Some(r) => nix::Error::from_i32(r),
+                None => return ErrorKind::Other,
+            },
+        };
+
+        // https://www.kernel.org/doc/html/latest/i2c/fault-codes.html
+        match errno {
+            EBUSY | EINVAL | EIO => ErrorKind::Bus,
+            EAGAIN => ErrorKind::ArbitrationLoss,
+            ENODEV => ErrorKind::NoAcknowledge(NoAcknowledgeSource::Data),
+            ENXIO => ErrorKind::NoAcknowledge(NoAcknowledgeSource::Address),
+            _ => ErrorKind::Other,
         }
     }
 }

src/serial.rs

@@ -62,51 +62,18 @@ pub struct SerialError {
     err: IoErrorKind,
 }
 
+impl SerialError {
+    /// Fetch inner (concrete) [`IoErrorKind`]
+    pub fn inner(&self) -> &IoErrorKind {
+        &self.err
+    }
+}
+
 impl embedded_hal::serial::Error for SerialError {
     fn kind(&self) -> embedded_hal::serial::ErrorKind {
         use embedded_hal::serial::ErrorKind::*;
         match &self.err {
-            // IoErrorKind::NotFound => todo!(),
-            // IoErrorKind::PermissionDenied => todo!(),
-            // IoErrorKind::ConnectionRefused => todo!(),
-            // IoErrorKind::ConnectionReset => todo!(),
-            // IoErrorKind::HostUnreachable => todo!(),
-            // IoErrorKind::NetworkUnreachable => todo!(),
-            // IoErrorKind::ConnectionAborted => todo!(),
-            // IoErrorKind::NotConnected => todo!(),
-            // IoErrorKind::AddrInUse => todo!(),
-            // IoErrorKind::AddrNotAvailable => todo!(),
-            // IoErrorKind::NetworkDown => todo!(),
-            // IoErrorKind::BrokenPipe => todo!(),
-            // IoErrorKind::AlreadyExists => todo!(),
-            // IoErrorKind::WouldBlock => todo!(),
-            // IoErrorKind::NotADirectory => todo!(),
-            // IoErrorKind::IsADirectory => todo!(),
-            // IoErrorKind::DirectoryNotEmpty => todo!(),
-            // IoErrorKind::ReadOnlyFilesystem => todo!(),
-            // IoErrorKind::FilesystemLoop => todo!(),
-            // IoErrorKind::StaleNetworkFileHandle => todo!(),
-            // IoErrorKind::InvalidInput => todo!(),
-            // IoErrorKind::InvalidData => todo!(),
-            // IoErrorKind::TimedOut => todo!(),
-            // IoErrorKind::WriteZero => todo!(),
-            // IoErrorKind::StorageFull => todo!(),
-            // IoErrorKind::NotSeekable => todo!(),
-            // IoErrorKind::FilesystemQuotaExceeded => todo!(),
-            // IoErrorKind::FileTooLarge => todo!(),
-            // IoErrorKind::ResourceBusy => todo!(),
-            // IoErrorKind::ExecutableFileBusy => todo!(),
-            // IoErrorKind::Deadlock => todo!(),
-            // IoErrorKind::CrossesDevices => todo!(),
-            // IoErrorKind::TooManyLinks => todo!(),
-            // IoErrorKind::FilenameTooLong => todo!(),
-            // IoErrorKind::ArgumentListTooLong => todo!(),
-            // IoErrorKind::Interrupted => todo!(),
-            // IoErrorKind::Unsupported => todo!(),
-            // IoErrorKind::UnexpectedEof => todo!(),
-            // IoErrorKind::OutOfMemory => todo!(),
-            // IoErrorKind::Other => todo!(),
-            // IoErrorKind::Uncategorized => todo!(),
+            // TODO: match any errors here if we can find any that are relevant
             _ => Other,
         }
     }

src/spi.rs

@@ -116,6 +116,13 @@ pub struct SPIError {
     err: io::Error,
 }
 
+impl SPIError {
+    /// Fetch inner (concrete) [`LinuxI2CError`]
+    pub fn inner(&self) -> &io::Error {
+        &self.err
+    }
+}
+
 impl From<io::Error> for SPIError {
     fn from(err: io::Error) -> Self {
         Self { err }
@@ -124,50 +131,10 @@ impl From<io::Error> for SPIError {
 
 impl embedded_hal::spi::Error for SPIError {
     fn kind(&self) -> embedded_hal::spi::ErrorKind {
-        use embedded_hal::spi::ErrorKind::*;
+        use embedded_hal::spi::ErrorKind;
         match self.err.kind() {
-            // io::ErrorKind::NotFound => todo!(),
-            // io::ErrorKind::PermissionDenied => todo!(),
-            // io::ErrorKind::ConnectionRefused => todo!(),
-            // io::ErrorKind::ConnectionReset => todo!(),
-            // io::ErrorKind::HostUnreachable => todo!(),
-            // io::ErrorKind::NetworkUnreachable => todo!(),
-            // io::ErrorKind::ConnectionAborted => todo!(),
-            // io::ErrorKind::NotConnected => todo!(),
-            // io::ErrorKind::AddrInUse => todo!(),
-            // io::ErrorKind::AddrNotAvailable => todo!(),
-            // io::ErrorKind::NetworkDown => todo!(),
-            // io::ErrorKind::BrokenPipe => todo!(),
-            // io::ErrorKind::AlreadyExists => todo!(),
-            // io::ErrorKind::WouldBlock => todo!(),
-            // io::ErrorKind::NotADirectory => todo!(),
-            // io::ErrorKind::IsADirectory => todo!(),
-            // io::ErrorKind::DirectoryNotEmpty => todo!(),
-            // io::ErrorKind::ReadOnlyFilesystem => todo!(),
-            // io::ErrorKind::FilesystemLoop => todo!(),
-            // io::ErrorKind::StaleNetworkFileHandle => todo!(),
-            // io::ErrorKind::InvalidInput => todo!(),
-            // io::ErrorKind::InvalidData => todo!(),
-            // io::ErrorKind::TimedOut => todo!(),
-            // io::ErrorKind::WriteZero => todo!(),
-            // io::ErrorKind::StorageFull => todo!(),
-            // io::ErrorKind::NotSeekable => todo!(),
-            // io::ErrorKind::FilesystemQuotaExceeded => todo!(),
-            // io::ErrorKind::FileTooLarge => todo!(),
-            // io::ErrorKind::ResourceBusy => todo!(),
-            // io::ErrorKind::ExecutableFileBusy => todo!(),
-            // io::ErrorKind::Deadlock => todo!(),
-            // io::ErrorKind::CrossesDevices => todo!(),
-            // io::ErrorKind::TooManyLinks => todo!(),
-            // io::ErrorKind::FilenameTooLong => todo!(),
-            // io::ErrorKind::ArgumentListTooLong => todo!(),
-            // io::ErrorKind::Interrupted => todo!(),
-            // io::ErrorKind::Unsupported => todo!(),
-            // io::ErrorKind::UnexpectedEof => todo!(),
-            // io::ErrorKind::OutOfMemory => todo!(),
-            // io::ErrorKind::Other => todo!(),
-            // io::ErrorKind::Uncategorized => todo!(),
-            _ => Other,
+            // TODO: match any errors here if we can find any that are relevant
+            _ => ErrorKind::Other,
         }
     }
 }