]> git.donarmstrong.com Git - kiibohd-controller.git/blobdiff - Macro/PartialMap/macro.c
DPH controller now working with the kishsaver and macros
[kiibohd-controller.git] / Macro / PartialMap / macro.c
index 3e31bb35d2f8338d863c0dd6c0ca5445a177a8ba..a86efb09e42b916069f8b6b2ca522f4cd2a4596e 100644 (file)
@@ -27,8 +27,7 @@
 
 // Keymaps
 #include "usb_hid.h"
-#include <defaultMap.h>
-#include "generatedKeymap.h" // TODO Use actual generated version
+#include <generatedKeymap.h> // Generated using kll at compile time, in build directory
 
 // Local Includes
 #include "macro.h"
@@ -56,12 +55,13 @@ void cliFunc_macroStep ( char* args );
 
 // Bit positions are important, passes (correct key) always trump incorrect key votes
 typedef enum TriggerMacroVote {
-       TriggerMacroVote_Release      = 0x8, // Correct key
-       TriggerMacroVote_PassRelease  = 0xC, // Correct key (both pass and release)
-       TriggerMacroVote_Pass         = 0x4, // Correct key
-       TriggerMacroVote_DoNothing    = 0x2, // Incorrect key
-       TriggerMacroVote_Fail         = 0x1, // Incorrect key
-       TriggerMacroVote_Invalid      = 0x0, // Invalid state
+       TriggerMacroVote_Release          = 0x10, // Correct key
+       TriggerMacroVote_PassRelease      = 0x18, // Correct key (both pass and release)
+       TriggerMacroVote_Pass             = 0x8,  // Correct key
+       TriggerMacroVote_DoNothingRelease = 0x4,  // Incorrect key
+       TriggerMacroVote_DoNothing        = 0x2,  // Incorrect key
+       TriggerMacroVote_Fail             = 0x1,  // Incorrect key
+       TriggerMacroVote_Invalid          = 0x0,  // Invalid state
 } TriggerMacroVote;
 
 typedef enum TriggerMacroEval {
@@ -106,7 +106,7 @@ uint8_t macroDebugMode = 0;
 uint8_t macroPauseMode = 0;
 
 // Macro step counter - If non-zero, the step counter counts down every time the macro module does one processing loop
-unsigned int macroStepCounter = 0;
+uint16_t macroStepCounter = 0;
 
 
 // Key Trigger List Buffer
@@ -118,45 +118,29 @@ uint8_t macroTriggerListBufferSize = 0;
 // TODO, figure out a good way to scale this array size without wasting too much memory, but not rejecting macros
 //       Possibly could be calculated by the KLL compiler
 // XXX It may be possible to calculate the worst case using the KLL compiler
-unsigned int macroTriggerMacroPendingList[ TriggerMacroNum ] = { 0 };
-unsigned int macroTriggerMacroPendingListSize = 0;
+uint16_t macroTriggerMacroPendingList[ TriggerMacroNum ] = { 0 };
+uint16_t macroTriggerMacroPendingListSize = 0;
 
 // Layer Index Stack
 //  * When modifying layer state and the state is non-0x0, the stack must be adjusted
-unsigned int macroLayerIndexStack[ LayerNum ] = { 0 };
-unsigned int macroLayerIndexStackSize = 0;
+uint16_t macroLayerIndexStack[ LayerNum + 1 ] = { 0 };
+uint16_t macroLayerIndexStackSize = 0;
 
 // Pending Result Macro Index List
 //  * Any result macro that needs processing from a previous macro processing loop
-unsigned int macroResultMacroPendingList[ ResultMacroNum ] = { 0 };
-unsigned int macroResultMacroPendingListSize = 0;
+uint16_t macroResultMacroPendingList[ ResultMacroNum ] = { 0 };
+uint16_t macroResultMacroPendingListSize = 0;
 
 
 
 // ----- Capabilities -----
 
-// Modifies the specified Layer control byte
-// Argument #1: Layer Index -> unsigned int
-// Argument #2: Toggle byte -> uint8_t
-void Macro_layerStateToggle_capability( uint8_t state, uint8_t stateType, uint8_t *args )
+// Sets the given layer with the specified layerState
+void Macro_layerState( uint8_t state, uint8_t stateType, uint16_t layer, uint8_t layerState )
 {
-       // Display capability name
-       if ( stateType == 0xFF && state == 0xFF )
-       {
-               print("Macro_layerState(layerIndex,toggleByte)");
-               return;
-       }
-
-       // Get layer index from arguments
-       // Cast pointer to uint8_t to unsigned int then access that memory location
-       unsigned int layer = *(unsigned int*)(&args[0]);
-
-       // Get layer toggle byte
-       uint8_t toggleByte = args[ sizeof(unsigned int) ];
-
        // Is layer in the LayerIndexStack?
        uint8_t inLayerIndexStack = 0;
-       unsigned int stackItem = 0;
+       uint16_t stackItem = 0;
        while ( stackItem < macroLayerIndexStackSize )
        {
                // Flag if layer is already in the LayerIndexStack
@@ -171,15 +155,15 @@ void Macro_layerStateToggle_capability( uint8_t state, uint8_t stateType, uint8_
        }
 
        // Toggle Layer State Byte
-       if ( LayerIndex[ layer ].state & toggleByte )
+       if ( LayerState[ layer ] & layerState )
        {
                // Unset
-               LayerIndex[ layer ].state &= ~toggleByte;
+               LayerState[ layer ] &= ~layerState;
        }
        else
        {
                // Set
-               LayerIndex[ layer ].state |= toggleByte;
+               LayerState[ layer ] |= layerState;
        }
 
        // If the layer was not in the LayerIndexStack add it
@@ -189,7 +173,7 @@ void Macro_layerStateToggle_capability( uint8_t state, uint8_t stateType, uint8_
        }
 
        // If the layer is in the LayerIndexStack and the state is 0x00, remove
-       if ( LayerIndex[ layer ].state == 0x00 && inLayerIndexStack )
+       if ( LayerState[ layer ] == 0x00 && inLayerIndexStack )
        {
                // Remove the layer from the LayerIndexStack
                // Using the already positioned stackItem variable from the loop above
@@ -204,55 +188,169 @@ void Macro_layerStateToggle_capability( uint8_t state, uint8_t stateType, uint8_
        }
 }
 
+// Modifies the specified Layer control byte
+// Argument #1: Layer Index -> uint16_t
+// Argument #2: Layer State -> uint8_t
+void Macro_layerState_capability( uint8_t state, uint8_t stateType, uint8_t *args )
+{
+       // Display capability name
+       if ( stateType == 0xFF && state == 0xFF )
+       {
+               print("Macro_layerState(layerIndex,layerState)");
+               return;
+       }
+
+       // Only use capability on press or release
+       // TODO Analog
+       // XXX This may cause issues, might be better to implement state table here to decide -HaaTa
+       if ( stateType == 0x00 && state == 0x02 ) // Hold condition
+               return;
+
+       // Get layer index from arguments
+       // Cast pointer to uint8_t to uint16_t then access that memory location
+       uint16_t layer = *(uint16_t*)(&args[0]);
+
+       // Get layer toggle byte
+       uint8_t layerState = args[ sizeof(uint16_t) ];
+
+       Macro_layerState( state, stateType, layer, layerState );
+}
+
+
+// Latches given layer
+// Argument #1: Layer Index -> uint16_t
+void Macro_layerLatch_capability( uint8_t state, uint8_t stateType, uint8_t *args )
+{
+       // Display capability name
+       if ( stateType == 0xFF && state == 0xFF )
+       {
+               print("Macro_layerLatch(layerIndex)");
+               return;
+       }
+
+       // Only use capability on press
+       // TODO Analog
+       // XXX To make sense, this code be on press or release. Or it could even be a sticky shift (why? dunno) -HaaTa
+       if ( stateType == 0x00 && state != 0x01 ) // All normal key conditions except press
+               return;
+
+       // Get layer index from arguments
+       // Cast pointer to uint8_t to uint16_t then access that memory location
+       uint16_t layer = *(uint16_t*)(&args[0]);
+
+       Macro_layerState( state, stateType, layer, 0x02 );
+}
+
+
+// Locks given layer
+// Argument #1: Layer Index -> uint16_t
+void Macro_layerLock_capability( uint8_t state, uint8_t stateType, uint8_t *args )
+{
+       // Display capability name
+       if ( stateType == 0xFF && state == 0xFF )
+       {
+               print("Macro_layerLock(layerIndex)");
+               return;
+       }
+
+       // Only use capability on press
+       // TODO Analog
+       // XXX Could also be on release, but that's sorta dumb -HaaTa
+       if ( stateType == 0x00 && state != 0x01 ) // All normal key conditions except press
+               return;
+
+       // Get layer index from arguments
+       // Cast pointer to uint8_t to uint16_t then access that memory location
+       uint16_t layer = *(uint16_t*)(&args[0]);
+
+       Macro_layerState( state, stateType, layer, 0x04 );
+}
+
+
+// Shifts given layer
+// Argument #1: Layer Index -> uint16_t
+void Macro_layerShift_capability( uint8_t state, uint8_t stateType, uint8_t *args )
+{
+       // Display capability name
+       if ( stateType == 0xFF && state == 0xFF )
+       {
+               print("Macro_layerShift(layerIndex)");
+               return;
+       }
+
+       // Only use capability on press or release
+       // TODO Analog
+       if ( stateType == 0x00 && ( state == 0x00 || state == 0x02 ) ) // Only pass press or release conditions
+               return;
+
+       // Get layer index from arguments
+       // Cast pointer to uint8_t to uint16_t then access that memory location
+       uint16_t layer = *(uint16_t*)(&args[0]);
+
+       Macro_layerState( state, stateType, layer, 0x01 );
+}
+
 
 
 // ----- Functions -----
 
 // Looks up the trigger list for the given scan code (from the active layer)
 // NOTE: Calling function must handle the NULL pointer case
-unsigned int *Macro_layerLookup( uint8_t scanCode )
+nat_ptr_t *Macro_layerLookup( uint8_t scanCode )
 {
        // If no trigger macro is defined at the given layer, fallthrough to the next layer
-       for ( unsigned int layerIndex = 0; layerIndex < macroLayerIndexStackSize; layerIndex++ )
+       for ( uint16_t layerIndex = 0; layerIndex < macroLayerIndexStackSize; layerIndex++ )
        {
                // Lookup Layer
-               Layer *layer = &LayerIndex[ macroLayerIndexStack[ layerIndex ] ];
+               const Layer *layer = &LayerIndex[ macroLayerIndexStack[ layerIndex ] ];
 
                // Check if latch has been pressed for this layer
                // XXX Regardless of whether a key is found, the latch is removed on first lookup
-               uint8_t latch = layer->state & 0x02;
+               uint8_t latch = LayerState[ layerIndex ] & 0x02;
                if ( latch )
                {
-                       layer->state &= ~0x02;
+                       LayerState[ layerIndex ] &= ~0x02;
                }
 
                // Only use layer, if state is valid
                // XOR each of the state bits
                // If only two are enabled, do not use this state
-               if ( (layer->state & 0x01) ^ (latch>>1) ^ ((layer->state & 0x04)>>2) )
+               if ( (LayerState[ macroLayerIndexStack[ layerIndex ] ] & 0x01) ^ (latch>>1) ^ ((LayerState[ macroLayerIndexStack[ layerIndex ] ] & 0x04)>>2) )
                {
                        // Lookup layer
-                       unsigned int **map = (unsigned int**)layer->triggerMap;
+                       nat_ptr_t **map = (nat_ptr_t**)layer->triggerMap;
 
                        // Determine if layer has key defined
-                       if ( map != 0 && *map[ scanCode ] != 0 )
-                               return map[ scanCode ];
+                       // Make sure scanCode is between layer first and last scancodes
+                       if ( map != 0
+                         && scanCode <= layer->last
+                         && scanCode >= layer->first
+                         && *map[ scanCode - layer->first ] != 0 )
+                       {
+                               return map[ scanCode - layer->first ];
+                       }
                }
        }
 
        // Do lookup on default layer
-       unsigned int **map = (unsigned int**)LayerIndex[0].triggerMap;
+       nat_ptr_t **map = (nat_ptr_t**)LayerIndex[0].triggerMap;
+
+       // Lookup default layer
+       const Layer *layer = &LayerIndex[0];
 
-       // Determine if layer has key defined
-       if ( map == 0 && *map[ scanCode ] == 0 )
+       // Make sure scanCode is between layer first and last scancodes
+       if ( map != 0
+         && scanCode <= layer->last
+         && scanCode >= layer->first
+         && *map[ scanCode - layer->first ] != 0 )
        {
-               erro_msg("Scan Code has no defined Trigger Macro: ");
-               printHex( scanCode );
-               return 0;
+               return map[ scanCode - layer->first ];
        }
 
-       // Return lookup result
-       return map[ scanCode ];
+       // Otherwise no defined Trigger Macro
+       erro_msg("Scan Code has no defined Trigger Macro: ");
+       printHex( scanCode );
+       return 0;
 }
 
 
@@ -320,10 +418,10 @@ inline void Macro_ledState( uint8_t ledCode, uint8_t state )
 inline void Macro_appendResultMacroToPendingList( TriggerMacro *triggerMacro )
 {
        // Lookup result macro index
-       unsigned int resultMacroIndex = triggerMacro->result;
+       var_uint_t resultMacroIndex = triggerMacro->result;
 
        // Iterate through result macro pending list, making sure this macro hasn't been added yet
-       for ( unsigned int macro = 0; macro < macroResultMacroPendingListSize; macro++ )
+       for ( var_uint_t macro = 0; macro < macroResultMacroPendingListSize; macro++ )
        {
                // If duplicate found, do nothing
                if ( macroResultMacroPendingList[ macro ] == resultMacroIndex )
@@ -334,7 +432,7 @@ inline void Macro_appendResultMacroToPendingList( TriggerMacro *triggerMacro )
        macroResultMacroPendingList[ macroResultMacroPendingListSize++ ] = resultMacroIndex;
 
        // Lookup scanCode of the last key in the last combo
-       unsigned int pos = 0;
+       var_uint_t pos = 0;
        for ( uint8_t comboLength = triggerMacro->guide[0]; comboLength > 0; )
        {
                pos += TriggerGuideSize * comboLength + 1;
@@ -364,8 +462,8 @@ inline uint8_t Macro_isLongResultMacro( ResultMacro *macro )
        // Check the second sequence combo length
        // If non-zero return non-zero (long sequence)
        // 0 otherwise (short sequence)
-       unsigned int position = 1;
-       for ( unsigned int result = 0; result < macro->guide[0]; result++ )
+       var_uint_t position = 1;
+       for ( var_uint_t result = 0; result < macro->guide[0]; result++ )
                position += ResultGuideSize( (ResultGuide*)&macro->guide[ position ] );
        return macro->guide[ position ];
 }
@@ -450,10 +548,13 @@ inline TriggerMacroVote Macro_evalLongTriggerMacroVote( TriggerGuide *key, Trigg
                        case 0x01:
                                return TriggerMacroVote_Fail;
 
-                       // Wrong key, held or released, do not pass (no effect)
+                       // Wrong key, held, do not pass (no effect)
                        case 0x02:
-                       case 0x03:
                                return TriggerMacroVote_DoNothing;
+
+                       // Wrong key released, fail out if pos == 0
+                       case 0x03:
+                               return TriggerMacroVote_DoNothing | TriggerMacroVote_DoNothingRelease;
                        }
                }
 
@@ -500,7 +601,7 @@ inline TriggerMacroVote Macro_evalLongTriggerMacroVote( TriggerGuide *key, Trigg
 
 
 // Evaluate/Update TriggerMacro
-inline TriggerMacroEval Macro_evalTriggerMacro( unsigned int triggerMacroIndex )
+inline TriggerMacroEval Macro_evalTriggerMacro( var_uint_t triggerMacroIndex )
 {
        // Lookup TriggerMacro
        TriggerMacro *macro = &TriggerMacroList[ triggerMacroIndex ];
@@ -513,7 +614,7 @@ inline TriggerMacroEval Macro_evalTriggerMacro( unsigned int triggerMacroIndex )
        }
 
        // Current Macro position
-       unsigned int pos = macro->pos;
+       var_uint_t pos = macro->pos;
 
        // Length of the combo being processed
        uint8_t comboLength = macro->guide[ pos ] * TriggerGuideSize;
@@ -572,6 +673,11 @@ inline TriggerMacroEval Macro_evalTriggerMacro( unsigned int triggerMacroIndex )
                overallVote |= vote;
        }
 
+       // If no pass vote was found after scanning the entire combo
+       // And this is the first position in the combo, just remove it (nothing important happened)
+       if ( longMacro && overallVote & TriggerMacroVote_DoNothingRelease && pos == 0 )
+               overallVote |= TriggerMacroVote_Fail;
+
        // Decide new state of macro after voting
        // Fail macro, remove from pending list
        if ( overallVote & TriggerMacroVote_Fail )
@@ -583,6 +689,16 @@ inline TriggerMacroEval Macro_evalTriggerMacro( unsigned int triggerMacroIndex )
        {
                // Just doing nothing :)
        }
+       // If ready for transition and in Press state, set to Waiting and increment combo position
+       // Position is incremented (and possibly remove the macro from the pending list) on the next iteration
+       else if ( overallVote & TriggerMacroVote_Release && macro->state == TriggerMacro_Press )
+       {
+               macro->state = TriggerMacro_Release;
+
+               // If this is the last combo in the sequence, remove from the pending list
+               if ( macro->guide[ macro->pos + macro->guide[ macro->pos ] * TriggerGuideSize + 1 ] == 0 )
+                       return TriggerMacroEval_DoResultAndRemove;
+       }
        // If passing and in Waiting state, set macro state to Press
        else if ( overallVote & TriggerMacroVote_Pass
             && ( macro->state == TriggerMacro_Waiting || macro->state == TriggerMacro_Press ) )
@@ -620,21 +736,11 @@ inline TriggerMacroEval Macro_evalTriggerMacro( unsigned int triggerMacroIndex )
                        }
                }
        }
-       // If ready for transition and in Press state, set to Waiting and increment combo position
-       // Position is incremented (and possibly remove the macro from the pending list) on the next iteration
-       else if ( overallVote & TriggerMacroVote_Release && macro->state == TriggerMacro_Press )
-       {
-               macro->state = TriggerMacro_Release;
-
-               // If this is the last combo in the sequence, remove from the pending list
-               if ( macro->guide[ macro->pos + macro->guide[ macro->pos ] * TriggerGuideSize + 1 ] == 0 )
-                       return TriggerMacroEval_Remove;
-       }
        // Otherwise, just remove the macro on key release
-       // XXX Might cause some issues
+       // One more result has to be called to indicate to the ResultMacro that the key transitioned to the release state
        else if ( overallVote & TriggerMacroVote_Release )
        {
-               return TriggerMacroEval_Remove;
+               return TriggerMacroEval_DoResultAndRemove;
        }
 
        // If this is a short macro, just remove it
@@ -647,22 +753,22 @@ inline TriggerMacroEval Macro_evalTriggerMacro( unsigned int triggerMacroIndex )
 
 
 // Evaluate/Update ResultMacro
-inline ResultMacroEval Macro_evalResultMacro( unsigned int resultMacroIndex )
+inline ResultMacroEval Macro_evalResultMacro( var_uint_t resultMacroIndex )
 {
        // Lookup ResultMacro
        ResultMacro *macro = &ResultMacroList[ resultMacroIndex ];
 
        // Current Macro position
-       unsigned int pos = macro->pos;
+       var_uint_t pos = macro->pos;
 
        // Length of combo being processed
        uint8_t comboLength = macro->guide[ pos ];
 
        // Function Counter, used to keep track of the combo items processed
-       unsigned int funcCount = 0;
+       var_uint_t funcCount = 0;
 
        // Combo Item Position within the guide
-       unsigned int comboItem = pos + 1;
+       var_uint_t comboItem = pos + 1;
 
        // Iterate through the Result Combo
        while ( funcCount < comboLength )
@@ -687,6 +793,7 @@ inline ResultMacroEval Macro_evalResultMacro( unsigned int resultMacroIndex )
        // If the ResultMacro is finished, remove
        if ( macro->guide[ comboItem ] == 0 )
        {
+               macro->pos = 0;
                return ResultMacroEval_Remove;
        }
 
@@ -708,21 +815,21 @@ inline void Macro_updateTriggerMacroPendingList()
                        continue;
 
                // Lookup Trigger List
-               unsigned int *triggerList = Macro_layerLookup( macroTriggerListBuffer[ key ].scanCode );
+               nat_ptr_t *triggerList = Macro_layerLookup( macroTriggerListBuffer[ key ].scanCode );
 
                // Number of Triggers in list
-               unsigned int triggerListSize = triggerList[0];
+               nat_ptr_t triggerListSize = triggerList[0];
 
                // Iterate over triggerList to see if any TriggerMacros need to be added
                // First item is the number of items in the TriggerList
-               for ( unsigned int macro = 1; macro < triggerListSize + 1; macro++ )
+               for ( var_uint_t macro = 1; macro < triggerListSize + 1; macro++ )
                {
                        // Lookup trigger macro index
-                       unsigned int triggerMacroIndex = triggerList[ macro ];
+                       var_uint_t triggerMacroIndex = triggerList[ macro ];
 
                        // Iterate over macroTriggerMacroPendingList to see if any macro in the scancode's
                        //  triggerList needs to be added
-                       unsigned int pending = 0;
+                       var_uint_t pending = 0;
                        for ( ; pending < macroTriggerMacroPendingListSize; pending++ )
                        {
                                // Stop scanning if the trigger macro index is found in the pending list
@@ -769,10 +876,10 @@ inline void Macro_process()
 
        // Tail pointer for macroTriggerMacroPendingList
        // Macros must be explicitly re-added
-       unsigned int macroTriggerMacroPendingListTail = 0;
+       var_uint_t macroTriggerMacroPendingListTail = 0;
 
        // Iterate through the pending TriggerMacros, processing each of them
-       for ( unsigned int macro = 0; macro < macroTriggerMacroPendingListSize; macro++ )
+       for ( var_uint_t macro = 0; macro < macroTriggerMacroPendingListSize; macro++ )
        {
                switch ( Macro_evalTriggerMacro( macroTriggerMacroPendingList[ macro ] ) )
                {
@@ -780,22 +887,18 @@ inline void Macro_process()
                case TriggerMacroEval_DoResult:
                        // Append ResultMacro to PendingList
                        Macro_appendResultMacroToPendingList( &TriggerMacroList[ macroTriggerMacroPendingList[ macro ] ] );
-                       print("D");
 
                default:
                        macroTriggerMacroPendingList[ macroTriggerMacroPendingListTail++ ] = macroTriggerMacroPendingList[ macro ];
-                       print("A");
                        break;
 
                // Trigger Result Macro and Remove (purposely falling through)
                case TriggerMacroEval_DoResultAndRemove:
                        // Append ResultMacro to PendingList
                        Macro_appendResultMacroToPendingList( &TriggerMacroList[ macroTriggerMacroPendingList[ macro ] ] );
-                       print("&");
 
                // Remove Macro from Pending List, nothing to do, removing by default
                case TriggerMacroEval_Remove:
-                       print("R");
                        break;
                }
        }
@@ -806,10 +909,10 @@ inline void Macro_process()
 
        // Tail pointer for macroResultMacroPendingList
        // Macros must be explicitly re-added
-       unsigned int macroResultMacroPendingListTail = 0;
+       var_uint_t macroResultMacroPendingListTail = 0;
 
        // Iterate through the pending ResultMacros, processing each of them
-       for ( unsigned int macro = 0; macro < macroResultMacroPendingListSize; macro++ )
+       for ( var_uint_t macro = 0; macro < macroResultMacroPendingListSize; macro++ )
        {
                switch ( Macro_evalResultMacro( macroResultMacroPendingList[ macro ] ) )
                {
@@ -861,14 +964,14 @@ inline void Macro_setup()
        macroTriggerListBufferSize = 0;
 
        // Initialize TriggerMacro states
-       for ( unsigned int macro = 0; macro < TriggerMacroNum; macro++ )
+       for ( var_uint_t macro = 0; macro < TriggerMacroNum; macro++ )
        {
                TriggerMacroList[ macro ].pos   = 0;
                TriggerMacroList[ macro ].state = TriggerMacro_Waiting;
        }
 
        // Initialize ResultMacro states
-       for ( unsigned int macro = 0; macro < ResultMacroNum; macro++ )
+       for ( var_uint_t macro = 0; macro < ResultMacroNum; macro++ )
        {
                ResultMacroList[ macro ].pos       = 0;
                ResultMacroList[ macro ].state     = 0;
@@ -886,7 +989,7 @@ void cliFunc_capList( char* args )
        printHex( CapabilitiesNum );
 
        // Iterate through all of the capabilities and display them
-       for ( unsigned int cap = 0; cap < CapabilitiesNum; cap++ )
+       for ( var_uint_t cap = 0; cap < CapabilitiesNum; cap++ )
        {
                print( NL "\t" );
                printHex( cap );
@@ -906,15 +1009,15 @@ void cliFunc_capSelect( char* args )
        char* arg2Ptr = args;
 
        // Total number of args to scan (must do a lookup if a keyboard capability is selected)
-       unsigned int totalArgs = 2; // Always at least two args
-       unsigned int cap = 0;
+       var_uint_t totalArgs = 2; // Always at least two args
+       var_uint_t cap = 0;
 
        // Arguments used for keyboard capability function
-       unsigned int argSetCount = 0;
+       var_uint_t argSetCount = 0;
        uint8_t *argSet = (uint8_t*)args;
 
        // Process all args
-       for ( unsigned int c = 0; argSetCount < totalArgs; c++ )
+       for ( var_uint_t c = 0; argSetCount < totalArgs; c++ )
        {
                curArgs = arg2Ptr;
                CLI_argumentIsolation( curArgs, &arg1Ptr, &arg2Ptr );
@@ -1052,7 +1155,7 @@ void cliFunc_layerList( char* args )
        info_msg("Layer List");
 
        // Iterate through all of the layers and display them
-       for ( unsigned int layer = 0; layer < LayerNum; layer++ )
+       for ( uint16_t layer = 0; layer < LayerNum; layer++ )
        {
                print( NL "\t" );
                printHex( layer );
@@ -1067,11 +1170,13 @@ void cliFunc_layerList( char* args )
 
                // Layer State
                print( NL "\t\t Layer State: " );
-               printHex( LayerIndex[ layer ].state );
+               printHex( LayerState[ layer ] );
 
-               // Max Index
-               print(" Max Index: ");
-               printHex( LayerIndex[ layer ].max );
+               // First -> Last Indices
+               print(" First -> Last Indices: ");
+               printHex( LayerIndex[ layer ].first );
+               print(" -> ");
+               printHex( LayerIndex[ layer ].last );
        }
 }
 
@@ -1116,7 +1221,7 @@ void cliFunc_layerState( char* args )
                        printHex( arg2 );
 
                        // Set the layer state
-                       LayerIndex[ arg1 ].state = arg2;
+                       LayerState[ arg1 ] = arg2;
                        break;
                }
        }
@@ -1150,7 +1255,7 @@ void cliFunc_macroList( char* args )
        info_msg("Pending Trigger Macros: ");
        printInt16( (uint16_t)macroTriggerMacroPendingListSize );
        print(" : ");
-       for ( unsigned int macro = 0; macro < macroTriggerMacroPendingListSize; macro++ )
+       for ( var_uint_t macro = 0; macro < macroTriggerMacroPendingListSize; macro++ )
        {
                printHex( macroTriggerMacroPendingList[ macro ] );
                print(" ");
@@ -1161,7 +1266,7 @@ void cliFunc_macroList( char* args )
        info_msg("Pending Result Macros: ");
        printInt16( (uint16_t)macroResultMacroPendingListSize );
        print(" : ");
-       for ( unsigned int macro = 0; macro < macroResultMacroPendingListSize; macro++ )
+       for ( var_uint_t macro = 0; macro < macroResultMacroPendingListSize; macro++ )
        {
                printHex( macroResultMacroPendingList[ macro ] );
                print(" ");
@@ -1180,7 +1285,7 @@ void cliFunc_macroList( char* args )
        // Show Trigger to Result Macro Links
        print( NL );
        info_msg("Trigger : Result Macro Pairs");
-       for ( unsigned int macro = 0; macro < TriggerMacroNum; macro++ )
+       for ( var_uint_t macro = 0; macro < TriggerMacroNum; macro++ )
        {
                print( NL );
                print("\tT");
@@ -1200,7 +1305,7 @@ void cliFunc_macroProc( char* args )
        printInt8( macroPauseMode );
 }
 
-void macroDebugShowTrigger( unsigned int index )
+void macroDebugShowTrigger( var_uint_t index )
 {
        // Only proceed if the macro exists
        if ( index >= TriggerMacroNum )
@@ -1215,14 +1320,14 @@ void macroDebugShowTrigger( unsigned int index )
        print( NL );
 
        // Read the comboLength for combo in the sequence (sequence of combos)
-       unsigned int pos = 0;
+       var_uint_t pos = 0;
        uint8_t comboLength = macro->guide[ pos ];
 
        // Iterate through and interpret the guide
        while ( comboLength != 0 )
        {
                // Initial position of the combo
-               unsigned int comboPos = ++pos;
+               var_uint_t comboPos = ++pos;
 
                // Iterate through the combo
                while ( pos < comboLength * TriggerGuideSize + comboPos )
@@ -1271,7 +1376,7 @@ void macroDebugShowTrigger( unsigned int index )
        }
 }
 
-void macroDebugShowResult( unsigned int index )
+void macroDebugShowResult( var_uint_t index )
 {
        // Only proceed if the macro exists
        if ( index >= ResultMacroNum )
@@ -1286,14 +1391,14 @@ void macroDebugShowResult( unsigned int index )
        print( NL );
 
        // Read the comboLength for combo in the sequence (sequence of combos)
-       unsigned int pos = 0;
+       var_uint_t pos = 0;
        uint8_t comboLength = macro->guide[ pos++ ];
 
        // Iterate through and interpret the guide
        while ( comboLength != 0 )
        {
                // Function Counter, used to keep track of the combos processed
-               unsigned int funcCount = 0;
+               var_uint_t funcCount = 0;
 
                // Iterate through the combo
                while ( funcCount < comboLength )
@@ -1306,7 +1411,7 @@ void macroDebugShowResult( unsigned int index )
                        print("|");
 
                        // Display Function Ptr Address
-                       printHex( (unsigned int)CapabilitiesList[ guide->index ].func );
+                       printHex( (nat_ptr_t)CapabilitiesList[ guide->index ].func );
                        print("|");
 
                        // Display/Lookup Capability Name (utilize debug mode of capability)
@@ -1315,7 +1420,7 @@ void macroDebugShowResult( unsigned int index )
 
                        // Display Argument(s)
                        print("(");
-                       for ( unsigned int arg = 0; arg < CapabilitiesList[ guide->index ].argCount; arg++ )
+                       for ( var_uint_t arg = 0; arg < CapabilitiesList[ guide->index ].argCount; arg++ )
                        {
                                // Arguments are only 8 bit values
                                printHex( (&guide->args)[ arg ] );
@@ -1397,7 +1502,7 @@ void cliFunc_macroStep( char* args )
        CLI_argumentIsolation( args, &arg1Ptr, &arg2Ptr );
 
        // Default to 1, if no argument given
-       unsigned int count = (unsigned int)numToInt( arg1Ptr );
+       var_uint_t count = (var_uint_t)numToInt( arg1Ptr );
 
        if ( count == 0 )
                count = 1;