michallenc commented on PR #9104:
URL: https://github.com/apache/nuttx/pull/9104#issuecomment-1523633751

   > > Btw this is something I proposed about a month ago in [this 
email](https://lists.apache.org/thread/y7zrsbf2k86n1qbjlt67hlcj10smjdfw) and a 
response from @liguiding suggests up_rtc_initalize() should be called even if 
`CONFIG_RTC_EXTERNAL` is defined. Although I am still not sure this is entirely 
true and I agree with putting it under `!defined(CONFIG_RTC_EXTERNAL)`
   > 
   > I don't think I agree with that email for the reason given in the docs I 
linked. The required subsystems may very well not be ready by the time the 
clocks are being initialized. It is better to instead make this the job of the 
board initialization logic.
   
   Yes, those were my thoughts.


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