Finish up the public API of the library

All the internal functions and definitions have been moved out of
lib/include/.  Includes are made with the semantically correct brackets
("" or <>) throughout the firmware and library.  That is to say, the
library is essentially done at this point, with only documentation
changes left to be made.

lib/include/pdb_dpm.h

@@ -21,9 +21,8 @@
 
 #include <stdbool.h>
 
+#include <pdb_fusb.h>
 #include <pdb_msg.h>
-/* TODO: improve this include */
-#include "fusb302b.h"
 
 
 /* Forward declaration of struct pdb_config */

lib/include/pdb_fusb.h

@@ -21,6 +21,12 @@
 
 #include <hal.h>
 
+/* I2C addresses of the FUSB302B chips */
+#define FUSB302B_ADDR 0x22
+#define FUSB302B01_ADDR 0x23
+#define FUSB302B10_ADDR 0x24
+#define FUSB302B11_ADDR 0x25
+
 
 /*
  * Configuration for the FUSB302B chip
@@ -32,5 +38,15 @@ struct pdb_fusb_config {
     i2caddr_t addr;
 };
 
+/*
+ * FUSB Type-C Current level enum
+ */
+enum fusb_typec_current {
+    fusb_tcc_none = 0,
+    fusb_tcc_default = 1,
+    fusb_tcc_1_5 = 2,
+    fusb_tcc_3_0 = 3
+};
+
 
 #endif /* PDB_FUSB_H */

lib/src/fusb302b.c

@@ -21,7 +21,7 @@
 #include <ch.h>
 #include <hal.h>
 
-#include "pd.h"
+#include <pd.h>
 
 
 /*

lib/include/fusb302b.h → lib/src/fusb302b.h

@@ -26,12 +26,6 @@
 #include <pdb_msg.h>
 
 
-/* I2C addresses of the FUSB302B chips */
-#define FUSB302B_ADDR 0x22
-#define FUSB302B01_ADDR 0x23
-#define FUSB302B10_ADDR 0x24
-#define FUSB302B11_ADDR 0x25
-
 /* Device ID register */
 #define FUSB_DEVICE_ID 0x01
 #define FUSB_DEVICE_ID_VERSION_ID_SHIFT 4
@@ -267,16 +261,6 @@ union fusb_status {
     };
 };
 
-/*
- * FUSB Type-C Current level enum
- */
-enum fusb_typec_current {
-    fusb_tcc_none = 0,
-    fusb_tcc_default = 1,
-    fusb_tcc_1_5 = 2,
-    fusb_tcc_3_0 = 3
-};
-
 
 /* FUSB functions */
 

lib/src/hard_reset.c

@@ -18,12 +18,12 @@
 
 #include "hard_reset.h"
 
+#include <pd.h>
 #include "priorities.h"
 #include "policy_engine.h"
 #include "protocol_rx.h"
 #include "protocol_tx.h"
 #include "fusb302b.h"
-#include "pd.h"
 
 
 /*

lib/src/policy_engine.c

@@ -20,11 +20,11 @@
 
 #include <stdbool.h>
 
+#include <pd.h>
 #include "priorities.h"
 #include "protocol_tx.h"
 #include "hard_reset.h"
 #include "fusb302b.h"
-#include "pd.h"
 
 
 enum policy_engine_state {

lib/src/protocol_rx.c

@@ -20,11 +20,11 @@
 
 #include <stdlib.h>
 
+#include <pd.h>
 #include "priorities.h"
 #include "policy_engine.h"
 #include "protocol_tx.h"
 #include "fusb302b.h"
-#include "pd.h"
 
 
 /*

lib/src/protocol_tx.c

@@ -18,11 +18,11 @@
 
 #include "protocol_tx.h"
 
+#include <pd.h>
 #include "priorities.h"
 #include "policy_engine.h"
 #include "protocol_rx.h"
 #include "fusb302b.h"
-#include "pd.h"
 
 
 /*

src/config.c

@@ -20,7 +20,7 @@
 
 #include "chprintf.h"
 
-#include "pd.h"
+#include <pd.h>
 
 
 /* Initialize the location of the configuration array.  PDBS_CONFIG_BASE is set

src/device_policy_manager.c

@@ -22,9 +22,10 @@
 
 #include <hal.h>
 
+#include <pd.h>
+
 #include "led.h"
 #include "config.h"
-#include "pd.h"
 
 
 /* The current draw when the output is disabled */

src/device_policy_manager.h

@@ -22,7 +22,6 @@
 #include <stdbool.h>
 
 #include <pdb.h>
-#include "fusb302b.h"
 
 
 struct pdbs_dpm_data {

src/main.c

@@ -39,9 +39,9 @@
 #include "usbcfg.h"
 
 #include <pdb.h>
+#include <pd.h>
 #include "led.h"
 #include "device_policy_manager.h"
-#include "pd.h"
 
 /*
  * I2C configuration object.

src/shell.c

@@ -40,11 +40,11 @@
 #include "chprintf.h"
 
 #include <pdb.h>
+#include <pd.h>
 #include "usbcfg.h"
 #include "config.h"
 #include "led.h"
 #include "device_policy_manager.h"
-#include "pd.h"
 
 
 /* Buffer for unwritten configuration */