i2c-axxia: check for error conditions first
authorAdamski, Krzysztof (Nokia - PL/Wroclaw) <krzysztof.adamski@nokia.com>
Mon, 10 Dec 2018 15:01:27 +0000 (15:01 +0000)
committerWolfram Sang <wsa@the-dreams.de>
Tue, 11 Dec 2018 19:59:17 +0000 (20:59 +0100)
commit4f5c85fe3a60ace555d09898166af372547f97fc
tree8da99d012d410a87b2784ef071d03f5b37dc84f6
parenteb9907789ac373fc122c4b3a3c2d3f061d9ee922
i2c-axxia: check for error conditions first

It was observed that when using seqentional mode contrary to the
documentation, the SS bit (which is supposed to only be set if
automatic/sequence command completed normally), is sometimes set
together with NA (NAK in address phase) causing transfer to falsely be
considered successful.

My assumption is that this does not happen during manual mode since the
controller is stopping its work the moment it sets NA/ND bit in status
register. This is not the case in Automatic/Sequentional mode where it
is still working to send STOP condition and the actual status we get
depends on the time when the ISR is run.

This patch changes the order of checking status bits in ISR - error
conditions are checked first and only if none of them occurred, the
transfer may be considered successful. This is required to introduce
using of sequentional mode in next patch.

Signed-off-by: Krzysztof Adamski <krzysztof.adamski@nokia.com>
Reviewed-by: Alexander Sverdlin <alexander.sverdlin@nokia.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
drivers/i2c/busses/i2c-axxia.c