feat: lighting effects, configuration, unsolicited gyro readings

.github/workflows/build.yml

@@ -82,3 +82,12 @@ jobs:
         with:
           name: zwa2_controller.hex
           path: artifact/zwa2_controller.hex
+
+      - name: Copy MAP file
+        run: cp build/release/nc_controller_ncp.map artifact/zwa2_controller.map
+
+      - name: Upload MAP
+        uses: actions/upload-artifact@v4
+        with:
+          name: zwa2_controller.map
+          path: artifact/zwa2_controller.map

app.c

@@ -42,6 +42,7 @@
 #include "ZAF_PrintAppInfo.h"
 #endif
 #include "ZAF_AppName.h"
+#include <ZAF_nvm_app.h>
 
 #include <assert.h>
 
@@ -66,7 +67,24 @@ static TaskHandle_t m_xTaskHandleBackgroundHw   = NULL;
 static StaticTask_t BackgroundHwTaskBuffer;
 static uint8_t BackgroundHwStackBuffer[HW_TASK_STACK_SIZE];
 
+// Gyro measurements
 bool bRequestGyroMeasurement = false;
+gyro_reading_t last_gyro_reading = {0};
+int stable_gyro_readings = 0;
+
+bool bAwaitingConnection = false;
+// Whether incorrect tilt was detected and should be indicated
+bool bTiltDetected = false;
+bool bEnableTiltDetection = true;
+
+// Default LED effect when no other effect is set
+LedEffect_t ledEffectDefault = {0};
+// LED state set by the user
+LedEffect_t ledEffectUser = {0};
+// Separate high priority LED state for system indication
+LedEffect_t ledEffectSystem = {0};
+// LED state to indicate incorrect tilt
+LedEffect_t ledEffectTilt = {0};
 
 static void ApplicationInitSW(void);
 static void ApplicationTask(SApplicationHandles *pAppHandles);
@@ -529,6 +547,11 @@ ApplicationTask(SApplicationHandles* pAppHandles)
 
 static void SerialAPICommandHandler(void)
 {
+  if (bAwaitingConnection) {
+    bAwaitingConnection = false;
+    zaf_event_distributor_enqueue_proprietary_app_event(EVENT_APP_CONNECTED, NULL);
+  }
+
   const bool handler_invoked = invoke_cmd_handler(serial_frame);
   if (!handler_invoked)
   {
@@ -785,49 +808,211 @@ ZCB_WakeupTimeout(__attribute__((unused)) SSwTimer *pTimer)
   DPRINT("ZCB_WakeupTimeout\n");
 }
 
+/* Returns the current effect that should be used for the LED */
+LedEffect_t*
+get_current_led_effect(void) {
+  if (ledEffectSystem.type != LED_EFFECT_NOT_SET) {
+    return &ledEffectSystem;
+  } else if (bEnableTiltDetection && bTiltDetected) {
+    return &ledEffectTilt;
+  } else if (ledEffectUser.type != LED_EFFECT_NOT_SET) {
+    return &ledEffectUser;
+  } else {
+    return &ledEffectDefault;
+  }
+}
 
+void
+trigger_led_effect_refresh(void) {
+  if (ledEffectUser.type == LED_EFFECT_SOLID) {
+    ledEffectUser.effect.solid.modified = true;
+  }
+  if (ledEffectDefault.type == LED_EFFECT_SOLID) {
+    ledEffectDefault.effect.solid.modified = true;
+  }
+}
 
 void
 zaf_event_distributor_app_proprietary(event_nc_t *event)
 {
   // Handles NC-specific proprietary events
   EVENT_APP event_nc = (EVENT_APP) event->event;
   switch (event_nc) {
-    case EVENT_APP_USERTASK_READY:
-      // Indicate that the firmware is ready by enabling the LED at low power
-      rgb_t color = {4, 0, 0};
-      set_color_buffer(color);
+    case EVENT_APP_USERTASK_READY: {
+      // Fade slowly to white
+      LedEffectFade_t fade = {
+        .color = white,
+        .brightness = 0xff,
+        .increasing = false,
+        .ticksPerStep = 2,
+        .tickCounter = 0
+      };
+      ledEffectDefault = (LedEffect_t) {
+        .type = LED_EFFECT_FADE,
+        .effect.fade = fade
+      };
+      bAwaitingConnection = true;
       break;
+    }
 
-    case EVENT_APP_USERTASK_GYRO_MEASUREMENT:
-      if (!bRequestGyroMeasurement) {
-        return;
+    case EVENT_APP_CONNECTED: {
+      if (ledEffectDefault.type == LED_EFFECT_FADE) {
+        // Stop the animation to indicate that we're connected
+        ledEffectDefault.effect.fade.stopAtMax = true;
+      } else {
+        // If we were not in fade mode, set the color to white
+        LedEffectSolid_t solid = {
+          .color = white,
+          .modified = true
+        };
+        ledEffectDefault = (LedEffect_t) {
+          .type = LED_EFFECT_SOLID,
+          .effect.solid = solid
+        };
       }
-      bRequestGyroMeasurement = false;
+      break;
+    }
 
+    case EVENT_APP_USERTASK_GYRO_MEASUREMENT: {
       // A gyro measurement was requested
       gyro_reading_t gyro_reading = event->payload->gyro_reading;
 
-      uint8_t cmd[8];
-      uint8_t i=0;
-      cmd[i++] = NABU_CASA_GYRO_MEASURE;
-      cmd[i++] = gyro_reading.x >> 8;
-      cmd[i++] = gyro_reading.x & 0xFF;
-      cmd[i++] = gyro_reading.y >> 8;
-      cmd[i++] = gyro_reading.y & 0xFF;
-      cmd[i++] = gyro_reading.z >> 8;
-      cmd[i++] = gyro_reading.z & 0xFF;
-      RequestUnsolicited(
-        FUNC_ID_NABU_CASA,
-        cmd,
-        i
-      );
+      // Debounce the readings
+      if (last_gyro_reading.x == 0 && last_gyro_reading.y == 0 && last_gyro_reading.z == 0) {
+        last_gyro_reading = gyro_reading;
+        return;
+      } else if (abs(gyro_reading.x - last_gyro_reading.x) < 25 &&
+                 abs(gyro_reading.y - last_gyro_reading.y) < 25 &&
+                 abs(gyro_reading.z - last_gyro_reading.z) < 25) {
+        stable_gyro_readings++;
+      } else {
+        stable_gyro_readings = 0;
+      }
+      last_gyro_reading = gyro_reading;
+
+      if (stable_gyro_readings < 3) {
+        // Not enough stable readings, ignore this one
+        return;
+      }
+
+      // Indicate bad orientation (more than 20° from vertical) in calibration mode
+      if (gyro_reading.z < -960 || gyro_reading.z > 960) {
+        if (bTiltDetected) {
+          // Mark the user LED effect as modified, so it gets used again
+          trigger_led_effect_refresh();
+        }
+        bTiltDetected = false;
+      } else if (!bTiltDetected) {
+        LedEffectFade_t fade = {
+          .color = red,
+          .brightness = 0xff,
+          .increasing = false,
+          .ticksPerStep = 1,
+          .tickCounter = 0
+        };
+        ledEffectTilt = (LedEffect_t) {
+          .type = LED_EFFECT_FADE,
+          .effect.fade = fade
+        };
+        bTiltDetected = true;
+      }
+
+      // Send a gyro measurement whenever the user requests it
+      // or the orientation has just stabilized
+      if (bRequestGyroMeasurement || stable_gyro_readings == 7) {
+        bRequestGyroMeasurement = false;
+
+        uint8_t cmd[8];
+        uint8_t i=0;
+        cmd[i++] = NABU_CASA_GYRO_MEASURE;
+        cmd[i++] = gyro_reading.x >> 8;
+        cmd[i++] = gyro_reading.x & 0xFF;
+        cmd[i++] = gyro_reading.y >> 8;
+        cmd[i++] = gyro_reading.y & 0xFF;
+        cmd[i++] = gyro_reading.z >> 8;
+        cmd[i++] = gyro_reading.z & 0xFF;
+        RequestUnsolicited(
+          FUNC_ID_NABU_CASA,
+          cmd,
+          i
+        );
+      }
+      break;
+    }
+
+    case EVENT_APP_USERTASK_TICK_LED: {
+      LedEffect_t* ledEffect = get_current_led_effect();
+
+      switch (ledEffect->type) {
+        case LED_EFFECT_SOLID: {
+          // For solid LED effect, set the color once
+          if (ledEffect->effect.solid.modified) {
+            ledEffect->effect.solid.modified = false;
+            set_color_buffer(ledEffect->effect.solid.color);
+          }
+          break;
+        }
+
+        case LED_EFFECT_FADE: {
+          // For fading, change the brightness every N ticks,
+          // and update the color
+          LedEffectFade_t fade = ledEffect->effect.fade;
+
+          if (fade.brightness > 0xe0 && fade.stopAtMax) {
+            // Switch to solid mode
+            LedEffectSolid_t solid = {
+              .color = fade.color,
+              .modified = true
+            };
+            *ledEffect = (LedEffect_t) {
+              .type = LED_EFFECT_SOLID,
+              .effect.solid = solid
+            };
+            break;
+          }
+
+          if (fade.tickCounter == 0) {
+            if (fade.increasing) {
+              fade.brightness++;
+              if (fade.brightness == 0xff) {
+                fade.increasing = false;
+              }
+            } else {
+              fade.brightness--;
+              if (fade.brightness == 0) {
+                fade.increasing = true;
+              }
+            }
+
+            uint16_t r = ((uint16_t) fade.color.R) * ((uint16_t) fade.brightness) / 0xff;
+            uint16_t g = ((uint16_t) fade.color.G) * ((uint16_t) fade.brightness) / 0xff;
+            uint16_t b = ((uint16_t) fade.color.B) * ((uint16_t) fade.brightness) / 0xff;
+
+            rgb_t color = {
+              (uint8_t) g,
+              (uint8_t) r,
+              (uint8_t) b
+            };
+
+            set_color_buffer(color);
+          }
+
+          fade.tickCounter = (fade.tickCounter + 1) % fade.ticksPerStep;
+          ledEffect->effect.fade = fade;
+          break;
+        }
+
+        default:
+          // Nothing to do
+          break;
+      } // switch (ledEffect->type)
       break;
+    }
 
     default:
       // Nothing to do
       break;
-  }
+  } // switch (event_nc)
 }
 
 // Called when the button next to the USB port is pressed or released
@@ -957,6 +1142,32 @@ ApplicationInit(
   initWs2812();
   initqma6100p();
 
+  // Try to restore the user-defined color from NVM
+  NabuCasaLedStorage_t ledStorage = {0};
+  if (
+    ZPAL_STATUS_OK == ZAF_nvm_app_read(FILE_ID_NABUCASA_LED, &ledStorage, sizeof(ledStorage))
+    && ledStorage.valid
+  ) {
+    rgb_t color = {
+      .G = ledStorage.g,
+      .R = ledStorage.r,
+      .B = ledStorage.b
+    };
+    ledEffectUser = (LedEffect_t) {
+      .type = LED_EFFECT_SOLID,
+      .effect.solid = {
+        .color = color,
+        .modified = true
+      }
+    };
+    set_color_buffer(color);
+  } else {
+    set_color_buffer(white);
+  }
+
+  // Try to restore settings from NVM
+  bEnableTiltDetection = nc_config_get(NC_CFG_ENABLE_TILT_INDICATOR);
+
   /*************************************************************************************
    * CREATE USER TASKS  -  ZW_ApplicationRegisterTask() and ZW_UserTask_CreateTask()
    *************************************************************************************

app.h

@@ -35,6 +35,8 @@
 #include <ZW_controller_api.h>
 #endif
 
+#include "drivers/ws2812.h"
+
 /* Serial API version */
 #define SERIAL_API_VER 10
 
@@ -270,4 +272,34 @@ extern void ApplicationNodeUpdate(uint8_t bStatus, uint16_t nodeID, uint8_t *pCm
 
 extern uint8_t compl_workbuf[BUF_SIZE_TX];
 
+void trigger_led_effect_refresh(void);
+
+typedef struct LedEffectSolid {
+  rgb_t color;
+  bool modified;
+} LedEffectSolid_t;
+
+typedef struct LedEffectFade {
+  rgb_t color;
+  uint8_t brightness;
+  uint8_t ticksPerStep;
+  uint8_t tickCounter;
+  bool increasing;
+  bool stopAtMax;
+} LedEffectFade_t;
+
+typedef enum {
+  LED_EFFECT_NOT_SET,
+  LED_EFFECT_SOLID,
+  LED_EFFECT_FADE
+} LedEffectType_t;
+
+typedef struct {
+  LedEffectType_t type;
+  union {
+    LedEffectSolid_t solid;
+    LedEffectFade_t fade;
+  } effect;
+} LedEffect_t;
+
 #endif /* _SERIALAPPL_H_ */

app_events.h

@@ -21,6 +21,8 @@ typedef enum EVENT_APP_NC
    */
   EVENT_APP_USERTASK_READY,
   EVENT_APP_USERTASK_GYRO_MEASUREMENT,
+  EVENT_APP_USERTASK_TICK_LED,
+  EVENT_APP_CONNECTED,
 }
 EVENT_APP;