Hi Ezequiel, On 16 May 2016 at 17:41, Ezequiel Garcia <[email protected]> wrote: > Commit f748be531d70 ("stmmac: support new GMAC4") reverted a previous fix > by mistake. This commit re-applies said fix: > > commit dec2165ff38a99f937fe61875d102c6c8596c815 > Author: Sonic Zhang <[email protected]> > Date: Thu Jan 22 14:55:57 2015 +0800 > stmmac: hardware TX COE doesn't work when force_thresh_dma_mode is set > > Clear the TX COE bit when force_thresh_dma_mode is set even hardware > dma capability says support. > > Tested on BF609. > > Signed-off-by: Sonic Zhang <[email protected]> > Acked-by: Giuseppe Cavallaro <[email protected]> > Signed-off-by: David S. Miller <[email protected]> > > Tested on LPC4350 Hitex board. > > Fixes: f748be531d70 ("stmmac: support new GMAC4") > Signed-off-by: Ezequiel Garcia <[email protected]> > --- > drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > index fd5ab7bfdb76..eac45d0c75e2 100644 > --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > @@ -3131,7 +3131,12 @@ static int stmmac_hw_init(struct stmmac_priv *priv) > priv->plat->enh_desc = priv->dma_cap.enh_desc; > priv->plat->pmt = priv->dma_cap.pmt_remote_wake_up; > > - priv->plat->tx_coe = priv->dma_cap.tx_coe; > + /* TXCOE doesn't work in thresh DMA mode */ > + if (priv->plat->force_thresh_dma_mode) > + priv->plat->tx_coe = 0; > + else > + priv->plat->tx_coe = priv->dma_cap.tx_coe; > + > /* In case of GMAC4 rx_coe is from HW cap register. */ > priv->plat->rx_coe = priv->dma_cap.rx_coe;
Works for me on EA4357 board together with another patch to fix the DT properties for DWMAC on lpc18xx. Tested-by: Joachim Eastwood <[email protected]> regards, Joachim Eastwood
