Andrew Lunn <and...@lunn.ch> writes: > The mv88e6xxx wait function can be upset of the system has nots of > other things to do and a sleep takes a lot longer than expected. Fix > this be using a fixed number of iterations, rather than a fixed > walkclock time. > > Witht that change made, it is possible to consoliate another > wait function. > > A wait actually timing out should not happen and when it does, it > means something serious is wrong. Make sure an error is logged, > since not all callers will log an error. > > Andrew Lunn (3): > dsa: mv88e6xxx: Timeout based on iterations, not time > dsa: mv88e6xxx: Use mv88e6xx_wait in mv88e6xxx_update() > dsa: mv88e6xxx: Make mv88e6xxx_wait() timeout verbose > > drivers/net/dsa/mv88e6xxx/chip.c | 32 +++++++++++--------------------- > 1 file changed, 11 insertions(+), 21 deletions(-)
Reviewed-by: Vivien Didelot <vivien.dide...@savoirfairelinux.com> David, this series superseeds http://patchwork.ozlabs.org/patch/660270/. Thanks, Vivien
