Revert "STM32L4: fix trng clock setting" #9029
Conversation
|
|
||
| /*Select PLLQ output as RNG clock source */ | ||
| PeriphClkInitStruct.PeriphClockSelection = RCC_PERIPHCLK_RNG; | ||
| PeriphClkInitStruct.RngClockSelection = RCC_RNGCLKSOURCE_PLL; |
There was a problem hiding this comment.
PeriphClkInitStruct.RngClockSelection = RCC_RNGCLKSOURCE_MSI;
Will resolve the issue.
There was a problem hiding this comment.
@jeromecoutant I tried this, but it will produce the same error as if the original fix.
There was a problem hiding this comment.
I see, clock configuration is different for MTB_ADV_WISE_1570 than for NUCLEO_L486RG.
There was a problem hiding this comment.
As previously was mentioned in original PR, should this adv wise target have own system clock config for this RNG selection?
There was a problem hiding this comment.
Could you check my commit in https://github.com/jeromecoutant/mbed/tree/PR_L4TRNG ?
There was a problem hiding this comment.
@jeromecoutant that works for me
There was a problem hiding this comment.
@jeromecoutant new PR and we close this revert?
|
@0xc0170, thank you for your changes. |
|
I set this to rc3 to keep track of this one (new PR might be up soon). |
|
Internal issue MBEDOSTEST-377 |
|
This is being replaced with #8867 |
Reverts #8867
@ARMmbed/team-st-mcd As noted in the PR itself, CI started failing because of this commit. I noticed TRNG tests were run for two L4 targets they did not fail, I am not certain why (opening this revert as the last resort, we shall fix this rather). Please review
cc @juhoeskeli