Retry the read if not successfull

AntonioLangiu · AntonioLangiu · commit 7d9aa735ddf0 · 2018-01-13T17:25:46.000+01:00
diff --git a/hal/hal_cc2650_i2c_contiki.c b/hal/hal_cc2650_i2c_contiki.c
@@ -81,10 +81,22 @@ ATCA_STATUS hal_i2c_send( ATCAIface iface, uint8_t* txdata, int txlength)
 ATCA_STATUS hal_i2c_receive( ATCAIface iface, uint8_t *rxdata, uint16_t *rxlength )
 {
     const ATCAIfaceCfg *cfg = atgetifacecfg(iface);
+    int retries = cfg->rx_retries;
+    bool success = false;
     board_i2c_select(DEFAULT_I2C_INTERFACE, cfg->atcai2c.slave_address >> 1);
-    bool success = board_i2c_read(rxdata, *rxlength);
+    while (retries-- > 0 && !success) {
+        // Wait 5 ms between one read and the other
+        atca_delay_ms(5);
+        success = board_i2c_read(rxdata, *rxlength);
+    }
     board_i2c_deselect();
-    return success? ATCA_SUCCESS: ATCA_COMM_FAIL;
+    if (!success) {
+        return ATCA_COMM_FAIL;
+    }
+    if (atCheckCrc(rxdata) != ATCA_SUCCESS) {
+        return ATCA_COMM_FAIL;
+    }
+    return ATCA_SUCCESS;
 }
 
 ATCA_STATUS hal_i2c_wake( ATCAIface iface) {
diff --git a/hal/hal_cc2650_timer_contiki.c b/hal/hal_cc2650_timer_contiki.c
@@ -1,6 +1,5 @@
 #include "atca_hal.h"
 #include "rtimer.h"
-#include <stdio.h>
 
 /** \defgroup hal_ Hardware abstraction layer (hal_)
  *

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,5 @@`
`1`	`1`	`#include "atca_hal.h"`
`2`	`2`	`#include "rtimer.h"`
`3`		`-#include <stdio.h>`
`4`	`3`
`5`	`4`	`/** \defgroup hal_ Hardware abstraction layer (hal_)`
`6`	`5`	`*`