Move sysfs Pin to its own module

Author: almusil

src/lib.rs

@@ -33,7 +33,11 @@ use spidev::SpidevTransfer;
 
 mod serial;
 
+/// Sysfs Pin wrapper module
+pub mod sysfs_pin;
+
 pub use serial::Serial;
+pub use sysfs_pin::Pin;
 
 /// Empty struct that provides delay functionality on top of `thread::sleep`
 pub struct Delay;
@@ -92,72 +96,6 @@ impl hal::blocking::delay::DelayMs<u64> for Delay {
     }
 }
 
-/// Newtype around [`sysfs_gpio::Pin`] that implements the `embedded-hal` traits
-///
-/// [`sysfs_gpio::Pin`]: https://docs.rs/sysfs_gpio/0.5.1/sysfs_gpio/struct.Pin.html
-pub struct Pin(pub sysfs_gpio::Pin);
-
-impl Pin {
-    /// See [`sysfs_gpio::Pin::new`][0] for details.
-    ///
-    /// [0]: https://docs.rs/sysfs_gpio/0.5.1/sysfs_gpio/struct.Pin.html#method.new
-    pub fn new(pin_num: u64) -> Pin {
-        Pin(sysfs_gpio::Pin::new(pin_num))
-    }
-
-    /// See [`sysfs_gpio::Pin::from_path`][0] for details.
-    ///
-    /// [0]: https://docs.rs/sysfs_gpio/0.5.1/sysfs_gpio/struct.Pin.html#method.from_path
-    pub fn from_path<P>(path: P) -> sysfs_gpio::Result<Pin>
-    where
-        P: AsRef<Path>,
-    {
-        sysfs_gpio::Pin::from_path(path).map(Pin)
-    }
-}
-
-impl hal::digital::v2::OutputPin for Pin {
-    type Error = sysfs_gpio::Error;
-
-    fn set_low(&mut self) -> Result<(), Self::Error> {
-        self.0.set_value(0)
-    }
-
-    fn set_high(&mut self) -> Result<(), Self::Error> {
-        self.0.set_value(1)
-    }
-}
-
-impl hal::digital::v2::InputPin for Pin {
-    type Error = sysfs_gpio::Error;
-
-    fn is_high(&self) -> Result<bool, Self::Error> {
-        if !self.0.get_active_low()? {
-            self.0.get_value().map(|val| val != 0)
-        } else {
-            self.0.get_value().map(|val| val == 0)
-        }
-    }
-
-    fn is_low(&self) -> Result<bool, Self::Error> {
-        self.is_high().map(|val| !val)
-    }
-}
-
-impl ops::Deref for Pin {
-    type Target = sysfs_gpio::Pin;
-
-    fn deref(&self) -> &Self::Target {
-        &self.0
-    }
-}
-
-impl ops::DerefMut for Pin {
-    fn deref_mut(&mut self) -> &mut Self::Target {
-        &mut self.0
-    }
-}
-
 /// Newtype around [`i2cdev::linux::LinuxI2CDevice`] that implements the `embedded-hal` traits
 ///
 /// [`i2cdev::linux::LinuxI2CDevice`]: https://docs.rs/i2cdev/0.3.1/i2cdev/linux/struct.LinuxI2CDevice.html

src/sysfs_pin.rs

@@ -0,0 +1,68 @@
+use std::ops;
+use std::path::Path;
+
+/// Newtype around [`sysfs_gpio::Pin`] that implements the `embedded-hal` traits
+///
+/// [`sysfs_gpio::Pin`]: https://docs.rs/sysfs_gpio/0.5.1/sysfs_gpio/struct.Pin.html
+pub struct Pin(pub sysfs_gpio::Pin);
+
+impl Pin {
+    /// See [`sysfs_gpio::Pin::new`][0] for details.
+    ///
+    /// [0]: https://docs.rs/sysfs_gpio/0.5.1/sysfs_gpio/struct.Pin.html#method.new
+    pub fn new(pin_num: u64) -> Pin {
+        Pin(sysfs_gpio::Pin::new(pin_num))
+    }
+
+    /// See [`sysfs_gpio::Pin::from_path`][0] for details.
+    ///
+    /// [0]: https://docs.rs/sysfs_gpio/0.5.1/sysfs_gpio/struct.Pin.html#method.from_path
+    pub fn from_path<P>(path: P) -> sysfs_gpio::Result<Pin>
+    where
+        P: AsRef<Path>,
+    {
+        sysfs_gpio::Pin::from_path(path).map(Pin)
+    }
+}
+
+impl hal::digital::v2::OutputPin for Pin {
+    type Error = sysfs_gpio::Error;
+
+    fn set_low(&mut self) -> Result<(), Self::Error> {
+        self.0.set_value(0)
+    }
+
+    fn set_high(&mut self) -> Result<(), Self::Error> {
+        self.0.set_value(1)
+    }
+}
+
+impl hal::digital::v2::InputPin for Pin {
+    type Error = sysfs_gpio::Error;
+
+    fn is_high(&self) -> Result<bool, Self::Error> {
+        if !self.0.get_active_low()? {
+            self.0.get_value().map(|val| val != 0)
+        } else {
+            self.0.get_value().map(|val| val == 0)
+        }
+    }
+
+    fn is_low(&self) -> Result<bool, Self::Error> {
+        self.is_high().map(|val| !val)
+    }
+}
+
+impl ops::Deref for Pin {
+    type Target = sysfs_gpio::Pin;
+
+    fn deref(&self) -> &Self::Target {
+        &self.0
+    }
+}
+
+impl ops::DerefMut for Pin {
+    fn deref_mut(&mut self) -> &mut Self::Target {
+        &mut self.0
+    }
+}