Thread (4 messages) 4 messages, 4 authors, 2016-04-01

RE: [PATCH] fec: Do not access unexisting register in Coldfire

From: Fugang Duan <hidden>
Date: 2016-04-01 01:54:58

From: Fabio Estevam <festevam@gmail.com> Sent: Thursday, March 31, 2016 11:05 PM
quoted hunk ↗ jump to hunk
To: davem@davemloft.net
Cc: Fugang Duan <redacted>; troy.kisky@boundarydevices.com;
gerg@uclinux.org; netdev@vger.kernel.org; Fabio Estevam
[off-list ref]
Subject: [PATCH] fec: Do not access unexisting register in Coldfire

From: Fabio Estevam <redacted>

Commit 55cd48c821de ("net: fec: stop the "rcv is not +last, " error
messages") introduces a write to a register that does not exist in Coldfire.

Move the FEC_FTRL register access inside the FEC_QUIRK_HAS_RACC 'if' block,
so that we guarantee it will not be used on Coldfire CPUs.

Reported-by: Greg Ungerer <redacted>
Signed-off-by: Fabio Estevam <redacted>
---
 drivers/net/ethernet/freescale/fec_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/freescale/fec_main.c
b/drivers/net/ethernet/freescale/fec_main.c
index 37c0815..08243c2 100644
--- a/drivers/net/ethernet/freescale/fec_main.c
+++ b/drivers/net/ethernet/freescale/fec_main.c
@@ -943,8 +943,8 @@ fec_restart(struct net_device *ndev)
 		else
 			val &= ~FEC_RACC_OPTIONS;
 		writel(val, fep->hwp + FEC_RACC);
+		writel(PKT_MAXBUF_SIZE, fep->hwp + FEC_FTRL);
 	}
-	writel(PKT_MAXBUF_SIZE, fep->hwp + FEC_FTRL);
 #endif

 	/*
--
1.9.1
If you stick to do it like this,  you must add comments on the quirk flag FEC_QUIRK_HAS_RACC. 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help