Add serial::Read implementation

Author: rnestler

Cargo.toml

@@ -13,6 +13,8 @@ embedded-hal = { version = "0.2.0", features = ["unproven"] }
 i2cdev = "0.4.1"
 spidev = "0.3.0"
 sysfs_gpio = "0.5.1"
+serial = "0.4.0"
+nb = "0.1.1"
 
 [dependencies.cast]
 # we don't need the `Error` implementation

src/lib.rs

@@ -17,6 +17,8 @@ extern crate embedded_hal as hal;
 pub extern crate i2cdev;
 pub extern crate spidev;
 pub extern crate sysfs_gpio;
+pub extern crate serial;
+pub extern crate nb;
 
 use std::io::{self, Write};
 use std::path::{Path, PathBuf};
@@ -27,6 +29,8 @@ use cast::{u32, u64};
 use i2cdev::core::I2CDevice;
 use spidev::SpidevTransfer;
 
+pub mod serial_impl;
+
 /// Empty struct that provides delay functionality on top of `thread::sleep`
 pub struct Delay;
 

src/serial_impl.rs

@@ -0,0 +1,39 @@
+//! Implementation of [`Serial`](https://docs.rs/embedded-hal/0.2.1/embedded_hal/serial/index.html)
+
+use hal::serial::Read;
+use nb;
+use serial;
+
+/// Newtype around [`serial::SystemPort`] that implements the `embedded-hal` traits
+pub struct Serial(pub serial::SystemPort);
+
+impl Read<u8> for Serial {
+    type Error = serial::Error;
+
+    fn read(&mut self) -> nb::Result<u8, Self::Error> {
+        use std::io::Read;
+        let mut buffer = [0; 1];
+        let bytes_read = self
+            .0
+            .read(&mut buffer)
+            .map_err(|err| nb::Error::Other(Self::Error::from(err)))?;
+        if bytes_read == 1 {
+            Ok(buffer[0])
+        } else {
+            Err(nb::Error::WouldBlock)
+        }
+    }
+}
+
+#[cfg(test)]
+mod test {
+    use super::*;
+
+    #[test]
+    fn test_empty() {
+        let mut port: Box<Read<u8, Error = serial::Error>> =
+            Box::new(Serial(serial::open("/dev/tty1").unwrap()));
+
+        port.read().unwrap();
+    }
+}