search

STMicroelectronics / STM32CubeH7

STM32Cube MCU Full Package for the STM32H7 series - (HAL + LL Drivers, CMSIS Core, CMSIS Device, MW libraries plus a set of Projects running on all boards provided by ST (Nucleo, Evaluation and Discovery Kits))
https://www.st.com/en/embedded-software/stm32cubeh7.html
Other
495 stars 303 forks source link

BUG: usbh_diskio, USBH_ioctl, cmd GET_SECTOR_SIZE: incorrect cast to DWORD instead of WORD #180

Closed iiinjoy closed 2 years ago

iiinjoy commented 2 years ago

BUG: usbh_diskio, USBH_ioctl, cmd GET_SECTOR_SIZE: incorrect cast to DWORD instead of WORD

https://github.com/STMicroelectronics/STM32CubeH7/blob/ccb11556044540590ca6e45056e6b65cdca2deb2/Middlewares/Third_Party/FatFs/src/drivers/usbh_diskio_template.c#L215

fatfs expects WORD for GET_SECTOR_SIZE command:

https://github.com/STMicroelectronics/STM32CubeH7/blob/5975bffae9358bc2b2890a35a203d940a395efef/Middlewares/Third_Party/FatFs/src/ff.c#L5314

https://github.com/STMicroelectronics/STM32CubeH7/blob/5975bffae9358bc2b2890a35a203d940a395efef/Middlewares/Third_Party/FatFs/src/ff.h#L95

This bug is also present in other MCU Firmware Packages.

ALABSTM commented 2 years ago

Hi @iiinjoy,

Thank you for your report. Your point is confirmed and has been logged into our internal database.

With regards,

ALABSTM commented 2 years ago

Hi @iiinjoy,

I hope you are doing well. Back to you about this point. Although confirmed, our development teams said it is minor and, as no future release of the FatFs lib is planned, they will take no action.

Please allow me then to close this issue. Thank you for your comprehension and thank you for having reported.

With regards,