Q: "the linux driver add these bits, why not we?" A: Because nobody activated the bits? Signed-off-by: Wolfram Sang <w.sang@xxxxxxxxxxxxxx> --- drivers/net/fec_imx.c | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/net/fec_imx.c b/drivers/net/fec_imx.c index 15c7a4a..45af78f 100644 --- a/drivers/net/fec_imx.c +++ b/drivers/net/fec_imx.c @@ -302,10 +302,8 @@ static int fec_init(struct eth_device *dev) if (fec->xcv_type == RMII) { if (cpu_is_mx28()) { - rcntl |= FEC_R_CNTRL_RMII_MODE; - /* the linux driver add these bits, why not we? */ - /* | FEC_R_CNTRL_FCE | */ - /* FEC_R_CNTRL_NO_LGTH_CHECK */ + rcntl |= FEC_R_CNTRL_RMII_MODE | FEC_R_CNTRL_FCE | + FEC_R_CNTRL_NO_LGTH_CHECK; } else { /* disable the gasket and wait */ writel(0, fec->regs + FEC_MIIGSK_ENR); -- 1.7.8.3 _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox