TI DaVinci EMAC: Fix incorrect reference to EMAC_CTRL registers.

The EMAC modules control registers vary as per the version of the
EMAC module. EMAC_CTRL_EWCTL,EMAC_CTRL_EWINTTCNT are available
only on EMAC_VERSION_1. The emac_dump_regs() function accesses
these indiscriminately. This patch fixes the issue.

Signed-off-by: Sriramakrishnan <srk@ti.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c
index 2ebf1a1..2fe709f 100644 (file)
--- a/drivers/net/davinci_emac.c
+++ b/drivers/net/davinci_emac.c
@@ -564,9 +564,11 @@ static void emac_dump_regs(struct emac_priv *priv)
 
        /* Print important registers in EMAC */
        dev_info(emac_dev, "EMAC Basic registers\n");
-       dev_info(emac_dev, "EMAC: EWCTL: %08X, EWINTTCNT: %08X\n",
-               emac_ctrl_read(EMAC_CTRL_EWCTL),
-               emac_ctrl_read(EMAC_CTRL_EWINTTCNT));
+       if (priv->version == EMAC_VERSION_1) {
+               dev_info(emac_dev, "EMAC: EWCTL: %08X, EWINTTCNT: %08X\n",
+                       emac_ctrl_read(EMAC_CTRL_EWCTL),
+                       emac_ctrl_read(EMAC_CTRL_EWINTTCNT));
+       }
        dev_info(emac_dev, "EMAC: TXID: %08X %s, RXID: %08X %s\n",
                emac_read(EMAC_TXIDVER),
                ((emac_read(EMAC_TXCONTROL)) ? "enabled" : "disabled"),