Added (default) pattern for mixed impedance electrodes.

Design.md

@@ -78,8 +78,9 @@ Conceptually, the firmware consists of two layers:
 2. The hardware-independent application logic, implementing the _policies_.
 
 ### About the code
-- All C modules, with the exception of the BSP, are less than 300 lines long.
+- All C modules, with the exception of the BSP, are less than 400 lines long.
 - Functions are short, or have low cyclomatic complexity.
+- Functions are _pure_ whenever possible.
 - There are no global variables, preventing unwanted coupling between modules and violation of invariants.
 - Most 'plumbing' is done through dependency injection.
 - Object types are opaque where possible.

firmware/inc/sequencer.h

@@ -15,6 +15,8 @@
 
 #include <stdint.h>
 
+#define DEFAULT_PRIMARY_VOLTAGE_mV      1500
+
 typedef struct _Sequencer Sequencer;            // Opaque type.
 
 

firmware/src/controller.c

@@ -104,7 +104,7 @@ void Controller_start(Controller *me)
 {
     BSP_logf("Starting NeoDK!\n");
     BSP_logf("Push the button to play or pause :-)\n");
-    BSP_setPrimaryVoltage_mV(2500);
+    BSP_setPrimaryVoltage_mV(DEFAULT_PRIMARY_VOLTAGE_mV);
     BSP_primaryVoltageEnable(true);
 }
 

firmware/src/debug_cli.c

@@ -85,7 +85,7 @@ static void interpretCommand(CmndInterp *me, char ch)
             BSP_changePrimaryVoltage_mV(+200);
             break;
         case 'v':
-            CLI_logf("Firmware V0.25-beta\n");
+            CLI_logf("Firmware v0.26-beta\n");
             break;
         default:
             CLI_logf("Unknown command '/%c'\n", ch);

firmware/src/pattern_iter.c

@@ -69,7 +69,7 @@ void PatternIterator_init(PatternIterator *me, PatternDescr const *pd)
     me->nr_of_reps = pd->nr_of_reps;
     me->nr_of_steps = pd->nr_of_steps;          // Length of a transition.
 
-    me->pulse_width_micros = 80;
+    me->pulse_width_micros = 120;
     me->elcon_nr = 0;
     M_ASSERT(me->nr_of_steps != 0);
     me->step_nr = 0;

firmware/src/sequencer.c

@@ -54,6 +54,14 @@ static uint8_t const pattern_toggle[][2] =
     {EL_D, EL_C},
 };
 
+static uint8_t const pattern_cross_toggle[][2] =
+{
+    {EL_A, EL_D},
+    {EL_B, EL_C},
+    {EL_C, EL_B},
+    {EL_D, EL_A},
+};
+
 static uint8_t const pattern_circle[][2] =
 {
     {EL_A,  EL_B},
@@ -95,6 +103,14 @@ static PatternDescr const pattern_descr[] =
         .nr_of_steps = 5,
         .nr_of_reps = 300,
     },
+    {
+        .name = "CrossToggle",
+        .pattern = pattern_cross_toggle,
+        .nr_of_elcons = M_DIM(pattern_cross_toggle),
+        .pace_ms = 20,
+        .nr_of_steps = 5,
+        .nr_of_reps = 200,
+    },
     {
         .name = "Circle",
         .pattern = pattern_circle,
@@ -108,7 +124,7 @@ static PatternDescr const pattern_descr[] =
 
 static void selectNextRoutine(Sequencer *me)
 {
-    BSP_setPrimaryVoltage_mV(2500);
+    BSP_setPrimaryVoltage_mV(DEFAULT_PRIMARY_VOLTAGE_mV);
     if (++me->pattern_index == M_DIM(pattern_descr)) me->pattern_index = 0;
     PatternDescr const *pd = &pattern_descr[me->pattern_index];
     CLI_logf("Switching to pattern '%s'\n", pd->name);
@@ -280,7 +296,7 @@ Sequencer *Sequencer_new()
     Sequencer *me = (Sequencer *)malloc(sizeof(Sequencer));
     EventQueue_init(&me->event_queue, me->event_storage, sizeof me->event_storage);
     me->state = &stateNop;
-    me->pattern_index = 0;
+    me->pattern_index = 2;
     BSP_registerPulseDelegate(&me->event_queue);
     return me;
 }