darrylring opened a new pull request, #19917:
URL: https://github.com/apache/nuttx/pull/19917

   ## Summary
   
   Existing handling of MCO (microcontroller clock output) pin and clock 
division is incorrect/incomplete. This adds the correct pin definitions and 
adds the necessary macros and function parameters to configure the MCO clock 
pins.
   
   This implementation now matches the STM32H7 which uses the same pins and 
configuration.
   
   ## Impact
   
   The existing macro was called MCO, but there are MCO1 and MCO2 pins. The 
functions `stm32_mco1config` and `stm32_mco2config` have an additional 
parameter. This is theoretically a breaking change, but I don't think it's 
really possible to use MCO1/MCO2 with the existing code.
   
   ## Testing
   
   Testing was conducted on a custom board using both MCO1 and MCO2 pins to 
provide clocks to other components. The clock frequency was verified using an 
oscilloscope, and found to match the expected value based on the PLL frequency, 
MCO clock source, and MCO clock dividers.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to