Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-28 Thread David Miller
From: Vivien Didelot Date: Tue, 28 Mar 2017 14:13:53 -0400 > Hi David, > > Vivien Didelot writes: > >> The ATU ageing time value programmed in the switch is rounded up to the >> nearest multiple of its coefficient (variable depending on the model.) >> >> Add a debug message to inform the user

Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-28 Thread Vivien Didelot
Hi David, Vivien Didelot writes: > The ATU ageing time value programmed in the switch is rounded up to the > nearest multiple of its coefficient (variable depending on the model.) > > Add a debug message to inform the user about the exact programmed value. > > On 6352, "brctl setageing br0 18" g

Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-14 Thread Andrew Lunn
On Tue, Mar 14, 2017 at 09:56:41AM -0400, Vivien Didelot wrote: > Hi Andrew, > > Andrew Lunn writes: > > >> The never ever seeing R/W failure on MDIO bus is not exactly accurate. > >> We had with art (atheros calibration tool) the problem that interrupts > >> were being disabled which lead to MD

Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-14 Thread Vivien Didelot
Hi Andrew, Andrew Lunn writes: >> The never ever seeing R/W failure on MDIO bus is not exactly accurate. >> We had with art (atheros calibration tool) the problem that interrupts >> were being disabled which lead to MDIO operations running into >> timout/failing. > > Yes, i've seen similar with

Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-14 Thread Andrew Lunn
> Hi > The never ever seeing R/W failure on MDIO bus is not exactly accurate. > We had with art (atheros calibration tool) the problem that interrupts > were being disabled which lead to MDIO operations running into > timout/failing. Yes, i've seen similar with power management bugs for the MDIO d

Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-14 Thread Matthias May
On 13/03/17 23:58, Andrew Lunn wrote: > On Mon, Mar 13, 2017 at 03:42:36PM -0700, Florian Fainelli wrote: >> On 03/13/2017 03:39 PM, Andrew Lunn wrote: >>> On Mon, Mar 13, 2017 at 03:20:43PM -0400, Vivien Didelot wrote: The ATU ageing time value programmed in the switch is rounded up to the >>

Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-13 Thread Andrew Lunn
On Mon, Mar 13, 2017 at 03:42:36PM -0700, Florian Fainelli wrote: > On 03/13/2017 03:39 PM, Andrew Lunn wrote: > > On Mon, Mar 13, 2017 at 03:20:43PM -0400, Vivien Didelot wrote: > >> The ATU ageing time value programmed in the switch is rounded up to the > >> nearest multiple of its coefficient (v

Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-13 Thread Florian Fainelli
On 03/13/2017 03:39 PM, Andrew Lunn wrote: > On Mon, Mar 13, 2017 at 03:20:43PM -0400, Vivien Didelot wrote: >> The ATU ageing time value programmed in the switch is rounded up to the >> nearest multiple of its coefficient (variable depending on the model.) >> >> Add a debug message to inform the u

Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-13 Thread Andrew Lunn
On Mon, Mar 13, 2017 at 03:20:43PM -0400, Vivien Didelot wrote: > The ATU ageing time value programmed in the switch is rounded up to the > nearest multiple of its coefficient (variable depending on the model.) > > Add a debug message to inform the user about the exact programmed value. > > On 63

[PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

2017-03-13 Thread Vivien Didelot
The ATU ageing time value programmed in the switch is rounded up to the nearest multiple of its coefficient (variable depending on the model.) Add a debug message to inform the user about the exact programmed value. On 6352, "brctl setageing br0 18" gives "AgeTime set to 0x01 (15000 ms)" while on