adds timeout to avr i2c

[qmk_firmware.git] / drivers / avr / i2c_master.c

diff --git a/drivers/avr/i2c_master.c b/drivers/avr/i2c_master.c
index f4a4bb7b0be474e9d486e5382d94042778ac2606..97f69004363f62e8c67fa53fb735c192065a5197 100755 (executable)
--- a/drivers/avr/i2c_master.c
+++ b/drivers/avr/i2c_master.c
@@ -6,6 +6,7 @@
 #include <util/twi.h>
 
 #include "i2c_master.h"
+#include "timer.h"
 
 #define F_SCL 400000UL // SCL frequency
 #define Prescaler 1
@@ -13,32 +14,54 @@
 
 void i2c_init(void)
 {
+  TWSR = 0;     /* no prescaler */
        TWBR = (uint8_t)TWBR_val;
+  //TWBR = 10;
 }
 
 uint8_t i2c_start(uint8_t address)
 {
        // reset TWI control register
        TWCR = 0;
-       // transmit START condition 
+       // transmit START condition
        TWCR = (1<<TWINT) | (1<<TWSTA) | (1<<TWEN);
-       // wait for end of transmission
-       while( !(TWCR & (1<<TWINT)) );
-       
+
+  #ifdef I2C_TIMEOUT
+    uint16_t timeout_timer = timer_read();
+    while( !(TWCR & (1<<TWINT)) ) {
+      if ((timer_read() - timeout_timer) > I2C_TIMEOUT) {
+        return 2; // should make these codes standard
+      }
+    }
+  #else
+  // wait for end of transmission
+    while( !(TWCR & (1<<TWINT)) );
+  #endif
+
        // check if the start condition was successfully transmitted
-       if((TWSR & 0xF8) != TW_START){ return 1; }
-       
+       if(((TW_STATUS & 0xF8) != TW_START) && ((TW_STATUS & 0xF8) != TW_REP_START)){ return 1; }
+
        // load slave address into data register
        TWDR = address;
        // start transmission of address
        TWCR = (1<<TWINT) | (1<<TWEN);
-       // wait for end of transmission
-       while( !(TWCR & (1<<TWINT)) );
-       
+
+  #ifdef I2C_TIMEOUT
+    timeout_timer = timer_read();
+    while( !(TWCR & (1<<TWINT)) ) {
+      if ((timer_read() - timeout_timer) > I2C_TIMEOUT) {
+        return 2; // should make these codes standard
+      }
+    }
+  #else
+  // wait for end of transmission
+    while( !(TWCR & (1<<TWINT)) );
+  #endif
+
        // check if the device has acknowledged the READ / WRITE mode
        uint8_t twst = TW_STATUS & 0xF8;
        if ( (twst != TW_MT_SLA_ACK) && (twst != TW_MR_SLA_ACK) ) return 1;
-       
+
        return 0;
 }
 
@@ -48,32 +71,64 @@ uint8_t i2c_write(uint8_t data)
        TWDR = data;
        // start transmission of data
        TWCR = (1<<TWINT) | (1<<TWEN);
+
+  #ifdef I2C_TIMEOUT
+    uint16_t timeout_timer = timer_read();
+    while( !(TWCR & (1<<TWINT)) ) {
+      if ((timer_read() - timeout_timer) > I2C_TIMEOUT) {
+        return 2; // should make these codes standard
+      }
+    }
+  #else
        // wait for end of transmission
-       while( !(TWCR & (1<<TWINT)) );
-       
-       if( (TWSR & 0xF8) != TW_MT_DATA_ACK ){ return 1; }
-       
+    while( !(TWCR & (1<<TWINT)) );
+  #endif
+
+       if( (TW_STATUS & 0xF8) != TW_MT_DATA_ACK ){ return 1; }
+
        return 0;
 }
 
 uint8_t i2c_read_ack(void)
 {
-       
+
        // start TWI module and acknowledge data after reception
-       TWCR = (1<<TWINT) | (1<<TWEN) | (1<<TWEA); 
-       // wait for end of transmission
-       while( !(TWCR & (1<<TWINT)) );
+       TWCR = (1<<TWINT) | (1<<TWEN) | (1<<TWEA);
+
+  #ifdef I2C_TIMEOUT
+    uint16_t timeout_timer = timer_read();
+    while( !(TWCR & (1<<TWINT)) ) {
+      if ((timer_read() - timeout_timer) > I2C_TIMEOUT) {
+        return 2; // should make these codes standard
+      }
+    }
+  #else
+  // wait for end of transmission
+    while( !(TWCR & (1<<TWINT)) );
+  #endif
+
        // return received data from TWDR
        return TWDR;
 }
 
 uint8_t i2c_read_nack(void)
 {
-       
+
        // start receiving without acknowledging reception
        TWCR = (1<<TWINT) | (1<<TWEN);
-       // wait for end of transmission
-       while( !(TWCR & (1<<TWINT)) );
+
+    #ifdef I2C_TIMEOUT
+    uint16_t timeout_timer = timer_read();
+    while( !(TWCR & (1<<TWINT)) ) {
+      if ((timer_read() - timeout_timer) > I2C_TIMEOUT) {
+        return 2; // should make these codes standard
+      }
+    }
+  #else
+  // wait for end of transmission
+    while( !(TWCR & (1<<TWINT)) );
+  #endif
+
        // return received data from TWDR
        return TWDR;
 }
@@ -81,29 +136,29 @@ uint8_t i2c_read_nack(void)
 uint8_t i2c_transmit(uint8_t address, uint8_t* data, uint16_t length)
 {
        if (i2c_start(address | I2C_WRITE)) return 1;
-       
+
        for (uint16_t i = 0; i < length; i++)
        {
                if (i2c_write(data[i])) return 1;
        }
-       
+
        i2c_stop();
-       
+
        return 0;
 }
 
 uint8_t i2c_receive(uint8_t address, uint8_t* data, uint16_t length)
 {
        if (i2c_start(address | I2C_READ)) return 1;
-       
+
        for (uint16_t i = 0; i < (length-1); i++)
        {
                data[i] = i2c_read_ack();
        }
        data[(length-1)] = i2c_read_nack();
-       
+
        i2c_stop();
-       
+
        return 0;
 }
 
@@ -142,8 +197,22 @@ uint8_t i2c_readReg(uint8_t devaddr, uint8_t regaddr, uint8_t* data, uint16_t le
        return 0;
 }
 
-void i2c_stop(void)
+uint8_t i2c_stop(void)
 {
        // transmit STOP condition
        TWCR = (1<<TWINT) | (1<<TWEN) | (1<<TWSTO);
+
+  #ifdef I2C_TIMEOUT
+    uint16_t timeout_timer = timer_read();
+    while(TWCR & (1<<TWSTO)) {
+        if ((timer_read() - timeout_timer) > I2C_TIMEOUT) {
+        return 2; // should make these codes standard
+      }
+    }
+  #else
+    // wait for end of transmission
+    while(TWCR & (1<<TWSTO));
+  #endif
+
+  return 0;
 }