Subject: Re: [PATCH net-next] drivers: net: davinci_mdio: prevent sprious timeout

From: Andrew Lunn <[email-masked]>
Date: Tue, 8 May 2018 14:48:56 +0200

On Tue, May 08, 2018 at 01:56:38PM +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 checking for status of hardware before returning
> timeout error.
> 
> Signed-off-by: Sekhar Nori <[email-masked]>

I've seen this with other drivers as well.

I suggest you make use of readx_poll_timeout(), or one of its
cousins. They get this right.

	 Andrew