Posted on November 23, 2017 at 11:49 Hello
s.mueller
,
The issue will be solved in the next official release (4.24).
Our apologies again for that regression impacting the series supporting FatFS R0.12c (F4, F7, H7 and L4).
Back to your questions, in fact, modifying generated code is useless unless code is added in what we call 'user sections'.
You will find different places in some files (such as sd_diskio.c) looking like:
/* USER CODE BEGIN sectionName */
/* place for user code */
/* USER CODE END sectionName */
The user code added there will be kept when generating again the project.
As a workaround, in order to fix the current issue before the 4.24 is available, I can suggest you a patch, using the existing user sections:
/* USER CODE BEGIN beforeFunctionSection */
/* can be used to modify / undefine following code or add new code */
#ifdef BROKEN_CODE /* Note: not defined, then following code will be ignored when preprocessing the C file */
/* USER CODE END beforeFunctionSection */
/* Private functions ---------------------------------------------------------*/
static DSTATUS SD_CheckStatus(BYTE lun)
{
Stat = STA_NOINIT;
if(BSP_SD_GetCardState() == MSD_OK)
{
Stat &= ~STA_NOINIT;
}
return Stat;
}
/**
* @brief Initializes a Drive
* @param lun : not used
* @retval DSTATUS: Operation status
*/
DSTATUS SD_initialize(BYTE lun)
{
return SD_CheckStatus(lun);
}
/**
* @brief Gets Disk Status
* @param lun : not used
* @retval DSTATUS: Operation status
*/
DSTATUS SD_status(BYTE lun)
{
return SD_CheckStatus(lun);
}
/* USER CODE BEGIN beforeReadSection */
#else
static DSTATUS SD_CheckStatus(BYTE lun)
{
Stat = STA_NOINIT;
if(BSP_SD_GetCardState() == MSD_OK)
{
Stat &= ~STA_NOINIT;
}
return Stat;
}
/**
* @brief Initializes a Drive
* @param lun : not used
* @retval DSTATUS: Operation status
*/
DSTATUS SD_initialize(BYTE lun)
{
Stat = STA_NOINIT;
if(BSP_SD_Init() == MSD_OK)
{
return SD_CheckStatus(lun);
}
return Stat;
}
/**
* @brief Gets Disk Status
* @param lun : not used
* @retval DSTATUS: Operation status
*/
DSTATUS SD_status(BYTE lun)
{
return SD_CheckStatus(lun);
}
#endif
/* USER CODE END beforeReadSection */
Not that beautiful, I agree, but should fix the issue (till the 4.2.4.0 solves it in a cleaner way).
Let us know if it does not.
Regards,
Fred
