Re: [PATCH v2 net-next] drivers: net: davinci_mdio: prevent spurious timeout

2018-05-09 Thread Sekhar Nori
On Wednesday 09 May 2018 07:00 PM, Andrew Lunn wrote: > On Wed, May 09, 2018 at 04:30:24PM +0530, Sekhar Nori wrote: >> A well timed kernel preemption in the time_after() loop >> in wait_for_idle() can result in a spurious timeout >> error to be returned. >> >> Fix it by using readl_poll_timeout()

Re: [PATCH v2 net-next] drivers: net: davinci_mdio: prevent spurious timeout

2018-05-09 Thread Andrew Lunn
On Wed, May 09, 2018 at 04:30:24PM +0530, Sekhar Nori wrote: > A well timed kernel preemption in the time_after() loop > in wait_for_idle() can result in a spurious timeout > error to be returned. > > Fix it by using readl_poll_timeout() which takes care of > this issue. > > Signed-off-by: Sekhar

[PATCH v2 net-next] drivers: net: davinci_mdio: prevent spurious timeout

2018-05-09 Thread Sekhar Nori
A well timed kernel preemption in the time_after() loop in wait_for_idle() can result in a spurious timeout error to be returned. Fix it by using readl_poll_timeout() which takes care of this issue. Signed-off-by: Sekhar Nori --- v2: use readl_poll_timeout() per suggestion from Andrew. The issu