Use bit based communication with drawers

drawers/src/main.cpp

@@ -3,9 +3,12 @@
 
 const uint8_t I2C_ADDR = 0x10;
 
-const uint8_t CMD_POWER_OFF    = 0x01;
+const uint8_t BITMASK_CMD = 0b00011000;
-const uint8_t CMD_LIGHT_INDEX  = 0x02;
+const uint8_t BITMASK_PAYLOAD = 0b00000111;
-const uint8_t CMD_DRAWER_INDEX = 0x03;
+
+const uint8_t CMD_POWER_OFF    = 0b00001000;
+const uint8_t CMD_LIGHT_INDEX  = 0b00010000;
+const uint8_t CMD_DRAWER_INDEX = 0b00011000;
 
 struct Schublade {
   int schlossPin;
@@ -47,22 +50,20 @@ void openLock(uint8_t indexOn) {
 
 void onI2CReceive(int len) {
   while (Wire.available()) {
-    uint8_t cmd = Wire.read();
+    uint8_t data = Wire.read();
 
-    switch (cmd) {
+    switch (data & BITMASK_CMD) {
 
       case CMD_POWER_OFF:
         turnAllLedsOf();
         break;
 
       case CMD_LIGHT_INDEX:
-        uint8_t lightIndex = Wire.read();
+        turnOnSingleLed(data & BITMASK_PAYLOAD);
-        turnOnSingleLed(lightIndex);
         break;
 
       case CMD_DRAWER_INDEX:
-        uint8_t drawerIndex = Wire.read();
+        openLock(data & BITMASK_PAYLOAD);
-        openLock(drawerIndex);
         break;
     }
   }

terminal/master/src/main.cpp

@@ -5,15 +5,18 @@ void resetPuzzles();
 
 /* ========================================================= */
 const uint8_t ARCHIV_ADDR = 0x10;
-const uint8_t CMD_POWER_OFF = 0x01;
+
-const uint8_t CMD_LIGHT_INDEX = 0x02;
+const uint8_t BITMASK_CMD = 0b00011000;
-const uint8_t CMD_DRAWER_INDEX = 0x03;
+const uint8_t BITMASK_PAYLOAD = 0b00000111;
+
+const uint8_t CMD_POWER_OFF    = 0b00001000;
+const uint8_t CMD_LIGHT_INDEX  = 0b00010000;
+const uint8_t CMD_DRAWER_INDEX = 0b00011000;
 
 void setDrawerLight(uint8_t index)
 {
   Wire.beginTransmission(ARCHIV_ADDR);
-  Wire.write(CMD_LIGHT_INDEX);
+  Wire.write(CMD_LIGHT_INDEX | (index & BITMASK_PAYLOAD));
-  Wire.write(index);
   Wire.endTransmission();
 }
 
@@ -27,8 +30,7 @@ void turnLightsOff()
 void openDrawer(uint8_t index)
 {
   Wire.beginTransmission(ARCHIV_ADDR);
-  Wire.write(CMD_DRAWER_INDEX);
+  Wire.write(CMD_DRAWER_INDEX | (index & BITMASK_PAYLOAD));
-  Wire.write(index);
   Wire.endTransmission();
 }
 /* ========================================================= */