utzig commented on pull request #2670:
URL: https://github.com/apache/mynewt-core/pull/2670#issuecomment-903101938


   > Original code used different settings for mcgConfig_BOARD_xxxx
   > Now one settings (from syscfg) is used in all places maybe it is OK, I 
don't know.
   
   Yeah, you got a point. It doesn't break anything apart from having a 
different configuration; only one of those is ever used, but there are more 
because this file is basically "auto-generated" and supports different running 
modes (high-speed, low-power). It would be better to remove that code and just 
do the clock configuration based on syscfgs for the changing fields; I'll take 
a look.


-- 
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: commits-unsubscr...@mynewt.apache.org

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


Reply via email to