Re: [PATCH] i2c: uniphier-f: fix misdetection of incomplete STOP condition

From: Masahiro Yamada
Date: Fri Sep 23 2016 - 09:34:00 EST


Hi Wolfram,


2016-09-23 21:12 GMT+09:00 Wolfram Sang <wsa@xxxxxxxxxxxxx>:
> On Fri, Sep 23, 2016 at 09:04:01PM +0900, Masahiro Yamada wrote:
>> Currently, the status register FI2C_SR is checked immediately after
>> a STOP condition is issued in case of the deferred STOP condition.
>> It takes typically 5-10 usec until the corresponding bits in the
>> register are set, so the error check for "stop condition was not
>> completed" is very likely to be false positive.
>>
>> Add wait code to relax the status register check.
>>
>> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
>
> Use readl_poll_timeout?
>

Yes, thanks for your advice!



--
Best Regards
Masahiro Yamada