Changeset acf1e5b2 in rtems for testsuites/validation/tc-clock.c

Timestamp:

09/22/22 11:19:34 (6 months ago)

Author:

Sebastian Huber <sebastian.huber@…>

Branches:

master

Children:

9ed7103

Parents:

962904c

git-author:

Sebastian Huber <sebastian.huber@…> (09/22/22 11:19:34)

git-committer:

Sebastian Huber <sebastian.huber@…> (09/23/22 09:14:58)

Message:

rtems: Add rtems_clock_get_ticks_since_boot() function

This function was declared, however, a definition was missing. Add a
validation test for it.

File:

• testsuites/validation/tc-clock.c (modified) (6 diffs)

testsuites/validation/tc-clock.c

@@ -8 +8 @@
 
 /*
- * Copyright (C) 2021 embedded brains GmbH (http://www.embedded-brains.de)
+ * Copyright (C) 2021, 2022 embedded brains GmbH (http://www.embedded-brains.de)
  *
  * Redistribution and use in source and binary forms, with or without
@@ -69 +69 @@
  * This test case performs the following actions:
  *
- * - Use the rtems_clock_get_ticks_since_boot() function.
+ * - Use the rtems_clock_get_ticks_since_boot() directive before and after
+ *   exactly one clock tick.
  *
  *   - Check that clock tick gets incremented.
  *
- * - Use the rtems_clock_get_ticks_per_second() function.
+ * - Use the rtems_clock_get_ticks_since_boot() directive before and after
+ *   exactly one clock tick.
+ *
+ *   - Check that clock tick gets incremented.
+ *
+ * - Use the rtems_clock_get_ticks_per_second() directive.
+ *
+ *   - Check that rtems_clock_get_ticks_per_second() actually returns 1us /
+ *     CONFIGURE_MICROSECONDS_PER_TICK.
+ *
+ * - Use the rtems_clock_get_ticks_per_second() directive.
  *
  *   - Check that rtems_clock_get_ticks_per_second() actually returns 1us /
@@ -82 +93 @@
 
 /**
- * @brief Use the rtems_clock_get_ticks_since_boot() function.
+ * @brief Use the rtems_clock_get_ticks_since_boot() directive before and after
+ *   exactly one clock tick.
  */
 static void RtemsClockValClock_Action_0( void )
@@ -88 +100 @@
   rtems_interval result_0;
   rtems_interval result_1;
-  int32_t difference; /* Note: rtems_interval = uint32_t (unsigned!) */
 
   result_0 = rtems_clock_get_ticks_since_boot();
   ClockTick();
   result_1 = rtems_clock_get_ticks_since_boot();
-  /*
-   * Because of the ones-complement, the overflow
-   * is handled correctly. result_0 = 0xFFFFFFFF will become -1
-   * and result_1 = 0x0 will become 0.
-   */
-  difference = (int32_t) result_1 - (int32_t) result_0;
 
   /*
    * Check that clock tick gets incremented.
    */
-  T_step_eq_i32( 0, difference, 1 );
+  T_step_eq_u32( 0, result_1 - result_0, 1 );
 }
 
 /**
- * @brief Use the rtems_clock_get_ticks_per_second() function.
+ * @brief Use the rtems_clock_get_ticks_since_boot() directive before and after
+ *   exactly one clock tick.
  */
 static void RtemsClockValClock_Action_1( void )
 {
+  rtems_interval result_0;
+  rtems_interval result_1;
+
+  #undef rtems_clock_get_ticks_since_boot
+
+  result_0 = rtems_clock_get_ticks_since_boot();
+  ClockTick();
+  result_1 = rtems_clock_get_ticks_since_boot();
+
+  /*
+   * Check that clock tick gets incremented.
+   */
+  T_step_eq_u32( 1, result_1 - result_0, 1 );
+}
+
+/**
+ * @brief Use the rtems_clock_get_ticks_per_second() directive.
+ */
+static void RtemsClockValClock_Action_2( void )
+{
   rtems_interval result;
+
   result = rtems_clock_get_ticks_per_second();
 
@@ -118 +145 @@
    * CONFIGURE_MICROSECONDS_PER_TICK.
    */
-  T_step_eq_u32( 1, result, 1000000UL / TEST_MICROSECONDS_PER_TICK );
+  T_step_eq_u32( 2, result, 1000000UL / TEST_MICROSECONDS_PER_TICK );
+}
+
+/**
+ * @brief Use the rtems_clock_get_ticks_per_second() directive.
+ */
+static void RtemsClockValClock_Action_3( void )
+{
+  rtems_interval result;
+
+  #undef rtems_clock_get_ticks_per_second
+
+  result = rtems_clock_get_ticks_per_second();
+
+  /*
+   * Check that rtems_clock_get_ticks_per_second() actually returns 1us /
+   * CONFIGURE_MICROSECONDS_PER_TICK.
+   */
+  T_step_eq_u32( 3, result, 1000000UL / TEST_MICROSECONDS_PER_TICK );
 }
 
@@ -126 +171 @@
 T_TEST_CASE( RtemsClockValClock )
 {
-  T_plan( 2 );
+  T_plan( 4 );
 
   RtemsClockValClock_Action_0();
   RtemsClockValClock_Action_1();
+  RtemsClockValClock_Action_2();
+  RtemsClockValClock_Action_3();
 }