Move LinkSpeed to phy because it's likely that more 10BaseT/100BaseT PHY will

be supported

Author: Johannes Draaijer

src/phy/lan87xxa.rs

@@ -1,29 +1,13 @@
 //! SMSC LAN87xxA (LAN8742A, LAN8720A) Ethernet PHYs
-
 use core::convert::TryFrom;
-use num_enum::{IntoPrimitive, TryFromPrimitive};
 
-use crate::mac::StationManagement;
+use crate::{mac::StationManagement, LinkSpeed};
 
 /// SMSC LAN8720A Ethernet PHY
 pub type LAN8720A<SMI> = LAN87xxA<SMI, false>;
 /// SMSC LAN8742A Ethernet PHY
 pub type LAN8742A<SMI> = LAN87xxA<SMI, true>;
 
-/// The link speeds supported by this PHY
-#[derive(Clone, Copy, Debug, IntoPrimitive, TryFromPrimitive)]
-#[repr(u8)]
-pub enum LinkSpeed {
-    /// 10BaseT - Half duplex
-    BaseT10HalfDuplex = 0b001,
-    /// 10BaseT - Full duplex
-    BaseT10FullDuplex = 0b101,
-    /// 100BaseT - Half duplex
-    BaseT100HalfDuplex = 0b010,
-    /// 100BaseT - Full duplex
-    BaseT100FullDuplex = 0b110,
-}
-
 use self::phy_consts::*;
 #[allow(dead_code)]
 mod phy_consts {

src/phy/mod.rs

@@ -1,8 +1,24 @@
+use num_enum::{IntoPrimitive, TryFromPrimitive};
+
 #[cfg(feature = "phy-lan87xxa")]
 mod lan87xxa;
 #[cfg(feature = "phy-lan87xxa")]
 pub use lan87xxa::*;
 
+/// The link speeds supported by this PHY
+#[derive(Clone, Copy, Debug, IntoPrimitive, TryFromPrimitive)]
+#[repr(u8)]
+pub enum LinkSpeed {
+    /// 10BaseT - Half duplex
+    BaseT10HalfDuplex = 0b001,
+    /// 10BaseT - Full duplex
+    BaseT10FullDuplex = 0b101,
+    /// 100BaseT - Half duplex
+    BaseT100HalfDuplex = 0b010,
+    /// 100BaseT - Full duplex
+    BaseT100FullDuplex = 0b110,
+}
+
 pub trait Phy {
     type LinkSpeed;
 
@@ -18,7 +34,7 @@ pub trait Phy {
 
     /// Poll the link status of this PHY. If it returns `true`, the
     /// link is up. If it returns `false`, the link is down.
-    /// 
+    ///
     // TODO: return an associated type with link information?
     fn poll_link(&mut self) -> bool;