On Wed, Feb 06, 2019 at 04:09:28PM +0100, Maxime Chevallier wrote:
> Hello Russell,
> 
> On Wed, 06 Feb 2019 11:35:07 +0000
> Russell King <rmk+ker...@armlinux.org.uk> wrote:
> 
> 
> >+    if (pp->comphy) {
> >+            enum phy_mode mode = PHY_MODE_INVALID;
> >+
> >+            switch (state->interface) {
> >+            case PHY_INTERFACE_MODE_SGMII:
> >+            case PHY_INTERFACE_MODE_1000BASEX:
> >+                    mode = PHY_MODE_SGMII;
> 
> PHY_MODE_SGMII no longer exists, the new PHY_MODE_ETHERNET must be used
> with the correct submode now, so this doesn't build unfortunately.
> 
> >+                    break;
> >+            case PHY_INTERFACE_MODE_2500BASEX:
> >+                    mode = PHY_MODE_2500SGMII;
> 
> Same here.

Of course, I knew there was some reason I hadn't sent them yet... thanks
for spotting.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up

Reply via email to