i2c-davinci: Fix race when setting up for TX
authorJon Povey <jon.povey@racelogic.co.uk>
Fri, 17 Sep 2010 03:02:11 +0000 (12:02 +0900)
committerBen Dooks <ben-linux@fluff.org>
Mon, 27 Sep 2010 23:05:08 +0000 (00:05 +0100)
commit4bba0fd8d1c6d405df666e2573e1a1f917098be0
treeaf3128819f126fc9333e7883af04952badef6ee6
parentcc33e54290ed845904dac4b047934207738f0205
i2c-davinci: Fix race when setting up for TX

When setting up to transmit, a race exists between the ISR and
i2c_davinci_xfer_msg() trying to load the first byte and adjust counters.
This is mostly visible for transmits > 1 byte long.

The hardware starts sending immediately that MDR is loaded. IMR trickery
doesn't work because if we start sending, finish the first byte and an
XRDY event occurs before we load IMR to unmask it, we never get an
interrupt, and we timeout.

Move the MDR load after DXR,IMR loads to avoid this race without locking.

Tested on DM355 connected to Techwell TW2836 and Wolfson WM8985

Signed-off-by: Jon Povey <jon.povey@racelogic.co.uk>
Signed-off-by: Ben Dooks <ben-linux@fluff.org>
drivers/i2c/busses/i2c-davinci.c