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.cb/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.