pbio/light_animation: Make API public.

Separation of powers is nice, but since we have #include "../src/light/light_matrix.h" there is clearly something not quite right. Let's be pragmatic and make the APIs for animations public.

Author: laurensvalk

lib/pbio/src/light/animation.h renamed to lib/pbio/include/pbio/light_animation.h

@@ -11,12 +11,32 @@
 
 #include <stdint.h>
 
+#include <pbdrv/led.h>
+
 #include <pbio/config.h>
 #include <pbio/error.h>
 #include <pbio/geometry.h>
+#include <pbio/light_animation.h>
 
 /** A light matrix instance. */
-typedef struct _pbio_light_matrix_t pbio_light_matrix_t;
+typedef struct {
+    /** Animation instance for background animation. */
+    pbio_light_animation_t animation;
+    /** Animation cell data. */
+    const uint8_t *animation_cells;
+    /** The number of cells in @p animation_cells */
+    uint8_t num_animation_cells;
+    /** The index of the currently displayed animation cell. */
+    uint8_t current_cell;
+    /** Animation update rate in milliseconds. */
+    uint16_t interval;
+    /** Size of the matrix (assumes matrix is square). */
+    uint8_t size;
+    /** Orientation of the matrix: which side is "up". */
+    pbio_geometry_side_t up_side;
+    /** The driver for this light matrix. */
+    pbdrv_led_array_dev_t *led_array_dev;
+} pbio_light_matrix_t;
 
 #if PBIO_CONFIG_LIGHT_MATRIX
 

lib/pbio/include/pbio/light_matrix.h

@@ -12,7 +12,7 @@
 
 #include <pbio/util.h>
 
-#include "animation.h"
+#include <pbio/light_animation.h>
 
 /**
  * This is used as a value for the next_animation field to indicate when an

lib/pbio/src/light/animation.c

@@ -14,7 +14,7 @@
 #include <pbio/light.h>
 #include <pbio/util.h>
 
-#include "animation.h"
+#include <pbio/light_animation.h>
 #include "color_light.h"
 
 /**

lib/pbio/src/light/color_light.c

@@ -7,7 +7,7 @@
 #include <pbio/error.h>
 #include <pbio/light.h>
 
-#include "animation.h"
+#include <pbio/light_animation.h>
 
 #ifndef _PBIO_LIGHT_COLOR_LIGHT_H_
 #define _PBIO_LIGHT_COLOR_LIGHT_H_

lib/pbio/src/light/color_light.h

@@ -11,8 +11,8 @@
 #include <pbio/light_matrix.h>
 #include <pbio/util.h>
 
-#include "animation.h"
-#include "light_matrix.h"
+#include <pbio/light_animation.h>
+#include <pbio/light_matrix.h>
 
 static pbio_light_matrix_t light_matrices[PBIO_CONFIG_LIGHT_MATRIX_NUM_DEV];
 

lib/pbio/src/light/light_matrix.c

@@ -15,11 +15,10 @@
 #include <pbio/battery.h>
 #include <pbio/image.h>
 #include <pbio/imu.h>
+#include <pbio/light_animation.h>
 #include <pbio/motor_process.h>
 #include <pbio/port_interface.h>
 
-#include "light/animation.h"
-
 /**
  * Initialize the Pybricks I/O Library. This function must be called once,
  * usually at the beginning of a program, before using any other functions in

lib/pbio/src/light/light_matrix.h

@@ -18,7 +18,7 @@
 #include <pbsys/config.h>
 #include <pbsys/status.h>
 
-#include "../src/light/light_matrix.h"
+#include <pbio/light_matrix.h>
 #include "hmi.h"
 
 #if PBSYS_CONFIG_HUB_LIGHT_MATRIX