GUIDINGLI opened a new pull request #5232: URL: https://github.com/apache/incubator-nuttx/pull/5232
## Summary rpmsg_rtc: resolve deadlock when the receive SYNC cmd. add `tp` param to up_rtc_set_lowerhalf() & clock_synchronize() Modify these for 2 reasons: 1. client CPU call `up_rtc_set_lowerhalf(rpmsg_rtc_initialize(0))`; up_rtc_set_lowerhalf() -> clock_synchronize -> clock_inittime -> clock_basetime -> up_rtc_getdatetime -> g_rtc_lower->ops->rdtime -> rpmsg if the server CPU hasn't start, and rptun hasn't setup done, then the rpmsg will be failed. So, `up_rtc_set_lowerhalf()` shouldn't call with `rpmsg_rtc_initialize()` 2. rtc rpmsg SYNC packet this is means: server set time, then sync the time to all the remote core. If the client receive the SYNC packet, then should call clock_synchronize(). Before this PR, clock_synchronize() will send IPC to server get time again. After this PR, SYNC packet with time, and clock_synchronize() don't need do IPC again. ## Impact ## Testing -- 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]
