Some register bits are not defined in the IMX CPUs using this FEC (at least I did not find anything about them in the related datasheets) and they will get (partially) a new meaning in the i.MX28 CPU. So, removing them from the source seems the best way to avoid any accidental usage on i.MX28. Signed-off-by: Juergen Beisert <jbe@xxxxxxxxxxxxxx> --- drivers/net/fec_imx.c | 3 +-- drivers/net/fec_imx.h | 4 ---- 2 files changed, 1 insertions(+), 6 deletions(-) diff --git a/drivers/net/fec_imx.c b/drivers/net/fec_imx.c index 9c8de77..91ef929 100644 --- a/drivers/net/fec_imx.c +++ b/drivers/net/fec_imx.c @@ -476,8 +476,7 @@ static int fec_recv(struct eth_device *dev) ievent = readl(fec->regs + FEC_IEVENT); writel(ievent, fec->regs + FEC_IEVENT); - if (ievent & (FEC_IEVENT_BABT | FEC_IEVENT_XFIFO_ERROR | - FEC_IEVENT_RFIFO_ERROR)) { + if (ievent & FEC_IEVENT_BABT) { /* BABT, Rx/Tx FIFO errors */ fec_halt(dev); fec_init(dev); diff --git a/drivers/net/fec_imx.h b/drivers/net/fec_imx.h index ce0fd89..f7fbdc9 100644 --- a/drivers/net/fec_imx.h +++ b/drivers/net/fec_imx.h @@ -73,8 +73,6 @@ #define FEC_IEVENT_LATE_COL 0x00200000 #define FEC_IEVENT_COL_RETRY_LIM 0x00100000 #define FEC_IEVENT_XFIFO_UN 0x00080000 -#define FEC_IEVENT_XFIFO_ERROR 0x00040000 -#define FEC_IEVENT_RFIFO_ERROR 0x00020000 #define FEC_IMASK_HBERR 0x80000000 #define FEC_IMASK_BABR 0x40000000 @@ -84,8 +82,6 @@ #define FEC_IMASK_LATE_COL 0x00200000 #define FEC_IMASK_COL_RETRY_LIM 0x00100000 #define FEC_IMASK_XFIFO_UN 0x00080000 -#define FEC_IMASK_XFIFO_ERROR 0x00040000 -#define FEC_IMASK_RFIFO_ERROR 0x00020000 #define FEC_RCNTRL_MAX_FL_SHIFT 16 #define FEC_RCNTRL_LOOP 0x01 -- 1.7.2.3 _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox