Update driver validation (#26)

- improved ETH driver validation
- removed support for ARM compiler 5
- changed timeout macro to milliseconds
- removed XML report type

Author: furbanc

ARM.CMSIS-Driver_Validation.pdsc

@@ -253,7 +253,7 @@
       </RTE_Components_h>
       <files>
         <file category="doc"    name="Documentation/html/group__dv__can.html" />
-        <file category="header" name="Config/DV_CAN_Config.h" attr="config" version = "1.0.0"/>
+        <file category="header" name="Config/DV_CAN_Config.h" attr="config" version = "1.1.0"/>
         <file category="source" name="Source/DV_CAN.c"/>
       </files>
     </component>

Config/DV_CAN_Config.h

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2015-2020 Arm Limited. All rights reserved.
+ * Copyright (c) 2015-2025 Arm Limited. All rights reserved.
  *
  * SPDX-License-Identifier: Apache-2.0
  *
@@ -17,7 +17,7 @@
  *
  * -----------------------------------------------------------------------------
  *
- * $Revision:   V1.0.0
+ * $Revision:   V1.1.0
  *
  * Project:     CMSIS-Driver Validation
  * Title:       Controller Area Network (CAN) driver validation 
@@ -60,8 +60,8 @@
 // <i> Set the minimum margin between measured and expected bitrates (%)
 #define MIN_BITRATE                     10
 // <o> Transfer timeout
-// <i> Set the transfer timeout (us)
-#define CAN_TRANSFER_TIMEOUT            1000000
+// <i> Set the transfer timeout (ms)
+#define CAN_TRANSFER_TIMEOUT            1000
 // <h> Tests
 // <i> Enable / disable tests.
 // <q> CAN_GetCapabilities

Include/cmsis_dv.h

@@ -32,16 +32,14 @@
 #include "RTE_Components.h"             // Component selection
 #endif
 
-#if defined(RTE_CMSIS_RTOS)
-#include "cmsis_os.h"
-#define GET_SYSTICK() osKernelSysTick()
-#define SYSTICK_MICROSEC(microsec) (((uint64_t)microsec * (osKernelSysTickFrequency)) / 1000000)
-
-#elif defined(RTE_CMSIS_RTOS2)
 #include "cmsis_os2.h"
-#define GET_SYSTICK() osKernelGetSysTimerCount()
-#define SYSTICK_MICROSEC(microsec) (((uint64_t)microsec *  osKernelGetSysTimerFreq()) / 1000000)
-#endif
+
+#define GET_SYSTICK()   	osKernelGetTickCount()
+#define SYSTICK_MS(ms)  	(((uint32_t)ms * osKernelGetTickFreq()) / 1000U)
+
+#define GET_SYSTIMER()      osKernelGetSysTimerCount()
+#define SYSTIMER_US(us)     (((uint64_t)us *  osKernelGetSysTimerFreq()) / 1000000)
+
 #include "cmsis_compiler.h"
 
 /* Expansion macro used to create CMSIS Driver references */

Source/DV_CAN.c

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2015-2020 Arm Limited. All rights reserved.
+ * Copyright (c) 2015-2025 Arm Limited. All rights reserved.
  *
  * SPDX-License-Identifier: Apache-2.0
  *
@@ -128,7 +128,7 @@ int8_t CAN_RunTransfer (uint32_t tx_obj_idx, ARM_CAN_MSG_INFO *tx_msg_info, cons
       return 0;
     }
   }
-  while ((GET_SYSTICK() - tick) < SYSTICK_MICROSEC(CAN_TRANSFER_TIMEOUT));
+  while ((GET_SYSTICK() - tick) < SYSTICK_MS(CAN_TRANSFER_TIMEOUT));
   return -1;
 }
 
@@ -386,10 +386,10 @@ void CAN_Loopback_CheckBitrate (void) {
         tx_data_msg_info.id = ARM_CAN_EXTENDED_ID(0x15555555U);
 
         /* Measure transfer time */
-        ticks_measured = GET_SYSTICK();
+        ticks_measured = GET_SYSTIMER();
         CAN_RunTransfer (tx_obj_idx, &tx_data_msg_info, buffer_out, rx_obj_idx, &rx_data_msg_info, buffer_in, CAN_MSG_SIZE);
-        ticks_measured = GET_SYSTICK() - ticks_measured;
-        ticks_expected = SYSTICK_MICROSEC((((CAN_MSG_SIZE * 8U) + CAN_EXT_FRAME_BITS) * 1000) / CAN_BR[bitrate]);
+        ticks_measured = GET_SYSTIMER() - ticks_measured;
+        ticks_expected = SYSTIMER_US((((CAN_MSG_SIZE * 8U) + CAN_EXT_FRAME_BITS) * 1000) / CAN_BR[bitrate]);
 
         rate = (double)ticks_measured/ticks_expected;
 
@@ -581,11 +581,11 @@ void CAN_Loopback_CheckBitrateFD (void) {
           tx_data_msg_info.id = ARM_CAN_EXTENDED_ID(0x15555555U);
 
           /* Measure transfer time */
-          ticks_measured = GET_SYSTICK();
+          ticks_measured = GET_SYSTIMER();
           CAN_RunTransfer (tx_obj_idx, &tx_data_msg_info, buffer_out, rx_obj_idx, &rx_data_msg_info, buffer_in, CAN_MSG_SIZE_FD);
-          ticks_measured = GET_SYSTICK() - ticks_measured;
-          ticks_expected = SYSTICK_MICROSEC((((((CAN_MSG_SIZE_FD * 8U) + CAN_EXT_FRAME_BITS_FD_DATA) * 1000) / (CAN_BR[bitrate] * CAN_DATA_ARB_RATIO)) +
-                                                     (((CAN_EXT_FRAME_BITS_NOMINAL)                         * 1000) /  CAN_BR[bitrate]                      ) ));
+          ticks_measured = GET_SYSTIMER() - ticks_measured;
+          ticks_expected = SYSTIMER_US((((((CAN_MSG_SIZE_FD * 8U) + CAN_EXT_FRAME_BITS_FD_DATA) * 1000) / (CAN_BR[bitrate] * CAN_DATA_ARB_RATIO)) +
+                                         (((CAN_EXT_FRAME_BITS_NOMINAL)                         * 1000) /  CAN_BR[bitrate]                      ) ));
 
           rate = (double)ticks_measured/ticks_expected;
 
@@ -667,7 +667,7 @@ void CAN_Loopback_Transfer (void) {
     }
 
     /* Set output buffer with random data */
-    srand(GET_SYSTICK());
+    srand(GET_SYSTIMER());
     for (cnt = 0; cnt<CAN_MSG_SIZE; cnt++) {
       buffer_out[cnt] = rand()%0x100U;
     }
@@ -865,7 +865,7 @@ void CAN_Loopback_TransferFD (void) {
       }
 
       /* Set output buffer with random data */
-      srand(GET_SYSTICK());
+      srand(GET_SYSTIMER());
       for (cnt = 0; cnt<CAN_MSG_SIZE_FD; cnt++) {
         buffer_out[cnt] = rand()%0x100;
       }