> @@ -2071,6 +2071,7 @@ void phy_advertise_eee_all(struct phy_device *phydev); > void phy_support_sym_pause(struct phy_device *phydev); > void phy_support_asym_pause(struct phy_device *phydev); > void phy_support_eee(struct phy_device *phydev); > +void phy_disable_eee(struct phy_device *phydev); So we have three states: MAC tells PHYLIB it does support EEE MAC tells PHYLIB it does not support EEE MAC says nothing. Do we really want this? For phylink, i think we have a nice new clean design and can say, if the MAC does not indicate it supports EEE, we turn it off in the PHY. For phylib, we have more of a mess, and there could be MACs actually doing EEE by default using default setting but with no user space control. Do we want to keep this, or should we say any MAC which does not call phy_support_eee() before phy_start() would have EEE disabled in the PHY? Andrew
