> +     if (eee->eee_enabled) {
> +             ret = phy_init_eee(phydev, 0);
> +             if (ret) {
> +                     netif_err(adapter, drv, adapter->netdev,
> +                               "EEE initialization failed\n");
> +                     return ret;
> +             }
> +
> +             buf = lan743x_csr_read(adapter, MAC_CR);
> +             buf |= MAC_CR_EEE_EN_;
> +             lan743x_csr_write(adapter, MAC_CR, buf);
> +
> +             phy_ethtool_set_eee(phydev, eee);

Hi Bryan

It could return an error, so please check the return value. Maybe move
this call out of the if to keep the code minimal?

     Andrew

Reply via email to