Refactoring: common.h with common typedefs etc.

firmware/src/commands.c

@@ -1,5 +1,5 @@
-#include "config.h"
 #include "commands.h"
+#include "common.h"
 #include "parsing.h"
 #include "uart.h"
 #include "eeprom.h"

firmware/src/commands.h

@@ -1,7 +1,7 @@
 #ifndef COMMANDS_H_
 #define COMMANDS_H_
 
-#include "config.h"
+#include "common.h"
 #include "parsing.h"
 
 void executeCommand(CommandLine cmdLine);

firmware/src/common.h

@@ -0,0 +1,21 @@
+#ifndef COMMON_H_
+#define COMMON_H_
+
+#include "config.h"
+#include <avr/io.h>
+#include <stdbool.h>
+
+// Define macro for NOP instruction
+#define _NOP() __asm__ __volatile__ ("nop");
+
+// Define type for EEPROM addresses
+typedef uint16_t address_t;
+
+// Type definition for address ranges (from-to)
+typedef struct {
+    bool isValid;
+    address_t from;
+    address_t to;
+} AddressRange;
+
+#endif /* COMMON_H_ */

firmware/src/eeprom.c

@@ -1,4 +1,3 @@
-#include "config.h"
 #include "eeprom.h"
 
 #include <avr/io.h>

firmware/src/eeprom.h

@@ -2,14 +2,9 @@
 #define EEPROM_H_
 
 #include "config.h"
+#include "common.h"
 #include <avr/io.h>
 
-// Define macro for NOP instruction
-#define _NOP() __asm__ __volatile__ ("nop");
-
-// Define type for (to be shifted) addresses
-typedef uint16_t address_t;
-
 // Define address length
 // TODO 15 or 16?
 #define ADDRESS_LENGTH 15

firmware/src/main.c

@@ -1,4 +1,5 @@
 #include "config.h"
+#include "common.h"
 #include "uart.h"
 #include "eeprom.h"
 #include "parsing.h"

firmware/src/parsing.h

@@ -1,8 +1,7 @@
 #ifndef PARSING_H_
 #define PARSING_H_
 
-#include "config.h"
+#include "common.h"
-#include "eeprom.h"
 #include <stdbool.h>
 
 // Type definition for a command line with optional argument (actually just two pointers to strings)
@@ -11,13 +10,6 @@ typedef struct {
     char* arg;
 } CommandLine;
 
-// Type definition for address ranges (from-to)
-typedef struct {
-    bool isValid;
-    address_t from;
-    address_t to;
-} AddressRange;
-
 void parseNextCommand();
 CommandLine readNextCommand(char* buffer, uint8_t bufferLength);
 CommandLine tokenizeCommand(char* cmd);