On Tue, Oct 22, 2019 at 11:11:07AM +0000, Thomas Hämmerle wrote: > + if (wol->wolopts & WAKE_MAGICSECURE) { > + phy_write_mmd(phydev, DP83867_DEVADDR, DP83867_RXFSOP1, > + (wol->sopass[1] << 8) | wol->sopass[0]); > + phy_write_mmd(phydev, DP83867_DEVADDR, DP83867_RXFSOP1, > + (wol->sopass[3] << 8) | wol->sopass[2]); > + phy_write_mmd(phydev, DP83867_DEVADDR, DP83867_RXFSOP1, > + (wol->sopass[5] << 8) | wol->sopass[3]);
Hi Thomas I see sopass[3] twice here. Is that a typo? Andrew