This is an automated email from the ASF dual-hosted git repository.
vipulrahane pushed a commit to branch vipul/juul-master-steam-10-18-2023
in repository https://gitbox.apache.org/repos/asf/mynewt-core.git
commit 9c9404c3a975d097acc225510a0f38e3f7bba130
Author: Will San Filippo <william.sanfili...@juul.com>
AuthorDate: Fri Aug 4 14:24:29 2023 -0400
hw/mcu/cmac: Modify pad to wakeup time.
In order to avoid cmac crashes due to compensation of LL timer
causing an EQ_X interrupt to fire off and generating an assert
in cmac_timer_slp_disable, we are adding a 250 usec pad to the
wakeup time. This should be sufficient until the calculations
are fixed (under investigation).
---
hw/mcu/dialog/cmac/src/cmac_sleep.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/hw/mcu/dialog/cmac/src/cmac_sleep.c
b/hw/mcu/dialog/cmac/src/cmac_sleep.c
index fa00cba02..1ed611ce3 100644
--- a/hw/mcu/dialog/cmac/src/cmac_sleep.c
+++ b/hw/mcu/dialog/cmac/src/cmac_sleep.c
@@ -198,11 +198,14 @@ cmac_sleep_calculate_wakeup_time(void)
T_LPTICK_U(2) + T_LPTICK_U(2) +
max(T_LPTICK_U(3), T_USEC(g_cmac_shared_data.xtal32m_settle_us)) +
T_LPTICK(2) + T_USEC(50) +
+
/*
- * Add an extra 500us of settle time due to occasional crashes due to
- * unknown additional wakeup delay.
+ * We are adding 250 usecs as a safeguard as we know the above
+ * calculations are not correct and need modification. This is
+ * to prevent EQ_X comparator from firing off and causing an
+ * assert when we compensate the LL timer
*/
- T_USEC(500);
+ T_USEC(250);
}
void
