net: thunderx: Don't set RX_PACKET_DIS while initializing
authorSunil Goutham <sgoutham@cavium.com>
Fri, 12 Aug 2016 11:21:44 +0000 (16:51 +0530)
committerDavid S. Miller <davem@davemloft.net>
Sat, 13 Aug 2016 18:59:33 +0000 (11:59 -0700)
Setting BGXX_SPUX_MISC_CONTROL::RX_PACKET_DIS is not needed as
packet reception is anyway disabled by BGXX_CMRX_CONFIG::DATA_PKT_RX_EN.
Also setting RX_PACKET_DIS causes a bogus remote fault condition
which delays link detection.

Signed-off-by: Sunil Goutham <sgoutham@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cavium/thunder/thunder_bgx.c

index ca58d7f52ecdddd83851b1631f6efdb2ae15d45d..8bbaedbb7b946353470f4bb57138f542025b068b 100644 (file)
@@ -459,12 +459,14 @@ static int bgx_lmac_xaui_init(struct bgx *bgx, struct lmac *lmac)
 
        bgx_reg_modify(bgx, lmacid, BGX_SPUX_CONTROL1, SPU_CTL_LOW_POWER);
        /* Set interleaved running disparity for RXAUI */
-       if (lmac->lmac_type != BGX_MODE_RXAUI)
-               bgx_reg_modify(bgx, lmacid,
-                              BGX_SPUX_MISC_CONTROL, SPU_MISC_CTL_RX_DIS);
-       else
+       if (lmac->lmac_type == BGX_MODE_RXAUI)
                bgx_reg_modify(bgx, lmacid, BGX_SPUX_MISC_CONTROL,
-                              SPU_MISC_CTL_RX_DIS | SPU_MISC_CTL_INTLV_RDISP);
+                              SPU_MISC_CTL_INTLV_RDISP);
+
+       /* Clear receive packet disable */
+       cfg = bgx_reg_read(bgx, lmacid, BGX_SPUX_MISC_CONTROL);
+       cfg &= ~SPU_MISC_CTL_RX_DIS;
+       bgx_reg_write(bgx, lmacid, BGX_SPUX_MISC_CONTROL, cfg);
 
        /* clear all interrupts */
        cfg = bgx_reg_read(bgx, lmacid, BGX_SMUX_RX_INT);
@@ -537,7 +539,6 @@ static int bgx_xaui_check_link(struct lmac *lmac)
        int lmac_type = lmac->lmac_type;
        u64 cfg;
 
-       bgx_reg_modify(bgx, lmacid, BGX_SPUX_MISC_CONTROL, SPU_MISC_CTL_RX_DIS);
        if (lmac->use_training) {
                cfg = bgx_reg_read(bgx, lmacid, BGX_SPUX_INT);
                if (!(cfg & (1ull << 13))) {
@@ -607,11 +608,6 @@ static int bgx_xaui_check_link(struct lmac *lmac)
                return -1;
        }
 
-       /* Clear receive packet disable */
-       cfg = bgx_reg_read(bgx, lmacid, BGX_SPUX_MISC_CONTROL);
-       cfg &= ~SPU_MISC_CTL_RX_DIS;
-       bgx_reg_write(bgx, lmacid, BGX_SPUX_MISC_CONTROL, cfg);
-
        /* Check for MAC RX faults */
        cfg = bgx_reg_read(bgx, lmacid, BGX_SMUX_RX_CTL);
        /* 0 - Link is okay, 1 - Local fault, 2 - Remote fault */