Removing serial print

.gitignore

@@ -71,4 +71,7 @@ types_c.taghl
 ctrlp.root
 
 #html
-*.html
+*.html
+
+# nixos files
+*.nix

Core/Inc/cerberus_conf.h

@@ -13,7 +13,6 @@
 #define SHUTDOWN_MONITOR_DELAY 500 /* ms */
 #define NERO_DELAY_TIME	       100 /* ms*/
 #define LV_READ_DELAY	       1000
-#define SERIAL_MONITOR_DELAY
 #define CAN_ROUTER_DELAY
 #define CAN_DISPATCH_DELAY    5
 #define BMS_CAN_MONITOR_DELAY 4000

Core/Src/bms.c

@@ -2,7 +2,6 @@
 #include "timer.h"
 #include "fault.h"
 #include "can.h"
-#include "serial_monitor.h"
 #include "cerberus_conf.h"
 #include <assert.h>
 #include <stdlib.h>

Core/Src/can_handler.c

@@ -13,7 +13,6 @@
 #include "cerberus_conf.h"
 #include "fault.h"
 #include "steeringio.h"
-#include "serial_monitor.h"
 #include "bms.h"
 #include <assert.h>
 #include <stdlib.h>

Core/Src/dti.c

@@ -20,7 +20,6 @@
 #include <string.h>
 #include <stdio.h>
 #include "bms.h"
-#include "serial_monitor.h"
 #include "nero.h"
 
 #define CAN_QUEUE_SIZE 5 /* messages */
@@ -66,7 +65,7 @@ void dti_set_torque(int16_t torque)
 	/* Motor controller expects AC current target to be received as multiplied by 10 */
 	int16_t ac_current = (((float)average / EMRAX_KT) * 10);
 
-	// serial_print("Commanded Current: %d \r\n", ac_current);
+	// printf("Commanded Current: %d \r\n", ac_current);
 
 	dti_set_current(ac_current);
 }

Core/Src/fault.c

@@ -1,5 +1,4 @@
 #include "fault.h"
-#include "serial_monitor.h"
 #include "task.h"
 #include <assert.h>
 #include <stdio.h>
@@ -55,9 +54,8 @@ void vFaultHandler(void *pv_params)
 			       sizeof(defcon));
 
 			queue_can_msg(msg);
-			serial_print(
-				"\r\nFault Handler! Diagnostic Info:\t%s\r\n\r\n",
-				fault_data.diag);
+			printf("\r\nFault Handler! Diagnostic Info:\t%s\r\n\r\n",
+			       fault_data.diag);
 
 			switch (fault_data.severity) {
 			case DEFCON1: /* Highest(1st) Priority */

Core/Src/main.c

@@ -31,7 +31,6 @@
 #include "queues.h"
 #include "fault.h"
 #include "can_handler.h"
-#include "serial_monitor.h"
 #include "state_machine.h"
 #include "bms.h"
 #include "pdu.h"
@@ -230,8 +229,6 @@ int main(void)
   assert(can_dispatch_handle);
   can_receive_thread = osThreadNew(vCanReceive, mc, &can_receive_attributes);
   assert(can_receive_thread);
-  serial_monitor_handle = osThreadNew(vSerialMonitor, NULL, &serial_monitor_attributes);
-  assert(serial_monitor_handle);
 
   /* Control Logic */
   fault_handle = osThreadNew(vFaultHandler, NULL, &fault_handle_attributes);

Core/Src/monitor.c

@@ -7,7 +7,6 @@
 #include "mpu.h"
 #include "pdu.h"
 #include "queues.h"
-#include "serial_monitor.h"
 #include "sht30.h"
 #include "state_machine.h"
 #include "steeringio.h"
@@ -266,7 +265,7 @@ void vTempMonitor(void *pv_params)
 			queue_fault(&fault_data);
 		}
 
-		serial_print("MPU Board Temperature:\t%d\r\n", temp);
+		printf("MPU Board Temperature:\t%d\r\n", temp);
 
 		temp_msg.data[0] = temp & 0xFF;
 		temp_msg.data[1] = (temp >> 8) & 0xFF;
@@ -362,7 +361,7 @@ void vIMUMonitor(void *pv_params)
 	mpu_t *mpu = (mpu_t *)pv_params;
 
 	for (;;) {
-		// serial_print("IMU Task\r\n");
+		// printf("IMU Task\r\n");
 		/* Take measurement */
 		if (read_accel(mpu)) {
 			fault_data.diag = "Failed to get IMU acceleration";

Core/Src/nero.c

@@ -3,7 +3,6 @@
 #include "stdbool.h"
 #include "can_handler.h"
 #include "state_machine.h"
-#include "serial_monitor.h"
 #include "queues.h"
 #include "c_utils.h"
 #include "stdio.h"

Core/Src/pdu.c

@@ -1,5 +1,4 @@
 #include "pdu.h"
-#include "serial_monitor.h"
 #include "fault.h"
 #include <assert.h>
 #include <stdio.h>

Core/Src/pedals.c

@@ -17,7 +17,6 @@
 #include "cerberus_conf.h"
 #include "dti.h"
 #include "queues.h"
-#include "serial_monitor.h"
 #include "bms.h"
 #include "emrax.h"
 #include "monitor.h"

Core/Src/state_machine.c

@@ -4,7 +4,6 @@
 #include "nero.h"
 #include "queues.h"
 #include "monitor.h"
-#include "serial_monitor.h"
 #include "nero.h"
 #include "queues.h"
 #include "pedals.h"
@@ -71,7 +70,7 @@ static int transition_functional_state(func_state_t new_state, pdu_t *pdu,
 		// write_fan_battbox(pdu, false);
 		write_pump(pdu, false);
 		write_fault(pdu, false);
-		serial_print("READY\r\n");
+		printf("READY\r\n");
 		break;
 	case F_PIT:
 	case F_PERFORMANCE:
@@ -92,7 +91,7 @@ static int transition_functional_state(func_state_t new_state, pdu_t *pdu,
 		// write_fan_battbox(pdu, true);
 		write_pump(pdu, true);
 		write_fault(pdu, false);
-		serial_print("ACTIVE STATE\r\n");
+		printf("ACTIVE STATE\r\n");
 		break;
 	case REVERSE:
 		/* Can only enter reverse mode if already in pit mode */
@@ -105,9 +104,10 @@ static int transition_functional_state(func_state_t new_state, pdu_t *pdu,
 		write_pump(pdu, false);
 		cerberus_state.nero =
 			(nero_state_t){ .nero_index = OFF, .home_mode = false };
+
 		osDelay(1000); /* Delay for 1 sec before faulting car */
 		write_fault(pdu, true);
-		serial_print("FAULTED\r\n");
+		printf("FAULTED\r\n");
 		break;
 	default:
 		// Do Nothing

Core/Src/steeringio.c

@@ -6,7 +6,6 @@
 #include <stdlib.h>
 #include <string.h>
 #include "state_machine.h"
-#include "serial_monitor.h"
 #include "nero.h"
 #include "stdio.h"
 #include "cerb_utils.h"
@@ -99,11 +98,11 @@ static void debounce_cb(void *arg)
 			paddle_right_cb();
 			break;
 		case NERO_BUTTON_UP:
-			serial_print("Up button pressed \r\n");
+			printf("Up button pressed \r\n");
 			decrement_nero_index();
 			break;
 		case NERO_BUTTON_DOWN:
-			serial_print("Down button pressed \r\n");
+			printf("Down button pressed \r\n");
 			increment_nero_index();
 			break;
 		case NERO_BUTTON_LEFT:
@@ -117,7 +116,7 @@ static void debounce_cb(void *arg)
 			select_nero_index();
 			break;
 		case NERO_HOME:
-			serial_print("Home button pressed \r\n");
+			printf("Home button pressed \r\n");
 			set_home_mode();
 			break;
 		default:

Makefile

@@ -49,7 +49,6 @@ Core/Src/stm32f4xx_hal_msp.c \
 Core/Src/monitor.c \
 Core/Src/fault.c \
 Core/Src/can_handler.c \
-Core/Src/serial_monitor.c \
 Core/Src/dti.c \
 Core/Src/state_machine.c \
 Core/Src/bms.c \