Re: [RFC PATCH] net: macb: Apply RXUBR workaround only to versions with errata

2018-11-29 Thread Claudiu.Beznea
On 29.11.2018 12:38, Harini Katakam wrote: > Hi Claudiu, > On Thu, Nov 29, 2018 at 3:51 PM wrote: >> >> >> >> On 23.11.2018 11:59, Harini Katakam wrote: > >>> - if (status & MACB_BIT(RXUBR)) { >>> + if ((bp->errata & MACB_ERRATA_RXLOCKUP) && >>> + (status

Re: [RFC PATCH] net: macb: Apply RXUBR workaround only to versions with errata

2018-11-29 Thread Harini Katakam
Hi Claudiu, On Thu, Nov 29, 2018 at 3:51 PM wrote: > > > > On 23.11.2018 11:59, Harini Katakam wrote: > > - if (status & MACB_BIT(RXUBR)) { > > + if ((bp->errata & MACB_ERRATA_RXLOCKUP) && > > + (status & MACB_BIT(RXUBR))) { > > Just asking, did you manage

Re: [RFC PATCH] net: macb: Apply RXUBR workaround only to versions with errata

2018-11-29 Thread Claudiu.Beznea
On 23.11.2018 11:59, Harini Katakam wrote: > The interrupt handler contains a workaround for RX hang applicable > to Zynq and AT91 only. Subsequent versions do not need this > workaround. This workaround unecessarily reset RX whenever RX used > bit read is observed, which can be often under heavy

Re: [RFC PATCH] net: macb: Apply RXUBR workaround only to versions with errata

2018-11-29 Thread Claudiu.Beznea
On 28.11.2018 23:09, Brandon Streiff wrote: > On 11/23/2018 3:59 AM, Harini Katakam wrote: >> +/* Errata mask bits */ >> +#define MACB_ERRATA_RXLOCKUP0x0001 >> + >> /* LSO settings */ >> #define MACB_LSO_UFO_ENABLE 0x01 >> #define MACB_LSO_TSO_ENABLE

Re: [RFC PATCH] net: macb: Apply RXUBR workaround only to versions with errata

2018-11-28 Thread Harini Katakam
Hi Brandon, On Thu, Nov 29, 2018 at 2:39 AM Brandon Streiff wrote: > > On 11/23/2018 3:59 AM, Harini Katakam wrote: > > +/* Errata mask bits */ > > +#define MACB_ERRATA_RXLOCKUP 0x0001 > > + > > /* LSO settings */ > > #define MACB_LSO_UFO_ENABLE 0x01 > > #d

Re: [RFC PATCH] net: macb: Apply RXUBR workaround only to versions with errata

2018-11-28 Thread Brandon Streiff
On 11/23/2018 3:59 AM, Harini Katakam wrote: > +/* Errata mask bits */ > +#define MACB_ERRATA_RXLOCKUP 0x0001 > + > /* LSO settings */ > #define MACB_LSO_UFO_ENABLE 0x01 > #define MACB_LSO_TSO_ENABLE 0x02 > @@ -1085,6 +1088,7 @@ struct macb_c

[RFC PATCH] net: macb: Apply RXUBR workaround only to versions with errata

2018-11-23 Thread Harini Katakam
The interrupt handler contains a workaround for RX hang applicable to Zynq and AT91 only. Subsequent versions do not need this workaround. This workaround unecessarily reset RX whenever RX used bit read is observed, which can be often under heavy traffic.Hence introduce an errata field and a check