stmhal: In HAL I2C driver, move DMA setup to after sending I2C address.

Previous to this patch the DMA was setup and then the I2C address sent.
If the I2C address sending failed (eg no I2C device on the bus) then the
DMA was left in an inconsistent state.

This patch moves the DMA setup to after a successful sending of the I2C
address(es).

See issue #1765.

1 files changed, 64 insertions, 52 deletions

diff --git a/stmhal/hal/f4/src/stm32f4xx_hal_i2c.c b/stmhal/hal/f4/src/stm32f4xx_hal_i2c.c
index 1c1a0d76d..e6774450a 100644
--- a/stmhal/hal/f4/src/stm32f4xx_hal_i2c.c
+++ b/stmhal/hal/f4/src/stm32f4xx_hal_i2c.c
@@ -1262,19 +1262,6 @@ HAL_StatusTypeDef HAL_I2C_Master_Transmit_DMA(I2C_HandleTypeDef *hi2c, uint16_t
     hi2c->State = HAL_I2C_STATE_BUSY_TX;

     hi2c->ErrorCode = HAL_I2C_ERROR_NONE;

 

-    hi2c->pBuffPtr = pData;

-    hi2c->XferSize = Size;

-    hi2c->XferCount = Size;

-

-    /* Set the I2C DMA transfert complete callback */

-    hi2c->hdmatx->XferCpltCallback = I2C_DMAMasterTransmitCplt;

-

-    /* Set the DMA error callback */

-    hi2c->hdmatx->XferErrorCallback = I2C_DMAError;

-

-    /* Enable the DMA Stream */

-    HAL_DMA_Start_IT(hi2c->hdmatx, (uint32_t)pData, (uint32_t)&hi2c->Instance->DR, Size);

-

     /* Send Slave Address */

     if(I2C_MasterRequestWrite(hi2c, DevAddress, I2C_TIMEOUT_FLAG) != HAL_OK)

     {

@@ -1292,6 +1279,22 @@ HAL_StatusTypeDef HAL_I2C_Master_Transmit_DMA(I2C_HandleTypeDef *hi2c, uint16_t
       }

     }

 

+    /* dpgeorge: DMA initialisation is moved to after sending the address */

+    {

+        hi2c->pBuffPtr = pData;

+        hi2c->XferSize = Size;

+        hi2c->XferCount = Size;

+

+        /* Set the I2C DMA transfert complete callback */

+        hi2c->hdmatx->XferCpltCallback = I2C_DMAMasterTransmitCplt;

+

+        /* Set the DMA error callback */

+        hi2c->hdmatx->XferErrorCallback = I2C_DMAError;

+

+        /* Enable the DMA Stream */

+        HAL_DMA_Start_IT(hi2c->hdmatx, (uint32_t)pData, (uint32_t)&hi2c->Instance->DR, Size);

+    }

+

     /* Enable DMA Request */

     hi2c->Instance->CR2 |= I2C_CR2_DMAEN;

 

@@ -1338,19 +1341,6 @@ HAL_StatusTypeDef HAL_I2C_Master_Receive_DMA(I2C_HandleTypeDef *hi2c, uint16_t D
     hi2c->State = HAL_I2C_STATE_BUSY_RX;

     hi2c->ErrorCode = HAL_I2C_ERROR_NONE;

 

-    hi2c->pBuffPtr = pData;

-    hi2c->XferSize = Size;

-    hi2c->XferCount = Size;

-

-    /* Set the I2C DMA transfert complete callback */

-    hi2c->hdmarx->XferCpltCallback = I2C_DMAMasterReceiveCplt;

-

-    /* Set the DMA error callback */

-    hi2c->hdmarx->XferErrorCallback = I2C_DMAError;

-

-    /* Enable the DMA Stream */

-    HAL_DMA_Start_IT(hi2c->hdmarx, (uint32_t)&hi2c->Instance->DR, (uint32_t)pData, Size);

-

     /* Send Slave Address */

     if(I2C_MasterRequestRead(hi2c, DevAddress, I2C_TIMEOUT_FLAG) != HAL_OK)

     {

@@ -1368,6 +1358,22 @@ HAL_StatusTypeDef HAL_I2C_Master_Receive_DMA(I2C_HandleTypeDef *hi2c, uint16_t D
       }

     }

 

+    /* dpgeorge: DMA initialisation is moved to after sending the address */

+    {

+        hi2c->pBuffPtr = pData;

+        hi2c->XferSize = Size;

+        hi2c->XferCount = Size;

+

+        /* Set the I2C DMA transfert complete callback */

+        hi2c->hdmarx->XferCpltCallback = I2C_DMAMasterReceiveCplt;

+

+        /* Set the DMA error callback */

+        hi2c->hdmarx->XferErrorCallback = I2C_DMAError;

+

+        /* Enable the DMA Stream */

+        HAL_DMA_Start_IT(hi2c->hdmarx, (uint32_t)&hi2c->Instance->DR, (uint32_t)pData, Size);

+    }

+

     if(Size == 1)

     {

       /* Disable Acknowledge */

@@ -2050,19 +2056,6 @@ HAL_StatusTypeDef HAL_I2C_Mem_Write_DMA(I2C_HandleTypeDef *hi2c, uint16_t DevAdd
     hi2c->State = HAL_I2C_STATE_MEM_BUSY_TX;

     hi2c->ErrorCode = HAL_I2C_ERROR_NONE;

 

-    hi2c->pBuffPtr = pData;

-    hi2c->XferSize = Size;

-    hi2c->XferCount = Size;

-

-    /* Set the I2C DMA transfert complete callback */

-    hi2c->hdmatx->XferCpltCallback = I2C_DMAMemTransmitCplt;

-

-    /* Set the DMA error callback */

-    hi2c->hdmatx->XferErrorCallback = I2C_DMAError;

-

-    /* Enable the DMA Stream */

-    HAL_DMA_Start_IT(hi2c->hdmatx, (uint32_t)pData, (uint32_t)&hi2c->Instance->DR, Size);

-

     /* Send Slave Address and Memory Address */

     if(I2C_RequestMemoryWrite(hi2c, DevAddress, MemAddress, MemAddSize, I2C_TIMEOUT_FLAG) != HAL_OK)

     {

@@ -2080,6 +2073,22 @@ HAL_StatusTypeDef HAL_I2C_Mem_Write_DMA(I2C_HandleTypeDef *hi2c, uint16_t DevAdd
       }

     }

 

+    /* dpgeorge: DMA initialisation is moved to after sending the addresses */

+    {

+        hi2c->pBuffPtr = pData;

+        hi2c->XferSize = Size;

+        hi2c->XferCount = Size;

+

+        /* Set the I2C DMA transfert complete callback */

+        hi2c->hdmatx->XferCpltCallback = I2C_DMAMemTransmitCplt;

+

+        /* Set the DMA error callback */

+        hi2c->hdmatx->XferErrorCallback = I2C_DMAError;

+

+        /* Enable the DMA Stream */

+        HAL_DMA_Start_IT(hi2c->hdmatx, (uint32_t)pData, (uint32_t)&hi2c->Instance->DR, Size);

+    }

+

     /* Enable DMA Request */

     hi2c->Instance->CR2 |= I2C_CR2_DMAEN;

 

@@ -2128,19 +2137,6 @@ HAL_StatusTypeDef HAL_I2C_Mem_Read_DMA(I2C_HandleTypeDef *hi2c, uint16_t DevAddr
     hi2c->State = HAL_I2C_STATE_MEM_BUSY_RX;

     hi2c->ErrorCode = HAL_I2C_ERROR_NONE;

 

-    hi2c->pBuffPtr = pData;

-    hi2c->XferSize = Size;

-    hi2c->XferCount = Size;

-

-    /* Set the I2C DMA transfert complete callback */

-    hi2c->hdmarx->XferCpltCallback = I2C_DMAMemReceiveCplt;

-

-    /* Set the DMA error callback */

-    hi2c->hdmarx->XferErrorCallback = I2C_DMAError;

-

-    /* Enable the DMA Stream */

-    HAL_DMA_Start_IT(hi2c->hdmarx, (uint32_t)&hi2c->Instance->DR, (uint32_t)pData, Size);

-

     /* Send Slave Address and Memory Address */

     if(I2C_RequestMemoryRead(hi2c, DevAddress, MemAddress, MemAddSize, I2C_TIMEOUT_FLAG) != HAL_OK)

     {

@@ -2158,6 +2154,22 @@ HAL_StatusTypeDef HAL_I2C_Mem_Read_DMA(I2C_HandleTypeDef *hi2c, uint16_t DevAddr
       }

     }

 

+    /* dpgeorge: DMA initialisation is moved to after sending the addresses */

+    {

+        hi2c->pBuffPtr = pData;

+        hi2c->XferSize = Size;

+        hi2c->XferCount = Size;

+

+        /* Set the I2C DMA transfert complete callback */

+        hi2c->hdmarx->XferCpltCallback = I2C_DMAMemReceiveCplt;

+

+        /* Set the DMA error callback */

+        hi2c->hdmarx->XferErrorCallback = I2C_DMAError;

+

+        /* Enable the DMA Stream */

+        HAL_DMA_Start_IT(hi2c->hdmarx, (uint32_t)&hi2c->Instance->DR, (uint32_t)pData, Size);

+    }

+

     if(Size == 1)

     {

       /* Disable Acknowledge */