Almost full circle display add

Author: brentru

src/Wippersnapper_demo.ino.cpp

@@ -0,0 +1,26 @@
+# 1 "/var/folders/ff/dmzflvf52tq9kzvt6g8jglxw0000gn/T/tmpkb6hfvfs"
+#include <Arduino.h>
+# 1 "/Users/brentrubell/Documents/Arduino/libraries/Adafruit_Wippersnapper_Arduino/src/Wippersnapper_demo.ino"
+# 16 "/Users/brentrubell/Documents/Arduino/libraries/Adafruit_Wippersnapper_Arduino/src/Wippersnapper_demo.ino"
+#include "Wippersnapper_Networking.h"
+Wippersnapper_WiFi wipper;
+
+
+#define WS_DEBUG
+void setup();
+void loop();
+#line 22 "/Users/brentrubell/Documents/Arduino/libraries/Adafruit_Wippersnapper_Arduino/src/Wippersnapper_demo.ino"
+void setup() {
+
+  wipper.provision();
+
+  Serial.begin(115200);
+
+
+  wipper.connect();
+
+}
+
+void loop() {
+  wipper.run();
+}

src/components/display/controller.cpp

@@ -28,13 +28,35 @@ DisplayController::DisplayController() {
   // TODO
 }
 
+/*!
+    @brief  Handles a Display_AddOrReplace message.
+    @param  msgAdd
+            Pointer to a DisplayAddOrReplace message structure.
+    @return True if the display was added or replaced successfully, false otherwise.
+*/
 bool DisplayController::Handle_Display_AddOrReplace(wippersnapper_display_v1_DisplayAddOrReplace *msgAdd) {
-    DisplayHardware *display = new DisplayHardware();
+    DisplayHardware *display = new DisplayHardware(msgAdd->name);
+
     // Configure display type
     display->setType(msgAdd->type);
-    // Configure display based on config type
+
+    // Attempt to initialize display hardware instance
+    bool did_begin = false;
     if (msgAdd->which_config == wippersnapper_display_v1_DisplayAddOrReplace_epd_config_tag) {
-        display->configureEPD(&msgAdd->config.epd_config, &msgAdd->interface_type.spi_epd);
+        did_begin = display->beginEPD(&msgAdd->config.epd_config, &msgAdd->interface_type.spi_epd);
+    } else {
+      WS_DEBUG_PRINTLN("[display] Unsupported display configuration type!");
+      return false;
     }
+
+    // Check if the display began successfully
+    if (!did_begin) {
+      WS_DEBUG_PRINTLN("[display] Failed to initialize display!");
+      delete display; // Clean up if initialization failed
+      return false;
+    }
+
+    _hw_instances.push_back(display); // Store the display instance
+    WS_DEBUG_PRINTLN("[display] Display added or replaced successfully!");
     return true; // Placeholder
 }

src/components/display/controller.h

@@ -32,10 +32,10 @@ class DisplayController {
   DisplayController();
   ~DisplayController();
   bool Handle_Display_AddOrReplace(wippersnapper_display_v1_DisplayAddOrReplace *msgAdd);
-  //bool Handle_Display_Write(pb_istream_t *stream);
-  //bool Handle_Display_Remove(pb_istream_t *stream);
+  //bool Handle_Display_Remove(...);
+  //bool Handle_Display_Write(...);
 private:
-  //DisplayHardware *_display_strands[MAX_DISPLAY_STRANDS] = {
+  std::vector<DisplayHardware*> _hw_instances; ///< Holds pointers to DisplayHardware instances
 };
 extern Wippersnapper Ws; ///< Global WS instance
 #endif

src/components/display/hardware.cpp

@@ -17,15 +17,16 @@
 /*!
     @brief  Constructs a new DisplayHardware object
 */
-DisplayHardware::DisplayHardware() {
+DisplayHardware::DisplayHardware(const char *name) {
+  _name = name; ///< Set the name of the hardware instance
   _type = wippersnapper_display_v1_DisplayType_DISPLAY_TYPE_UNSPECIFIED;
 }
 
 /*!
     @brief  Destructor
 */
-DisplayHardware::DisplayHardware() {
-  // TODO: Clean up display drivers
+DisplayHardware::~DisplayHardware() {
+  // TODO
 }
 
 /*!

src/components/display/hardware.h

@@ -28,18 +28,18 @@
 /**************************************************************************/
 class DisplayHardware {
 public:
-  DisplayHardware();
+  DisplayHardware(const char *name);
   ~DisplayHardware();
   // High-level API functions
   void setType(wippersnapper_display_v1_DisplayType type);
   wippersnapper_display_v1_DisplayType getType();
-  void beginEPD(wippersnapper_display_v1_EPDConfig *config,
-                wippersnapper_display_v1_EpdSpiConfig *spi_config);
+  bool beginEPD(wippersnapper_display_v1_EPDConfig *config, wippersnapper_display_v1_EpdSpiConfig *spi_config);
   bool begin(bool reset = true);
   // API functions to abstract Adafruit_GFX
   void setTextSize(uint8_t sz);
 
 private:
+  const char *_name; ///< Identifies the hardware instance
   wippersnapper_display_v1_DisplayType _type; ///< Display type
   wippersnapper_display_v1_EPDThinkInkPanel
       _thinkink_driver; ///< ThinkInk driver type