Saving the settings of EEE to avoid they become the default settings
after reset_resume().

Signed-off-by: Hayes Wang <hayesw...@realtek.com>
---
 drivers/net/usb/r8152.c | 80 +++++++++++++++++++++++++----------------
 1 file changed, 50 insertions(+), 30 deletions(-)

diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c
index 1aa61610f0bb..a7aa48bee732 100644
--- a/drivers/net/usb/r8152.c
+++ b/drivers/net/usb/r8152.c
@@ -751,6 +751,7 @@ struct r8152 {
 
        atomic_t rx_count;
 
+       bool eee_en;
        int intr_interval;
        u32 saved_wolopts;
        u32 msg_enable;
@@ -762,6 +763,7 @@ struct r8152 {
 
        u16 ocp_base;
        u16 speed;
+       u16 eee_adv;
        u8 *intr_buff;
        u8 version;
        u8 duplex;
@@ -3202,8 +3204,13 @@ static void r8152_eee_en(struct r8152 *tp, bool enable)
 
 static void r8152b_enable_eee(struct r8152 *tp)
 {
-       r8152_eee_en(tp, true);
-       r8152_mmd_write(tp, MDIO_MMD_AN, MDIO_AN_EEE_ADV, MDIO_EEE_100TX);
+       if (tp->eee_en) {
+               r8152_eee_en(tp, true);
+               r8152_mmd_write(tp, MDIO_MMD_AN, MDIO_AN_EEE_ADV, tp->eee_adv);
+       } else {
+               r8152_eee_en(tp, false);
+               r8152_mmd_write(tp, MDIO_MMD_AN, MDIO_AN_EEE_ADV, 0);
+       }
 }
 
 static void r8152b_enable_fc(struct r8152 *tp)
@@ -3495,8 +3502,10 @@ static void r8153_hw_phy_cfg(struct r8152 *tp)
        sram_write(tp, SRAM_10M_AMP1, 0x00af);
        sram_write(tp, SRAM_10M_AMP2, 0x0208);
 
-       r8153_eee_en(tp, true);
-       ocp_reg_write(tp, OCP_EEE_ADV, MDIO_EEE_1000T | MDIO_EEE_100TX);
+       if (tp->eee_en) {
+               r8153_eee_en(tp, true);
+               ocp_reg_write(tp, OCP_EEE_ADV, tp->eee_adv);
+       }
 
        r8153_aldps_en(tp, true);
        r8152b_enable_fc(tp);
@@ -3599,8 +3608,10 @@ static void r8153b_hw_phy_cfg(struct r8152 *tp)
 
        r8153b_ups_flags_w1w0(tp, ups_flags, 0);
 
-       r8153b_eee_en(tp, true);
-       ocp_reg_write(tp, OCP_EEE_ADV, MDIO_EEE_1000T | MDIO_EEE_100TX);
+       if (tp->eee_en) {
+               r8153b_eee_en(tp, true);
+               ocp_reg_write(tp, OCP_EEE_ADV, tp->eee_adv);
+       }
 
        r8153b_aldps_en(tp, true);
        r8153b_enable_fc(tp);
@@ -4891,7 +4902,7 @@ static void rtl8152_get_strings(struct net_device *dev, 
u32 stringset, u8 *data)
 
 static int r8152_get_eee(struct r8152 *tp, struct ethtool_eee *eee)
 {
-       u32 ocp_data, lp, adv, supported = 0;
+       u32 lp, adv, supported = 0;
        u16 val;
 
        val = r8152_mmd_read(tp, MDIO_MMD_PCS, MDIO_PCS_EEE_ABLE);
@@ -4903,13 +4914,10 @@ static int r8152_get_eee(struct r8152 *tp, struct 
ethtool_eee *eee)
        val = r8152_mmd_read(tp, MDIO_MMD_AN, MDIO_AN_EEE_LPABLE);
        lp = mmd_eee_adv_to_ethtool_adv_t(val);
 
-       ocp_data = ocp_read_word(tp, MCU_TYPE_PLA, PLA_EEE_CR);
-       ocp_data &= EEE_RX_EN | EEE_TX_EN;
-
-       eee->eee_enabled = !!ocp_data;
+       eee->eee_enabled = tp->eee_en;
        eee->eee_active = !!(supported & adv & lp);
        eee->supported = supported;
-       eee->advertised = adv;
+       eee->advertised = tp->eee_adv;
        eee->lp_advertised = lp;
 
        return 0;
@@ -4919,19 +4927,22 @@ static int r8152_set_eee(struct r8152 *tp, struct 
ethtool_eee *eee)
 {
        u16 val = ethtool_adv_to_mmd_eee_adv_t(eee->advertised);
 
-       r8152_eee_en(tp, eee->eee_enabled);
+       tp->eee_en = eee->eee_enabled;
+       tp->eee_adv = val;
 
-       if (!eee->eee_enabled)
-               val = 0;
+       r8152_eee_en(tp, eee->eee_enabled);
 
-       r8152_mmd_write(tp, MDIO_MMD_AN, MDIO_AN_EEE_ADV, val);
+       if (eee->eee_enabled)
+               r8152_mmd_write(tp, MDIO_MMD_AN, MDIO_AN_EEE_ADV, val);
+       else
+               r8152_mmd_write(tp, MDIO_MMD_AN, MDIO_AN_EEE_ADV, 0);
 
        return 0;
 }
 
 static int r8153_get_eee(struct r8152 *tp, struct ethtool_eee *eee)
 {
-       u32 ocp_data, lp, adv, supported = 0;
+       u32 lp, adv, supported = 0;
        u16 val;
 
        val = ocp_reg_read(tp, OCP_EEE_ABLE);
@@ -4943,13 +4954,10 @@ static int r8153_get_eee(struct r8152 *tp, struct 
ethtool_eee *eee)
        val = ocp_reg_read(tp, OCP_EEE_LPABLE);
        lp = mmd_eee_adv_to_ethtool_adv_t(val);
 
-       ocp_data = ocp_read_word(tp, MCU_TYPE_PLA, PLA_EEE_CR);
-       ocp_data &= EEE_RX_EN | EEE_TX_EN;
-
-       eee->eee_enabled = !!ocp_data;
+       eee->eee_enabled = tp->eee_en;
        eee->eee_active = !!(supported & adv & lp);
        eee->supported = supported;
-       eee->advertised = adv;
+       eee->advertised = tp->eee_adv;
        eee->lp_advertised = lp;
 
        return 0;
@@ -4959,12 +4967,15 @@ static int r8153_set_eee(struct r8152 *tp, struct 
ethtool_eee *eee)
 {
        u16 val = ethtool_adv_to_mmd_eee_adv_t(eee->advertised);
 
-       r8153_eee_en(tp, eee->eee_enabled);
+       tp->eee_en = eee->eee_enabled;
+       tp->eee_adv = val;
 
-       if (!eee->eee_enabled)
-               val = 0;
+       r8153_eee_en(tp, eee->eee_enabled);
 
-       ocp_reg_write(tp, OCP_EEE_ADV, val);
+       if (eee->eee_enabled)
+               ocp_reg_write(tp, OCP_EEE_ADV, val);
+       else
+               ocp_reg_write(tp, OCP_EEE_ADV, 0);
 
        return 0;
 }
@@ -4973,12 +4984,15 @@ static int r8153b_set_eee(struct r8152 *tp, struct 
ethtool_eee *eee)
 {
        u16 val = ethtool_adv_to_mmd_eee_adv_t(eee->advertised);
 
-       r8153b_eee_en(tp, eee->eee_enabled);
+       tp->eee_en = eee->eee_enabled;
+       tp->eee_adv = val;
 
-       if (!eee->eee_enabled)
-               val = 0;
+       r8153b_eee_en(tp, eee->eee_enabled);
 
-       ocp_reg_write(tp, OCP_EEE_ADV, val);
+       if (eee->eee_enabled)
+               ocp_reg_write(tp, OCP_EEE_ADV, val);
+       else
+               ocp_reg_write(tp, OCP_EEE_ADV, 0);
 
        return 0;
 }
@@ -5353,6 +5367,8 @@ static int rtl_ops_init(struct r8152 *tp)
                ops->hw_phy_cfg         = r8152b_hw_phy_cfg;
                ops->autosuspend_en     = rtl_runtime_suspend_enable;
                tp->rx_buf_sz           = 16 * 1024;
+               tp->eee_en              = true;
+               tp->eee_adv             = MDIO_EEE_100TX;
                break;
 
        case RTL_VER_03:
@@ -5371,6 +5387,8 @@ static int rtl_ops_init(struct r8152 *tp)
                ops->hw_phy_cfg         = r8153_hw_phy_cfg;
                ops->autosuspend_en     = rtl8153_runtime_enable;
                tp->rx_buf_sz           = 32 * 1024;
+               tp->eee_en              = true;
+               tp->eee_adv             = MDIO_EEE_1000T | MDIO_EEE_100TX;
                break;
 
        case RTL_VER_08:
@@ -5387,6 +5405,8 @@ static int rtl_ops_init(struct r8152 *tp)
                ops->hw_phy_cfg         = r8153b_hw_phy_cfg;
                ops->autosuspend_en     = rtl8153b_runtime_enable;
                tp->rx_buf_sz           = 32 * 1024;
+               tp->eee_en              = true;
+               tp->eee_adv             = MDIO_EEE_1000T | MDIO_EEE_100TX;
                break;
 
        default:
-- 
2.21.0

Reply via email to