stm32cube: stm32g431: wrong clock setup

STMicroelectronics / STM32CubeG4

STM32Cube MCU Full Package for the STM32G4 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))

Other

182 stars 98 forks source link

stm32cube: stm32g431: wrong clock setup #3

Closed FRASTM closed 4 years ago

FRASTM commented 4 years ago

When configuring the PLL > 80MHz, an intermediate step is needed with AHB prescaler set to 2 before setting the actual value. Then the AHB prescaler 1 must be set, though

Signed-off-by: Francois Ramu francois.ramu@st.com

IMPORTANT INFORMATION

Contributor License Agreement (CLA)

The Pull Request feature will be considered by STMicroelectronics only after a Contributor License Agreement (CLA) mechanism has been deployed.
We are currently working on the set-up of this procedure.

intercreate commented 4 years ago

Hi @FRASTM , this PR was already integrated in https://github.com/zephyrproject-rtos/hal_stm32.

Please read this post from me on the ST community page https://community.st.com/s/feed/0D50X0000C5TOy8

Mainly SystemCoreClock gets divided by 2. I think that's a bug.

should the patch in this post be integrated in this PR?

ALABSTM commented 4 years ago

Hi InterCreate,

Thank you for your involvement. For the moment, we do not process pull-requests. Hopefully, this will change soon enough.

In the meanwhile, I would suggest that you create an issue to report what you pointed out. Mentioning the link to your post on the ST Community would be helpful too.

With regards,

ALABSTM commented 4 years ago

Hi @FRASTM,

Thank you for this report. Pull-requests are not processed for the moment. However, @intercreate created an issue where he mentioned your pull-request.

Please allow me to close this pull-request. Further discussions regarding this aspect will continue in the frame of the issue above mentioned.

With regards,

ALABSTM commented 4 years ago

ST Internal Reference: 79947