more structure to the package

tmk_core/protocol/lufa/lufa.c

@@ -1147,10 +1147,9 @@ void send_nibble(uint8_t number) {
     }
 }
 
-uint8_t midi_buffer[16] = {0};
+uint8_t midi_buffer[MIDI_SYSEX_BUFFER] = {0};
 
-void sysex_callback(MidiDevice * device,
+void sysex_callback(MidiDevice * device, uint16_t start, uint8_t length, uint8_t * data) {
-    uint16_t start, uint8_t length, uint8_t * data) {
   // for (int i = 0; i < length; i++)
   //   midi_send_cc(device, 15, 0x7F & data[i], 0x7F & (start + i));
     // if (start == 0x27) {
@@ -1169,7 +1168,7 @@ void sysex_callback(MidiDevice * device,
 
 }
 
-uint32_t decode_4byte_chunk(uint8_t * data) {
+uint32_t decode_uint32_chunk(uint8_t * data) {
     uint32_t part1 = *data++;
     uint32_t part2 = *data++;
     uint32_t part3 = *data++;
@@ -1178,7 +1177,13 @@ uint32_t decode_4byte_chunk(uint8_t * data) {
     return ((part1 & 0x1FUL) << 28) | (part2 << 21) | (part3 << 14) | (part4 << 7) | part5;
 }
 
-void encode_4byte_chunk(uint32_t data, uint8_t * pointer) {
+uint32_t decode_uint8_chunk(uint8_t * data) {
+    uint32_t part4 = *data++;
+    uint32_t part5 = *data++;
+    return (part4 << 7) | part5;
+}
+
+void encode_uint32_chunk(uint32_t data, uint8_t * pointer) {
     *pointer++ = (data >> 28) & 0x7F;
     *pointer++ = (data >> 21) & 0x7F;
     *pointer++ = (data >> 14) & 0x7F;
@@ -1186,6 +1191,11 @@ void encode_4byte_chunk(uint32_t data, uint8_t * pointer) {
     *pointer++ = (data) & 0x7F;
 }
 
+void encode_uint8_chunk(uint8_t data, uint8_t * pointer) {
+    *pointer++ = (data >> 7) & 0x7F;
+    *pointer++ = (data) & 0x7F;
+}
+
 void sysex_buffer_callback(MidiDevice * device, uint8_t length, uint8_t * data) {
     uint8_t * pointer_copy = data;
 
@@ -1197,28 +1207,77 @@ void sysex_buffer_callback(MidiDevice * device, uint8_t length, uint8_t * data)
     data++;
 
     switch (*data++) {
-        case 0x13: ; // Get info from keyboard
+        case 0x12: ; // Set info on keyboard
             switch (*data++) {
-                case 0x00: ; // Get layer state
+                case 0x02: ; // set default layer
-                    // send_dword(layer_state);
+                    uint8_t default_layer = decode_uint8_chunk(data);
-                    uint8_t chunk[5];
+                    eeconfig_update_default_layer(default_layer);
-                    encode_4byte_chunk(layer_state | default_layer_state, &chunk);
+                    default_layer_set((uint32_t)default_layer);
-
+                    break;
-                    uint8_t array[] = {0xF0, 0x00, 0x00, 0x00, 0x00, chunk[0], chunk[1], chunk[2], chunk[3], chunk[4], 0xF7};
+                case 0x08: ; // set keymap options
-                    midi_send_array(&midi_device, 11, &array);
+                    uint8_t keymap_options = decode_uint8_chunk(data);
-                    // midi_send_data(device, 3, 0x00, layer_state >> 24 & 0x7f, layer_state >> 16 & 0x7f);
+                    eeconfig_update_keymap(keymap_options);
-                    // midi_send_data(device, 6, layer_state >> 8 & 0x7f, layer_state & 0x7f, 0xF7);
                     break;
             }
+            break;
+        case 0x13: ; // Get info from keyboard
+            switch (*data++) {
+                case 0x00: ; // Handshake
+                    send_bytes_sysex(0x00, NULL, 0);
+                    break;
+                case 0x01: ; // Get debug state
+                    uint8_t debug[2];
+                    encode_uint8_chunk(eeprom_read_byte(EECONFIG_DEBUG), &debug);
+                    send_bytes_sysex(0x01, &debug, 2);
+                    break;
+                case 0x02: ; // Get default layer
+                    uint8_t default_layer[2];
+                    encode_uint8_chunk(eeprom_read_byte(EECONFIG_DEFAULT_LAYER), &default_layer);
+                    send_bytes_sysex(0x02, &default_layer, 2);
+                    break;
+                #ifdef AUDIO_ENABLE
+                case 0x03: ; // Get backlight state
+                    uint8_t audio[2];
+                    encode_uint8_chunk(eeprom_read_byte(EECONFIG_AUDIO), &audio);
+                    send_bytes_sysex(0x03, &audio, 2);
+                #endif
+                case 0x04: ; // Get layer state
+                    uint8_t layers[5];
+                    encode_uint32_chunk(layer_state, &layers);
+                    send_bytes_sysex(0x04, &layers, 5);
+                    break;
+                #ifdef BACKLIGHT_ENABLE
+                case 0x06: ; // Get backlight state
+                    uint8_t backlight[2];
+                    encode_uint8_chunk(eeprom_read_byte(EECONFIG_BACKLIGHT), &backlight);
+                    send_bytes_sysex(0x06, &backlight, 2);
+                #endif
+                #ifdef RGBLIGHT_ENABLE
+                case 0x07: ; // Get rgblight state
+                    uint8_t rgblight[2];
+                    encode_uint32_chunk(eeprom_read_dword(EECONFIG_RGBLIGHT), &rgblight);
+                    send_bytes_sysex(0x07, &rgblight, 5);
+                #endif
+                case 0x08: ; // Keymap options
+                    uint8_t keymap_options[2];
+                    encode_uint8_chunk(eeconfig_read_keymap(), &keymap_options);
+                    send_bytes_sysex(0x08, &keymap_options, 2);
+                    break;
+            }
+            break;
         #ifdef RGBLIGHT_ENABLE
         case 0x27: ; // RGB LED functions
             switch (*data++) {
                 case 0x00: ; // Update HSV
-                    uint32_t chunk = decode_4byte_chunk(data);
+                    uint32_t hsv = decode_uint32_chunk(data);
-                    rgblight_sethsv(((chunk >> 16) & 0xFFFF) % 360, (chunk >> 8) & 0xFF, chunk & 0xFF);
+                    rgblight_sethsv(((hsv >> 16) & 0xFFFF) % 360, (hsv >> 8) & 0xFF, hsv & 0xFF);
                     break;
                 case 0x01: ; // Update RGB
                     break;
+                case 0x02: ; // Update mode
+                    uint8_t rgb_mode = decode_uint8_chunk(data);
+                    rgblight_mode(rgb_mode);
+                    break;
             }
             break;
         #endif
@@ -1234,10 +1293,20 @@ void sysex_buffer_callback(MidiDevice * device, uint8_t length, uint8_t * data)
 
 void send_unicode_midi(uint32_t unicode) {
     uint8_t chunk[5];
-    encode_4byte_chunk(unicode, &chunk);
+    encode_uint32_chunk(unicode, &chunk);
+    send_bytes_sysex(0x05, &chunk, 5);
+}
 
-    uint8_t array[] = {0xF0, 0x00, 0x00, 0x00, 0x05, chunk[0], chunk[1], chunk[2], chunk[3], chunk[4], 0xF7};
+void send_bytes_sysex(uint8_t type, uint8_t * bytes, uint8_t length) {
-    midi_send_array(&midi_device, 11, &array);
+    uint8_t * array = malloc(sizeof(uint8_t) * (length + 6));
+    array[0] = 0xF0;
+    array[1] = 0x00;
+    array[2] = 0x00;
+    array[3] = 0x00;
+    array[4] = type;
+    array[length + 5] = 0xF7;
+    memcpy(array + 5, bytes, length);
+    midi_send_array(&midi_device, length + 6, array);
 }
 
 #endif

tmk_core/protocol/lufa/lufa.h

@@ -68,9 +68,18 @@ typedef struct {
 } __attribute__ ((packed)) report_extra_t;
 
 #ifdef MIDI_ENABLE
-void MIDI_Task(void);
+  #define MIDI_SYSEX_BUFFER 16 
-MidiDevice midi_device;
+  void MIDI_Task(void);
-void send_unicode_midi(uint32_t unicode);
+  MidiDevice midi_device;
+
+  void sysex_callback(MidiDevice * device, uint16_t start, uint8_t length, uint8_t * data);
+  uint32_t decode_uint32_chunk(uint8_t * data);
+  uint32_t decode_uint8_chunk(uint8_t * data);
+  void encode_uint32_chunk(uint32_t data, uint8_t * pointer);
+  void encode_uint8_chunk(uint8_t data, uint8_t * pointer);
+  void sysex_buffer_callback(MidiDevice * device, uint8_t length, uint8_t * data);
+  void send_unicode_midi(uint32_t unicode);
+  void send_bytes_sysex(uint8_t type, uint8_t * bytes, uint8_t length);
 #endif
 
 // #if LUFA_VERSION_INTEGER < 0x120730