added i2c support for stm32l475

Author: diogo

lib/atca_command.c

@@ -788,9 +788,9 @@ void deleteATCACommand(ATCACommand *ca_cmd)    // destructor
  * \param[out] crc_le  Pointer to the place where the two-bytes of CRC will be
  *                     returned in little-endian byte order.
  */
-void atCRC(size_t length, const uint8_t *data, uint8_t *crc_le)
+void atCRC(uint32_t length, const uint8_t *data, uint8_t *crc_le)
 {
-    size_t counter;
+    uint32_t counter;
     uint16_t crc_register = 0;
     uint16_t polynom = 0x8005;
     uint8_t shift_register;

lib/atca_command.h

@@ -184,7 +184,7 @@ void deleteATCACommand(ATCACommand *);        // destructor
 /*---- end of ATCACommand ----*/
 
 // command helpers
-void atCRC(size_t length, const uint8_t *data, uint8_t *crc_le);
+void atCRC(uint32_t length, const uint8_t *data, uint8_t *crc_le);
 void atCalcCrc(ATCAPacket *pkt);
 ATCA_STATUS atCheckCrc(const uint8_t *response);
 

lib/hal/atca_hal.c

@@ -1,8 +1,11 @@
 /**
  * \file
- * \brief low-level HAL - methods used to setup indirection to physical layer interface.
- * this level does the dirty work of abstracting the higher level ATCAIFace methods from the
- * low-level physical interfaces.  Its main goal is to keep low-level details from bleeding into
+ * \brief low-level HAL - methods used to setup indirection to physical layer
+ * interface.
+ * this level does the dirty work of abstracting the higher level ATCAIFace
+ * methods from the
+ * low-level physical interfaces.  Its main goal is to keep low-level details
+ * from bleeding into
  * the logical interface implemetation.
  *
  * \copyright (c) 2017 Microchip Technology Inc. and its subsidiaries.
@@ -32,165 +35,172 @@
  * TERMS.
  */
 
+/* when incorporating ATCA HAL into your application, you need to adjust the
+ * #defines in atca_hal.h to include
+ * and exclude appropriate interfaces - this optimizes memory use when not using
+ * a specific iface implementation in your application */
 
-/* when incorporating ATCA HAL into your application, you need to adjust the #defines in atca_hal.h to include
- * and exclude appropriate interfaces - this optimizes memory use when not using a specific iface implementation in your application */
-
-#include "cryptoauthlib.h"
 #include "atca_hal.h"
+#include "cryptoauthlib.h"
+#include "hal_i2c_stm32l475.h"
+
+#define ATCA_HAL_I2C
+void atca_delay_us(uint32_t delay) { HAL_Delay(0); }
+void atca_delay_10us(uint32_t delay) { HAL_Delay(0); }
+void atca_delay_ms(uint32_t delay) { HAL_Delay(0); }
 
 /** \brief Standard HAL API for ATCA to initialize a physical interface
  * \param[in] cfg pointer to ATCAIfaceCfg object
  * \param[in] hal pointer to ATCAHAL_t intermediate data structure
  *  \return ATCA_SUCCESS on success, otherwise an error code.
  */
 
-ATCA_STATUS hal_iface_init(ATCAIfaceCfg *cfg, ATCAHAL_t *hal)
-{
-    // Because C isn't a real object oriented language or dynamically typed, some switch in the overall system is unavoidable
-    // The key here is to provide the flexibility to include just the types of interfaces you want/need without breaking the
-    // object model.  The former is needed so in an embedded, constrained memory environment, you don't have to pay the price
-    // (in terms of memory) for interfaces you don't use in your application.
-    ATCA_STATUS status = ATCA_COMM_FAIL;
+ATCA_STATUS hal_iface_init(ATCAIfaceCfg *cfg, ATCAHAL_t *hal) {
+  // Because C isn't a real object oriented language or dynamically typed, some
+  // switch in the overall system is unavoidable
+  // The key here is to provide the flexibility to include just the types of
+  // interfaces you want/need without breaking the
+  // object model.  The former is needed so in an embedded, constrained memory
+  // environment, you don't have to pay the price
+  // (in terms of memory) for interfaces you don't use in your application.
+  ATCA_STATUS status = ATCA_COMM_FAIL;
 
-    switch (cfg->iface_type)
-    {
-    case ATCA_I2C_IFACE:
-        #ifdef ATCA_HAL_I2C
-        hal->halinit = &hal_i2c_init;
-        hal->halpostinit = &hal_i2c_post_init;
-        hal->halreceive = &hal_i2c_receive;
-        hal->halsend = &hal_i2c_send;
-        hal->halsleep = &hal_i2c_sleep;
-        hal->halwake = &hal_i2c_wake;
-        hal->halidle = &hal_i2c_idle;
-        hal->halrelease = &hal_i2c_release;
-        hal->hal_data = NULL;
+  switch (cfg->iface_type) {
+  case ATCA_I2C_IFACE:
+#ifdef ATCA_HAL_I2C
+    hal->halinit = &hal_i2c_init;
+    hal->halpostinit = &hal_i2c_post_init;
+    hal->halreceive = &hal_i2c_receive;
+    hal->halsend = &hal_i2c_send;
+    hal->halsleep = &hal_i2c_sleep;
+    hal->halwake = &hal_i2c_wake;
+    hal->halidle = &hal_i2c_idle;
+    hal->halrelease = &hal_i2c_release;
+    hal->hal_data = NULL;
 
-        status = ATCA_SUCCESS;
-        #endif
-        break;
-    case ATCA_SWI_IFACE:
-        #ifdef ATCA_HAL_SWI
-        hal->halinit = &hal_swi_init;
-        hal->halpostinit = &hal_swi_post_init;
-        hal->halreceive = &hal_swi_receive;
-        hal->halsend = &hal_swi_send;
-        hal->halsleep = &hal_swi_sleep;
-        hal->halwake = &hal_swi_wake;
-        hal->halidle = &hal_swi_idle;
-        hal->halrelease = &hal_swi_release;
-        hal->hal_data = NULL;
+    status = ATCA_SUCCESS;
+#endif
+    break;
+  case ATCA_SWI_IFACE:
+#ifdef ATCA_HAL_SWI
+    hal->halinit = &hal_swi_init;
+    hal->halpostinit = &hal_swi_post_init;
+    hal->halreceive = &hal_swi_receive;
+    hal->halsend = &hal_swi_send;
+    hal->halsleep = &hal_swi_sleep;
+    hal->halwake = &hal_swi_wake;
+    hal->halidle = &hal_swi_idle;
+    hal->halrelease = &hal_swi_release;
+    hal->hal_data = NULL;
 
-        status = ATCA_SUCCESS;
-        #endif
-        break;
-    case ATCA_UART_IFACE:
-        #ifdef ATCA_HAL_UART
-        // TODO - initialize UART iface
-        #endif
-        #ifdef ATCA_HAL_KIT_CDC
-        hal->halinit = &hal_kit_cdc_init;
-        hal->halpostinit = &hal_kit_cdc_post_init;
-        hal->halreceive = &hal_kit_cdc_receive;
-        hal->halsend = &hal_kit_cdc_send;
-        hal->halsleep = &hal_kit_cdc_sleep;
-        hal->halwake = &hal_kit_cdc_wake;
-        hal->halidle = &hal_kit_cdc_idle;
-        hal->halrelease = &hal_kit_cdc_release;
-        hal->hal_data = NULL;
+    status = ATCA_SUCCESS;
+#endif
+    break;
+  case ATCA_UART_IFACE:
+#ifdef ATCA_HAL_UART
+// TODO - initialize UART iface
+#endif
+#ifdef ATCA_HAL_KIT_CDC
+    hal->halinit = &hal_kit_cdc_init;
+    hal->halpostinit = &hal_kit_cdc_post_init;
+    hal->halreceive = &hal_kit_cdc_receive;
+    hal->halsend = &hal_kit_cdc_send;
+    hal->halsleep = &hal_kit_cdc_sleep;
+    hal->halwake = &hal_kit_cdc_wake;
+    hal->halidle = &hal_kit_cdc_idle;
+    hal->halrelease = &hal_kit_cdc_release;
+    hal->hal_data = NULL;
 
-        status = ATCA_SUCCESS;
-        #endif
-        break;
-    case ATCA_SPI_IFACE:
-        #ifdef ATCA_HAL_SPI
-        // TODO - initialize SPI iface
-        #endif
-        break;
-    case ATCA_HID_IFACE:
-        #ifdef ATCA_HAL_KIT_HID
-        hal->halinit = &hal_kit_hid_init;
-        hal->halpostinit = &hal_kit_hid_post_init;
-        hal->halreceive = &hal_kit_hid_receive;
-        hal->halsend = &hal_kit_hid_send;
-        hal->halsleep = &hal_kit_hid_sleep;
-        hal->halwake = &hal_kit_hid_wake;
-        hal->halidle = &hal_kit_hid_idle;
-        hal->halrelease = &hal_kit_hid_release;
-        hal->hal_data = NULL;
+    status = ATCA_SUCCESS;
+#endif
+    break;
+  case ATCA_SPI_IFACE:
+#ifdef ATCA_HAL_SPI
+// TODO - initialize SPI iface
+#endif
+    break;
+  case ATCA_HID_IFACE:
+#ifdef ATCA_HAL_KIT_HID
+    hal->halinit = &hal_kit_hid_init;
+    hal->halpostinit = &hal_kit_hid_post_init;
+    hal->halreceive = &hal_kit_hid_receive;
+    hal->halsend = &hal_kit_hid_send;
+    hal->halsleep = &hal_kit_hid_sleep;
+    hal->halwake = &hal_kit_hid_wake;
+    hal->halidle = &hal_kit_hid_idle;
+    hal->halrelease = &hal_kit_hid_release;
+    hal->hal_data = NULL;
 
-        status = ATCA_SUCCESS;
-        #endif
-        break;
-    case ATCA_CUSTOM_IFACE:
-        #ifdef ATCA_HAL_CUSTOM
-        hal->halinit = cfg->atcacustom.halinit;
-        hal->halpostinit = cfg->atcacustom.halpostinit;
-        hal->halreceive = cfg->atcacustom.halreceive;
-        hal->halsend = cfg->atcacustom.halsend;
-        hal->halsleep = cfg->atcacustom.halsleep;
-        hal->halwake = cfg->atcacustom.halwake;
-        hal->halidle = cfg->atcacustom.halidle;
-        hal->halrelease = cfg->atcacustom.halrelease;
-        hal->hal_data = NULL;
+    status = ATCA_SUCCESS;
+#endif
+    break;
+  case ATCA_CUSTOM_IFACE:
+#ifdef ATCA_HAL_CUSTOM
+    hal->halinit = cfg->atcacustom.halinit;
+    hal->halpostinit = cfg->atcacustom.halpostinit;
+    hal->halreceive = cfg->atcacustom.halreceive;
+    hal->halsend = cfg->atcacustom.halsend;
+    hal->halsleep = cfg->atcacustom.halsleep;
+    hal->halwake = cfg->atcacustom.halwake;
+    hal->halidle = cfg->atcacustom.halidle;
+    hal->halrelease = cfg->atcacustom.halrelease;
+    hal->hal_data = NULL;
 
-        status = ATCA_SUCCESS;
-        #endif
-        break;
-    default:
-        break;
-    }
-    return status;
+    status = ATCA_SUCCESS;
+#endif
+    break;
+  default:
+    break;
+  }
+  return status;
 }
 
 /** \brief releases a physical interface, HAL knows how to interpret hal_data
  * \param[in] iface_type - the type of physical interface to release
- * \param[in] hal_data - pointer to opaque hal data maintained by HAL implementation for this interface type
+ * \param[in] hal_data - pointer to opaque hal data maintained by HAL
+ * implementation for this interface type
  *  \return ATCA_SUCCESS on success, otherwise an error code.
  */
 
-ATCA_STATUS hal_iface_release(ATCAIfaceType iface_type, void *hal_data)
-{
-    ATCA_STATUS status = ATCA_GEN_FAIL;
+ATCA_STATUS hal_iface_release(ATCAIfaceType iface_type, void *hal_data) {
+  ATCA_STATUS status = ATCA_GEN_FAIL;
 
-    switch (iface_type)
-    {
-    case ATCA_I2C_IFACE:
+  switch (iface_type) {
+  case ATCA_I2C_IFACE:
 #ifdef ATCA_HAL_I2C
-        status = hal_i2c_release(hal_data);
+    status = hal_i2c_release(hal_data);
 #endif
-        break;
-    case ATCA_SWI_IFACE:
- #ifdef ATCA_HAL_SWI
-        status = hal_swi_release(hal_data);
+    break;
+  case ATCA_SWI_IFACE:
+#ifdef ATCA_HAL_SWI
+    status = hal_swi_release(hal_data);
 #endif
-        break;
-    case ATCA_UART_IFACE:
- #ifdef ATCA_HAL_UART
-        // TODO - release HAL UART
+    break;
+  case ATCA_UART_IFACE:
+#ifdef ATCA_HAL_UART
+// TODO - release HAL UART
 #endif
 #ifdef ATCA_HAL_KIT_CDC
-        status = hal_kit_cdc_release(hal_data);
+    status = hal_kit_cdc_release(hal_data);
 #endif
-        break;
-    case ATCA_SPI_IFACE:
+    break;
+  case ATCA_SPI_IFACE:
 #ifdef ATCA_HAL_SPI
-        // TODO - release HAL SPI
+// TODO - release HAL SPI
 #endif
-        break;
-    case ATCA_HID_IFACE:
+    break;
+  case ATCA_HID_IFACE:
 #ifdef ATCA_HAL_KIT_HID
-        status = hal_kit_hid_release(hal_data);
+    status = hal_kit_hid_release(hal_data);
 #endif
-        break;
-    case ATCA_CUSTOM_IFACE:
+    break;
+  case ATCA_CUSTOM_IFACE:
 #ifdef ATCA_HAL_CUSTOM
 #endif
-        break;
-    default:
-        break;
-    }
+    break;
+  default:
+    break;
+  }
 
-    return status;
+  return status;
 }