Make it possible to not use our short name override, which we need wh…

…en not using keyboardiohid
keyboardio · Mar 25, 2024 · 1a1ab9c · 1a1ab9c
1 parent 1f8d465
commit 1a1ab9c
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/src/kaleidoscope/driver/hid/Keyboardio.h b/src/kaleidoscope/driver/hid/Keyboardio.h
@@ -22,6 +22,8 @@
 #include "kaleidoscope/driver/hid/keyboardio/Keyboard.h"       // for Keyboard, KeyboardProps
 #include "kaleidoscope/driver/hid/keyboardio/Mouse.h"          // for Mouse, MouseProps
 
+#define KALEIDOSCOPE_HID_DRIVER_KEYBOARDIOHID
+
 namespace kaleidoscope {
 namespace driver {
 namespace hid {

diff --git a/src/kaleidoscope/layers.h b/src/kaleidoscope/layers.h
@@ -56,6 +56,7 @@
                                                                         __NL__ \
   _INIT_SKETCH_EXPLORATION                                              __NL__ \
   _INIT_HID_GETSHORTNAME
+  // TODO(jesse): re-enable GETSHORTNAME once we figure out how to do that for non-keyboardiohid devices
 
 // Macro for defining the keymap. This should be used in the sketch
 // file (*.ino) to define the keymap[] array that holds the user's

diff --git a/src/kaleidoscope_internal/shortname.h b/src/kaleidoscope_internal/shortname.h
@@ -30,9 +30,14 @@
 
 #ifndef _INIT_HID_GETSHORTNAME
 
-#define _INIT_HID_GETSHORTNAME                        __NL__ \
+  #ifdef KALEIDOSCOPE_HID_DRIVER_KEYBOARDIOHID
+
+    #define _INIT_HID_GETSHORTNAME                        __NL__ \
   uint8_t HID_::getShortName(char *name) {            __NL__ \
     return Kaleidoscope.device().getShortName(name);  __NL__ \
   }
 
+  #else 
+    #define _INIT_HID_GETSHORTNAME            
+  #endif
 #endif