Boards/LilygoTLoraPager/Source/Init.cpp

@@ -1,4 +1,3 @@
-#include <Bq27220.h>
 #include <Tactility/TactilityCore.h>
 #include <Tactility/kernel/SystemEvents.h>
 #include <Tactility/service/gps/GpsService.h>
@@ -22,18 +21,6 @@ bool tpagerInit() {
     }
 
     tt::kernel::subscribeSystemEvent(tt::kernel::SystemEvent::BootSplash, [](auto) {
-        tt::hal::findDevices([](auto device) {
-            if (device->getName() == "BQ27220") {
-                auto bq27220 = std::reinterpret_pointer_cast<Bq27220>(device);
-                if (bq27220 != nullptr) {
-                    bq27220->configureCapacity(1500, 1500);
-                    return false;
-                }
-            }
-
-            return true;
-        });
-
         auto gps_service = tt::service::gps::findGpsService();
         if (gps_service != nullptr) {
             std::vector<tt::hal::gps::GpsConfiguration> gps_configurations;

Boards/LilygoTLoraPager/Source/LilygoTloraPager.cpp

@@ -18,6 +18,7 @@ using namespace tt::hal;
 
 DeviceVector createDevices() {
     auto bq27220 = std::make_shared<Bq27220>(I2C_NUM_0);
+    bq27220->configureCapacity(1500, 1500);
     auto power = std::make_shared<TpagerPower>(bq27220);
 
     auto tca8418 = std::make_shared<Tca8418>(I2C_NUM_0);
@@ -41,10 +42,10 @@ extern const Configuration lilygo_tlora_pager = {
     .createDevices = createDevices,
     .i2c = {
         i2c::Configuration {
-            .name = "Internal",
+            .name = "Shared",
             .port = I2C_NUM_0,
             .initMode = i2c::InitMode::ByTactility,
-            .isMutable = false,
+            .isMutable = true,
             .config = (i2c_config_t) {
                 .mode = I2C_MODE_MASTER,
                 .sda_io_num = GPIO_NUM_3,

Boards/LilygoTLoraPager/Source/hal/TpagerKeyboard.cpp

@@ -43,6 +43,7 @@ void TpagerKeyboard::readCallback(lv_indev_t* indev, lv_indev_data_t* data) {
     char keypress = 0;
 
     if (xQueueReceive(keyboard->queue, &keypress, pdMS_TO_TICKS(50)) == pdPASS) {
+        TT_LOG_I(TAG, "Keypress: %c", keypress);
         data->key = keypress;
         data->state = LV_INDEV_STATE_PRESSED;
     } else {

Documentation/ideas.md

@@ -16,7 +16,7 @@
 
 ## Lower Priority
 
-- Support hot-plugging SD card (note: this is not possible if they require the CS pin hack)
+- Support hot-plugging SD card
 - Create more unit tests for `tactility`
 - Explore LVGL9's FreeRTOS functionality
 - CrashHandler: use "corrupted" flag

Drivers/BQ27220/Source/Bq27220.cpp

@@ -127,7 +127,7 @@ bool Bq27220::getCurrent(int16_t &value) {
     return false;
 }
 
-bool Bq27220::getBatteryStatus(BatteryStatus &batt_sta) {
+bool Bq27220::getBatteryStatus(Bq27220::BatteryStatus &batt_sta) {
     if (readRegister16(registers::CMD_BATTERY_STATUS, batt_sta.full)) {
         swapEndianess(batt_sta.full);
         return true;

Drivers/DRV2605/Source/Drv2605.cpp

@@ -18,8 +18,15 @@ bool Drv2605::init() {
 
     writeRegister(Register::RealtimePlaybackInput, 0x00); // Disable
 
+    writeRegister(Register::WaveSequence1, 1); // Strong click
+    writeRegister(Register::WaveSequence2, 0); // End sequence
 
-    setWaveFormForClick();
+    writeRegister(Register::OverdriveTimeOffset, 0); // No overdrive
+
+    writeRegister(Register::SustainTimeOffsetPostivie, 0);
+    writeRegister(Register::SustainTimeOffsetNegative, 0);
+    writeRegister(Register::BrakeTimeOffset, 0);
+    writeRegister(Register::AudioInputLevelMax, 0x64);
 
     // ERM open loop
 
@@ -36,34 +43,6 @@ bool Drv2605::init() {
     return true;
 }
 
-void Drv2605::setWaveFormForBuzz() {
-    writeRegister(Register::WaveSequence1, 1); // Strong click
-    writeRegister(Register::WaveSequence2, 1); // Strong click
-    writeRegister(Register::WaveSequence3, 1); // Strong click
-    writeRegister(Register::WaveSequence4, 0); // End sequence
-
-    writeRegister(Register::OverdriveTimeOffset, 0); // No overdrive
-
-    writeRegister(Register::SustainTimeOffsetPostivie, 0);
-    writeRegister(Register::SustainTimeOffsetNegative, 0);
-    writeRegister(Register::BrakeTimeOffset, 0);
-
-    writeRegister(Register::AudioInputLevelMax, 0x64);
-}
-
-void Drv2605::setWaveFormForClick() {
-    writeRegister(Register::WaveSequence1, 1); // Strong click
-    writeRegister(Register::WaveSequence2, 0); // End sequence
-
-    writeRegister(Register::OverdriveTimeOffset, 0); // No overdrive
-
-    writeRegister(Register::SustainTimeOffsetPostivie, 0);
-    writeRegister(Register::SustainTimeOffsetNegative, 0);
-    writeRegister(Register::BrakeTimeOffset, 0);
-
-    writeRegister(Register::AudioInputLevelMax, 0x64);
-}
-
 void Drv2605::setWaveForm(uint8_t slot, uint8_t waveform) {
     writeRegister8(static_cast<uint8_t>(Register::WaveSequence1) + slot, waveform);
 }

Drivers/DRV2605/Source/Drv2605.h

@@ -7,8 +7,6 @@ class Drv2605 : public tt::hal::i2c::I2cDevice {
     static constexpr auto* TAG = "DRV2605";
     static constexpr auto ADDRESS = 0x5A;
 
-    bool autoPlayStartupBuzz;
-
     // Chip IDs
     enum class ChipId {
         DRV2604 = 0x04,     // Has RAM. Doesn't havew licensed ROM library.
@@ -63,15 +61,10 @@ class Drv2605 : public tt::hal::i2c::I2cDevice {
 
 public:
 
-    explicit Drv2605(i2c_port_t port, bool autoPlayStartupBuzz = true) : I2cDevice(port, ADDRESS), autoPlayStartupBuzz(autoPlayStartupBuzz) {
+    explicit Drv2605(i2c_port_t port) : I2cDevice(port, ADDRESS) {
         if (!init()) {
             TT_LOG_E(TAG, "Failed to initialize DRV2605");
         }
-
-        if (autoPlayStartupBuzz) {
-            setWaveFormForBuzz();
-            startPlayback();
-        }
     }
 
     std::string getName() const final { return "DRV2605"; }
@@ -79,14 +72,6 @@ public:
 
     bool init();
 
-    void setWaveFormForBuzz();
-    void setWaveFormForClick();
-
-    /**
-     *
-     * @param slot a value from 0 to 7
-     * @param waveform
-     */
     void setWaveForm(uint8_t slot, uint8_t waveform);
     void selectLibrary(uint8_t library);
     void startPlayback();

Drivers/ST7796/Source/St7796Display.cpp

@@ -133,14 +133,7 @@ lvgl_port_display_cfg_t St7796Display::getLvglPortDisplayConfig(esp_lcd_panel_io
             .mirror_y = configuration->mirrorY,
         },
         .color_format = LV_COLOR_FORMAT_NATIVE,
-        .flags = {
+        .flags = {.buff_dma = true, .buff_spiram = false, .sw_rotate = false, .swap_bytes = true, .full_refresh = false, .direct_mode = false}
-            .buff_dma = true,
-            .buff_spiram = false,
-            .sw_rotate = false,
-            .swap_bytes = true,
-            .full_refresh = false,
-            .direct_mode = false
-        }
     };
 }
 

Tactility/Private/Tactility/hal/sdcard/SdCardMounting.h

@@ -1,7 +0,0 @@
-#pragma once
-
-namespace tt::hal::sdcard {
-
-void mountAll();
-
-}

Tactility/Source/Tactility.cpp

@@ -114,7 +114,7 @@ static void registerSystemApps() {
     addApp(app::development::manifest);
 #endif
 
-    if (hal::findDevices(hal::Device::Type::Power).size() > 0) {
+    if (getConfiguration()->hardware->power != nullptr) {
         addApp(app::power::manifest);
     }
 }

Tactility/Source/TactilityHeadless.cpp

@@ -6,7 +6,6 @@
 #include "Tactility/service/ServiceRegistration.h"
 
 #include <Tactility/Dispatcher.h>
-#include <Tactility/hal/sdcard/SdCardMounting.h>
 #include <Tactility/settings/TimePrivate.h>
 
 #ifdef ESP_PLATFORM
@@ -15,7 +14,7 @@
 
 namespace tt {
 
-constexpr auto* TAG = "Tactility";
+#define TAG "tactility"
 
 namespace service::gps { extern const ServiceManifest manifest; }
 namespace service::wifi { extern const ServiceManifest manifest; }
@@ -48,11 +47,11 @@ void initHeadless(const hal::Configuration& config) {
     hardwareConfig = &config;
     settings::initTimeZone();
     hal::init(config);
-    hal::sdcard::mountAll();
     network::ntp::init();
     registerAndStartSystemServices();
 }
 
+
 Dispatcher& getMainDispatcher() {
     return mainDispatcher;
 }

Tactility/Source/app/boot/Boot.cpp

@@ -22,9 +22,9 @@
 #define CONFIG_TT_SPLASH_DURATION 0
 #endif
 
-namespace tt::app::boot {
+#define TAG "boot"
 
-constexpr auto* TAG = "Boot";
+namespace tt::app::boot {
 
 static std::shared_ptr<hal::display::DisplayDevice> getHalDisplay() {
     return hal::findFirstDevice<hal::display::DisplayDevice>(hal::Device::Type::Display);
@@ -60,7 +60,6 @@ class BootApp : public App {
         }
     }
 
-
     static bool setupUsbBootMode() {
         if (!hal::usb::isUsbBootMode()) {
             return false;
@@ -87,7 +86,7 @@ class BootApp : public App {
 
         kernel::publishSystemEvent(kernel::SystemEvent::BootSplash);
 
-        setupDisplay(); // Set backlight
+        setupDisplay();
 
         if (!setupUsbBootMode()) {
             initFromBootApp();

Tactility/Source/hal/Hal.cpp

@@ -8,13 +8,13 @@
 
 #include <Tactility/kernel/SystemEvents.h>
 
+#define TAG "hal"
+
+#define TT_SDCARD_MOUNT_POINT "/sdcard"
+
 namespace tt::hal {
 
-constexpr auto* TAG = "hal";
+void initDevices(const Configuration& configuration) {
-
-void registerDevices(const Configuration& configuration) {
-    TT_LOG_I(TAG, "Registering devices");
-
     if (configuration.sdcard != nullptr) {
         registerDevice(configuration.sdcard);
     }
@@ -35,6 +35,29 @@ void registerDevices(const Configuration& configuration) {
     for (auto& device : devices) {
         registerDevice(device);
     }
+
+    // TODO: Move
+    auto sdcards = hal::findDevices<sdcard::SdCardDevice>(Device::Type::SdCard);
+    if (!sdcards.empty()) {
+        if (sdcards.size() == 1) {
+            // Fixed mount path name
+            auto sdcard = sdcards[0];
+            TT_LOG_I(TAG, "Mounting sdcard at %s", TT_SDCARD_MOUNT_POINT);
+            if (!sdcard->mount(TT_SDCARD_MOUNT_POINT)) {
+                TT_LOG_W(TAG, "SD card mount failed (init can continue)");
+            }
+        } else {
+            // Numbered mount path name
+            for (int i = 0; i < sdcards.size(); i++) {
+                auto sdcard = sdcards[i];
+                std::string mount_path = TT_SDCARD_MOUNT_POINT + std::to_string(i);
+                TT_LOG_I(TAG, "Mounting sdcard at %d", mount_path.c_str());
+                if (!sdcard->mount(mount_path)) {
+                    TT_LOG_W(TAG, "SD card mount failed (init can continue)");
+                }
+            }
+        }
+    }
 }
 
 void init(const Configuration& configuration) {
@@ -57,7 +80,7 @@ void init(const Configuration& configuration) {
         tt_check(configuration.initBoot(), "Init power failed");
     }
 
-    registerDevices(configuration);
+    initDevices(configuration);
 
     kernel::publishSystemEvent(kernel::SystemEvent::BootInitHalEnd);
 }

Tactility/Source/hal/sdcard/SdCardMounting.cpp

@@ -1,40 +0,0 @@
-#include <Tactility/hal/sdcard/SdCardMounting.h>
-#include <Tactility/hal/sdcard/SdCardDevice.h>
-
-namespace tt::hal::sdcard {
-
-constexpr auto* TAG = "SdCardMounting";
-constexpr auto* TT_SDCARD_MOUNT_POINT = "/sdcard";
-
-static void mount(const std::shared_ptr<SdCardDevice>& sdcard, const std::string& path) {
-    sdcard->getLock()->withLock([&sdcard, &path] {
-       TT_LOG_I(TAG, "Mounting sdcard at %s", path.c_str());
-       if (!sdcard->mount(path)) {
-           TT_LOG_W(TAG, "SD card mount failed for %s (init can continue)", path.c_str());
-       }
-   });
-}
-
-void mountAll() {
-    auto sdcards = hal::findDevices<SdCardDevice>(Device::Type::SdCard);
-    if (!sdcards.empty()) {
-        if (sdcards.size() == 1) {
-            // Fixed mount path name
-            auto sdcard = sdcards[0];
-            if (!sdcard->isMounted()) {
-                mount(sdcard, TT_SDCARD_MOUNT_POINT);
-            }
-        } else {
-            // Numbered mount path name
-            for (int i = 0; i < sdcards.size(); i++) {
-                auto sdcard = sdcards[i];
-                if (!sdcard->isMounted()) {
-                    std::string mount_path = TT_SDCARD_MOUNT_POINT + std::to_string(i);
-                    mount(sdcard, mount_path);
-                }
-            }
-        }
-    }
-}
-
-}

Tactility/Source/hal/usb/Usb.cpp

@@ -21,29 +21,30 @@ static Mode currentMode = Mode::Default;
 static RTC_NOINIT_ATTR BootMode bootMode;
 
 sdmmc_card_t* _Nullable getCard() {
-    auto sdcards = findDevices<sdcard::SpiSdCardDevice>(Device::Type::SdCard);
+    auto sdcard = getConfiguration()->sdcard;
-
+    if (sdcard == nullptr) {
-    std::shared_ptr<sdcard::SpiSdCardDevice> usable_sdcard;
+        TT_LOG_W(TAG, "No SD card configuration found");
-    for (auto& sdcard : sdcards) {
-        auto sdcard_candidate = std::static_pointer_cast<sdcard::SpiSdCardDevice>(sdcard);
-        if (sdcard_candidate != nullptr && sdcard_candidate->isMounted() && sdcard_candidate->getCard() != nullptr) {
-            usable_sdcard = sdcard_candidate;
-            break;
-        }
-    }
-
-    if (usable_sdcard == nullptr) {
-        TT_LOG_W(TAG, "Couldn't find a mounted SpiSdCard");
         return nullptr;
     }
 
-    auto* sdmmc_card = usable_sdcard->getCard();
+    if (!sdcard->isMounted()) {
-    if (sdmmc_card == nullptr) {
+        TT_LOG_W(TAG, "SD card not mounted");
+        return nullptr;
+    }
+
+    auto spi_sdcard = std::static_pointer_cast<sdcard::SpiSdCardDevice>(sdcard);
+    if (spi_sdcard == nullptr) {
+        TT_LOG_W(TAG, "SD card interface is not supported (must be SpiSdCard)");
+        return nullptr;
+    }
+
+    auto* card = spi_sdcard->getCard();
+    if (card == nullptr) {
         TT_LOG_W(TAG, "SD card has no card object available");
         return nullptr;
     }
 
-    return sdmmc_card;
+    return card;
 }
 
 static bool canStartNewMode() {

Tactility/Source/lvgl/Lvgl.cpp

@@ -178,7 +178,7 @@ void start() {
 }
 
 void stop() {
-    TT_LOG_I(TAG, "Stopping LVGL");
+    TT_LOG_I(TAG, "Stop LVGL");
 
     if (!started) {
         TT_LOG_W(TAG, "Can't stop LVGL: not started");
@@ -195,7 +195,6 @@ void stop() {
 
     // Stop keyboards
 
-    TT_LOG_I(TAG, "Stopping keyboards");
     auto keyboards = hal::findDevices<hal::keyboard::KeyboardDevice>(hal::Device::Type::Keyboard);
     for (auto keyboard : keyboards) {
         if (keyboard->getLvglIndev() != nullptr) {
@@ -205,7 +204,6 @@ void stop() {
 
     // Stop touch
 
-    TT_LOG_I(TAG, "Stopping touch");
     // The display generally stops their own touch devices, but we'll clean up anything that didn't
     auto touch_devices = hal::findDevices<hal::touch::TouchDevice>(hal::Device::Type::Touch);
     for (auto touch_device : touch_devices) {
@@ -214,19 +212,8 @@ void stop() {
         }
     }
 
-    // Stop encoders
-
-    TT_LOG_I(TAG, "Stopping encoders");
-    // The display generally stops their own touch devices, but we'll clean up anything that didn't
-    auto encoder_devices = hal::findDevices<hal::encoder::EncoderDevice>(hal::Device::Type::Encoder);
-    for (auto encoder_device : encoder_devices) {
-        if (encoder_device->getLvglIndev() != nullptr) {
-            encoder_device->stopLvgl();
-        }
-    }
     // Stop displays (and their touch devices)
 
-    TT_LOG_I(TAG, "Stopping displays");
     auto displays = hal::findDevices<hal::display::DisplayDevice>(hal::Device::Type::Display);
     for (auto display : displays) {
         if (display->supportsLvgl() && display->getLvglDisplay() != nullptr && !display->stopLvgl()) {
@@ -237,8 +224,6 @@ void stop() {
     started = false;
 
     kernel::publishSystemEvent(kernel::SystemEvent::LvglStopped);
-
-    TT_LOG_I(TAG, "Stopped LVGL");
 }
 
 } // namespace

Tactility/Source/service/statusbar/Statusbar.cpp

@@ -89,20 +89,13 @@ static const char* getSdCardStatusIcon(hal::sdcard::SdCardDevice::State state) {
 }
 
 static _Nullable const char* getPowerStatusIcon() {
-    // TODO: Support multiple power devices?
+    auto get_power = getConfiguration()->hardware->power;
-    std::shared_ptr<hal::power::PowerDevice> power;
+    if (get_power == nullptr) {
-    hal::findDevices<hal::power::PowerDevice>(hal::Device::Type::Power, [&power](const auto& device) {
-        if (device->supportsMetric(hal::power::PowerDevice::MetricType::ChargeLevel)) {
-            power = device;
-            return false;
-        }
-        return true;
-    });
-
-    if (power == nullptr) {
         return nullptr;
     }
 
+    auto power = get_power();
+
     hal::power::PowerDevice::MetricData charge_level;
     if (!power->getMetric(hal::power::PowerDevice::MetricType::ChargeLevel, charge_level)) {
         return nullptr;