search

STMicroelectronics / stm32f7xx_hal_driver

Provides the STM32Cube MCU Component "hal_driver" of the STM32F7 series.
BSD 3-Clause "New" or "Revised" License
27 stars 11 forks source link

Bug in stm32f7xx_hal_hcd.c using callback #2

Closed micheledarold closed 2 years ago

micheledarold commented 3 years ago

Missing use of registered callback in some part of the code.

I have enabled USE_HAL_HCD_REGISTER_CALLBACKS but I find some part of the code not using the correct way to call the function HAL_HCD_HC_NotifyURBChange_Callback. hcd_patch.txt

ALABSTM commented 3 years ago

Hi @micheledarold,

Thank you for this report. Your point is correct. It will be forwarded to our development teams. May I ask you first to create a pull-request? This will allow a far more easy way to view the changes proposed. Thank you very much.

With regards,

ALABSTM commented 3 years ago

ST Internal Reference: 111775

ALABSTM commented 3 years ago

Hi @micheledarold,

The issue you reported has been forwarded to our development teams. No more need to create a pull-request this time. Next times maybe, as it allows to visualize the changes far more easily and, more important, to integrate them faster into the repository. Thank you again for having reported this one.

With regards,

micheledarold commented 3 years ago

Hi @ALABSTM I'm on vacation and back in office only today, so thank you for your help, next time I'll provide the pull request.

ALABSTM commented 2 years ago

Hi @micheledarold,

I hope you are fine. The issue you reported has been fixed in the frame of version v1.2.10 of the stm32f7xx_hal_driver published recently on GitHub. Thank you again for having reported.

With regards,