STM32CubeMX 6.7.0 does not configure SDMMC DMA correctly

[FPlow.1]

I am having an issue similar to https://community.st.com/s/question/0D53W000027iXpbSAE/spi-dma-configuration-generated-by-stm32cubemx-does-not-work but with the SDMMC peripheral rather than SPI.

I have an STM32CubeMX project targeting the 32F746GDISCOVERY with the following relevant peripherals and settings:

• SDMMC SD 4 Bits Wide bus
  • SDMMC1 global interrupt enabled
  • SDMMC1 bidirectional DMA request enabled on DMA2 stream 3
• FatFS enabled with DMA template

With the above STM32CubeMX configuration and the minimum working example below:

#define TEST_DATA_SIZE 32768
 
// -- snip
 
static FRESULT fatfs_err;
static const unsigned char test_data[TEST_DATA_SIZE] __attribute__ ((section (".dtcm"), aligned (4))) = {0};
static unsigned int bytes;
 
// -- snip
 
int main(void) {
  // -- snip
 
  fatfs_err = f_mount(&SDFatFS, SDPath, 1);
  if (fatfs_err != FR_OK) {
    printf("failed to mount card, code: %i.\n", fatfs_err);
    if (fatfs_err != FR_NOT_READY) {
      printf("exiting.\n");
      exit(fatfs_err);
    }
    printf("continuing.\n");
  }
 
  // Open file
  fatfs_err = f_open(&SDFile, "test.txt", FA_WRITE | FA_CREATE_ALWAYS);
  if (fatfs_err != FR_OK) {
    printf("failed to open file, code: %i.\n", fatfs_err);
    printf("exiting.\n");
    exit(fatfs_err);
  }
 
  // Write file
  fatfs_err = f_write(&SDFile,
                      test_data,
                      TEST_DATA_SIZE,
                      &bytes);
  if (fatfs_err != FR_OK) {
    printf("failed to write file, code: %i\n", fatfs_err);
    printf("exiting.\n");
    exit(fatfs_err);
  }
  if (bytes != TEST_DATA_SIZE) {
    printf("wrote %i bytes, less than the expected %i.\n",
           bytes, TEST_DATA_SIZE);
  }
 
  // Close file
  fatfs_err = f_close(&SDFile);
  if (fatfs_err != FR_OK) {
    exit(fatfs_err);
  }
  
  // -- snip

The project works fine on 6.6.1, but as soon as I migrate to 6.7.0 the SDMMC peripheral times out. Specifically, it hangs on line 220 of sd_diskio.c, while waiting for a DMA callback.

[FPlow.1 · Accepted solution]

Where 661 and 670 are STM32CubeIDE projects generated by STM32CubeMX 6.6.1 and 6.7.0, respectively, this command:

diff -r 661 670 --exclude ".project" --exclude ".cproject" --exclude ".settings" --exclude "Debug"

produces the following output

Only in 661: 661 Debug.launch
Only in 661: 661.ioc
Only in 670: 670 Debug.launch
Only in 670: 670.ioc
diff -r --ex 661/Core/Inc/main.h 670/Core/Inc/main.h
383a384
> 
diff -r --ex 661/Core/Inc/stm32f7xx_hal_conf.h 670/Core/Inc/stm32f7xx_hal_conf.h
40,62c40,61
<   /* #define HAL_ADC_MODULE_ENABLED   */
< /* #define HAL_CRYP_MODULE_ENABLED   */
< /* #define HAL_CAN_MODULE_ENABLED   */
< /* #define HAL_CEC_MODULE_ENABLED   */
< /* #define HAL_CRC_MODULE_ENABLED   */
< /* #define HAL_CRYP_MODULE_ENABLED   */
< /* #define HAL_DAC_MODULE_ENABLED   */
< /* #define HAL_DCMI_MODULE_ENABLED   */
< /* #define HAL_DMA2D_MODULE_ENABLED   */
< /* #define HAL_ETH_MODULE_ENABLED   */
< /* #define HAL_NAND_MODULE_ENABLED   */
< /* #define HAL_NOR_MODULE_ENABLED   */
< /* #define HAL_SRAM_MODULE_ENABLED   */
< /* #define HAL_SDRAM_MODULE_ENABLED   */
< /* #define HAL_HASH_MODULE_ENABLED   */
< /* #define HAL_I2S_MODULE_ENABLED   */
< /* #define HAL_IWDG_MODULE_ENABLED   */
< /* #define HAL_LPTIM_MODULE_ENABLED   */
< /* #define HAL_LTDC_MODULE_ENABLED   */
< /* #define HAL_QSPI_MODULE_ENABLED   */
< /* #define HAL_RNG_MODULE_ENABLED   */
< /* #define HAL_RTC_MODULE_ENABLED   */
< /* #define HAL_SAI_MODULE_ENABLED   */
---
>   /* #define HAL_CRYP_MODULE_ENABLED */
> /* #define HAL_ADC_MODULE_ENABLED */
> /* #define HAL_CAN_MODULE_ENABLED */
> /* #define HAL_CEC_MODULE_ENABLED */
> /* #define HAL_CRC_MODULE_ENABLED */
> /* #define HAL_DAC_MODULE_ENABLED */
> /* #define HAL_DCMI_MODULE_ENABLED */
> /* #define HAL_DMA2D_MODULE_ENABLED */
> /* #define HAL_ETH_MODULE_ENABLED */
> /* #define HAL_NAND_MODULE_ENABLED */
> /* #define HAL_NOR_MODULE_ENABLED */
> /* #define HAL_SRAM_MODULE_ENABLED */
> /* #define HAL_SDRAM_MODULE_ENABLED */
> /* #define HAL_HASH_MODULE_ENABLED */
> /* #define HAL_I2S_MODULE_ENABLED */
> /* #define HAL_IWDG_MODULE_ENABLED */
> /* #define HAL_LPTIM_MODULE_ENABLED */
> /* #define HAL_LTDC_MODULE_ENABLED */
> /* #define HAL_QSPI_MODULE_ENABLED */
> /* #define HAL_RNG_MODULE_ENABLED */
> /* #define HAL_RTC_MODULE_ENABLED */
> /* #define HAL_SAI_MODULE_ENABLED */
64,80c63,79
< /* #define HAL_MMC_MODULE_ENABLED   */
< /* #define HAL_SPDIFRX_MODULE_ENABLED   */
< /* #define HAL_SPI_MODULE_ENABLED   */
< /* #define HAL_TIM_MODULE_ENABLED   */
< /* #define HAL_UART_MODULE_ENABLED   */
< /* #define HAL_USART_MODULE_ENABLED   */
< /* #define HAL_IRDA_MODULE_ENABLED   */
< /* #define HAL_SMARTCARD_MODULE_ENABLED   */
< /* #define HAL_WWDG_MODULE_ENABLED   */
< /* #define HAL_PCD_MODULE_ENABLED   */
< /* #define HAL_HCD_MODULE_ENABLED   */
< /* #define HAL_DFSDM_MODULE_ENABLED   */
< /* #define HAL_DSI_MODULE_ENABLED   */
< /* #define HAL_JPEG_MODULE_ENABLED   */
< /* #define HAL_MDIOS_MODULE_ENABLED   */
< /* #define HAL_SMBUS_MODULE_ENABLED   */
< /* #define HAL_EXTI_MODULE_ENABLED   */
---
> /* #define HAL_MMC_MODULE_ENABLED */
> /* #define HAL_SPDIFRX_MODULE_ENABLED */
> /* #define HAL_SPI_MODULE_ENABLED */
> /* #define HAL_TIM_MODULE_ENABLED */
> /* #define HAL_UART_MODULE_ENABLED */
> /* #define HAL_USART_MODULE_ENABLED */
> /* #define HAL_IRDA_MODULE_ENABLED */
> /* #define HAL_SMARTCARD_MODULE_ENABLED */
> /* #define HAL_WWDG_MODULE_ENABLED */
> /* #define HAL_PCD_MODULE_ENABLED */
> /* #define HAL_HCD_MODULE_ENABLED */
> /* #define HAL_DFSDM_MODULE_ENABLED */
> /* #define HAL_DSI_MODULE_ENABLED */
> /* #define HAL_JPEG_MODULE_ENABLED */
> /* #define HAL_MDIOS_MODULE_ENABLED */
> /* #define HAL_SMBUS_MODULE_ENABLED */
> /* #define HAL_EXTI_MODULE_ENABLED */
213c212
< #define ETH_RX_BUF_SIZE                ETH_MAX_PACKET_SIZE /* buffer size for receive               */
---
> #define ETH_RX_BUF_SIZE                 /* buffer size for receive               */
diff -r --ex 661/Core/Src/main.c 670/Core/Src/main.c
218c218
<   hsd1.Init.BusWide = SDMMC_BUS_WIDE_1B;
---
>   hsd1.Init.BusWide = SDMMC_BUS_WIDE_4B;
diff -r --ex 661/Core/Src/syscalls.c 670/Core/Src/syscalls.c
13c13
<  * Copyright (c) 2022 STMicroelectronics.
---
>  * Copyright (c) 2020-2022 STMicroelectronics.
50c50
< 	return 1;
---
>   return 1;
55,56c55,58
< 	errno = EINVAL;
< 	return -1;
---
>   (void)pid;
>   (void)sig;
>   errno = EINVAL;
>   return -1;
61,62c63,64
< 	_kill(status, -1);
< 	while (1) {}		/* Make sure we hang here */
---
>   _kill(status, -1);
>   while (1) {}    /* Make sure we hang here */
67c69,70
< 	int DataIdx;
---
>   (void)file;
>   int DataIdx;
69,72c72,75
< 	for (DataIdx = 0; DataIdx < len; DataIdx++)
< 	{
< 		*ptr++ = __io_getchar();
< 	}
---
>   for (DataIdx = 0; DataIdx < len; DataIdx++)
>   {
>     *ptr++ = __io_getchar();
>   }
74c77
< return len;
---
>   return len;
79c82,83
< 	int DataIdx;
---
>   (void)file;
>   int DataIdx;
81,85c85,89
< 	for (DataIdx = 0; DataIdx < len; DataIdx++)
< 	{
< 		__io_putchar(*ptr++);
< 	}
< 	return len;
---
>   for (DataIdx = 0; DataIdx < len; DataIdx++)
>   {
>     __io_putchar(*ptr++);
>   }
>   return len;
90c94,95
< 	return -1;
---
>   (void)file;
>   return -1;
96,97c101,103
< 	st->st_mode = S_IFCHR;
< 	return 0;
---
>   (void)file;
>   st->st_mode = S_IFCHR;
>   return 0;
102c108,109
< 	return 1;
---
>   (void)file;
>   return 1;
107c114,117
< 	return 0;
---
>   (void)file;
>   (void)ptr;
>   (void)dir;
>   return 0;
112,113c122,125
< 	/* Pretend like we always fail */
< 	return -1;
---
>   (void)path;
>   (void)flags;
>   /* Pretend like we always fail */
>   return -1;
118,119c130,132
< 	errno = ECHILD;
< 	return -1;
---
>   (void)status;
>   errno = ECHILD;
>   return -1;
124,125c137,139
< 	errno = ENOENT;
< 	return -1;
---
>   (void)name;
>   errno = ENOENT;
>   return -1;
130c144,145
< 	return -1;
---
>   (void)buf;
>   return -1;
135,136c150,152
< 	st->st_mode = S_IFCHR;
< 	return 0;
---
>   (void)file;
>   st->st_mode = S_IFCHR;
>   return 0;
141,142c157,160
< 	errno = EMLINK;
< 	return -1;
---
>   (void)old;
>   (void)new;
>   errno = EMLINK;
>   return -1;
147,148c165,166
< 	errno = EAGAIN;
< 	return -1;
---
>   errno = EAGAIN;
>   return -1;
153,154c171,175
< 	errno = ENOMEM;
< 	return -1;
---
>   (void)name;
>   (void)argv;
>   (void)env;
>   errno = ENOMEM;
>   return -1;

Based on this, I tried changing

hsd1.Init.BusWide = SDMMC_BUS_WIDE_4B;

to

hsd1.Init.BusWide = SDMMC_BUS_WIDE_1B;

in the 6.7.0 project and it fixed the issue. Note this is not the same as running the card in 1-bit mode, the FatFS Disk I/O driver switches it to 4-bit mode later in initialisation:

~/SDMMC_STM32CubeMX_Test/661 % rg "SDMMC_BUS_WIDE_4B"
Drivers/STM32F7xx_HAL_Driver/Src/stm32f7xx_hal_sd.c
2180:  *            @arg SDMMC_BUS_WIDE_4B: 4-bit data transfer
2202:    else if(WideMode == SDMMC_BUS_WIDE_4B)
 
Drivers/STM32F7xx_HAL_Driver/Inc/stm32f7xx_ll_sdmmc.h
365:#define SDMMC_BUS_WIDE_4B                      SDMMC_CLKCR_WIDBUS_0
369:                                        ((WIDE) == SDMMC_BUS_WIDE_4B) || \
 
Debug/661.list
10552:  *            @arg SDMMC_BUS_WIDE_4B: 4-bit data transfer
10597:    else if(WideMode == SDMMC_BUS_WIDE_4B)
14765:    if (HAL_SD_ConfigWideBusOperation(&hsd1, SDMMC_BUS_WIDE_4B) != HAL_OK)
 
FATFS/Target/bsp_driver_sd.c
62:    if (HAL_SD_ConfigWideBusOperation(&hsd1, SDMMC_BUS_WIDE_4B) != HAL_OK)

I'm not quite sure what the whole sequence is for initialisation in the SD specification, but I think the card has to be in 1-bit mode first in order to set up 4-bit mode.

[Amel NASRI]

Hi @FPlow.1​ ,

An explanation of what was updated is provided by @Ghofrane GSOURI​ in this thread: SDCARD does not work (FR_NOT_READY) when migrating from FW_F4 V1.27.0 to newer FW_F4 V1.27.1.

-Amel