Export LVGL canvas symbols for side-loaded GameBoy app

Buildscripts/DevicetreeCompiler/source/binding_parser.py

@@ -45,7 +45,6 @@ def parse_binding(file_path: str, binding_dirs: list[str]) -> Binding:
             required=details.get('required', False),
             description=details.get('description', '').strip(),
             default=details.get('default', None),
-            element_type=details.get('element-type', None),
         )
         properties_dict[name] = prop
     filename = os.path.basename(file_path)

Buildscripts/DevicetreeCompiler/source/generator.py

@@ -80,7 +80,7 @@ def property_to_string(property: DeviceProperty, devices: list[Device]) -> str:
         return "{ " + ",".join(value_list) + " }"
     elif type == "phandle":
         return find_phandle(devices, property.value)
-    elif type == "phandles":
+    elif type == "phandle-array":
         value_list = list()
         if isinstance(property.value, list):
             for item in property.value:
@@ -93,26 +93,10 @@ def property_to_string(property: DeviceProperty, devices: list[Device]) -> str:
             # If it's a string, assume it's a #define and show it as-is
             return property.value
         else:
-            raise Exception(f"Unsupported phandles type for {property.name} with value {property.value} ")
+            raise Exception(f"Unsupported phandle-array type for {property.value}")
     else:
         raise DevicetreeException(f"property_to_string() has an unsupported type: {type}")
 
-def resolve_phandle_array_entries(device_property, devices):
-    """Convert a phandle-array DTS property into a list of C initializer strings."""
-    entries = []
-    if device_property.type == "phandle-array":
-        items = device_property.value
-    elif device_property.type == "values":
-        items = [PropertyValue(type="values", value=device_property.value)]
-    else:
-        return []
-    for item in items:
-        if isinstance(item, PropertyValue):
-            entries.append(property_to_string(DeviceProperty(name="", type=item.type, value=item.value), devices))
-        else:
-            entries.append(str(item))
-    return entries
-
 def resolve_parameters_from_bindings(device: Device, bindings: list[Binding], devices: list[Device]) -> list:
     compatible_property = find_device_property(device, "compatible")
     if compatible_property is None:
@@ -135,32 +119,11 @@ def resolve_parameters_from_bindings(device: Device, bindings: list[Binding], de
         if device_property.name not in binding_property_names:
             raise DevicetreeException(f"Device '{device.node_name}' has invalid property '{device_property.name}'")
 
-    node_name = get_device_node_name_safe(device)
-    result = []
-    phandle_arrays = []
-    for binding_property in binding_properties:
+    # Allocate total expected configuration arguments
+    result = [0] * len(binding_properties)
+    for index, binding_property in enumerate(binding_properties):
         device_property = find_device_property(device, binding_property.name)
-
-        if binding_property.type == "phandle-array":
-            if binding_property.element_type is None:
-                raise DevicetreeException(f"phandle-array property '{binding_property.name}' requires 'element-type' in binding")
-            prop_safe = binding_property.name.replace("-", "_")
-            array_var = f"{node_name}_{prop_safe}"
-            if device_property is not None:
-                entries = resolve_phandle_array_entries(device_property, devices)
-                phandle_arrays.append((array_var, binding_property.element_type, entries))
-                result.append(f"({binding_property.element_type}*){array_var}")
-                result.append(str(len(entries)))
-            elif binding_property.default is not None:
-                result.append("NULL")
-                result.append("0")
-            elif binding_property.required:
-                raise DevicetreeException(f"device {device.node_name} doesn't have property '{binding_property.name}'")
-            else:
-                result.append("NULL")
-                result.append("0")
-            continue
-
+        # No property specified in DTS, use binding defaults
         if device_property is None:
             if binding_property.default is not None:
                 temp_prop = DeviceProperty(
@@ -168,38 +131,30 @@ def resolve_parameters_from_bindings(device: Device, bindings: list[Binding], de
                     type=binding_property.type,
                     value=binding_property.default
                 )
-                result.append(property_to_string(temp_prop, devices))
+                result[index] = property_to_string(temp_prop, devices)
             elif binding_property.required:
                 raise DevicetreeException(f"device {device.node_name} doesn't have property '{binding_property.name}'")
             elif binding_property.type == "bool" or binding_property.type == "boolean":
                 if binding_property.default == "true" or binding_property.default == None:
-                    result.append("true")
-                else:
-                    result.append("false")
+                    result[index] = "true"
+                else: # Explicit or implied false
+                    result[index] = "false"
             else:
                 raise DevicetreeException(f"Device {device.node_name} doesn't have property '{binding_property.name}' and no default value is set")
         else:
-            result.append(property_to_string(device_property, devices))
-
-    return result, phandle_arrays
+            result[index] = property_to_string(device_property, devices)
+    return result
 
 def write_config(file, device: Device, bindings: list[Binding], devices: list[Device], type_name: str):
     node_name = get_device_node_name_safe(device)
     config_type = f"{type_name}_config_dt"
     config_variable_name = f"{node_name}_config"
-
-    config_params, phandle_arrays = resolve_parameters_from_bindings(device, bindings, devices)
-
-    # Write phandle-array variables before the config struct
-    for array_var, element_type, entries in phandle_arrays:
-        entries_str = ", ".join(entries)
-        file.write(f"static {element_type} {array_var}[] = {{ {entries_str} }};\n")
-
     file.write(f"static const {config_type} {config_variable_name}" " = {\n")
+    config_params = resolve_parameters_from_bindings(device, bindings, devices)
     # Indent all params
     for index, config_param in enumerate(config_params):
         config_params[index] = f"\t{config_param}"
-    # Join with comma and newline
+    # Join with command and newline
     if len(config_params) > 0:
         config_params_joined = ",\n".join(config_params)
         file.write(f"{config_params_joined}\n")
@@ -223,9 +178,7 @@ def write_device_structs(file, device: Device, parent_device: Device, bindings:
     # Write config struct
     write_config(file, device, bindings, devices, type_name)
     # Write device struct
-    address_value = device.node_address if device.node_address is not None else "0"
     file.write(f"static struct Device {node_name}" " = {\n")
-    file.write(f"\t.address = {address_value},\n")
     file.write(f"\t.name = \"{device.node_name}\",\n") # Use original name
     file.write(f"\t.config = &{config_variable_name},\n")
     file.write(f"\t.parent = {parent_value},\n")

Buildscripts/DevicetreeCompiler/source/grammar.lark

@@ -37,17 +37,13 @@ value: VALUE
 values: VALUE+
 array: NUMBER+
 
-phandle_array_entry: "<" values ">"
-phandle_array: phandle_array_entry ("," phandle_array_entry)+
-
-property_value: quoted_text_array | QUOTED_TEXT | phandle_array | "<" value ">" | "<" values ">" | "[" array "]" | PHANDLE
+property_value: quoted_text_array | QUOTED_TEXT | "<" value ">" | "<" values ">" | "[" array "]" | PHANDLE
 device_property: PROPERTY_NAME ["=" property_value] ";"
 
-NODE_ALIAS: /[a-zA-Z0-9_\-\/]+/
-NODE_NAME: /[a-zA-Z0-9_\-\/]+/
-NODE_ADDRESS: /[0-9a-zA-Z_]+/
+NODE_ALIAS: /[a-zA-Z0-9_\-\/@]+/
+NODE_NAME: /[a-zA-Z0-9_\-\/@]+/
 
-device: (NODE_ALIAS ":")? NODE_NAME ("@" NODE_ADDRESS)? "{" (device | device_property)* "};"
+device: (NODE_ALIAS ":")? NODE_NAME "{" (device | device_property)* "};"
 
 dts_version: /[0-9a-zA-Z\-]+/
 

Buildscripts/DevicetreeCompiler/source/models.py

@@ -8,7 +8,6 @@ class DtsVersion:
 class Device:
     node_name: str
     node_alias: str
-    node_address: str
     status: str
     properties: list
     devices: list
@@ -39,7 +38,6 @@ class BindingProperty:
     required: bool
     description: str
     default: object = None
-    element_type: str = None
 
 @dataclass
 class Binding:

Buildscripts/DevicetreeCompiler/source/transformer.py

@@ -29,7 +29,6 @@ class DtsTransformer(Transformer):
     def device(self, tokens: list):
         node_name = None
         node_alias = None
-        node_address = None
         status = None
         properties = list()
         child_devices = list()
@@ -38,8 +37,6 @@ class DtsTransformer(Transformer):
                 node_name = item.value
             elif type(item) is Token and item.type == 'NODE_ALIAS':
                 node_alias = item.value
-            elif type(item) is Token and item.type == 'NODE_ADDRESS':
-                node_address = item.value
             elif type(item) is DeviceProperty:
                 if item.name == "status":
                     status = item.value
@@ -47,7 +44,7 @@ class DtsTransformer(Transformer):
                     properties.append(item)
             elif type(item) is Device:
                 child_devices.append(item)
-        return Device(node_name, node_alias, node_address, status, properties, child_devices)
+        return Device(node_name, node_alias, status, properties, child_devices)
     def device_property(self, objects: List[object]):
         name = objects[0]
         # Boolean property has no value as the value is implied to be true
@@ -70,10 +67,6 @@ class DtsTransformer(Transformer):
         if isinstance(object[0], PropertyValue):
             return object[0]
         return PropertyValue(type="value", value=object[0])
-    def phandle_array_entry(self, tokens: list):
-        return tokens[0]
-    def phandle_array(self, tokens: list):
-        return PropertyValue(type="phandle-array", value=tokens)
     def array(self, object):
         return PropertyValue(type="array", value=object)
     def VALUE(self, token: Token):

Buildscripts/DevicetreeCompiler/tests/data/expected_devicetree.c

@@ -10,23 +10,21 @@ static const root_config_dt root_config = {
 };
 
 static struct Device root = {
-	.address = 0,
 	.name = "/",
 	.config = &root_config,
 	.parent = NULL,
 	.internal = NULL
 };
 
-static const generic_device_config_dt test_device_config = {
+static const generic_device_config_dt test_device@0_config = {
 	0,
 	42,
 	"hello"
 };
 
-static struct Device test_device = {
-	.address = 0,
-	.name = "test-device",
-	.config = &test_device_config,
+static struct Device test_device@0 = {
+	.name = "test-device@0",
+	.config = &test_device@0_config,
 	.parent = &root,
 	.internal = NULL
 };
@@ -40,7 +38,6 @@ static const bool_device_config_dt bool_test_device_config = {
 };
 
 static struct Device bool_test_device = {
-	.address = 0,
 	.name = "bool-test-device",
 	.config = &bool_test_device_config,
 	.parent = &root,
@@ -49,7 +46,7 @@ static struct Device bool_test_device = {
 
 struct DtsDevice dts_devices[] = {
 	{ &root, "test,root", DTS_DEVICE_STATUS_OKAY },
-	{ &test_device, "test,generic-device", DTS_DEVICE_STATUS_OKAY },
+	{ &test_device@0, "test,generic-device", DTS_DEVICE_STATUS_OKAY },
 	{ &bool_test_device, "test,bool-device", DTS_DEVICE_STATUS_OKAY },
 	DTS_DEVICE_TERMINATOR
 };

CMakeLists.txt

@@ -54,6 +54,8 @@ if (DEFINED ENV{ESP_IDF_VERSION})
 
     set(EXCLUDE_COMPONENTS "Simulator")
 
+    idf_build_set_property(LINK_OPTIONS "-Wl,--allow-multiple-definition" APPEND)
+
     # Panic handler wrapping is only available on Xtensa architecture
     if (CONFIG_IDF_TARGET_ARCH_XTENSA)
         idf_build_set_property(LINK_OPTIONS "-Wl,--wrap=esp_panic_handler" APPEND)

Devices/cyd-2432s024c/Source/Configuration.cpp

@@ -1,7 +1,9 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
 #include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
+#include <Tactility/lvgl/LvglSync.h>
 #include <PwmBacklight.h>
 
 static bool initBoot() {
@@ -21,6 +23,7 @@ static bool initBoot() {
 static tt::hal::DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/cyd-2432s024c/Source/devices/SdCard.cpp

@@ -0,0 +1,34 @@
+#include "SdCard.h"
+
+#define TAG "twodotfour_sdcard"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+#include <Tactility/RecursiveMutex.h>
+
+constexpr auto SDCARD_SPI_HOST = SPI3_HOST;
+constexpr auto SDCARD_PIN_CS = GPIO_NUM_5;
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        SDCARD_PIN_CS,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SDCARD_SPI_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}
+

Devices/cyd-2432s024c/Source/devices/SdCard.h

@@ -0,0 +1,8 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();
+

Devices/cyd-2432s024c/cyd,2432s024c.dts

@@ -4,8 +4,6 @@
 #include <tactility/bindings/esp32_gpio.h>
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
 
 / {
 	compatible = "root";
@@ -24,29 +22,18 @@
 		pin-scl = <&gpio0 32 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
-	
-		display {
-			compatible = "display-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 };

Devices/cyd-2432s024r/Source/Configuration.cpp

@@ -1,6 +1,9 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
+#include <Tactility/lvgl/LvglSync.h>
 #include <PwmBacklight.h>
 
 using namespace tt::hal;
@@ -11,7 +14,8 @@ static bool initBoot() {
 
 static DeviceVector createDevices() {
     return {
-        createDisplay()
+        createDisplay(),
+        createSdCard()
     };
 }
 

Devices/cyd-2432s024r/Source/devices/SdCard.cpp

@@ -0,0 +1,23 @@
+#include "SdCard.h"
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto config = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_5,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SPI3_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(std::move(config), spi_controller);
+}

Devices/cyd-2432s024r/Source/devices/SdCard.h

@@ -0,0 +1,8 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();
+

Devices/cyd-2432s024r/cyd,2432s024r.dts

@@ -3,9 +3,6 @@
 #include <tactility/bindings/root.h>
 #include <tactility/bindings/esp32_gpio.h>
 #include <tactility/bindings/esp32_spi.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
-#include <tactility/bindings/touch_placeholder.h>
 #include <tactility/bindings/esp32_uart.h>
 
 / {
@@ -17,36 +14,20 @@
         gpio-count = <40>;
     };
 
-    spi0 {
+    display_spi: spi0 {
         compatible = "espressif,esp32-spi";
         host = <SPI2_HOST>;
         pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
         pin-miso = <&gpio0 12 GPIO_FLAG_NONE>;
         pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
-        cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>, // Display
-                   <&gpio0 33 GPIO_FLAG_NONE>; // Touch
-
-		display@0 {
-			compatible = "display-placeholder";
-		};
-
-		touch@1 {
-			compatible = "touch-placeholder";
-		};
     };
 
-    spi1 {
+    sdcard_spi: spi1 {
         compatible = "espressif,esp32-spi";
         host = <SPI3_HOST>;
         pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
         pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
         pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
-
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
     };
 
     uart1 {

Devices/cyd-2432s028r/Source/Configuration.cpp

@@ -1,7 +1,9 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
 #include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
+#include <Tactility/lvgl/LvglSync.h>
 #include <PwmBacklight.h>
 
 using namespace tt::hal;
@@ -23,6 +25,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/cyd-2432s028r/Source/devices/SdCard.cpp

@@ -0,0 +1,25 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto config = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_5,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SPI3_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(std::move(config), spi_controller);
+}
+

Devices/cyd-2432s028r/Source/devices/SdCard.h

@@ -0,0 +1,8 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();
+

Devices/cyd-2432s028r/cyd,2432s028r.dts

@@ -5,9 +5,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
-#include <tactility/bindings/touch_placeholder.h>
 
 / {
 	compatible = "root";
@@ -26,36 +23,20 @@
 		pin-scl = <&gpio0 22 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>, // Display
-		           <&gpio0 33 GPIO_FLAG_NONE>; // Touch
 		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 12 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
-	
-		display@0 {
-			compatible = "display-placeholder";
-		};
-
-		touch@1 {
-			compatible = "touch-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
 		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
-
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart1 {

Devices/cyd-2432s028rv3/Source/Configuration.cpp

@@ -1,7 +1,9 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
 #include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
+#include <Tactility/lvgl/LvglSync.h>
 #include <PwmBacklight.h>
 
 using namespace tt::hal;
@@ -23,6 +25,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/cyd-2432s028rv3/Source/devices/SdCard.cpp

@@ -0,0 +1,25 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto config = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_5,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SPI3_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(std::move(config), spi_controller);
+}
+

Devices/cyd-2432s028rv3/Source/devices/SdCard.h

@@ -0,0 +1,8 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();
+

Devices/cyd-2432s028rv3/cyd,2432s028rv3.dts

@@ -5,9 +5,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_spi.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
-#include <tactility/bindings/touch_placeholder.h>
 
 / {
 	compatible = "root";
@@ -26,36 +23,20 @@
 		pin-scl = <&gpio0 22 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>, // Display
-		           <&gpio0 33 GPIO_FLAG_NONE>; // Touch
 		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 12 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
-	
-		display@0 {
-			compatible = "display-placeholder";
-		};
-
-		touch@1 {
-			compatible = "touch-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart1 {

Devices/cyd-2432s032c/Source/Configuration.cpp

@@ -1,4 +1,5 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
 #include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
@@ -38,6 +39,7 @@ static bool initBoot() {
 static tt::hal::DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/cyd-2432s032c/Source/devices/SdCard.cpp

@@ -0,0 +1,33 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+#include <Tactility/lvgl/LvglSync.h>
+#include <Tactility/RecursiveMutex.h>
+
+constexpr auto SDCARD_SPI_HOST = SPI3_HOST;
+constexpr auto SDCARD_PIN_CS = GPIO_NUM_5;
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        SDCARD_PIN_CS,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        std::make_shared<tt::RecursiveMutex>(),
+        std::vector<gpio_num_t>(),
+        SDCARD_SPI_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}
+

Devices/cyd-2432s032c/Source/devices/SdCard.h

@@ -0,0 +1,8 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();
+

Devices/cyd-2432s032c/cyd,2432s032c.dts

@@ -4,14 +4,12 @@
 #include <tactility/bindings/esp32_gpio.h>
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
 
 / {
 	compatible = "root";
 	model = "CYD 2432S032C";
 
-	gpio0 {
+	gpio1 {
 		compatible = "espressif,esp32-gpio";
 		gpio-count = <40>;
 	};
@@ -20,33 +18,22 @@
 		compatible = "espressif,esp32-i2c";
 		port = <I2C_NUM_0>;
 		clock-frequency = <400000>;
-		pin-sda = <&gpio0 33 GPIO_FLAG_NONE>;
-		pin-scl = <&gpio0 32 GPIO_FLAG_NONE>;
+		pin-sda = <&gpio1 33 GPIO_FLAG_NONE>;
+		pin-scl = <&gpio1 32 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>;
-		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
-		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
-	
-		display {
-			compatible = "display-placeholder";
-		};
+		pin-mosi = <&gpio1 13 GPIO_FLAG_NONE>;
+		pin-sclk = <&gpio1 14 GPIO_FLAG_NONE>;
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
-		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
-		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
-		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
+		pin-mosi = <&gpio1 23 GPIO_FLAG_NONE>;
+		pin-miso = <&gpio1 19 GPIO_FLAG_NONE>;
+		pin-sclk = <&gpio1 18 GPIO_FLAG_NONE>;
 	};
 };

Devices/cyd-3248s035c/Source/Configuration.cpp

@@ -1,32 +1,34 @@
-#include "devices/Display.h"
-#include <driver/gpio.h>
-
-#include <PwmBacklight.h>
-#include <Tactility/hal/Configuration.h>
-
-using namespace tt::hal;
-
-static bool initBoot() {
-    //Set the RGB Led Pins to output and turn them off
-    ESP_ERROR_CHECK(gpio_set_direction(GPIO_NUM_4, GPIO_MODE_OUTPUT)); //Red
-    ESP_ERROR_CHECK(gpio_set_direction(GPIO_NUM_16, GPIO_MODE_OUTPUT)); //Green
-    ESP_ERROR_CHECK(gpio_set_direction(GPIO_NUM_17, GPIO_MODE_OUTPUT)); //Blue
-
-    //0 on, 1 off... yep it's backwards.
-    ESP_ERROR_CHECK(gpio_set_level(GPIO_NUM_4, 1)); //Red
-    ESP_ERROR_CHECK(gpio_set_level(GPIO_NUM_16, 1)); //Green
-    ESP_ERROR_CHECK(gpio_set_level(GPIO_NUM_17, 1)); //Blue
-
-    return driver::pwmbacklight::init(LCD_PIN_BACKLIGHT);
-}
-
-static DeviceVector createDevices() {
-    return {
-        createDisplay(),
-    };
-}
-
-extern const Configuration hardwareConfiguration = {
-    .initBoot = initBoot,
-    .createDevices = createDevices
-};
+#include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
+
+#include <PwmBacklight.h>
+#include <Tactility/hal/Configuration.h>
+
+using namespace tt::hal;
+
+static bool initBoot() {
+    //Set the RGB Led Pins to output and turn them off
+    ESP_ERROR_CHECK(gpio_set_direction(GPIO_NUM_4, GPIO_MODE_OUTPUT)); //Red
+    ESP_ERROR_CHECK(gpio_set_direction(GPIO_NUM_16, GPIO_MODE_OUTPUT)); //Green
+    ESP_ERROR_CHECK(gpio_set_direction(GPIO_NUM_17, GPIO_MODE_OUTPUT)); //Blue
+
+    //0 on, 1 off... yep it's backwards.
+    ESP_ERROR_CHECK(gpio_set_level(GPIO_NUM_4, 1)); //Red
+    ESP_ERROR_CHECK(gpio_set_level(GPIO_NUM_16, 1)); //Green
+    ESP_ERROR_CHECK(gpio_set_level(GPIO_NUM_17, 1)); //Blue
+
+    return driver::pwmbacklight::init(LCD_PIN_BACKLIGHT);
+}
+
+static DeviceVector createDevices() {
+    return {
+        createDisplay(),
+        createSdCard()
+    };
+}
+
+extern const Configuration hardwareConfiguration = {
+    .initBoot = initBoot,
+    .createDevices = createDevices
+};

Devices/cyd-3248s035c/Source/devices/SdCard.cpp

@@ -0,0 +1,31 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+constexpr auto SDCARD_SPI_HOST = SPI3_HOST;
+constexpr auto SDCARD_PIN_CS = GPIO_NUM_5;
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        SDCARD_PIN_CS,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SDCARD_SPI_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}
+

Devices/cyd-3248s035c/Source/devices/SdCard.h

@@ -0,0 +1,8 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();
+

Devices/cyd-3248s035c/cyd,3248s035c.dts

@@ -1,71 +1,58 @@
-/dts-v1/;
-
-#include <tactility/bindings/root.h>
-#include <tactility/bindings/esp32_gpio.h>
-#include <tactility/bindings/esp32_i2c.h>
-#include <tactility/bindings/esp32_spi.h>
-#include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
-
-/ {
-	compatible = "root";
-	model = "CYD 3248S035C";
-
-	gpio0 {
-		compatible = "espressif,esp32-gpio";
-		gpio-count = <40>;
-	};
-
-	i2c_internal: i2c0 {
-		compatible = "espressif,esp32-i2c";
-		port = <I2C_NUM_0>;
-		clock-frequency = <400000>;
-		pin-sda = <&gpio0 33 GPIO_FLAG_NONE>;
-		pin-scl = <&gpio0 32 GPIO_FLAG_NONE>;
-	};
-
-	// CN1 header
-	i2c_external: i2c1 {
-		compatible = "espressif,esp32-i2c";
-		port = <I2C_NUM_1>;
-		clock-frequency = <400000>;
-		pin-sda = <&gpio0 21 GPIO_FLAG_NONE>;
-		pin-scl = <&gpio0 22 GPIO_FLAG_NONE>;
-	};
-
-	spi0 {
-		compatible = "espressif,esp32-spi";
-		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>;
-		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
-		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
-	
-		display {
-			compatible = "display-placeholder";
-		};
-	};
-
-	spi1 {
-		compatible = "espressif,esp32-spi";
-		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
-		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
-		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
-		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
-	};
-
-	// CN1 header, JST SH 1.25, GND / IO22 / IO21 / 3.3V
-	uart1 {
-		compatible = "espressif,esp32-uart";
-		status = "disabled";
-		port = <UART_NUM_1>;
-		pin-tx = <&gpio0 22 GPIO_FLAG_NONE>;
-		pin-rx = <&gpio0 21 GPIO_FLAG_NONE>;
-	};
-};
+/dts-v1/;
+
+#include <tactility/bindings/root.h>
+#include <tactility/bindings/esp32_gpio.h>
+#include <tactility/bindings/esp32_i2c.h>
+#include <tactility/bindings/esp32_spi.h>
+#include <tactility/bindings/esp32_uart.h>
+
+/ {
+	compatible = "root";
+	model = "CYD 3248S035C";
+
+	gpio0 {
+		compatible = "espressif,esp32-gpio";
+		gpio-count = <40>;
+	};
+
+	i2c_internal: i2c0 {
+		compatible = "espressif,esp32-i2c";
+		port = <I2C_NUM_0>;
+		clock-frequency = <400000>;
+		pin-sda = <&gpio0 33 GPIO_FLAG_NONE>;
+		pin-scl = <&gpio0 32 GPIO_FLAG_NONE>;
+	};
+
+	// CN1 header
+	i2c_external: i2c1 {
+		compatible = "espressif,esp32-i2c";
+		port = <I2C_NUM_1>;
+		clock-frequency = <400000>;
+		pin-sda = <&gpio0 21 GPIO_FLAG_NONE>;
+		pin-scl = <&gpio0 22 GPIO_FLAG_NONE>;
+	};
+
+	display_spi: spi0 {
+		compatible = "espressif,esp32-spi";
+		host = <SPI2_HOST>;
+		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
+		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
+	};
+
+	sdcard_spi: spi1 {
+		compatible = "espressif,esp32-spi";
+		host = <SPI3_HOST>;
+		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
+		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
+		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
+	};
+
+	// CN1 header, JST SH 1.25, GND / IO22 / IO21 / 3.3V
+	uart1 {
+		compatible = "espressif,esp32-uart";
+		status = "disabled";
+		port = <UART_NUM_1>;
+		pin-tx = <&gpio0 22 GPIO_FLAG_NONE>;
+		pin-rx = <&gpio0 21 GPIO_FLAG_NONE>;
+	};
+};

Devices/cyd-4848s040c/Source/Configuration.cpp

@@ -1,6 +1,10 @@
 #include "devices/St7701Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
+#include <Tactility/kernel/SystemEvents.h>
+#include <Tactility/lvgl/LvglSync.h>
 #include <PwmBacklight.h>
 
 using namespace tt::hal;
@@ -12,6 +16,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         std::make_shared<St7701Display>(),
+        createSdCard()
     };
 }
 

Devices/cyd-4848s040c/Source/devices/SdCard.cpp

@@ -0,0 +1,28 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+#include <Tactility/lvgl/LvglSync.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto config = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_42,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        tt::lvgl::getSyncLock(),
+        std::vector { GPIO_NUM_39 }
+    );
+
+    auto* spi_controller = device_find_by_name("spi0");
+    check(spi_controller, "spi0 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(config),
+        spi_controller
+    );
+}
+

Devices/cyd-4848s040c/Source/devices/SdCard.h

@@ -0,0 +1,8 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();
+

Devices/cyd-4848s040c/cyd,4848s040c.dts

@@ -5,7 +5,6 @@
 #include <tactility/bindings/esp32_gpio.h>
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
-#include <tactility/bindings/esp32_sdspi.h>
 
 / {
 	compatible = "root";
@@ -31,14 +30,9 @@
 	spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 42 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 47 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 41 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 48 GPIO_FLAG_NONE>;
-
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
+		pin-hd = <&gpio0 42 GPIO_FLAG_NONE>;
 	};
 };

Devices/cyd-8048s043c/Source/Configuration.cpp

@@ -1,5 +1,7 @@
 #include "PwmBacklight.h"
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
 
@@ -13,6 +15,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/cyd-8048s043c/Source/devices/SdCard.cpp

@@ -0,0 +1,25 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+#include <Tactility/lvgl/LvglSync.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto config = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_10,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot
+    );
+
+    auto* spi_controller = device_find_by_name("spi0");
+    check(spi_controller, "spi0 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(config),
+        spi_controller
+    );
+}

Devices/cyd-8048s043c/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/cyd-8048s043c/cyd,8048s043c.dts

@@ -6,7 +6,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_spi.h>
-#include <tactility/bindings/esp32_sdspi.h>
 
 / {
 	compatible = "root";
@@ -40,15 +39,9 @@
 	spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 10 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 11 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 12 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart1 {

Devices/cyd-e32r28t/Source/Configuration.cpp

@@ -1,6 +1,8 @@
+#include "devices/SdCard.h"
 #include "devices/Display.h"
 
 #include <Tactility/hal/Configuration.h>
+#include <Tactility/lvgl/LvglSync.h>
 #include <PwmBacklight.h>
 
 static bool initBoot() {
@@ -10,6 +12,7 @@ static bool initBoot() {
 static tt::hal::DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/cyd-e32r28t/Source/devices/SdCard.cpp

@@ -0,0 +1,28 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+#include <Tactility/RecursiveMutex.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_5,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SPI3_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}

Devices/cyd-e32r28t/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/cyd-e32r28t/cyd,e32r28t.dts

@@ -3,9 +3,6 @@
 #include <tactility/bindings/root.h>
 #include <tactility/bindings/esp32_gpio.h>
 #include <tactility/bindings/esp32_spi.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
-#include <tactility/bindings/touch_placeholder.h>
 
 / {
 	compatible = "root";
@@ -16,35 +13,19 @@
 		gpio-count = <40>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>, // Display
-		            <&gpio0 33 GPIO_FLAG_NONE>; // Touch
 		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 12 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
-	
-		display@0 {
-			compatible = "display-placeholder";
-		};
-
-		touch@1 {
-			compatible = "touch-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 };

Devices/cyd-e32r32p/Source/Configuration.cpp

@@ -1,7 +1,10 @@
+#include "devices/SdCard.h"
 #include "devices/Display.h"
 #include "devices/Power.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
+#include <Tactility/lvgl/LvglSync.h>
 #include <PwmBacklight.h>
 
 using namespace tt::hal;
@@ -14,6 +17,7 @@ static tt::hal::DeviceVector createDevices() {
     return {
         createPower(),
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/cyd-e32r32p/Source/devices/SdCard.cpp

@@ -0,0 +1,29 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+#include <Tactility/RecursiveMutex.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+using SdCardDevice = tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        SD_CS_PIN,                         // CS pin (IO5 on the module)
+        GPIO_NUM_NC,                           // MOSI override: leave NC to use SPI host pins
+        GPIO_NUM_NC,                           // MISO override: leave NC
+        GPIO_NUM_NC,                           // SCLK override: leave NC
+        SdCardDevice::MountBehaviour::AtBoot,
+        std::make_shared<tt::RecursiveMutex>(),
+        std::vector<gpio_num_t>(),
+        SD_SPI_HOST                        // SPI host for SD card (SPI3_HOST)
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}

Devices/cyd-e32r32p/Source/devices/SdCard.h

@@ -0,0 +1,13 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+#include "driver/gpio.h"
+#include "driver/spi_common.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+// SD card (microSD)
+constexpr auto SD_CS_PIN = GPIO_NUM_5;
+constexpr auto SD_SPI_HOST = SPI3_HOST;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/cyd-e32r32p/cyd,e32r32p.dts

@@ -4,9 +4,6 @@
 #include <tactility/bindings/esp32_gpio.h>
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
-#include <tactility/bindings/touch_placeholder.h>
 
 / {
 	compatible = "root";
@@ -25,35 +22,19 @@
 		pin-scl = <&gpio0 25 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>, // Display
-		            <&gpio0 33 GPIO_FLAG_NONE>; // Touch
 		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 12 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
-	
-		display@0 {
-			compatible = "display-placeholder";
-		};
-
-		touch@1 {
-			compatible = "touch-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 };

Devices/elecrow-crowpanel-advance-28/Source/Configuration.cpp

@@ -1,5 +1,7 @@
 #include "PwmBacklight.h"
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
 
@@ -12,6 +14,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/elecrow-crowpanel-advance-28/Source/devices/SdCard.cpp

@@ -0,0 +1,29 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+constexpr auto CROWPANEL_SDCARD_PIN_CS = GPIO_NUM_7;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        CROWPANEL_SDCARD_PIN_CS,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SPI3_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}

Devices/elecrow-crowpanel-advance-28/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/elecrow-crowpanel-advance-28/elecrow,crowpanel-advance-28.dts

@@ -6,8 +6,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
 
 / {
 	compatible = "root";
@@ -30,30 +28,19 @@
 		pin-scl = <&gpio0 16 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 40 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 39 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 42 GPIO_FLAG_NONE>;
-	
-		display {
-			compatible = "display-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 7 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 6 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 4 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 5 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart0 {

Devices/elecrow-crowpanel-advance-35/Source/Configuration.cpp

@@ -1,4 +1,6 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
 #include <PwmBacklight.h>
@@ -12,6 +14,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/elecrow-crowpanel-advance-35/Source/devices/SdCard.cpp

@@ -0,0 +1,29 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+constexpr auto CROWPANEL_SDCARD_PIN_CS = GPIO_NUM_7;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        CROWPANEL_SDCARD_PIN_CS,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SPI3_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}

Devices/elecrow-crowpanel-advance-35/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/elecrow-crowpanel-advance-35/elecrow,crowpanel-advance-35.dts

@@ -6,8 +6,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
 
 / {
 	compatible = "root";
@@ -30,30 +28,19 @@
 		pin-scl = <&gpio0 16 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 40 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 39 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 42 GPIO_FLAG_NONE>;
-	
-		display {
-			compatible = "display-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 7 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 6 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 4 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 5 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart0 {

Devices/elecrow-crowpanel-advance-50/Source/Configuration.cpp

@@ -1,4 +1,6 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
 #include <TCA9534.h>
@@ -23,6 +25,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/elecrow-crowpanel-advance-50/Source/devices/SdCard.cpp

@@ -0,0 +1,25 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        // See https://github.com/Elecrow-RD/CrowPanel-Advance-HMI-ESP32-AI-Display/blob/master/5.0/factory_code/factory_code.ino
+        GPIO_NUM_0, // It's actually not connected, but in the demo pin 0 is used
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot
+    );
+
+    auto* spi_controller = device_find_by_name("spi0");
+    check(spi_controller, "spi0 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}

Devices/elecrow-crowpanel-advance-50/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/elecrow-crowpanel-advance-50/elecrow,crowpanel-advance-50.dts

@@ -6,7 +6,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
 
 / {
 	compatible = "root";
@@ -29,18 +28,12 @@
 		pin-scl = <&gpio0 16 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	sdcard_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 0 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 6 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 4 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 5 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart0 {

Devices/elecrow-crowpanel-basic-28/Source/Configuration.cpp

@@ -1,5 +1,7 @@
 #include "PwmBacklight.h"
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
 
@@ -12,6 +14,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/elecrow-crowpanel-basic-28/Source/devices/SdCard.cpp

@@ -0,0 +1,28 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/lvgl/LvglSync.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_5,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        tt::lvgl::getSyncLock(),
+        std::vector<gpio_num_t>(),
+        SPI3_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}

Devices/elecrow-crowpanel-basic-28/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/elecrow-crowpanel-basic-28/elecrow,crowpanel-basic-28.dts

@@ -5,9 +5,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
-#include <tactility/bindings/touch_placeholder.h>
 
 / {
 	compatible = "root";
@@ -26,37 +23,21 @@
 		pin-scl = <&gpio0 21 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>, // Display
-		            <&gpio0 33 GPIO_FLAG_NONE>; // Touch
 		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 12 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
 		max-transfer-size = <65536>;
-	
-		display@0 {
-			compatible = "display-placeholder";
-		};
-
-		touch@1 {
-			compatible = "touch-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart1 {

Devices/elecrow-crowpanel-basic-35/Source/Configuration.cpp

@@ -1,21 +1,20 @@
 #include "PwmBacklight.h"
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
 
 using namespace tt::hal;
 
 static bool initBoot() {
-    driver::pwmbacklight::init(GPIO_NUM_27);
-    // Delay is required, or GUI will lock up.
-    // It's possibly related to the increased power draw when turning on the backlight.
-    vTaskDelay(100);
-    return true;
+    return driver::pwmbacklight::init(GPIO_NUM_27);
 }
 
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/elecrow-crowpanel-basic-35/Source/devices/SdCard.cpp

@@ -0,0 +1,28 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/lvgl/LvglSync.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_5,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        tt::lvgl::getSyncLock(),
+        std::vector<gpio_num_t>(),
+        SPI3_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}

Devices/elecrow-crowpanel-basic-35/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/elecrow-crowpanel-basic-35/elecrow,crowpanel-basic-35.dts

@@ -5,9 +5,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
-#include <tactility/bindings/touch_placeholder.h>
 
 / {
 	compatible = "root";
@@ -26,37 +23,21 @@
 		pin-scl = <&gpio0 21 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>, // Display
-		            <&gpio0 12 GPIO_FLAG_NONE>; // Touch
 		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 33 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
 		max-transfer-size = <65536>;
-	
-		display@0 {
-			compatible = "display-placeholder";
-		};
-
-		touch@1 {
-			compatible = "touch-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart1 {

Devices/elecrow-crowpanel-basic-50/Source/Configuration.cpp

@@ -1,4 +1,6 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
 #include <PwmBacklight.h>
@@ -13,6 +15,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard(),
     };
 }
 

Devices/elecrow-crowpanel-basic-50/Source/devices/SdCard.cpp

@@ -0,0 +1,24 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_10,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot
+    );
+
+    auto* spi_controller = device_find_by_name("spi0");
+    check(spi_controller, "spi0 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}

Devices/elecrow-crowpanel-basic-50/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/elecrow-crowpanel-basic-50/elecrow,crowpanel-basic-50.dts

@@ -6,7 +6,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
 
 / {
 	compatible = "root";
@@ -29,18 +28,12 @@
 		pin-scl = <&gpio0 20 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	sdcard_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 10 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 11 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 12 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart0 {

Devices/es3c28p/CMakeLists.txt

@@ -0,0 +1,7 @@
+file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+
+idf_component_register(
+    SRCS ${SOURCE_FILES}
+    INCLUDE_DIRS "Source"
+    REQUIRES Tactility esp_lvgl_port ILI934x FT6x36 PwmBacklight driver vfs fatfs
+)

Devices/es3c28p/Source/Configuration.cpp

@@ -0,0 +1,105 @@
+#include "PwmBacklight.h"
+#include "devices/Display.h"
+#include <driver/gpio.h>
+
+#include <tactility/device.h>
+#include <tactility/drivers/i2c_controller.h>
+#include <tactility/log.h>
+#include <tactility/error.h>
+#include <tactility/freertos/freertos.h>
+
+#include <Tactility/hal/Configuration.h>
+
+using namespace tt::hal;
+
+static constexpr auto* TAG = "ES3C28P";
+static constexpr uint8_t ES8311_I2C_ADDR = 0x18;
+
+static error_t initSound(::Device* i2c_controller) {
+    static constexpr uint8_t ENABLED_BULK_DATA[] = {
+        0x00, 0x80,  // RESET/CSM POWER ON
+        0x01, 0x3F,  // CLOCK_MANAGER/ use MCLK pin (external), all clocks on
+        0x02, 0x00,  // CLOCK_MANAGER/ pre_div=1 pre_multi=1 (256x MCLK is correct ratio)
+        0x0D, 0x01,  // SYSTEM/ Power up analog circuitry
+        0x12, 0x00,  // SYSTEM/ power-up DAC
+        0x13, 0x10,  // SYSTEM/ Enable output to HP drive
+        0x32, 0xBF,  // DAC/ DAC volume (0xBF == ±0 dB)
+        0x37, 0x08,  // DAC/ Bypass DAC equalizer
+    };
+
+    return i2c_controller_write_register_array(
+        i2c_controller,
+        ES8311_I2C_ADDR,
+        ENABLED_BULK_DATA,
+        sizeof(ENABLED_BULK_DATA),
+        pdMS_TO_TICKS(1000)
+    );
+}
+
+static error_t initMicrophone(::Device* i2c_controller) {
+    static constexpr uint8_t ENABLED_BULK_DATA[] = {
+        0x00, 0x80,  // RESET/CSM POWER ON
+        0x01, 0x3F,  // CLOCK_MANAGER/ use MCLK pin (external), all clocks on
+        0x02, 0x00,  // CLOCK_MANAGER/ pre_div=1 pre_multi=1
+        0x0D, 0x01,  // SYSTEM/ Power up analog circuitry
+        0x0E, 0x02,  // SYSTEM/ Enable analog PGA, enable ADC modulator
+        0x14, 0x10,  // ADC_REG14: select Mic1p-Mic1n / PGA GAIN (minimum)
+        0x17, 0xFF,  // ADC_REG17: ADC_VOLUME (MAXGAIN)
+        0x1C, 0x6A,  // ADC_REG1C: ADC Equalizer bypass, cancel DC offset
+    };
+
+    return i2c_controller_write_register_array(
+        i2c_controller,
+        ES8311_I2C_ADDR,
+        ENABLED_BULK_DATA,
+        sizeof(ENABLED_BULK_DATA),
+        pdMS_TO_TICKS(1000)
+    );
+}
+
+static bool initBoot() {
+    // 1. Initialize display backlight
+    if (!driver::pwmbacklight::init(LCD_PIN_BACKLIGHT)) {
+        return false;
+    }
+
+    // 2. Initialize and power on the FM8002E Audio Amplifier (GPIO 1, Active LOW)
+    gpio_config_t io_conf = {};
+    io_conf.intr_type = GPIO_INTR_DISABLE;
+    io_conf.mode = GPIO_MODE_OUTPUT;
+    io_conf.pin_bit_mask = (1ULL << GPIO_NUM_1);
+    io_conf.pull_down_en = GPIO_PULLDOWN_DISABLE;
+    io_conf.pull_up_en = GPIO_PULLUP_DISABLE;
+    gpio_config(&io_conf);
+    
+    // Drive LOW to enable the speaker amplifier
+    gpio_set_level(GPIO_NUM_1, 0);
+
+    // 3. Configure the ES8311 Codec over the I2C bus
+    auto* i2c_bus = device_find_by_name("i2c0");
+    if (i2c_bus != nullptr) {
+        error_t error = initSound(i2c_bus);
+        if (error != ERROR_NONE) {
+            LOG_E(TAG, "Failed to enable ES8311 speaker: %s", error_to_string(error));
+        }
+        error = initMicrophone(i2c_bus);
+        if (error != ERROR_NONE) {
+            LOG_E(TAG, "Failed to enable ES8311 microphone: %s", error_to_string(error));
+        }
+    } else {
+        LOG_E(TAG, "i2c0 bus not found, skipping ES8311 initialization");
+    }
+
+    return true;
+}
+
+static DeviceVector createDevices() {
+    return {
+        createDisplay()
+    };
+}
+
+extern const Configuration hardwareConfiguration = {
+    .initBoot = initBoot,
+    .createDevices = createDevices
+};

Devices/es3c28p/Source/devices/Display.cpp

@@ -0,0 +1,49 @@
+#include "Display.h"
+
+#include <Ft6x36Touch.h>
+#include <Ili934xDisplay.h>
+#include <PwmBacklight.h>
+
+static std::shared_ptr<tt::hal::touch::TouchDevice> createTouch() {
+    auto configuration = std::make_unique<Ft6x36Touch::Configuration>(
+        I2C_NUM_0,
+        240, // xMax (native portrait width)
+        320, // yMax (native portrait height)
+        true, // swapXy
+        true, // mirrorX
+        false, // mirrorY
+        GPIO_NUM_18, // Touch Reset (RST)
+        GPIO_NUM_17  // Touch Interrupt (INT)
+    );
+
+    return std::make_shared<Ft6x36Touch>(std::move(configuration));
+}
+
+std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay() {
+    Ili934xDisplay::Configuration panel_configuration = {
+        .horizontalResolution = LCD_HORIZONTAL_RESOLUTION,
+        .verticalResolution = LCD_VERTICAL_RESOLUTION,
+        .gapX = 0,
+        .gapY = 0,
+        .swapXY = true,
+        .mirrorX = false,
+        .mirrorY = false,
+        .invertColor = false,
+        .swapBytes = true,
+        .bufferSize = LCD_BUFFER_SIZE,
+        .touch = createTouch(),
+        .backlightDutyFunction = driver::pwmbacklight::setBacklightDuty,
+        .resetPin = GPIO_NUM_NC,
+        .rgbElementOrder = LCD_RGB_ELEMENT_ORDER_BGR
+    };
+
+    auto spi_configuration = std::make_shared<Ili934xDisplay::SpiConfiguration>(Ili934xDisplay::SpiConfiguration {
+        .spiHostDevice = LCD_SPI_HOST,
+        .csPin = LCD_PIN_CS,
+        .dcPin = LCD_PIN_DC,
+        .pixelClockFrequency = 40'000'000,
+        .transactionQueueDepth = 10
+    });
+
+    return std::make_shared<Ili934xDisplay>(panel_configuration, spi_configuration, true);
+}

Devices/es3c28p/Source/devices/Display.h

@@ -0,0 +1,20 @@
+#pragma once
+
+#include <Tactility/hal/display/DisplayDevice.h>
+#include <driver/gpio.h>
+#include <driver/spi_common.h>
+#include <memory>
+
+// Display
+constexpr auto LCD_SPI_HOST = SPI2_HOST;
+constexpr auto LCD_PIN_CS = GPIO_NUM_10;
+constexpr auto LCD_PIN_DC = GPIO_NUM_46;
+constexpr auto LCD_HORIZONTAL_RESOLUTION = 320;
+constexpr auto LCD_VERTICAL_RESOLUTION = 240;
+constexpr auto LCD_BUFFER_HEIGHT = LCD_VERTICAL_RESOLUTION / 10;
+constexpr auto LCD_BUFFER_SIZE = LCD_HORIZONTAL_RESOLUTION * LCD_BUFFER_HEIGHT;
+
+// Backlight pin
+constexpr auto LCD_PIN_BACKLIGHT = GPIO_NUM_45;
+
+std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay();

Devices/es3c28p/Source/module.cpp

@@ -0,0 +1,23 @@
+#include <tactility/module.h>
+
+extern "C" {
+
+static error_t start() {
+    // Empty for now
+    return ERROR_NONE;
+}
+
+static error_t stop() {
+    // Empty for now
+    return ERROR_NONE;
+}
+
+struct Module es3c28p_module = {
+    .name = "es3c28p",
+    .start = start,
+    .stop = stop,
+    .symbols = nullptr,
+    .internal = nullptr
+};
+
+}

Devices/es3c28p/device.properties

@@ -0,0 +1,24 @@
+[general]
+vendor=LCDWIKI
+name=ES3C28P
+
+[apps]
+launcherAppId=Launcher
+
+[hardware]
+target=ESP32S3
+flashSize=16MB
+spiRam=true
+spiRamMode=OCT
+spiRamSpeed=120M
+tinyUsb=true
+esptoolFlashFreq=120M
+bluetooth=true
+
+[display]
+size=2.8"
+shape=rectangle
+dpi=143
+
+[lvgl]
+colorDepth=16

Devices/es3c28p/devicetree.yaml

@@ -0,0 +1,3 @@
+dependencies:
+  - Platforms/platform-esp32
+dts: es3c28p.dts

Devices/es3c28p/es3c28p.dts

@@ -0,0 +1,59 @@
+/dts-v1/;
+
+#include <tactility/bindings/root.h>
+#include <tactility/bindings/esp32_ble.h>
+#include <tactility/bindings/esp32_gpio.h>
+#include <tactility/bindings/esp32_i2c.h>
+#include <tactility/bindings/esp32_sdmmc.h>
+#include <tactility/bindings/esp32_spi.h>
+#include <tactility/bindings/esp32_i2s.h>
+
+/ {
+	compatible = "root";
+	model = "LCDWIKI ES3C28P";
+
+	ble0 {
+		compatible = "espressif,esp32-ble";
+	};
+
+	gpio0 {
+		compatible = "espressif,esp32-gpio";
+		gpio-count = <49>;
+	};
+
+	i2c0 {
+		compatible = "espressif,esp32-i2c";
+		port = <I2C_NUM_0>;
+		clock-frequency = <400000>;
+		pin-sda = <&gpio0 16 GPIO_FLAG_NONE>;
+		pin-scl = <&gpio0 15 GPIO_FLAG_NONE>;
+	};
+
+	i2s0 {
+		compatible = "espressif,esp32-i2s";
+		port = <I2S_NUM_0>;
+		pin-bclk = <&gpio0 5 GPIO_FLAG_NONE>;
+		pin-ws = <&gpio0 7 GPIO_FLAG_NONE>;
+		pin-data-out = <&gpio0 6 GPIO_FLAG_NONE>;
+		pin-data-in = <&gpio0 8 GPIO_FLAG_NONE>;
+		pin-mclk = <&gpio0 4 GPIO_FLAG_NONE>;
+	};
+
+	display_spi: spi0 {
+		compatible = "espressif,esp32-spi";
+		host = <SPI2_HOST>;
+		pin-mosi = <&gpio0 11 GPIO_FLAG_NONE>;
+		pin-sclk = <&gpio0 12 GPIO_FLAG_NONE>;
+	};
+
+	sdmmc0 {
+		compatible = "espressif,esp32-sdmmc";
+		pin-clk = <&gpio0 38 GPIO_FLAG_NONE>;
+		pin-cmd = <&gpio0 40 GPIO_FLAG_NONE>;
+		pin-d0 = <&gpio0 39 GPIO_FLAG_NONE>;
+		pin-d1 = <&gpio0 41 GPIO_FLAG_NONE>;
+		pin-d2 = <&gpio0 48 GPIO_FLAG_NONE>;
+		pin-d3 = <&gpio0 47 GPIO_FLAG_NONE>;
+		bus-width = <4>;
+	};
+};

Devices/guition-jc1060p470ciwy/Source/Configuration.cpp

@@ -1,5 +1,6 @@
 #include "devices/Display.h"
 #include "devices/Power.h"
+#include "devices/SdCard.h"
 
 #include <Tactility/hal/Configuration.h>
 
@@ -8,6 +9,7 @@ using namespace tt::hal;
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard(),
         createPower()
     };
 }

Devices/guition-jc1060p470ciwy/Source/devices/SdCard.cpp

@@ -0,0 +1,128 @@
+#include "SdCard.h"
+
+#include <Tactility/Logger.h>
+#include <Tactility/Mutex.h>
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+#include <driver/sdmmc_defs.h>
+#include <driver/sdmmc_host.h>
+#include <esp_check.h>
+#include <esp_ldo_regulator.h>
+#include <esp_vfs_fat.h>
+#include <sdmmc_cmd.h>
+
+using tt::hal::sdcard::SdCardDevice;
+
+static const auto LOGGER = tt::Logger("JcSdCard");
+
+// ESP32-P4 Slot 0 uses IO MUX (fixed pins, not manually configurable)
+// CLK=43, CMD=44, D0=39, D1=40, D2=41, D3=42 (defined automatically by hardware)
+
+// TODO: Migrate to "espressif,esp32-sdmmc" driver (needs LDO code porting)
+class SdCardDeviceImpl final : public SdCardDevice {
+
+    class NoLock final : public tt::Lock {
+        bool lock(TickType_t timeout) const override { return true; }
+        void unlock() const override { /* NO-OP */ }
+    };
+
+    std::shared_ptr<tt::Lock> lock = std::make_shared<NoLock>();
+    sdmmc_card_t* card = nullptr;
+    bool mounted = false;
+    std::string mountPath;
+
+public:
+    SdCardDeviceImpl() : SdCardDevice(MountBehaviour::AtBoot) {}
+    ~SdCardDeviceImpl() override {
+        if (mounted) {
+            unmount();
+        }
+    }
+
+    std::string getName() const override { return "SD Card"; }
+    std::string getDescription() const override { return "SD card via SDMMC host"; }
+
+    bool mount(const std::string& newMountPath) override {
+        if (mounted) {
+            return true;
+        }
+
+        esp_vfs_fat_sdmmc_mount_config_t mount_config = {
+            .format_if_mount_failed = false,
+            .max_files = 5,
+            .allocation_unit_size = 64 * 1024,
+            .disk_status_check_enable = false,
+            .use_one_fat = false,
+        };
+
+        sdmmc_host_t host = SDMMC_HOST_DEFAULT();
+        host.slot = SDMMC_HOST_SLOT_0;
+        host.max_freq_khz = SDMMC_FREQ_DEFAULT; // 20MHz - more stable for initialization
+        host.flags = SDMMC_HOST_FLAG_4BIT; // Force 4-bit mode
+        // Configure LDO power supply for SD card (critical on ESP32-P4)
+        esp_ldo_channel_handle_t ldo_handle = nullptr;
+        esp_ldo_channel_config_t ldo_config = {
+            .chan_id = 4,  // LDO channel 4 for SD power
+            .voltage_mv = 3300,  // 3.3V
+            .flags {
+                .adjustable = 0,
+                .owned_by_hw = 0,
+                .bypass = 0
+            }
+        };
+
+        esp_err_t ldo_ret = esp_ldo_acquire_channel(&ldo_config, &ldo_handle);
+        if (ldo_ret != ESP_OK) {
+            LOGGER.warn("Failed to acquire LDO for SD power: {} (continuing anyway)", esp_err_to_name(ldo_ret));
+        }
+
+        // Slot 0 uses IO MUX - pins are fixed and not specified
+        sdmmc_slot_config_t slot_config = SDMMC_SLOT_CONFIG_DEFAULT();
+        slot_config.width = 4;
+        slot_config.cd = SDMMC_SLOT_NO_CD;  // No card detect
+        slot_config.wp = SDMMC_SLOT_NO_WP;  // No write protect
+        slot_config.flags = 0;
+
+        esp_err_t ret = esp_vfs_fat_sdmmc_mount(newMountPath.c_str(), &host, &slot_config, &mount_config, &card);
+        if (ret != ESP_OK) {
+            LOGGER.error("Failed to mount SD card: {}", esp_err_to_name(ret));
+            card = nullptr;
+            return false;
+        }
+
+        mountPath = newMountPath;
+        mounted = true;
+        LOGGER.info("SD card mounted at {}", mountPath);
+        return true;
+    }
+
+    bool unmount() override {
+        if (!mounted) {
+            return true;
+        }
+
+        esp_err_t ret = esp_vfs_fat_sdcard_unmount(mountPath.c_str(), card);
+        if (ret != ESP_OK) {
+            LOGGER.error("Failed to unmount SD card: {}", esp_err_to_name(ret));
+            return false;
+        }
+        card = nullptr;
+        mounted = false;
+        LOGGER.info("SD card unmounted");
+        return true;
+    }
+
+    std::string getMountPath() const override {
+        return mountPath;
+    }
+
+    std::shared_ptr<tt::Lock> getLock() const override { return lock; }
+
+    State getState(TickType_t /*timeout*/) const override {
+        return mounted ? State::Mounted : State::Unmounted;
+    }
+};
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    return std::make_shared<SdCardDeviceImpl>();
+}

Devices/guition-jc1060p470ciwy/Source/devices/SdCard.h

@@ -0,0 +1,6 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+// Create SD card device for jc1060p470ciwy using SDMMC slot 0 (4-bit)
+std::shared_ptr<tt::hal::sdcard::SdCardDevice> createSdCard();

Devices/guition-jc1060p470ciwy/guition,jc1060p470ciwy.dts

@@ -5,7 +5,6 @@
 #include <tactility/bindings/esp32_gpio.h>
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_i2s.h>
-#include <tactility/bindings/esp32_sdmmc.h>
 
 /**
  * For future reference:
@@ -45,17 +44,4 @@
 		pin-data-in = <&gpio0 48 GPIO_FLAG_NONE>;
 		pin-mclk = <&gpio0 13 GPIO_FLAG_NONE>;
 	};
-
-	sdcard {
-		compatible = "espressif,esp32-sdmmc";
-		pin-clk = <&gpio0 43 GPIO_FLAG_NONE>;
-		pin-cmd = <&gpio0 44 GPIO_FLAG_NONE>;
-		pin-d0 = <&gpio0 39 GPIO_FLAG_NONE>;
-		pin-d1 = <&gpio0 40 GPIO_FLAG_NONE>;
-		pin-d2 = <&gpio0 41 GPIO_FLAG_NONE>;
-		pin-d3 = <&gpio0 42 GPIO_FLAG_NONE>;
-		slot = <SDMMC_HOST_SLOT_0>;
-		bus-width = <4>;
-		on-chip-ldo-chan = <4>;
-	};
 };

Devices/guition-jc2432w328c/Source/Configuration.cpp

@@ -1,4 +1,5 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
 #include <driver/gpio.h>
 
 #include <PwmBacklight.h>
@@ -23,6 +24,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/guition-jc2432w328c/Source/devices/SdCard.cpp

@@ -0,0 +1,31 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+constexpr auto SDCARD_SPI_HOST = SPI3_HOST;
+constexpr auto SDCARD_PIN_CS = GPIO_NUM_5;
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        SDCARD_PIN_CS,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SDCARD_SPI_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}
+

Devices/guition-jc2432w328c/Source/devices/SdCard.h

@@ -0,0 +1,8 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();
+

Devices/guition-jc2432w328c/guition,jc2432w328c.dts

@@ -5,8 +5,6 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
 
 / {
 	compatible = "root";
@@ -34,30 +32,19 @@
 		pin-scl = <&gpio0 22 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 15 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 14 GPIO_FLAG_NONE>;
-	
-		display {
-			compatible = "display-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 5 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 23 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 18 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	// CN1 header, JST SH 1.25, GND / IO22 / IO21 / 3.3V

Devices/guition-jc3248w535c/Source/Axs15231b/Axs15231bDisplay.cpp

@@ -2,6 +2,7 @@
 
 #include <Tactility/Logger.h>
 #include <Tactility/hal/touch/TouchDevice.h>
+#include <Tactility/lvgl/LvglSync.h>
 #include <Axs15231b/Axs15231bTouch.h>
 #include <EspLcdDisplayDriver.h>
 

Devices/guition-jc3248w535c/Source/Configuration.cpp

@@ -1,4 +1,6 @@
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
 #include <PwmBacklight.h>
@@ -8,6 +10,7 @@ using namespace tt::hal;
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/guition-jc3248w535c/Source/devices/SdCard.cpp

@@ -0,0 +1,30 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+constexpr auto SDCARD_SPI_HOST = SPI3_HOST;
+constexpr auto SDCARD_PIN_CS = GPIO_NUM_10;
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        SDCARD_PIN_CS,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        nullptr,
+        std::vector<gpio_num_t>(),
+        SDCARD_SPI_HOST
+    );
+
+    auto* spi_controller = device_find_by_name("spi1");
+    check(spi_controller, "spi1 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spi_controller
+    );
+}

Devices/guition-jc3248w535c/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/guition-jc3248w535c/guition,jc3248w535c.dts

@@ -7,8 +7,6 @@
 #include <tactility/bindings/esp32_i2s.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/display_placeholder.h>
 
 / {
 	compatible = "root";
@@ -47,33 +45,22 @@
 		pin-data-out = <&gpio0 41 GPIO_FLAG_NONE>;
 	};
 
-	spi0 {
+	display_spi: spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 45 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 21 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 48 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 47 GPIO_FLAG_NONE>;
 		pin-wp = <&gpio0 40 GPIO_FLAG_NONE>;
 		pin-hd = <&gpio0 39 GPIO_FLAG_NONE>;
-
-		display {
-			compatible = "display-placeholder";
-		};
 	};
 
-	spi1 {
+	sdcard_spi: spi1 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI3_HOST>;
-		cs-gpios = <&gpio0 10 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 11 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 12 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	// P1 header

Devices/guition-jc8048w550c/Source/Configuration.cpp

@@ -1,5 +1,7 @@
 #include "PwmBacklight.h"
 #include "devices/Display.h"
+#include "devices/SdCard.h"
+#include <driver/gpio.h>
 
 #include <Tactility/hal/Configuration.h>
 
@@ -12,6 +14,7 @@ static bool initBoot() {
 static DeviceVector createDevices() {
     return {
         createDisplay(),
+        createSdCard()
     };
 }
 

Devices/guition-jc8048w550c/Source/devices/SdCard.cpp

@@ -0,0 +1,24 @@
+#include "SdCard.h"
+
+#include <tactility/device.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto config = std::make_unique<SpiSdCardDevice::Config>(
+        GPIO_NUM_10,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot
+    );
+
+    auto* spi_controller = device_find_by_name("spi0");
+    check(spi_controller, "spi0 not found");
+
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(config),
+        spi_controller
+    );
+}

Devices/guition-jc8048w550c/Source/devices/SdCard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include <Tactility/hal/sdcard/SdCardDevice.h>
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();

Devices/guition-jc8048w550c/guition,jc8048w550c.dts

@@ -7,7 +7,6 @@
 #include <tactility/bindings/esp32_i2s.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_sdspi.h>
 
 / {
 	compatible = "root";
@@ -50,15 +49,9 @@
 	spi0 {
 		compatible = "espressif,esp32-spi";
 		host = <SPI2_HOST>;
-		cs-gpios = <&gpio0 10 GPIO_FLAG_NONE>;
 		pin-mosi = <&gpio0 11 GPIO_FLAG_NONE>;
 		pin-miso = <&gpio0 13 GPIO_FLAG_NONE>;
 		pin-sclk = <&gpio0 12 GPIO_FLAG_NONE>;
-	
-		sdcard@0 {
-			compatible = "espressif,esp32-sdspi";
-			frequency-khz = <20000>;
-		};
 	};
 
 	uart1 {

Devices/lilygo-tdeck/Source/Configuration.cpp

@@ -1,10 +1,12 @@
 #include "devices/Display.h"
 #include "devices/KeyboardBacklight.h"
 #include "devices/Power.h"
+#include "devices/Sdcard.h"
 #include "devices/TdeckKeyboard.h"
 #include "devices/TrackballDevice.h"
 
 #include <Tactility/hal/Configuration.h>
+#include <Tactility/lvgl/LvglSync.h>
 #include <tactility/check.h>
 #include <tactility/device.h>
 
@@ -21,6 +23,7 @@ static std::vector<std::shared_ptr<tt::hal::Device>> createDevices() {
         std::make_shared<TdeckKeyboard>(i2c_internal),
         std::make_shared<KeyboardBacklightDevice>(),
         std::make_shared<TrackballDevice>(),
+        createSdCard()
     };
 }
 

Devices/lilygo-tdeck/Source/Init.cpp

@@ -11,6 +11,7 @@
 
 static const auto LOGGER = tt::Logger("T-Deck");
 
+// Power on
 constexpr auto TDECK_POWERON_GPIO = GPIO_NUM_10;
 
 static bool powerOn() {

Devices/lilygo-tdeck/Source/devices/Sdcard.cpp

@@ -0,0 +1,34 @@
+#include "Sdcard.h"
+
+#include <tactility/device.h>
+
+#include <Tactility/lvgl/LvglSync.h>
+#include <Tactility/hal/sdcard/SpiSdCardDevice.h>
+
+using tt::hal::sdcard::SpiSdCardDevice;
+
+constexpr auto TDECK_SDCARD_PIN_CS = GPIO_NUM_39;
+constexpr auto TDECK_LCD_PIN_CS = GPIO_NUM_12;
+constexpr auto TDECK_RADIO_PIN_CS = GPIO_NUM_9;
+
+std::shared_ptr<SdCardDevice> createSdCard() {
+    auto configuration = std::make_unique<SpiSdCardDevice::Config>(
+        TDECK_SDCARD_PIN_CS,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        GPIO_NUM_NC,
+        SdCardDevice::MountBehaviour::AtBoot,
+        tt::lvgl::getSyncLock(),
+        std::vector {
+            TDECK_RADIO_PIN_CS,
+            TDECK_LCD_PIN_CS
+        }
+    );
+
+    ::Device* spiController = device_find_by_name("spi0");
+    check(spiController != nullptr, "spi0 not found");
+    return std::make_shared<SpiSdCardDevice>(
+        std::move(configuration),
+        spiController
+    );
+}

Devices/lilygo-tdeck/Source/devices/Sdcard.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include "Tactility/hal/sdcard/SdCardDevice.h"
+
+using tt::hal::sdcard::SdCardDevice;
+
+std::shared_ptr<SdCardDevice> createSdCard();