> +int ksz9477_set_wol(struct ksz_device *dev, int port, > + struct ethtool_wolinfo *wol) > +{ > + u8 pme_conf, pme_ctrl = 0; > + int ret; > + > + if (wol->wolopts & ~WAKE_PHY) > + return -EINVAL; EOPNOTSUPP might be better here. I'm assuming there is no other way WoL can be supported, since this is a combined MAC/PHY device. Reviewed-by: Andrew Lunn <andrew@xxxxxxx> Andrew