Various fixes

Remove mbed assert.
Change location of warning so it only warn once
Use MINAR_TEST_CLOCK_OVERFLOW as the timer mask

Author: Liyou Zhou

minar-platform/minar_platform_types.h

@@ -29,9 +29,8 @@ enum Constants{
     Time_Base = MINAR_PLATFORM_TIME_BASE,
 
 #if YOTTA_CFG_MINAR_TEST_CLOCK_OVERFLOW
-#warning "testing clock overflow"
     // use only lower 16bits of timer for testing timer overflow
-	Time_Mask = UINT32_MAX >> 16
+	Time_Mask = YOTTA_CFG_MINAR_TEST_CLOCK_OVERFLOW
 #else
     // 32 bits of time for mbed platforms
     Time_Mask = 0xFFFFFFFFu

source/mbed_platform.cpp

@@ -21,10 +21,6 @@
 #include "mbed-hal/sleep_api.h"
 #include "cmsis-core/core_generic.h"
 
-#if YOTTA_CFG_MINAR_TEST_CLOCK_OVERFLOW
-#include "mbed-drivers/test_env.h"
-#endif
-
 /// @name Local Constants
 const static minar::platform::tick_t Minimum_Sleep = MINAR_PLATFORM_MINIMUM_SLEEP; // in Platform_Time_Base units
 
@@ -58,6 +54,7 @@ void sleep(){
 
 tick_t getTime() {
 #if YOTTA_CFG_MINAR_TEST_CLOCK_OVERFLOW
+    #warning "testing clock overflow"
     return lp_ticker_read() & Time_Mask;
 #else
     return lp_ticker_read();
@@ -71,7 +68,6 @@ uint32_t getTimeOverflows(){
 void sleepFromUntil(tick_t now, tick_t until){
 
 #if YOTTA_CFG_MINAR_TEST_CLOCK_OVERFLOW
-    MBED_HOSTTEST_ASSERT(now <= Time_Mask && until <= Time_Mask);
     tick_t timer_top_bits = lp_ticker_read() & ~Time_Mask;
     now += timer_top_bits;
     until += timer_top_bits;
@@ -81,7 +77,6 @@ void sleepFromUntil(tick_t now, tick_t until){
     }
 
     const tick_t real_now = timer_top_bits + getTime();
-    //printf("sleepFromUntil %lx %lx real_now %lx\r\n", now, until, real_now);
 #else
     // use real-now for front-most end of do-not-sleep range check
     const tick_t real_now = getTime();