Change define name

- "DEBUG" is also used by STM32 for... something...

Core/Inc/main.h

@@ -1,85 +1,76 @@
-/* USER CODE BEGIN Header */
+/* USER CODE BEGIN Header */
-/**
+/**
-  ******************************************************************************
+  ******************************************************************************
-  * @file           : main.h
+  * @file           : main.h
-  * @brief          : Header for main.c file.
+  * @brief          : Header for main.c file.
-  *                   This file contains the common defines of the application.
+  *                   This file contains the common defines of the application.
-  ******************************************************************************
+  ******************************************************************************
-  * @attention
+  * @attention
-  *
+  *
-  * Copyright (c) 2026 STMicroelectronics.
+  * Copyright (c) 2026 STMicroelectronics.
-  * Copyright (c) 2026 Erick Ahmed.
+  * Copyright (c) 2026 Erick Ahmed.
-  *
+  *
-  * SPDX-License-Identifier: GPL-3.0-or-later
+  * SPDX-License-Identifier: GPL-3.0-or-later
-  *
+  *
-  ******************************************************************************
+  ******************************************************************************
-  */
+  */
-/* USER CODE END Header */
+/* USER CODE END Header */
-
+
-/* Define to prevent recursive inclusion -------------------------------------*/
+/* Define to prevent recursive inclusion -------------------------------------*/
-#ifndef __MAIN_H
+#ifndef __MAIN_H
-#define __MAIN_H
+#define __MAIN_H
-
+
-#ifdef __cplusplus
+#ifdef __cplusplus
-extern "C" {
+extern "C" {
-#endif
+#endif
-
+
-/* Includes ------------------------------------------------------------------*/
+/* Includes ------------------------------------------------------------------*/
-#include "stm32f4xx_hal.h"
+#include "stm32f4xx_hal.h"
-
+
-
+/* Private includes ----------------------------------------------------------*/
-/* Private includes ----------------------------------------------------------*/
+/* USER CODE BEGIN Includes */
-/* USER CODE BEGIN Includes */
+#include <stdint.h>
-#include <stdint.h>
+#include "canlog.h"
-#include "canlog.h"
+/* USER CODE END Includes */
-/* USER CODE END Includes */
+
-
+/* Exported types ------------------------------------------------------------*/
-/* Exported types ------------------------------------------------------------*/
+/* USER CODE BEGIN ET */
-/* USER CODE BEGIN ET */
+
-
+/* USER CODE END ET */
-/* USER CODE END ET */
+
-
+/* Exported constants --------------------------------------------------------*/
-/* Exported constants --------------------------------------------------------*/
+/* USER CODE BEGIN EC */
-/* USER CODE BEGIN EC */
+
-
+/* USER CODE END EC */
-/* USER CODE END EC */
+
-
+/* Exported macro ------------------------------------------------------------*/
-/* Exported macro ------------------------------------------------------------*/
+/* USER CODE BEGIN EM */
-/* USER CODE BEGIN EM */
+#define DEBUG_ITM
-#define DEBUG
+#define DEBUG_DUMMY_FRAME
-#ifdef DEBUG
+
-static inline void ITM_Print(const char *str) {
+#ifdef DEBUG_ITM
-    while (*str) {
+#include <stdio.h>
-        uint32_t timeout = 1000;
+#define DEBUG_PRINT(...) printf(__VA_ARGS__)
-        while ((ITM->PORT[0].u32 == 0) && (timeout-- > 0));
+#else
-
+#define DEBUG_PRINT(...)
-        if (timeout > 0) {
+#endif
-            ITM->PORT[0].u8 = (uint8_t)(*str);
+/* USER CODE END EM */
-        }
+
-        str++;
+/* Exported functions prototypes ---------------------------------------------*/
-    }
+void Error_Handler(void);
-}
+
-#define DEBUG_PRINT(x) ITM_Print(x)
+/* USER CODE BEGIN EFP */
-#else
+
-#define DEBUG_PRINT(x)
+/* USER CODE END EFP */
-#endif
+
-/* USER CODE END EM */
+/* Private defines -----------------------------------------------------------*/
-
+
-/* Exported functions prototypes ---------------------------------------------*/
+/* USER CODE BEGIN Private defines */
-void Error_Handler(void);
+
-
+/* USER CODE END Private defines */
-/* USER CODE BEGIN EFP */
+
-
+#ifdef __cplusplus
-/* USER CODE END EFP */
+}
-
+#endif
-/* Private defines -----------------------------------------------------------*/
+
-
+#endif /* __MAIN_H */
-/* USER CODE BEGIN Private defines */
-
-/* USER CODE END Private defines */
-
-#ifdef __cplusplus
-}
-#endif
-
-#endif /* __MAIN_H */

Core/Inc/stm32f4xx_it.h

@@ -56,6 +56,7 @@ void SDIO_IRQHandler(void);
 void TIM6_DAC_IRQHandler(void);
 void DMA2_Stream3_IRQHandler(void);
 void DMA2_Stream6_IRQHandler(void);
+void DMA2_Stream7_IRQHandler(void);
 /* USER CODE BEGIN EFP */
 
 /* USER CODE END EFP */

Core/Src/canlog.c

@@ -117,13 +117,15 @@ void vCANListener(void *argument)
 
   for (;;)
   {
-    if (osMessageQueueGet(queue, &message, NULL, osWaitForever) == osOK)
+    if (osMessageQueueGet(queue, &message, NULL, 1000) == osOK)
     {
           // J1939 decoding
 
           // RELAY: Push to the UART queue
       osMessageQueuePut(xUARTQueue, &message, 0U, 0U);
     }
+    else DEBUG_PRINT("No CAN yet!\r\n");
+
   }
 }
 /* END vCANListener */

Core/Src/cansend.c

@@ -32,7 +32,7 @@ extern osMessageQueueId_t xUARTQueue;
   * @param  arguments: buffer, source, message
   * @retval None
   */
-static void format_can_message(char *buf, uint8_t source, const CanMessage_t *message)
+static int format_can_message(char *buf, uint8_t source, const CanMessage_t *message)
 {
   const char hex[] = "0123456789ABCDEF";
 
@@ -72,6 +72,9 @@ static void format_can_message(char *buf, uint8_t source, const CanMessage_t *me
 
   buf[idx++] = '\r';
   buf[idx++] = '\n';
+  buf[idx] = '\0';
+
+  return idx;
 }
 /* END format_can_message */
 
@@ -101,17 +104,37 @@ void vUARTLogger(void *argument)
     CanMessage_t message;
     char tx_buffer[45];
 
-    for (;;)
+  #ifdef DEBUG_DUMMY_FRAME
+  CanMessage_t dummy_frame = {
+    .id = 0x0CF00400,       // J1939 EEC1 ID
+    .dlc = 8,               // 8 bytes of data
+    .isExtended = 1,        // 29-bit Extended ID
+    .source = 1,            // CAN1
+    .payload = {0xAA, 0xBB, 0xCC, 0xDD, 0xEE, 0xFF, 0x11, 0x22}
+  };
+  #endif
+
+  for (;;)
+  {
+    DEBUG_PRINT("Waiting for CAN traffic...\r\n");
+
+    #ifdef DEBUG_DUMMY_FRAME
+    osMessageQueuePut(xUARTQueue, &dummy_frame, 0U, 0U);
+    osDelay(1000);
+    #endif
+
+    if (osMessageQueueGet(xUARTQueue, &message, NULL, osWaitForever) == osOK)
     {
       DEBUG_PRINT("Waiting for CAN traffic...\r\n");
       if (osMessageQueueGet(xUARTQueue, &message, NULL, osWaitForever) == osOK)
       {
         DEBUG_PRINT("CAN frame detected\r\n");
-        format_can_message(tx_buffer, message.source, &message);
+        int len = format_can_message(tx_buffer, message.source, &message);
-        HAL_UART_Transmit_DMA(&huart1, (uint8_t*)tx_buffer, 44);
+        HAL_UART_Transmit_DMA(&huart1, (uint8_t*)tx_buffer, len);
         osSemaphoreAcquire(xUARTDMASemaphore, osWaitForever);
         DEBUG_PRINT("CAN frame sent via UART\r\n");
       }
     }
+  }
 }
 /* END vUARTLoggerListen */

Core/Src/stm32f4xx_hal_msp.c

@@ -26,6 +26,8 @@ extern DMA_HandleTypeDef hdma_sdio_rx;
 
 extern DMA_HandleTypeDef hdma_sdio_tx;
 
+extern DMA_HandleTypeDef hdma_usart1_tx;
+
 /* Private typedef -----------------------------------------------------------*/
 /* USER CODE BEGIN TD */
 
@@ -371,6 +373,25 @@ void HAL_UART_MspInit(UART_HandleTypeDef* huart)
     GPIO_InitStruct.Alternate = GPIO_AF7_USART1;
     HAL_GPIO_Init(GPIOA, &GPIO_InitStruct);
 
+    /* USART1 DMA Init */
+    /* USART1_TX Init */
+    hdma_usart1_tx.Instance = DMA2_Stream7;
+    hdma_usart1_tx.Init.Channel = DMA_CHANNEL_4;
+    hdma_usart1_tx.Init.Direction = DMA_MEMORY_TO_PERIPH;
+    hdma_usart1_tx.Init.PeriphInc = DMA_PINC_DISABLE;
+    hdma_usart1_tx.Init.MemInc = DMA_MINC_ENABLE;
+    hdma_usart1_tx.Init.PeriphDataAlignment = DMA_PDATAALIGN_BYTE;
+    hdma_usart1_tx.Init.MemDataAlignment = DMA_MDATAALIGN_BYTE;
+    hdma_usart1_tx.Init.Mode = DMA_NORMAL;
+    hdma_usart1_tx.Init.Priority = DMA_PRIORITY_LOW;
+    hdma_usart1_tx.Init.FIFOMode = DMA_FIFOMODE_DISABLE;
+    if (HAL_DMA_Init(&hdma_usart1_tx) != HAL_OK)
+    {
+      Error_Handler();
+    }
+
+    __HAL_LINKDMA(huart,hdmatx,hdma_usart1_tx);
+
     /* USER CODE BEGIN USART1_MspInit 1 */
 
     /* USER CODE END USART1_MspInit 1 */
@@ -401,6 +422,8 @@ void HAL_UART_MspDeInit(UART_HandleTypeDef* huart)
     */
     HAL_GPIO_DeInit(GPIOA, GPIO_PIN_9|GPIO_PIN_10);
 
+    /* USART1 DMA DeInit */
+    HAL_DMA_DeInit(huart->hdmatx);
     /* USER CODE BEGIN USART1_MspDeInit 1 */
 
     /* USER CODE END USART1_MspDeInit 1 */

Core/Src/stm32f4xx_it.c

@@ -58,6 +58,7 @@
 extern DMA_HandleTypeDef hdma_sdio_rx;
 extern DMA_HandleTypeDef hdma_sdio_tx;
 extern SD_HandleTypeDef hsd;
+extern DMA_HandleTypeDef hdma_usart1_tx;
 extern TIM_HandleTypeDef htim6;
 
 /* USER CODE BEGIN EV */
@@ -218,6 +219,20 @@ void DMA2_Stream6_IRQHandler(void)
   /* USER CODE END DMA2_Stream6_IRQn 1 */
 }
 
+/**
+  * @brief This function handles DMA2 stream7 global interrupt.
+  */
+void DMA2_Stream7_IRQHandler(void)
+{
+  /* USER CODE BEGIN DMA2_Stream7_IRQn 0 */
+
+  /* USER CODE END DMA2_Stream7_IRQn 0 */
+  HAL_DMA_IRQHandler(&hdma_usart1_tx);
+  /* USER CODE BEGIN DMA2_Stream7_IRQn 1 */
+
+  /* USER CODE END DMA2_Stream7_IRQn 1 */
+}
+
 /* USER CODE BEGIN 1 */
 
 /* USER CODE END 1 */

can_logger.ioc

@@ -20,7 +20,8 @@ CAN2.Mode=CAN_MODE_SILENT
 CAN2.Prescaler=8
 Dma.Request0=SDIO_RX
 Dma.Request1=SDIO_TX
-Dma.RequestsNb=2
+Dma.Request2=USART1_TX
+Dma.RequestsNb=3
 Dma.SDIO_RX.0.Direction=DMA_PERIPH_TO_MEMORY
 Dma.SDIO_RX.0.FIFOMode=DMA_FIFOMODE_ENABLE
 Dma.SDIO_RX.0.FIFOThreshold=DMA_FIFO_THRESHOLD_FULL
@@ -47,6 +48,16 @@ Dma.SDIO_TX.1.PeriphDataAlignment=DMA_PDATAALIGN_WORD
 Dma.SDIO_TX.1.PeriphInc=DMA_PINC_DISABLE
 Dma.SDIO_TX.1.Priority=DMA_PRIORITY_LOW
 Dma.SDIO_TX.1.RequestParameters=Instance,Direction,PeriphInc,MemInc,PeriphDataAlignment,MemDataAlignment,Mode,Priority,FIFOMode,FIFOThreshold,MemBurst,PeriphBurst
+Dma.USART1_TX.2.Direction=DMA_MEMORY_TO_PERIPH
+Dma.USART1_TX.2.FIFOMode=DMA_FIFOMODE_DISABLE
+Dma.USART1_TX.2.Instance=DMA2_Stream7
+Dma.USART1_TX.2.MemDataAlignment=DMA_MDATAALIGN_BYTE
+Dma.USART1_TX.2.MemInc=DMA_MINC_ENABLE
+Dma.USART1_TX.2.Mode=DMA_NORMAL
+Dma.USART1_TX.2.PeriphDataAlignment=DMA_PDATAALIGN_BYTE
+Dma.USART1_TX.2.PeriphInc=DMA_PINC_DISABLE
+Dma.USART1_TX.2.Priority=DMA_PRIORITY_LOW
+Dma.USART1_TX.2.RequestParameters=Instance,Direction,PeriphInc,MemInc,PeriphDataAlignment,MemDataAlignment,Mode,Priority,FIFOMode
 FREERTOS.FootprintOK=true
 FREERTOS.IPParameters=Tasks01,configUSE_NEWLIB_REENTRANT,FootprintOK
 FREERTOS.Tasks01=DefaultTask,49,128,can_listen,Default,NULL,Dynamic,NULL,NULL
@@ -68,30 +79,26 @@ Mcu.IP8=USART1
 Mcu.IPNb=9
 Mcu.Name=STM32F405RGTx
 Mcu.Package=LQFP64
-Mcu.Pin0=PA0-WKUP
+Mcu.Pin0=PB12
-Mcu.Pin1=PB12
+Mcu.Pin1=PB13
-Mcu.Pin10=PC10
+Mcu.Pin10=PC12
-Mcu.Pin11=PC11
+Mcu.Pin11=PD2
-Mcu.Pin12=PC12
+Mcu.Pin12=PB5
-Mcu.Pin13=PD2
+Mcu.Pin13=PB6
-Mcu.Pin14=PB3
+Mcu.Pin14=PB7
-Mcu.Pin15=PB4
+Mcu.Pin15=PB8
-Mcu.Pin16=PB5
+Mcu.Pin16=PB9
-Mcu.Pin17=PB6
+Mcu.Pin17=VP_FREERTOS_VS_CMSIS_V2
-Mcu.Pin18=PB7
+Mcu.Pin18=VP_SYS_VS_tim6
-Mcu.Pin19=PB8
+Mcu.Pin2=PC8
-Mcu.Pin2=PB13
+Mcu.Pin3=PC9
-Mcu.Pin20=PB9
+Mcu.Pin4=PA9
-Mcu.Pin21=VP_FREERTOS_VS_CMSIS_V2
+Mcu.Pin5=PA10
-Mcu.Pin22=VP_SYS_VS_tim6
+Mcu.Pin6=PA13
-Mcu.Pin3=PC8
+Mcu.Pin7=PA14
-Mcu.Pin4=PC9
+Mcu.Pin8=PC10
-Mcu.Pin5=PA9
+Mcu.Pin9=PC11
-Mcu.Pin6=PA10
+Mcu.PinsNb=19
-Mcu.Pin7=PA13
-Mcu.Pin8=PA14
-Mcu.Pin9=PA15
-Mcu.PinsNb=23
 Mcu.ThirdPartyNb=0
 Mcu.UserConstants=
 Mcu.UserName=STM32F405RGTx
@@ -100,6 +107,7 @@ MxDb.Version=DB.6.0.170
 NVIC.BusFault_IRQn=true\:0\:0\:false\:false\:true\:false\:false\:false\:false
 NVIC.DMA2_Stream3_IRQn=true\:5\:0\:false\:false\:true\:true\:false\:true\:true
 NVIC.DMA2_Stream6_IRQn=true\:5\:0\:false\:false\:true\:true\:false\:true\:true
+NVIC.DMA2_Stream7_IRQn=true\:5\:0\:false\:false\:true\:true\:false\:true\:true
 NVIC.DebugMonitor_IRQn=true\:0\:0\:false\:false\:true\:false\:false\:false\:false
 NVIC.ForceEnableDMAVector=true
 NVIC.HardFault_IRQn=true\:0\:0\:false\:false\:true\:false\:false\:false\:false
@@ -117,26 +125,18 @@ NVIC.TIM6_DAC_IRQn=true\:15\:0\:false\:false\:true\:false\:false\:true\:true
 NVIC.TimeBase=TIM6_DAC_IRQn
 NVIC.TimeBaseIP=TIM6
 NVIC.UsageFault_IRQn=true\:0\:0\:false\:false\:true\:false\:false\:false\:false
-PA0-WKUP.Mode=SYS_WakeUp0
-PA0-WKUP.Signal=SYS_WKUP
 PA10.Mode=Asynchronous
 PA10.Signal=USART1_RX
-PA13.Mode=JTAG_5_pins
+PA13.Mode=Serial_Wire
 PA13.Signal=SYS_JTMS-SWDIO
-PA14.Mode=JTAG_5_pins
+PA14.Mode=Serial_Wire
 PA14.Signal=SYS_JTCK-SWCLK
-PA15.Mode=JTAG_5_pins
-PA15.Signal=SYS_JTDI
 PA9.Mode=Asynchronous
 PA9.Signal=USART1_TX
 PB12.Mode=CAN_Activate
 PB12.Signal=CAN2_RX
 PB13.Mode=CAN_Activate
 PB13.Signal=CAN2_TX
-PB3.Mode=JTAG_5_pins
-PB3.Signal=SYS_JTDO-SWO
-PB4.Mode=JTAG_5_pins
-PB4.Signal=SYS_JTRST
 PB5.Locked=true
 PB5.Signal=GPIO_Output
 PB6.Locked=true