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>

1 files changed, 5 insertions, 3 deletions

diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c
index 2ebf1a1dd1e1..2fe709f98cc4 100644
--- 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"),