[GitHub] [incubator-nuttx] slorquet edited a comment on issue #1568: STM32F4 core over-drive enable/disable sequence problem.

[GitBox <gi...@apache.org>]

slorquet edited a comment on issue #1568:
URL: https://github.com/apache/incubator-nuttx/issues/1568#issuecomment-694772173


   Is this function really broken or hard to use?
   
   it has some benefit, it allows stm32f4 to run at 180 Mhz instead of being restricted to lower speed. If I understand correctly, it just has to be called early in the clock setup process?
   
   Why not do this instead of removing a useful performance feature?
   
   Instead of having an overly and unecessary complex process like disabling all clocks, enabling this and reenabling, the function can just be restricted so it has no effect(or returns error) if called when clocks and plls have been enabled.
   If called before enabling PLLs, the function should work, with a useful performance improvement effect.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org