Use define instead of hardcoded value

Core/Inc/canlog.h

@@ -18,9 +18,13 @@
 /* Define to prevent recursive inclusion -------------------------------------*/
 #ifndef CANLOG_H
 #define CANLOG_H
-
+/* Includes ------------------------------------------------------------------*/
 #include "main.h"
 #include "cmsis_os.h"
+/* Private define ------------------------------------------------------------*/
+/* USER CODE BEGIN PD */
+#define LED_BLINK_MS 25U
+/* USER CODE END PD */
 
 extern osMessageQueueId_t xCAN1RxQueue;
 extern osMessageQueueId_t xCAN2RxQueue;

Core/Src/canlog.c

@@ -134,7 +134,7 @@ void vLEDHeartbeat(void *argument)
 
   for (;;)
   {
-    uint32_t notification = osThreadFlagsWait(0x01, osFlagsWaitAny, 25U);
+    uint32_t notification = osThreadFlagsWait(0x01, osFlagsWaitAny, LED_BLINK_MS);
 
     if (notification & 0x01) HAL_GPIO_WritePin(led->port, led->pin, GPIO_PIN_SET);
     else HAL_GPIO_WritePin(led->port, led->pin, GPIO_PIN_RESET);