Fixes and new apps (#30)

- M5 Unit Modules library + M5 Unit Test app - Minor fixes for TodoList, TwoEleven, Snake, Brainfuck and Breakout - Fixed SerialConsole to use the uart controller as it was broken in one of the many updates - Fixed keyboard input in TwoEleven, Breakout, Magic8Ball and Snake - Bluetooth Media Keys app, supports pressing physical keys to trigger the corresponding buttonmatrix button - Epub Reader app
2026-06-07 23:56:32 +10:00
parent 1f959c8bbf
commit dbf850c434
127 changed files with 11156 additions and 174 deletions
@@ -0,0 +1,56 @@
+/*
+ * UnitCommon.h - shared I2C helpers for M5Stack STM32-based units
+ *
+ * All M5Stack Grove units with STM32 firmware require a STOP condition between
+ * the register-pointer write and the data read (the STM32 HAL receive callback
+ * populates tx_buffer after STOP, before the next START). Using a repeated-START
+ * (the default i2c_controller_read_register) causes bus errors on every read.
+ *
+ * Pattern: write(reg) → STOP → vTaskDelay(2ms) → read(data)
+ */
+#pragma once
+
+#include <tactility/device.h>
+#include <tactility/drivers/i2c_controller.h>
+#include <freertos/FreeRTOS.h>
+#include <freertos/task.h>
+#include <cstring>
+
+static constexpr uint32_t UNIT_I2C_TIMEOUT_MS       = 20;
+static constexpr uint32_t UNIT_I2C_PROBE_TIMEOUT_MS = 10;  // shorter timeout for presence checks
+static constexpr uint32_t UNIT_STM32_READ_DELAY_MS  =  2;  // STM32 needs STOP + delay before read
+static constexpr uint16_t UNIT_MAX_WRITE_PAYLOAD     = 32;  // max bytes in a single I2C write
+
+// Probe whether a device exists at the given address on the given I2C bus.
+inline bool unitProbe(Device* dev, uint8_t addr) {
+    return i2c_controller_has_device_at_address(
+        dev, addr, pdMS_TO_TICKS(UNIT_I2C_PROBE_TIMEOUT_MS)) == ERROR_NONE;
+}
+
+// Write [reg, buf...] as a single transaction (STOP at end).
+inline bool unitWriteReg(Device* dev, uint8_t addr, uint8_t reg,
+                         const uint8_t* buf, uint16_t len) {
+    if (len > UNIT_MAX_WRITE_PAYLOAD) return false;
+    if (len > 0 && buf == nullptr) return false;
+    uint8_t pkt[33];
+    pkt[0] = reg;
+    if (len) memcpy(pkt + 1, buf, len);
+    return i2c_controller_write(dev, addr, pkt, (uint16_t)(len + 1),
+                                pdMS_TO_TICKS(UNIT_I2C_TIMEOUT_MS)) == ERROR_NONE;
+}
+
+// Write register pointer only (no data payload) - convenience wrapper.
+inline bool unitWriteRegPtr(Device* dev, uint8_t addr, uint8_t reg) {
+    return i2c_controller_write(dev, addr, &reg, 1,
+                                pdMS_TO_TICKS(UNIT_I2C_TIMEOUT_MS)) == ERROR_NONE;
+}
+
+// Read from a register: write pointer → STOP → 2ms delay → read.
+inline bool unitReadReg(Device* dev, uint8_t addr, uint8_t reg,
+                        uint8_t* buf, uint16_t len) {
+    if (len > 0 && buf == nullptr) return false;
+    if (!unitWriteRegPtr(dev, addr, reg)) return false;
+    vTaskDelay(pdMS_TO_TICKS(UNIT_STM32_READ_DELAY_MS));
+    return i2c_controller_read(dev, addr, buf, len,
+                               pdMS_TO_TICKS(UNIT_I2C_TIMEOUT_MS)) == ERROR_NONE;
+}