From 02dcc6ec74469523f2dc5502b1900bdf927ee1a3 Mon Sep 17 00:00:00 2001 From: Sandeep Mistry Date: Fri, 28 Aug 2015 10:25:56 -0400 Subject: check TWWC (Write Collision Flag) bit after setting TWDR as suggested by @earlyprogrammer in #2173, to ensure TWDR value is set if there is a write collision --- libraries/Wire/utility/twi.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'libraries/Wire') diff --git a/libraries/Wire/utility/twi.c b/libraries/Wire/utility/twi.c index 9fb87f1..2af0597 100644 --- a/libraries/Wire/utility/twi.c +++ b/libraries/Wire/utility/twi.c @@ -167,7 +167,9 @@ uint8_t twi_readFrom(uint8_t address, uint8_t* data, uint8_t length, uint8_t sen // up. Also, don't enable the START interrupt. There may be one pending from the // repeated start that we sent outselves, and that would really confuse things. twi_inRepStart = false; // remember, we're dealing with an ASYNC ISR - TWDR = twi_slarw; + do { + TWDR = twi_slarw; + } while(TWCR & _BV(TWWC)); TWCR = _BV(TWINT) | _BV(TWEA) | _BV(TWEN) | _BV(TWIE); // enable INTs, but not START } else @@ -247,7 +249,9 @@ uint8_t twi_writeTo(uint8_t address, uint8_t* data, uint8_t length, uint8_t wait // up. Also, don't enable the START interrupt. There may be one pending from the // repeated start that we sent outselves, and that would really confuse things. twi_inRepStart = false; // remember, we're dealing with an ASYNC ISR - TWDR = twi_slarw; + do { + TWDR = twi_slarw; + } while(TWCR & _BV(TWWC)); TWCR = _BV(TWINT) | _BV(TWEA) | _BV(TWEN) | _BV(TWIE); // enable INTs, but not START } else -- cgit v1.2.3-18-g5258