Device migrations, drivers and fixes

* **New Features**
  * Added native display support for ST7796, ILI9341, and ST7789 panels across supported boards.
  * Added FT5x06 and FT6x36 touchscreen support.
  * Generic PWM driver
  * ESP32 PWM driver
  * Generic RGB LED driver
  * RGB PWM LED driver
  * RGB GPIO LED driver
  * Implementation of RGB LED for various boards

* **Improvements**
  * Updated board hardware descriptions to use explicit display/touch/backlight device-tree bindings and disabled deprecated HAL usage.
  * Improved display and touch-driver cleanup to prevent stale resources and improve shutdown reliability.
  * Pinned esp-hosted library to a fixed version
 
* **Deletions**
  * Obsolete placeholder display
  * Legacy ILI9488 support.
  * ESP32-specific LEDC PWM implementation

Buildscripts/DevicetreeCompiler/source/generator.py

@@ -161,7 +161,7 @@ def resolve_parameters_from_bindings(device: Device, bindings: list[Binding], de
 
     node_name = get_device_node_name_safe(device)
     result = []
-    phandle_arrays = []
+    array_decls = []
     for binding_property in binding_properties:
         device_property = find_device_property(device, binding_property.name)
 
@@ -172,7 +172,35 @@ def resolve_parameters_from_bindings(device: Device, bindings: list[Binding], de
             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))
+                array_decls.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
+
+        if binding_property.type == "array":
+            # A flat literal array (DTS `[ ... ]` syntax, e.g. a byte blob), as opposed to
+            # phandle-array's list of resolved device references. Emits the same
+            # (pointer, length) parameter pair, backed by a plain data array instead of one
+            # holding phandle-derived initializers.
+            if binding_property.element_type is None:
+                raise DevicetreeException(f"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:
+                if device_property.type != "array":
+                    raise DevicetreeException(
+                        f"Device '{device.node_name}' property '{binding_property.name}' must use '[ ... ]' array syntax"
+                    )
+                entries = [str(value) for value in device_property.value]
+                array_decls.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:
@@ -207,17 +235,17 @@ def resolve_parameters_from_bindings(device: Device, bindings: list[Binding], de
             validate_property_range(device, binding_property, device_property.value)
             result.append(property_to_string(device_property, devices))
 
-    return result, phandle_arrays
+    return result, array_decls
 
 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)
+    config_params, array_decls = resolve_parameters_from_bindings(device, bindings, devices)
 
-    # Write phandle-array variables before the config struct
-    for array_var, element_type, entries in phandle_arrays:
+    # Write phandle-array/array variables before the config struct
+    for array_var, element_type, entries in array_decls:
         entries_str = ", ".join(entries)
         file.write(f"static {element_type} {array_var}[] = {{ {entries_str} }};\n")
 

Buildscripts/DevicetreeCompiler/tests/test_integration.py

@@ -209,6 +209,92 @@ def test_minmax_symbolic_value_skips_validation():
         print("PASSED")
         return True
 
+def write_array_config(tmp_dir, device_property_line):
+    config_dir = os.path.join(tmp_dir, "array_data")
+    bindings_dir = os.path.join(config_dir, "bindings")
+    os.makedirs(bindings_dir)
+
+    with open(os.path.join(config_dir, "devicetree.yaml"), "w") as f:
+        f.write("dts: test.dts\nbindings: bindings")
+
+    with open(os.path.join(config_dir, "test.dts"), "w") as f:
+        f.write(f"""/dts-v1/;
+
+/ {{
+    compatible = "test,root";
+    model = "Test Model";
+
+    test-device@0 {{
+        compatible = "test,array-device";
+        {device_property_line}
+    }};
+}};
+""")
+
+    with open(os.path.join(bindings_dir, "test,root.yaml"), "w") as f:
+        f.write("description: Test root binding\ncompatible: \"test,root\"\nproperties:\n  model:\n    type: string\n")
+
+    with open(os.path.join(bindings_dir, "test,array-device.yaml"), "w") as f:
+        f.write("""description: Test array binding
+compatible: "test,array-device"
+properties:
+  init-sequence:
+    type: array
+    element-type: uint8_t
+""")
+
+    return config_dir
+
+def test_array_property_generates_static_array_and_length():
+    print("Running test_array_property_generates_static_array_and_length...")
+    with tempfile.TemporaryDirectory() as tmp_dir:
+        config_dir = write_array_config(tmp_dir, "init-sequence = [0xFF 0x01 0x00 0x00 0x10 5 0];")
+        output_dir = os.path.join(tmp_dir, "output")
+        os.makedirs(output_dir)
+
+        result = run_compiler(config_dir, output_dir)
+
+        if result.returncode != 0:
+            print(f"FAILED: Compilation should have succeeded: {result.stderr} {result.stdout}")
+            return False
+
+        with open(os.path.join(output_dir, "devicetree.c")) as f:
+            generated = f.read()
+
+        if "static uint8_t test_device_init_sequence[] = { 0xFF, 0x01, 0x00, 0x00, 0x10, 5, 0 };" not in generated:
+            print(f"FAILED: Expected static array declaration not found:\n{generated}")
+            return False
+
+        if "(uint8_t*)test_device_init_sequence" not in generated or "\t7\n" not in generated:
+            print(f"FAILED: Expected (pointer, length) config params not found:\n{generated}")
+            return False
+
+        print("PASSED")
+        return True
+
+def test_array_property_defaults_to_null_when_absent():
+    print("Running test_array_property_defaults_to_null_when_absent...")
+    with tempfile.TemporaryDirectory() as tmp_dir:
+        config_dir = write_array_config(tmp_dir, "")
+        output_dir = os.path.join(tmp_dir, "output")
+        os.makedirs(output_dir)
+
+        result = run_compiler(config_dir, output_dir)
+
+        if result.returncode != 0:
+            print(f"FAILED: Compilation should have succeeded: {result.stderr} {result.stdout}")
+            return False
+
+        with open(os.path.join(output_dir, "devicetree.c")) as f:
+            generated = f.read()
+
+        if "NULL,\n\t0" not in generated:
+            print(f"FAILED: Expected NULL/0 defaults not found:\n{generated}")
+            return False
+
+        print("PASSED")
+        return True
+
 def test_compile_missing_config():
     print("Running test_compile_missing_config...")
     with tempfile.TemporaryDirectory() as output_dir:
@@ -234,7 +320,9 @@ if __name__ == "__main__":
         test_minmax_below_minimum_fails,
         test_minmax_above_maximum_fails,
         test_minmax_out_of_range_default_fails,
-        test_minmax_symbolic_value_skips_validation
+        test_minmax_symbolic_value_skips_validation,
+        test_array_property_generates_static_array_and_length,
+        test_array_property_defaults_to_null_when_absent
     ]
     
     failed = 0

Devices/btt-panda-touch/bigtreetech,panda-touch.dts

@@ -6,7 +6,8 @@
 #include <tactility/bindings/esp32_gpio.h>
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_usbhost.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
 #include <bindings/rgb_display.h>
 #include <bindings/gt911.h>
 
@@ -55,9 +56,17 @@
 		pin-scl = <&gpio0 3 GPIO_FLAG_NONE>;
 	};
 
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 21 GPIO_FLAG_NONE>;
+		period-ns = <33333>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
-		pin-backlight = <&gpio0 21 GPIO_FLAG_NONE>;
+		compatible = "pwm-backlight";
+		pwm = <&display_backlight_pwm>;
 	};
 
 	display0 {

Devices/cyd-2432s024c/CMakeLists.txt

@@ -2,5 +2,5 @@ file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    REQUIRES TactilityKernel driver
+    REQUIRES TactilityKernel
 )

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

@@ -6,7 +6,9 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <tactility/bindings/rgb_led_pwm.h>
 #include <bindings/ili9341.h>
 #include <bindings/cst816s.h>
 
@@ -39,13 +41,57 @@
 		};
 	};
 
+	rgb_led_channel_red {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 4 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+		inverted;
+	};
+
+	rgb_led_channel_green {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 16 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <1>;
+		inverted;
+	};
+
+	rgb_led_channel_blue {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 17 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <2>;
+		inverted;
+	};
+
+	rgb_led_pwm {
+		compatible = "rgb-led-pwm";
+		pwm-red = <&rgb_led_channel_red>;
+		pwm-green = <&rgb_led_channel_green>;
+		pwm-blue = <&rgb_led_channel_blue>;
+		// Default is red, and we want to reset it to off by default
+		default-color = <0 0 0>;
+		enabled;
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 27 GPIO_FLAG_NONE>;
+		period-ns = <1953125>;
+		ledc-timer = <1>;
+		ledc-channel = <3>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
+		compatible = "pwm-backlight";
 		// Off by default so display power-on won't show the screen from before the last power loss.
 		// The display backlight is turned on during the boot process.
 		status = "disabled";
-		pin-backlight = <&gpio0 27 GPIO_FLAG_NONE>;
-		frequency-hz = <512>;
+		pwm = <&display_backlight_pwm>;
 	};
 
 	spi0 {

Devices/cyd-2432s024c/source/module.cpp

@@ -1,20 +1,8 @@
 #include <tactility/module.h>
-#include <tactility/error.h>
-
-#include <driver/gpio.h>
 
 extern "C" {
 
 static error_t start() {
-    // Set the RGB LED pins to output and turn them off (0 on, 1 off)
-    gpio_set_direction(GPIO_NUM_4, GPIO_MODE_OUTPUT); // Red
-    gpio_set_direction(GPIO_NUM_16, GPIO_MODE_OUTPUT); // Green
-    gpio_set_direction(GPIO_NUM_17, GPIO_MODE_OUTPUT); // Blue
-
-    gpio_set_level(GPIO_NUM_4, 1); // Red
-    gpio_set_level(GPIO_NUM_16, 1); // Green
-    gpio_set_level(GPIO_NUM_17, 1); // Blue
-
     return ERROR_NONE;
 }
 

Devices/cyd-2432s024r/CMakeLists.txt

@@ -2,5 +2,5 @@ file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    REQUIRES TactilityKernel driver
+    REQUIRES TactilityKernel
 )

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

@@ -6,7 +6,9 @@
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_sdspi.h>
 #include <tactility/bindings/esp32_uart.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <tactility/bindings/rgb_led_pwm.h>
 #include <bindings/ili9341.h>
 #include <bindings/xpt2046.h>
 
@@ -24,13 +26,57 @@
         gpio-count = <40>;
     };
 
+	rgb_led_channel_red {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 4 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+		inverted;
+	};
+
+	rgb_led_channel_green {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 16 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <1>;
+		inverted;
+	};
+
+	rgb_led_channel_blue {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 17 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <2>;
+		inverted;
+	};
+
+	rgb_led_pwm {
+		compatible = "rgb-led-pwm";
+		pwm-red = <&rgb_led_channel_red>;
+		pwm-green = <&rgb_led_channel_green>;
+		pwm-blue = <&rgb_led_channel_blue>;
+		// Default is red, and we want to reset it to off by default
+		default-color = <0 0 0>;
+		enabled;
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 27 GPIO_FLAG_NONE>;
+		period-ns = <1953125>;
+		ledc-timer = <1>;
+		ledc-channel = <3>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
+		compatible = "pwm-backlight";
 		// Off by default so display power-on won't show the screen from before the last power loss.
 		// The display backlight is turned on during the boot process.
 		status = "disabled";
-		pin-backlight = <&gpio0 27 GPIO_FLAG_NONE>;
-		frequency-hz = <512>;
+		pwm = <&display_backlight_pwm>;
 	};
 
     spi0 {

Devices/cyd-2432s024r/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/cyd-2432s028r/CMakeLists.txt

@@ -2,5 +2,5 @@ file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    REQUIRES TactilityKernel driver
+    REQUIRES TactilityKernel
 )

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

@@ -7,7 +7,9 @@
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <tactility/bindings/rgb_led_pwm.h>
 #include <bindings/ili9341.h>
 #include <bindings/xpt2046_softspi.h>
 
@@ -33,13 +35,57 @@
 		pin-scl = <&gpio0 22 GPIO_FLAG_NONE>;
 	};
 
+	rgb_led_channel_red {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 4 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+		inverted;
+	};
+
+	rgb_led_channel_green {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 16 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <1>;
+		inverted;
+	};
+
+	rgb_led_channel_blue {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 17 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <2>;
+		inverted;
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 21 GPIO_FLAG_NONE>;
+		period-ns = <1953125>;
+		ledc-timer = <1>;
+		ledc-channel = <3>;
+	};
+
+	rgb_led_pwm {
+		compatible = "rgb-led-pwm";
+		pwm-red = <&rgb_led_channel_red>;
+		pwm-green = <&rgb_led_channel_green>;
+		pwm-blue = <&rgb_led_channel_blue>;
+		// Default is red, and we want to reset it to off by default
+		default-color = <0 0 0>;
+		enabled;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
+		compatible = "pwm-backlight";
 		// Off by default so display power-on won't show the screen from before the last power loss.
 		// The display backlight is turned on during the boot process.
 		status = "disabled";
-		pin-backlight = <&gpio0 21 GPIO_FLAG_NONE>;
-		frequency-hz = <512>;
+		pwm = <&display_backlight_pwm>;
 	};
 
 	touch {

Devices/cyd-2432s028r/source/module.cpp

@@ -1,25 +1,12 @@
 #include <tactility/module.h>
-#include <tactility/error.h>
-
-#include <driver/gpio.h>
 
 extern "C" {
 
 static error_t start() {
-    // Set the RGB LED pins to output and turn them off (0 on, 1 off)
-    gpio_set_direction(GPIO_NUM_4, GPIO_MODE_OUTPUT); // Red
-    gpio_set_direction(GPIO_NUM_16, GPIO_MODE_OUTPUT); // Green
-    gpio_set_direction(GPIO_NUM_17, GPIO_MODE_OUTPUT); // Blue
-
-    gpio_set_level(GPIO_NUM_4, 1); // Red
-    gpio_set_level(GPIO_NUM_16, 1); // Green
-    gpio_set_level(GPIO_NUM_17, 1); // Blue
-
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/cyd-2432s028rv3/CMakeLists.txt

@@ -2,5 +2,5 @@ file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    REQUIRES TactilityKernel driver
+    REQUIRES TactilityKernel
 )

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

@@ -7,7 +7,9 @@
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <tactility/bindings/rgb_led_pwm.h>
 #include <bindings/st7789.h>
 #include <bindings/xpt2046_softspi.h>
 
@@ -33,13 +35,57 @@
 		pin-scl = <&gpio0 22 GPIO_FLAG_NONE>;
 	};
 
+	rgb_led_channel_red {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 4 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+		inverted;
+	};
+
+	rgb_led_channel_green {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 16 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <1>;
+		inverted;
+	};
+
+	rgb_led_channel_blue {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 17 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <2>;
+		inverted;
+	};
+
+	rgb_led_pwm {
+		compatible = "rgb-led-pwm";
+		pwm-red = <&rgb_led_channel_red>;
+		pwm-green = <&rgb_led_channel_green>;
+		pwm-blue = <&rgb_led_channel_blue>;
+		// Default is red, and we want to reset it to off by default
+		default-color = <0 0 0>;
+		enabled;
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 21 GPIO_FLAG_NONE>;
+		period-ns = <1953125>;
+		ledc-timer = <1>;
+		ledc-channel = <3>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
+		compatible = "pwm-backlight";
 		// Off by default so display power-on won't show the screen from before the last power loss.
 		// The display backlight is turned on during the boot process.
 		status = "disabled";
-		pin-backlight = <&gpio0 21 GPIO_FLAG_NONE>;
-		frequency-hz = <512>;
+		pwm = <&display_backlight_pwm>;
 	};
 
 	touch {

Devices/cyd-2432s028rv3/source/module.cpp

@@ -1,20 +1,8 @@
 #include <tactility/module.h>
-#include <tactility/error.h>
-
-#include <driver/gpio.h>
 
 extern "C" {
 
 static error_t start() {
-    // Set the RGB LED pins to output and turn them off (0 on, 1 off)
-    gpio_set_direction(GPIO_NUM_4, GPIO_MODE_OUTPUT); // Red
-    gpio_set_direction(GPIO_NUM_16, GPIO_MODE_OUTPUT); // Green
-    gpio_set_direction(GPIO_NUM_17, GPIO_MODE_OUTPUT); // Blue
-
-    gpio_set_level(GPIO_NUM_4, 1); // Red
-    gpio_set_level(GPIO_NUM_16, 1); // Green
-    gpio_set_level(GPIO_NUM_17, 1); // Blue
-
     return ERROR_NONE;
 }
 

Devices/cyd-2432s032c/CMakeLists.txt

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

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

@@ -1,47 +0,0 @@
-#include "devices/Display.h"
-#include <driver/gpio.h>
-
-#include <Tactility/hal/Configuration.h>
-#include <Tactility/SystemEvents.h>
-#include <Tactility/lvgl/LvglSync.h>
-
-#include <PwmBacklight.h>
-
-static bool init_boot() {
-    if (!driver::pwmbacklight::init(LCD_PIN_BACKLIGHT)) {
-        return false;
-    }
-
-    // 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
-    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
-
-    // This display has a weird glitch with gamma during boot, which results in uneven dark gray colours.
-    // Setting gamma curve index to 0 doesn't work at boot for an unknown reason, so we set the curve index to 1:
-    tt::kernel::subscribeSystemEvent(tt::kernel::SystemEvent::BootSplash, [](auto) {
-        auto display = tt::hal::findFirstDevice<tt::hal::display::DisplayDevice>(tt::hal::Device::Type::Display);
-        assert(display != nullptr);
-        tt::lvgl::lock(portMAX_DELAY);
-        display->setGammaCurve(1U);
-        tt::lvgl::unlock();
-    });
-
-    return true;
-}
-
-static tt::hal::DeviceVector createDevices() {
-    return {
-        createDisplay(),
-    };
-}
-
-extern const tt::hal::Configuration hardwareConfiguration = {
-    .initBoot = init_boot,
-    .createDevices = createDevices
-};

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

@@ -1,48 +0,0 @@
-#include "Display.h"
-
-#include <Gt911Touch.h>
-#include <Ili934xDisplay.h>
-#include <PwmBacklight.h>
-#include <tactility/check.h>
-#include <tactility/device.h>
-
-static std::shared_ptr<tt::hal::touch::TouchDevice> createTouch() {
-    auto* i2c = device_find_by_name("i2c0");
-    check(i2c);
-    auto configuration = std::make_unique<Gt911Touch::Configuration>(
-        i2c,
-        LCD_HORIZONTAL_RESOLUTION,
-        LCD_VERTICAL_RESOLUTION
-    );
-
-    return std::make_shared<Gt911Touch>(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 = true,
-        .mirrorY = true,
-        .invertColor = true,
-        .swapBytes = true,
-        .bufferSize = LCD_BUFFER_SIZE,
-        .touch = createTouch(),
-        .backlightDutyFunction = driver::pwmbacklight::setBacklightDuty,
-        .resetPin = GPIO_NUM_NC,
-        .rgbElementOrder = LCD_RGB_ELEMENT_ORDER_RGB
-    };
-
-    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/cyd-2432s032c/Source/devices/Display.h

@@ -1,18 +0,0 @@
-#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_15;
-constexpr auto LCD_PIN_DC = GPIO_NUM_2;
-constexpr auto LCD_PIN_BACKLIGHT = GPIO_NUM_27;
-constexpr auto LCD_HORIZONTAL_RESOLUTION = 240;
-constexpr auto LCD_VERTICAL_RESOLUTION = 320;
-constexpr auto LCD_BUFFER_HEIGHT = LCD_VERTICAL_RESOLUTION / 10;
-constexpr auto LCD_BUFFER_SIZE = LCD_HORIZONTAL_RESOLUTION * LCD_BUFFER_HEIGHT;
-
-std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay();

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

@@ -6,7 +6,11 @@
 #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/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <tactility/bindings/rgb_led_pwm.h>
+#include <bindings/ili9341.h>
+#include <bindings/gt911.h>
 
 / {
 	compatible = "root";
@@ -28,6 +32,66 @@
 		clock-frequency = <400000>;
 		pin-sda = <&gpio0 33 GPIO_FLAG_NONE>;
 		pin-scl = <&gpio0 32 GPIO_FLAG_NONE>;
+
+		touch {
+			compatible = "goodix,gt911";
+			reg = <0x5D>;
+			x-max = <240>;
+			y-max = <320>;
+		};
+	};
+
+	rgb_led_channel_red {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 4 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+		inverted;
+	};
+
+	rgb_led_channel_green {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 16 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <1>;
+		inverted;
+	};
+
+	rgb_led_channel_blue {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 17 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <2>;
+		inverted;
+	};
+
+	rgb_led_pwm {
+		compatible = "rgb-led-pwm";
+		pwm-red = <&rgb_led_channel_red>;
+		pwm-green = <&rgb_led_channel_green>;
+		pwm-blue = <&rgb_led_channel_blue>;
+		// Default is red, and we want to reset it to off by default
+		default-color = <0 0 0>;
+		enabled;
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 27 GPIO_FLAG_NONE>;
+		period-ns = <33333>;
+		ledc-timer = <1>;
+		ledc-channel = <3>;
+	};
+
+	display_backlight {
+		compatible = "pwm-backlight";
+		// Off by default so display power-on won't show the screen from before the last power loss.
+		// The display backlight is turned on during the boot process.
+		status = "disabled";
+		pwm = <&display_backlight_pwm>;
 	};
 
 	spi0 {
@@ -36,9 +100,20 @@
 		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";
+
+		display@0 {
+			compatible = "ilitek,ili9341";
+			horizontal-resolution = <240>;
+			vertical-resolution = <320>;
+			swap-xy;
+			mirror-x;
+			mirror-y;
+			invert-color;
+			// Curve 0 doesn't apply cleanly at boot on this panel (uneven dark-gray gamma glitch); 1 does.
+			gamma-curve = <1>;
+			pixel-clock-hz = <40000000>;
+			pin-dc = <&gpio0 2 GPIO_FLAG_NONE>;
+			backlight = <&display_backlight>;
 		};
 	};
 

Devices/cyd-2432s032c/device.properties

@@ -7,6 +7,8 @@ hardware.target=ESP32
 hardware.flashSize=4MB
 hardware.spiRam=false
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=SD
 
 display.size=3.2"

Devices/cyd-2432s032c/devicetree.yaml

@@ -1,3 +1,5 @@
 dependencies:
   - Platforms/platform-esp32
+  - Drivers/ili9341-module
+  - Drivers/gt911-module
 dts: cyd,2432s032c.dts

Devices/cyd-2432s032c/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/cyd-3248s035c/CMakeLists.txt

@@ -1,7 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility esp_lvgl_port ST7796 GT911 PwmBacklight driver vfs fatfs
+    REQUIRES TactilityKernel
 )

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

@@ -1,32 +0,0 @@
-#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
-};

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

@@ -1,40 +0,0 @@
-#include "Display.h"
-
-#include <Gt911Touch.h>
-#include <PwmBacklight.h>
-#include <St7796Display.h>
-#include <tactility/check.h>
-#include <tactility/device.h>
-
-static std::shared_ptr<tt::hal::touch::TouchDevice> createTouch() {
-    auto* i2c = device_find_by_name("i2c0");
-    check(i2c);
-    auto configuration = std::make_unique<Gt911Touch::Configuration>(
-        i2c,
-        LCD_HORIZONTAL_RESOLUTION,
-        LCD_VERTICAL_RESOLUTION
-    );
-
-    return std::make_shared<Gt911Touch>(std::move(configuration));
-}
-
-std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay() {
-    auto touch = createTouch();
-    auto configuration = std::make_unique<St7796Display::Configuration>(
-        LCD_SPI_HOST,
-        LCD_PIN_CS,
-        LCD_PIN_DC,
-        LCD_HORIZONTAL_RESOLUTION,
-        LCD_VERTICAL_RESOLUTION,
-        touch,
-        false,
-        true,
-        false,
-        false
-    );
-
-    configuration->backlightDutyFunction = driver::pwmbacklight::setBacklightDuty;
-
-    auto display = std::make_shared<St7796Display>(std::move(configuration));
-    return std::reinterpret_pointer_cast<tt::hal::display::DisplayDevice>(display);
-}

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

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

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

@@ -7,7 +7,11 @@
 #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/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <tactility/bindings/rgb_led_pwm.h>
+#include <bindings/st7796.h>
+#include <bindings/gt911.h>
 
 / {
 	compatible = "root";
@@ -29,6 +33,13 @@
 		clock-frequency = <400000>;
 		pin-sda = <&gpio0 33 GPIO_FLAG_NONE>;
 		pin-scl = <&gpio0 32 GPIO_FLAG_NONE>;
+
+		touch {
+			compatible = "goodix,gt911";
+			reg = <0x5D>;
+			x-max = <320>;
+			y-max = <480>;
+		};
 	};
 
 	// CN1 header
@@ -40,15 +51,74 @@
 		pin-scl = <&gpio0 22 GPIO_FLAG_NONE>;
 	};
 
+	rgb_led_channel_red {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 4 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+		inverted;
+	};
+
+	rgb_led_channel_green {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 16 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <1>;
+		inverted;
+	};
+
+	rgb_led_channel_blue {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 17 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <2>;
+		inverted;
+	};
+
+	rgb_led_pwm {
+		compatible = "rgb-led-pwm";
+		pwm-red = <&rgb_led_channel_red>;
+		pwm-green = <&rgb_led_channel_green>;
+		pwm-blue = <&rgb_led_channel_blue>;
+		// Default is red, and we want to reset it to off by default
+		default-color = <0 0 0>;
+		enabled;
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 27 GPIO_FLAG_NONE>;
+		period-ns = <33333>;
+		ledc-timer = <1>;
+		ledc-channel = <3>;
+	};
+
+	display_backlight {
+		compatible = "pwm-backlight";
+		// Off by default so display power-on won't show the screen from before the last power loss.
+		// The display backlight is turned on during the boot process.
+		status = "disabled";
+		pwm = <&display_backlight_pwm>;
+	};
+
 	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";
+
+		display@0 {
+			compatible = "sitronix,st7796";
+			horizontal-resolution = <320>;
+			vertical-resolution = <480>;
+			mirror-x;
+			pixel-clock-hz = <80000000>;
+			pin-dc = <&gpio0 2 GPIO_FLAG_NONE>;
+			backlight = <&display_backlight>;
 		};
 	};
 

Devices/cyd-3248s035c/device.properties

@@ -7,6 +7,8 @@ hardware.target=ESP32
 hardware.flashSize=4MB
 hardware.spiRam=false
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=SD
 
 display.size=3.5"

Devices/cyd-3248s035c/devicetree.yaml

@@ -1,3 +1,5 @@
 dependencies:
-- Platforms/platform-esp32
+  - Platforms/platform-esp32
+  - Drivers/st7796-module
+  - Drivers/gt911-module
 dts: cyd,3248s035c.dts

Devices/cyd-3248s035c/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/cyd-4848s040c/CMakeLists.txt

@@ -1,7 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility EspLcdCompat esp_lcd_st7701 esp_lcd_panel_io_additions GT911 PwmBacklight driver vfs fatfs
+    REQUIRES TactilityKernel driver
 )

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

@@ -1,21 +0,0 @@
-#include "devices/St7701Display.h"
-
-#include <Tactility/hal/Configuration.h>
-#include <PwmBacklight.h>
-
-using namespace tt::hal;
-
-static bool initBoot() {
-    return driver::pwmbacklight::init(GPIO_NUM_38, 1000);
-}
-
-static DeviceVector createDevices() {
-    return {
-        std::make_shared<St7701Display>(),
-    };
-}
-
-extern const Configuration hardwareConfiguration = {
-    .initBoot = initBoot,
-    .createDevices = createDevices
-};

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

@@ -1,260 +0,0 @@
-#include "St7701Display.h"
-
-#include <Gt911Touch.h>
-#include <PwmBacklight.h>
-#include <tactility/check.h>
-#include <tactility/device.h>
-#include <tactility/log.h>
-
-#include <driver/gpio.h>
-#include <esp_err.h>
-#include <esp_lcd_panel_rgb.h>
-#include <esp_lcd_panel_ops.h>
-#include <esp_lcd_panel_io_additions.h>
-#include <esp_lcd_st7701.h>
-#include <esp_rom_gpio.h>
-#include <soc/spi_periph.h>
-#include <driver/spi_master.h>
-
-constexpr auto* TAG = "St7701Display";
-
-// GPIO47/48 are physically shared between this bit-banged 3-wire command bus
-// and the SD card's real SPI2 bus (no alternate pins exist on this PCB).
-// esp_lcd_new_panel_io_3wire_spi() reconfigures them as plain GPIO via
-// gpio_config(), severing their SPI2 matrix routing. Reconnect them here once
-// the vendor init sequence is done, so SD card reads/writes keep working.
-// Safe only because nothing else calls into the ST7701 IO handle after boot.
-static void reclaimSpiPinsForSdCard() {
-    esp_rom_gpio_connect_out_signal(GPIO_NUM_47, spi_periph_signal[SPI2_HOST].spid_out, false, false);
-    esp_rom_gpio_connect_out_signal(GPIO_NUM_48, spi_periph_signal[SPI2_HOST].spiclk_out, false, false);
-    gpio_set_direction(GPIO_NUM_47, GPIO_MODE_OUTPUT);
-    gpio_set_direction(GPIO_NUM_48, GPIO_MODE_OUTPUT);
-}
-
-static const st7701_lcd_init_cmd_t st7701_lcd_init_cmds[] = {
-    //  {cmd, { data }, data_size, delay_ms}
-    {0xFF, (uint8_t[]) {0x77, 0x01, 0x00, 0x00, 0x10}, 5, 0},
-    {0xC0, (uint8_t[]) {0x3B, 0x00}, 2, 0},
-    {0xC1, (uint8_t[]) {0x0D, 0x02}, 2, 0},
-    {0xC2, (uint8_t[]) {0x31, 0x05}, 2, 0},
-    {0xCD, (uint8_t[]) {0x00}, 1, 0}, //0x08
-    //Positive Voltage Gamma Control
-    {0xB0, (uint8_t[]) {0x00, 0x11, 0x18, 0x0E, 0x11, 0x06, 0x07, 0x08, 0x07, 0x22, 0x04, 0x12, 0x0F, 0xAA, 0x31, 0x18}, 16, 0},
-    //Negative Voltage Gamma Control
-    {0xB1, (uint8_t[]) {0x00, 0x11, 0x19, 0x0E, 0x12, 0x07, 0x08, 0x08, 0x08, 0x22, 0x04, 0x11, 0x11, 0xA9, 0x32, 0x18}, 16, 0},
-    //Page1
-    {0xFF, (uint8_t[]) {0x77, 0x01, 0x00, 0x00, 0x11}, 5, 0},
-    {0xB0, (uint8_t[]) {0x60}, 1, 0}, //Vop=4.7375v
-    {0xB1, (uint8_t[]) {0x32}, 1, 0}, //VCOM=32
-    {0xB2, (uint8_t[]) {0x07}, 1, 0}, //VGH=15v
-    {0xB3, (uint8_t[]) {0x80}, 1, 0},
-    {0xB5, (uint8_t[]) {0x49}, 1, 0}, //VGL=-10.17v
-    {0xB7, (uint8_t[]) {0x85}, 1, 0},
-    {0xB8, (uint8_t[]) {0x21}, 1, 0}, //AVDD=6.6 & AVCL=-4.6
-    {0xC1, (uint8_t[]) {0x78}, 1, 0},
-    {0xC2, (uint8_t[]) {0x78}, 1, 0},
-    {0xE0, (uint8_t[]) {0x00, 0x1B, 0x02}, 3, 0},
-    {0xE1, (uint8_t[]) {0x08, 0xA0, 0x00, 0x00, 0x07, 0xA0, 0x00, 0x00, 0x00, 0x44, 0x44}, 11, 0},
-    {0xE2, (uint8_t[]) {0x11, 0x11, 0x44, 0x44, 0xED, 0xA0, 0x00, 0x00, 0xEC, 0xA0, 0x00, 0x00}, 12, 0},
-    {0xE3, (uint8_t[]) {0x00, 0x00, 0x11, 0x11}, 4, 0},
-    {0xE4, (uint8_t[]) {0x44, 0x44}, 2, 0},
-    {0xE5, (uint8_t[]) {0x0A, 0xE9, 0xD8, 0xA0, 0x0C, 0xEB, 0xD8, 0xA0, 0x0E, 0xED, 0xD8, 0xA0, 0x10, 0xEF, 0xD8, 0xA0}, 16, 0},
-    {0xE6, (uint8_t[]) {0x00, 0x00, 0x11, 0x11}, 4, 0},
-    {0xE7, (uint8_t[]) {0x44, 0x44}, 2, 0},
-    {0xE8, (uint8_t[]) {0x09, 0xE8, 0xD8, 0xA0, 0x0B, 0xEA, 0xD8, 0xA0, 0x0D, 0xEC, 0xD8, 0xA0, 0x0F, 0xEE, 0xD8, 0xA0}, 16, 0},
-    {0xEB, (uint8_t[]) {0x02, 0x00, 0xE4, 0xE4, 0x88, 0x00, 0x40}, 7, 0},
-    {0xEC, (uint8_t[]) {0x3C, 0x00}, 2, 0},
-    {0xED, (uint8_t[]) {0xAB, 0x89, 0x76, 0x54, 0x02, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0x20, 0x45, 0x67, 0x98, 0xBA}, 16, 0},
-    //VAP & VAN
-    {0xFF, (uint8_t[]) {0x77, 0x01, 0x00, 0x00, 0x13}, 5, 0},
-    {0xE5, (uint8_t[]) {0xE4}, 1, 0},
-    {0xFF, (uint8_t[]) {0x77, 0x01, 0x00, 0x00, 0x00}, 5, 0},
-    {0x3A, (uint8_t[]) {0x60}, 1, 10}, //0x70 RGB888, 0x60 RGB666, 0x50 RGB565
-    {0x11, (uint8_t[]) {0x00}, 0, 120}, //Sleep Out
-    {0x29, (uint8_t[]) {0x00}, 0, 0}, //Display On
-};
-
-bool St7701Display::createIoHandle(esp_lcd_panel_io_handle_t& outHandle) {
-    spi_line_config_t line_config = {
-        .cs_io_type = IO_TYPE_GPIO,
-        .cs_gpio_num = GPIO_NUM_39,
-        .scl_io_type = IO_TYPE_GPIO,
-        .scl_gpio_num = GPIO_NUM_48,
-        .sda_io_type = IO_TYPE_GPIO,
-        .sda_gpio_num = GPIO_NUM_47,
-        .io_expander = nullptr,
-    };
-    esp_lcd_panel_io_3wire_spi_config_t panel_io_config = ST7701_PANEL_IO_3WIRE_SPI_CONFIG(line_config, 0);
-    return esp_lcd_new_panel_io_3wire_spi(&panel_io_config, &outHandle) == ESP_OK;
-}
-
-bool St7701Display::createPanelHandle(esp_lcd_panel_io_handle_t ioHandle, esp_lcd_panel_handle_t& panelHandle) {
-    const esp_lcd_rgb_panel_config_t rgb_config = {
-        .clk_src = LCD_CLK_SRC_DEFAULT,
-        .timings = {
-            .pclk_hz = 14000000,
-            .h_res = 480,
-            .v_res = 480,
-            .hsync_pulse_width = 10,
-            .hsync_back_porch = 10,
-            .hsync_front_porch = 20,
-            .vsync_pulse_width = 10,
-            .vsync_back_porch = 10,
-            .vsync_front_porch = 10,
-            .flags = {
-                .hsync_idle_low = false,
-                .vsync_idle_low = false,
-                .de_idle_high = false,
-                .pclk_active_neg = false,
-                .pclk_idle_high = false
-            }
-        },
-        .data_width = 16,
-        .bits_per_pixel = 16,
-        .num_fbs = 2,
-        .bounce_buffer_size_px = 480 * 10,
-        .sram_trans_align = 8,
-        .psram_trans_align = 64,
-        .hsync_gpio_num = GPIO_NUM_16,
-        .vsync_gpio_num = GPIO_NUM_17,
-        .de_gpio_num = GPIO_NUM_18,
-        .pclk_gpio_num = GPIO_NUM_21,
-        .disp_gpio_num = GPIO_NUM_NC,
-        .data_gpio_nums = {
-            GPIO_NUM_4, // B1
-            GPIO_NUM_5, // B2
-            GPIO_NUM_6, // B3
-            GPIO_NUM_7, // B4
-            GPIO_NUM_15, // B5
-            GPIO_NUM_8, // G1
-            GPIO_NUM_20, // G2
-            GPIO_NUM_3, // G3
-            GPIO_NUM_46, // G4
-            GPIO_NUM_9, // G5
-            GPIO_NUM_10, // G6
-            GPIO_NUM_11, // R1
-            GPIO_NUM_12, // R2
-            GPIO_NUM_13, // R3
-            GPIO_NUM_14, // R4
-            GPIO_NUM_0 // R5
-        },
-        .flags = {
-            .disp_active_low = false,
-            .refresh_on_demand = false,
-            .fb_in_psram = true,
-            .double_fb = true,
-            .no_fb = false,
-            .bb_invalidate_cache = true
-        }
-    };
-
-    st7701_vendor_config_t vendor_config = {
-        .init_cmds = st7701_lcd_init_cmds,
-        .init_cmds_size = sizeof(st7701_lcd_init_cmds) / sizeof(st7701_lcd_init_cmd_t),
-        .rgb_config = &rgb_config,
-        .flags = {
-            .use_mipi_interface = 0,
-            .mirror_by_cmd = 1,
-            .auto_del_panel_io = 0,
-        },
-    };
-
-    const esp_lcd_panel_dev_config_t panel_config = {
-        .reset_gpio_num = GPIO_NUM_NC,
-        .rgb_ele_order = LCD_RGB_ELEMENT_ORDER_RGB,
-        .data_endian = LCD_RGB_DATA_ENDIAN_LITTLE,
-        .bits_per_pixel = 16,
-        .flags = {
-            .reset_active_high = false,
-        },
-        .vendor_config = &vendor_config,
-    };
-
-    if (esp_lcd_new_panel_st7701(ioHandle, &panel_config, &panelHandle) != ESP_OK) {
-        LOG_E(TAG, "Failed to create panel");
-        return false;
-    }
-
-    if (esp_lcd_panel_reset(panelHandle) != ESP_OK) {
-        LOG_E(TAG, "Failed to reset panel");
-        return false;
-    }
-
-    if (esp_lcd_panel_init(panelHandle) != ESP_OK) {
-        LOG_E(TAG, "Failed to init panel");
-        return false;
-    }
-
-    if (esp_lcd_panel_invert_color(panelHandle, false) != ESP_OK) {
-        LOG_E(TAG, "Failed to invert color");
-        return false;
-    }
-
-    esp_lcd_panel_set_gap(panelHandle, 0, 0);
-
-    if (esp_lcd_panel_disp_on_off(panelHandle, true) != ESP_OK) {
-        LOG_E(TAG, "Failed to turn display on");
-        return false;
-    }
-
-    reclaimSpiPinsForSdCard();
-
-    return true;
-}
-
-lvgl_port_display_cfg_t St7701Display::getLvglPortDisplayConfig(esp_lcd_panel_io_handle_t ioHandle, esp_lcd_panel_handle_t panelHandle) {
-    return {
-        .io_handle = ioHandle,
-        .panel_handle = panelHandle,
-        .control_handle = nullptr,
-        .buffer_size = (480 * 480),
-        .double_buffer = true,
-        .trans_size = 0,
-        .hres = 480,
-        .vres = 480,
-        .monochrome = false,
-        .rotation = {
-            .swap_xy = false,
-            .mirror_x = false,
-            .mirror_y = false,
-        },
-        .color_format = LV_COLOR_FORMAT_RGB565,
-        .flags = {
-            .buff_dma = false,
-            .buff_spiram = true,
-            .sw_rotate = false,
-            .swap_bytes = false,
-            .full_refresh = false,
-            .direct_mode = false
-        }
-    };
-}
-
-lvgl_port_display_rgb_cfg_t St7701Display::getLvglPortDisplayRgbConfig(esp_lcd_panel_io_handle_t ioHandle, esp_lcd_panel_handle_t panelHandle) {
-    return {
-        .flags = {
-            .bb_mode = true,
-            .avoid_tearing = false
-        }
-    };
-}
-
-std::shared_ptr<tt::hal::touch::TouchDevice> St7701Display::getTouchDevice() {
-    if (touchDevice == nullptr) {
-        auto* i2c = device_find_by_name("i2c0");
-        check(i2c);
-        auto configuration = std::make_unique<Gt911Touch::Configuration>(
-           i2c,
-           480,
-           480
-       );
-
-        touchDevice = std::make_shared<Gt911Touch>(std::move(configuration));
-    }
-
-    return touchDevice;
-}
-
-void St7701Display::setBacklightDuty(uint8_t backlightDuty) {
-    driver::pwmbacklight::setBacklightDuty(backlightDuty);
-}

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

@@ -1,36 +0,0 @@
-#pragma once
-
-#include <EspLcdDisplay.h>
-#include <lvgl.h>
-
-class St7701Display final : public EspLcdDisplay {
-
-    std::shared_ptr<tt::hal::touch::TouchDevice> touchDevice;
-
-    bool createIoHandle(esp_lcd_panel_io_handle_t& outHandle) override;
-
-    bool createPanelHandle(esp_lcd_panel_io_handle_t ioHandle, esp_lcd_panel_handle_t& panelHandle) override;
-
-    lvgl_port_display_cfg_t getLvglPortDisplayConfig(esp_lcd_panel_io_handle_t ioHandle, esp_lcd_panel_handle_t panelHandle) override;
-
-    bool isRgbPanel() const override { return true; }
-
-    lvgl_port_display_rgb_cfg_t getLvglPortDisplayRgbConfig(esp_lcd_panel_io_handle_t ioHandle, esp_lcd_panel_handle_t panelHandle) override;
-
-public:
-
-    St7701Display() : EspLcdDisplay() {}
-
-    std::string getName() const override { return "ST7701S"; }
-
-    std::string getDescription() const override { return "ST7701S RGB display"; }
-
-    std::shared_ptr<tt::hal::touch::TouchDevice> getTouchDevice() override;
-
-    void setBacklightDuty(uint8_t backlightDuty) override;
-
-    bool supportsBacklightDuty() const override { return true; }
-
-    // TODO: Find out why it crashes
-    bool supportsDisplayDriver() const override { return false; }
-};

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

@@ -5,8 +5,12 @@
 #include <tactility/bindings/esp32_wifi_pinned.h>
 #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/esp32_spi.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <bindings/st7701.h>
+#include <bindings/gt911.h>
 
 / {
 	compatible = "root";
@@ -33,5 +37,137 @@
 		clock-frequency = <400000>;
 		pin-sda = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-scl = <&gpio0 45 GPIO_FLAG_NONE>;
+
+		touch0 {
+			// No reset/interrupt pin wired up here, matching the original deprecated-HAL
+			// config (Gt911Touch::Configuration was constructed with no reset/interrupt pin
+			// arguments on this board).
+			compatible = "goodix,gt911";
+			reg = <0x5D>;
+			x-max = <480>;
+			y-max = <480>;
+		};
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 38 GPIO_FLAG_NONE>;
+		period-ns = <1000000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+	};
+
+	display_backlight {
+		compatible = "pwm-backlight";
+		pwm = <&display_backlight_pwm>;
+	};
+
+	display0 {
+		compatible = "sitronix,st7701";
+		horizontal-resolution = <480>;
+		vertical-resolution = <480>;
+		pixel-clock-hz = <14000000>;
+		hsync-pulse-width = <10>;
+		hsync-back-porch = <10>;
+		hsync-front-porch = <20>;
+		vsync-pulse-width = <10>;
+		vsync-back-porch = <10>;
+		vsync-front-porch = <10>;
+		num-fbs = <2>;
+		double-fb;
+		bounce-buffer-size-px = <4800>;
+		bb-invalidate-cache;
+		pin-hsync = <&gpio0 16 GPIO_FLAG_NONE>;
+		pin-vsync = <&gpio0 17 GPIO_FLAG_NONE>;
+		pin-de = <&gpio0 18 GPIO_FLAG_NONE>;
+		pin-pclk = <&gpio0 21 GPIO_FLAG_NONE>;
+		pin-data0 = <&gpio0 4 GPIO_FLAG_NONE>; // B1
+		pin-data1 = <&gpio0 5 GPIO_FLAG_NONE>; // B2
+		pin-data2 = <&gpio0 6 GPIO_FLAG_NONE>; // B3
+		pin-data3 = <&gpio0 7 GPIO_FLAG_NONE>; // B4
+		pin-data4 = <&gpio0 15 GPIO_FLAG_NONE>; // B5
+		pin-data5 = <&gpio0 8 GPIO_FLAG_NONE>; // G1
+		pin-data6 = <&gpio0 20 GPIO_FLAG_NONE>; // G2
+		pin-data7 = <&gpio0 3 GPIO_FLAG_NONE>; // G3
+		pin-data8 = <&gpio0 46 GPIO_FLAG_NONE>; // G4
+		pin-data9 = <&gpio0 9 GPIO_FLAG_NONE>; // G5
+		pin-data10 = <&gpio0 10 GPIO_FLAG_NONE>; // G6
+		pin-data11 = <&gpio0 11 GPIO_FLAG_NONE>; // R1
+		pin-data12 = <&gpio0 12 GPIO_FLAG_NONE>; // R2
+		pin-data13 = <&gpio0 13 GPIO_FLAG_NONE>; // R3
+		pin-data14 = <&gpio0 14 GPIO_FLAG_NONE>; // R4
+		pin-data15 = <&gpio0 0 GPIO_FLAG_NONE>; // R5
+
+		// 3-wire bit-banged command bus. GPIO47/48 are physically shared with this board's SD
+		// card SPI2 bus (MOSI/SCLK) - no alternate pins exist on this PCB. No SD card device is
+		// declared in this devicetree yet (CS/MISO pins are unverified), but if one is added on
+		// SPI2_HOST later, declare it *after* this node: starting that SPI bus unconditionally
+		// reprograms the GPIO matrix for its MOSI/SCLK pins (see esp32_spi.cpp's start()),
+		// which reclaims them from this display's bit-banged use without any extra code.
+		pin-cs = <&gpio0 39 GPIO_FLAG_NONE>;
+		pin-scl = <&gpio0 48 GPIO_FLAG_NONE>;
+		pin-sda = <&gpio0 47 GPIO_FLAG_NONE>;
+		mirror-by-cmd;
+
+		init-sequence = [
+			// Vendor bring-up sequence for this panel. Framing is
+			// [cmd, data-len, delay-ms, data-len bytes of data...] - see st7701-module's
+			// init-sequence binding property for the encoding.
+			0xFF 5  0    0x77 0x01 0x00 0x00 0x10
+			0xC0 2  0    0x3B 0x00
+			0xC1 2  0    0x0D 0x02
+			0xC2 2  0    0x31 0x05
+			0xCD 1  0    0x00
+			// Positive Voltage Gamma Control
+			0xB0 16 0    0x00 0x11 0x18 0x0E 0x11 0x06 0x07 0x08 0x07 0x22 0x04 0x12 0x0F 0xAA 0x31 0x18
+			// Negative Voltage Gamma Control
+			0xB1 16 0    0x00 0x11 0x19 0x0E 0x12 0x07 0x08 0x08 0x08 0x22 0x04 0x11 0x11 0xA9 0x32 0x18
+			// Page1
+			0xFF 5  0    0x77 0x01 0x00 0x00 0x11
+			0xB0 1  0    0x60 // Vop=4.7375v
+			0xB1 1  0    0x32 // VCOM=32
+			0xB2 1  0    0x07 // VGH=15v
+			0xB3 1  0    0x80
+			0xB5 1  0    0x49 // VGL=-10.17v
+			0xB7 1  0    0x85
+			0xB8 1  0    0x21 // AVDD=6.6 & AVCL=-4.6
+			0xC1 1  0    0x78
+			0xC2 1  0    0x78
+			0xE0 3  0    0x00 0x1B 0x02
+			0xE1 11 0    0x08 0xA0 0x00 0x00 0x07 0xA0 0x00 0x00 0x00 0x44 0x44
+			0xE2 12 0    0x11 0x11 0x44 0x44 0xED 0xA0 0x00 0x00 0xEC 0xA0 0x00 0x00
+			0xE3 4  0    0x00 0x00 0x11 0x11
+			0xE4 2  0    0x44 0x44
+			0xE5 16 0    0x0A 0xE9 0xD8 0xA0 0x0C 0xEB 0xD8 0xA0 0x0E 0xED 0xD8 0xA0 0x10 0xEF 0xD8 0xA0
+			0xE6 4  0    0x00 0x00 0x11 0x11
+			0xE7 2  0    0x44 0x44
+			0xE8 16 0    0x09 0xE8 0xD8 0xA0 0x0B 0xEA 0xD8 0xA0 0x0D 0xEC 0xD8 0xA0 0x0F 0xEE 0xD8 0xA0
+			0xEB 7  0    0x02 0x00 0xE4 0xE4 0x88 0x00 0x40
+			0xEC 2  0    0x3C 0x00
+			0xED 16 0    0xAB 0x89 0x76 0x54 0x02 0xFF 0xFF 0xFF 0xFF 0xFF 0xFF 0x20 0x45 0x67 0x98 0xBA
+			// VAP & VAN
+			0xFF 5  0    0x77 0x01 0x00 0x00 0x13
+			0xE5 1  0    0xE4
+			0xFF 5  0    0x77 0x01 0x00 0x00 0x00
+			0x3A 1  10   0x60 // 0x70 RGB888, 0x60 RGB666, 0x50 RGB565
+			0x11 0  120       // Sleep Out
+			0x29 0  0         // Display On
+		];
+
+		backlight = <&display_backlight>;
+	};
+
+	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>;
+		};
 	};
 };

Devices/cyd-4848s040c/device.properties

@@ -10,6 +10,8 @@ hardware.spiRamMode=OCT
 hardware.spiRamSpeed=80M
 hardware.bluetooth=true
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=SD
 
 display.size=4"

Devices/cyd-4848s040c/devicetree.yaml

@@ -1,3 +1,5 @@
 dependencies:
   - Platforms/platform-esp32
+  - Drivers/st7701-module
+  - Drivers/gt911-module
 dts: cyd,4848s040c.dts

Devices/cyd-4848s040c/source/module.cpp

@@ -3,16 +3,14 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 
-Module cyd_4848s040c_module = {
+struct Module cyd_4848s040c_module = {
     .name = "cyd-4848s040c",
     .start = start,
     .stop = stop,

Devices/cyd-8048s043c/CMakeLists.txt

@@ -1,7 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility esp_lvgl_port esp_lcd RgbDisplay GT911 PwmBacklight driver vfs fatfs
+    REQUIRES TactilityKernel driver
 )

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

@@ -1,22 +0,0 @@
-#include "PwmBacklight.h"
-#include "devices/Display.h"
-
-#include <Tactility/hal/Configuration.h>
-
-using namespace tt::hal;
-
-static bool initBoot() {
-    // Display backlight
-    return driver::pwmbacklight::init(GPIO_NUM_2, 200);
-}
-
-static DeviceVector createDevices() {
-    return {
-        createDisplay(),
-    };
-}
-
-extern const Configuration hardwareConfiguration = {
-    .initBoot = initBoot,
-    .createDevices = createDevices
-};

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

@@ -1,108 +0,0 @@
-#include "Display.h"
-
-#include <Gt911Touch.h>
-#include <PwmBacklight.h>
-#include <RgbDisplay.h>
-#include <tactility/check.h>
-#include <tactility/device.h>
-
-std::shared_ptr<tt::hal::touch::TouchDevice> createTouch() {
-    // Note for future changes: Reset pin is 38 and interrupt pin is 18
-    // or INT = NC, schematic and other info floating around is kinda conflicting...
-    auto* i2c = device_find_by_name("i2c_internal");
-    check(i2c);
-    auto configuration = std::make_unique<Gt911Touch::Configuration>(
-        i2c,
-        800,
-        480
-    );
-
-    return std::make_shared<Gt911Touch>(std::move(configuration));
-}
-
-std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay() {
-    auto touch = createTouch();
-
-    constexpr uint32_t bufferPixels = 800 * 10;
-
-    esp_lcd_rgb_panel_config_t rgb_panel_config = {
-        .clk_src = LCD_CLK_SRC_DEFAULT,
-        .timings = {
-            .pclk_hz = 16000000,
-            .h_res = 800,
-            .v_res = 480,
-            .hsync_pulse_width = 4,
-            .hsync_back_porch = 8,
-            .hsync_front_porch = 8,
-            .vsync_pulse_width = 4,
-            .vsync_back_porch = 8,
-            .vsync_front_porch = 8,
-            .flags = {
-                .hsync_idle_low = false,
-                .vsync_idle_low = false,
-                .de_idle_high = false,
-                .pclk_active_neg = true,
-                .pclk_idle_high = false
-            }
-        },
-        .data_width = 16,
-        .bits_per_pixel = 0,
-        .num_fbs = 2,
-        .bounce_buffer_size_px = bufferPixels,
-        .sram_trans_align = 8,
-        .psram_trans_align = 64,
-        .hsync_gpio_num = GPIO_NUM_39,
-        .vsync_gpio_num = GPIO_NUM_41,
-        .de_gpio_num = GPIO_NUM_40 ,
-        .pclk_gpio_num = GPIO_NUM_42,
-        .disp_gpio_num = GPIO_NUM_NC,
-        .data_gpio_nums = {
-            GPIO_NUM_8, // B3
-            GPIO_NUM_3, // B4
-            GPIO_NUM_46, // B5
-            GPIO_NUM_9, // B6
-            GPIO_NUM_1, // B7
-            GPIO_NUM_5, // G2
-            GPIO_NUM_6, // G3
-            GPIO_NUM_7, // G4
-            GPIO_NUM_15, // G5
-            GPIO_NUM_16, // G6
-            GPIO_NUM_4, // G7
-            GPIO_NUM_45, // R3
-            GPIO_NUM_48, // R4
-            GPIO_NUM_47, // R5
-            GPIO_NUM_21, // R6
-            GPIO_NUM_14, // R7
-        },
-        .flags = {
-            .disp_active_low = false,
-            .refresh_on_demand = false,
-            .fb_in_psram = true,
-            .double_fb = true,
-            .no_fb = false,
-            .bb_invalidate_cache = false
-        }
-    };
-
-    RgbDisplay::BufferConfiguration buffer_config = {
-        .size = (800 * 480),
-        .useSpi = true,
-        .doubleBuffer = true,
-        .bounceBufferMode = true,
-        .avoidTearing = false
-    };
-
-    auto configuration = std::make_unique<RgbDisplay::Configuration>(
-        rgb_panel_config,
-        buffer_config,
-        touch,
-        LV_COLOR_FORMAT_RGB565,
-        false,
-        false,
-        false,
-        false,
-        driver::pwmbacklight::setBacklightDuty
-    );
-
-    return std::make_shared<RgbDisplay>(std::move(configuration));
-}

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

@@ -1,5 +0,0 @@
-#pragma once
-
-#include "Tactility/hal/display/DisplayDevice.h"
-
-std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay();

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

@@ -8,6 +8,10 @@
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_sdspi.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <bindings/rgb_display.h>
+#include <bindings/gt911.h>
 
 / {
 	compatible = "root";
@@ -34,6 +38,16 @@
 		clock-frequency = <400000>;
 		pin-sda = <&gpio0 19 GPIO_FLAG_NONE>;
 		pin-scl = <&gpio0 20 GPIO_FLAG_NONE>;
+
+		touch0 {
+			// Reset pin 38 and interrupt pin 18 (or INT = NC) exist on the board but are not
+			// wired up here - conflicting schematic info, unverified (matches the original
+			// deprecated-HAL config).
+			compatible = "goodix,gt911";
+			reg = <0x5D>;
+			x-max = <800>;
+			y-max = <480>;
+		};
 	};
 
 	i2c_external {
@@ -51,13 +65,64 @@
 		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>;
 		};
 	};
 
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 2 GPIO_FLAG_NONE>;
+		period-ns = <33333>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+	};
+
+	display_backlight {
+		compatible = "pwm-backlight";
+		pwm = <&display_backlight_pwm>;
+	};
+
+	display0 {
+		compatible = "espressif,esp32-rgb-display";
+		horizontal-resolution = <800>;
+		vertical-resolution = <480>;
+		pixel-clock-hz = <16000000>;
+		hsync-pulse-width = <4>;
+		hsync-back-porch = <8>;
+		hsync-front-porch = <8>;
+		vsync-pulse-width = <4>;
+		vsync-back-porch = <8>;
+		vsync-front-porch = <8>;
+		pclk-active-neg;
+		num-fbs = <2>;
+		double-fb;
+		bounce-buffer-size-px = <8000>;
+		pin-hsync = <&gpio0 39 GPIO_FLAG_NONE>;
+		pin-vsync = <&gpio0 41 GPIO_FLAG_NONE>;
+		pin-de = <&gpio0 40 GPIO_FLAG_NONE>;
+		pin-pclk = <&gpio0 42 GPIO_FLAG_NONE>;
+		pin-data0 = <&gpio0 8 GPIO_FLAG_NONE>; // B3
+		pin-data1 = <&gpio0 3 GPIO_FLAG_NONE>; // B4
+		pin-data2 = <&gpio0 46 GPIO_FLAG_NONE>; // B5
+		pin-data3 = <&gpio0 9 GPIO_FLAG_NONE>; // B6
+		pin-data4 = <&gpio0 1 GPIO_FLAG_NONE>; // B7
+		pin-data5 = <&gpio0 5 GPIO_FLAG_NONE>; // G2
+		pin-data6 = <&gpio0 6 GPIO_FLAG_NONE>; // G3
+		pin-data7 = <&gpio0 7 GPIO_FLAG_NONE>; // G4
+		pin-data8 = <&gpio0 15 GPIO_FLAG_NONE>; // G5
+		pin-data9 = <&gpio0 16 GPIO_FLAG_NONE>; // G6
+		pin-data10 = <&gpio0 4 GPIO_FLAG_NONE>; // G7
+		pin-data11 = <&gpio0 45 GPIO_FLAG_NONE>; // R3
+		pin-data12 = <&gpio0 48 GPIO_FLAG_NONE>; // R4
+		pin-data13 = <&gpio0 47 GPIO_FLAG_NONE>; // R5
+		pin-data14 = <&gpio0 21 GPIO_FLAG_NONE>; // R6
+		pin-data15 = <&gpio0 14 GPIO_FLAG_NONE>; // R7
+		backlight = <&display_backlight>;
+	};
+
 	uart1 {
 		compatible = "espressif,esp32-uart";
 		status = "disabled";

Devices/cyd-8048s043c/device.properties

@@ -12,6 +12,8 @@ hardware.spiRamSpeed=80M
 hardware.esptoolFlashFreq=80M
 hardware.bluetooth=true
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=SD
 
 display.size=4.3"

Devices/cyd-8048s043c/devicetree.yaml

@@ -1,3 +1,5 @@
 dependencies:
   - Platforms/platform-esp32
+  - Drivers/rgb-display-module
+  - Drivers/gt911-module
 dts: cyd,8048s043c.dts

Devices/cyd-8048s043c/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/cyd-e32r28t/CMakeLists.txt

@@ -2,5 +2,5 @@ file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    REQUIRES TactilityKernel driver
+    REQUIRES TactilityKernel
 )

Devices/cyd-e32r28t/cyd,e32r28t.dts

@@ -5,7 +5,9 @@
 #include <tactility/bindings/esp32_gpio.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <tactility/bindings/rgb_led_pwm.h>
 #include <bindings/ili9341.h>
 #include <bindings/xpt2046_softspi.h>
 
@@ -23,13 +25,58 @@
 		gpio-count = <40>;
 	};
 
+
+	rgb_led_channel_red {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 22 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+		inverted;
+	};
+
+	rgb_led_channel_green {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 16 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <1>;
+		inverted;
+	};
+
+	rgb_led_channel_blue {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 17 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <2>;
+		inverted;
+	};
+
+	rgb_led_pwm {
+		compatible = "rgb-led-pwm";
+		pwm-red = <&rgb_led_channel_red>;
+		pwm-green = <&rgb_led_channel_green>;
+		pwm-blue = <&rgb_led_channel_blue>;
+		// Default is red, and we want to reset it to off by default
+		default-color = <0 0 0>;
+		enabled;
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 21 GPIO_FLAG_NONE>;
+		period-ns = <1953125>;
+		ledc-timer = <1>;
+		ledc-channel = <3>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
+		compatible = "pwm-backlight";
 		// Off by default so display power-on won't show the screen from before the last power loss.
 		// The display backlight is turned on during the boot process.
 		status = "disabled";
-		pin-backlight = <&gpio0 21 GPIO_FLAG_NONE>;
-		frequency-hz = <512>;
+		pwm = <&display_backlight_pwm>;
 	};
 
 	touch {

Devices/cyd-e32r28t/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/cyd-e32r32p/CMakeLists.txt

@@ -2,5 +2,5 @@ file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    REQUIRES TactilityKernel driver
+    REQUIRES TactilityKernel
 )

Devices/cyd-e32r32p/cyd,e32r32p.dts

@@ -8,7 +8,9 @@
 #include <tactility/bindings/esp32_i2c.h>
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <tactility/bindings/rgb_led_pwm.h>
 #include <bindings/st7789.h>
 #include <bindings/xpt2046.h>
 
@@ -40,6 +42,43 @@
 		channels = <ADC_CHANNEL_6 ADC_ATTEN_DB_12 ADC_BITWIDTH_DEFAULT>;
 	};
 
+	rgb_led_channel_red {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 22 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+		inverted;
+	};
+
+	rgb_led_channel_green {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 16 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <1>;
+		inverted;
+	};
+
+	rgb_led_channel_blue {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 17 GPIO_FLAG_NONE>;
+		period-ns = <500000>;
+		ledc-timer = <0>;
+		ledc-channel = <2>;
+		inverted;
+	};
+
+	rgb_led_pwm {
+		compatible = "rgb-led-pwm";
+		pwm-red = <&rgb_led_channel_red>;
+		pwm-green = <&rgb_led_channel_green>;
+		pwm-blue = <&rgb_led_channel_blue>;
+		// Default is red, and we want to reset it to off by default
+		default-color = <0 0 0>;
+		enabled;
+	};
+
 	// Matches the deprecated HAL's old ChargeFromAdcVoltage config: adcMultiplier=2.11,
 	// adcRefVoltage=3.3 (default), voltageMin/Max=3.2/4.2 (default, same as battery-sense's own
 	// fixed curve - see TactilityKernel/source/drivers/battery_sense.cpp).
@@ -50,13 +89,20 @@
 		multiplier = <2110>;
 	};
 
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 27 GPIO_FLAG_NONE>;
+		period-ns = <25000>;
+		ledc-timer = <1>;
+		ledc-channel = <3>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
+		compatible = "pwm-backlight";
 		// Off by default so display power-on won't show the screen from before the last power loss.
 		// The display backlight is turned on during the boot process.
 		status = "disabled";
-		pin-backlight = <&gpio0 27 GPIO_FLAG_NONE>;
-		frequency-hz = <40000>;
+		pwm = <&display_backlight_pwm>;
 	};
 
 	spi0 {

Devices/cyd-e32r32p/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/elecrow-crowpanel-advance-28/CMakeLists.txt

@@ -1,7 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility FT5x06 ST7789 PwmBacklight driver
+    REQUIRES TactilityKernel
 )

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

@@ -1,21 +0,0 @@
-#include "PwmBacklight.h"
-#include "devices/Display.h"
-
-#include <Tactility/hal/Configuration.h>
-
-using namespace tt::hal;
-
-static bool initBoot() {
-    return driver::pwmbacklight::init(GPIO_NUM_38);
-}
-
-static DeviceVector createDevices() {
-    return {
-        createDisplay(),
-    };
-}
-
-extern const Configuration hardwareConfiguration = {
-    .initBoot = initBoot,
-    .createDevices = createDevices
-};

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

@@ -1,46 +0,0 @@
-#include "Display.h"
-
-#include <Ft5x06Touch.h>
-#include <PwmBacklight.h>
-#include <St7789Display.h>
-
-static std::shared_ptr<tt::hal::touch::TouchDevice> createTouch() {
-    auto configuration = std::make_unique<Ft5x06Touch::Configuration>(
-        I2C_NUM_0,
-        LCD_HORIZONTAL_RESOLUTION,
-        LCD_VERTICAL_RESOLUTION,
-        false,
-        false,
-        false
-    );
-
-    return std::make_shared<Ft5x06Touch>(std::move(configuration));
-}
-
-std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay() {
-    St7789Display::Configuration panel_configuration = {
-        .horizontalResolution = LCD_HORIZONTAL_RESOLUTION,
-        .verticalResolution = LCD_VERTICAL_RESOLUTION,
-        .gapX = 0,
-        .gapY = 0,
-        .swapXY = false,
-        .mirrorX = false,
-        .mirrorY = false,
-        .invertColor = true,
-        .bufferSize = LCD_BUFFER_SIZE,
-        .touch = createTouch(),
-        .backlightDutyFunction = driver::pwmbacklight::setBacklightDuty,
-        .resetPin = GPIO_NUM_NC,
-        .lvglSwapBytes = false
-    };
-
-    auto spi_configuration = std::make_shared<St7789Display::SpiConfiguration>(St7789Display::SpiConfiguration {
-        .spiHostDevice = LCD_SPI_HOST,
-        .csPin = LCD_PIN_CS,
-        .dcPin = LCD_PIN_DC,
-        .pixelClockFrequency = 62'500'000,
-        .transactionQueueDepth = 10
-    });
-
-    return std::make_shared<St7789Display>(panel_configuration, spi_configuration);
-}

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

@@ -1,16 +0,0 @@
-#pragma once
-
-#include <Tactility/hal/display/DisplayDevice.h>
-
-#include <driver/gpio.h>
-#include <driver/spi_common.h>
-
-constexpr auto LCD_SPI_HOST = SPI2_HOST;
-constexpr auto LCD_PIN_CS = GPIO_NUM_40;
-constexpr auto LCD_PIN_DC = GPIO_NUM_41; // RS
-constexpr auto LCD_HORIZONTAL_RESOLUTION = 240;
-constexpr auto LCD_VERTICAL_RESOLUTION = 320;
-constexpr auto LCD_BUFFER_HEIGHT = LCD_VERTICAL_RESOLUTION / 10;
-constexpr auto LCD_BUFFER_SIZE = LCD_HORIZONTAL_RESOLUTION * LCD_BUFFER_HEIGHT;
-
-std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay();

Devices/elecrow-crowpanel-advance-28/device.properties

@@ -12,6 +12,8 @@ hardware.tinyUsb=true
 hardware.esptoolFlashFreq=120M
 hardware.bluetooth=true
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=SD
 
 display.size=2.8"

Devices/elecrow-crowpanel-advance-28/devicetree.yaml

@@ -1,3 +1,5 @@
 dependencies:
   - Platforms/platform-esp32
+  - Drivers/st7789-module
+  - Drivers/ft5x06-module
 dts: elecrow,crowpanel-advance-28.dts

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

@@ -8,7 +8,10 @@
 #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/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <bindings/st7789.h>
+#include <bindings/ft5x06.h>
 
 / {
 	compatible = "root";
@@ -35,6 +38,26 @@
 		clock-frequency = <400000>;
 		pin-sda = <&gpio0 15 GPIO_FLAG_NONE>;
 		pin-scl = <&gpio0 16 GPIO_FLAG_NONE>;
+
+		touch0 {
+			compatible = "focaltech,ft5x06";
+			reg = <0x38>;
+			x-max = <240>;
+			y-max = <320>;
+		};
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 38 GPIO_FLAG_NONE>;
+		period-ns = <33333>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+	};
+
+	display_backlight {
+		compatible = "pwm-backlight";
+		pwm = <&display_backlight_pwm>;
 	};
 
 	spi0 {
@@ -43,9 +66,15 @@
 		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";
+
+		display@0 {
+			compatible = "sitronix,st7789";
+			horizontal-resolution = <240>;
+			vertical-resolution = <320>;
+			invert-color;
+			pixel-clock-hz = <62500000>;
+			pin-dc = <&gpio0 41 GPIO_FLAG_NONE>;
+			backlight = <&display_backlight>;
 		};
 	};
 

Devices/elecrow-crowpanel-advance-28/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/elecrow-crowpanel-advance-35/CMakeLists.txt

@@ -2,5 +2,5 @@ file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    REQUIRES TactilityKernel driver
+    REQUIRES TactilityKernel
 )

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

@@ -8,7 +8,8 @@
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
 #include <bindings/ili9488.h>
 #include <bindings/gt911.h>
 
@@ -46,13 +47,20 @@
 		};
 	};
 
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 38 GPIO_FLAG_NONE>;
+		period-ns = <1953125>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
+		compatible = "pwm-backlight";
 		// Off by default so display power-on won't show the screen from before the last power loss.
 		// The display backlight is turned on during the boot process.
 		status = "disabled";
-		pin-backlight = <&gpio0 38 GPIO_FLAG_NONE>;
-		frequency-hz = <512>;
+		pwm = <&display_backlight_pwm>;
 	};
 
 	spi0 {

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

@@ -8,7 +8,7 @@
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_gpio_backlight.h>
+#include <tactility/bindings/gpio_backlight.h>
 #include <bindings/rgb_display.h>
 #include <bindings/gt911.h>
 #include <bindings/tca9534.h>
@@ -70,8 +70,8 @@
 	};
 
 	display_backlight {
-		compatible = "espressif,esp32-gpio-backlight";
-		pin-backlight = <&io_expander0 1 GPIO_FLAG_NONE>;
+		compatible = "gpio-backlight";
+		pin = <&io_expander0 1 GPIO_FLAG_NONE>;
 	};
 
 	display0 {

Devices/elecrow-crowpanel-basic-28/CMakeLists.txt

@@ -2,5 +2,5 @@ file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    REQUIRES TactilityKernel driver
+    REQUIRES TactilityKernel
 )

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

@@ -7,7 +7,8 @@
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
 #include <bindings/ili9341.h>
 #include <bindings/xpt2046.h>
 
@@ -33,13 +34,20 @@
 		pin-scl = <&gpio0 21 GPIO_FLAG_NONE>;
 	};
 
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 27 GPIO_FLAG_NONE>;
+		period-ns = <25000>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
+		compatible = "pwm-backlight";
 		// Off by default so display power-on won't show the screen from before the last power loss.
 		// The display backlight is turned on during the boot process.
 		status = "disabled";
-		pin-backlight = <&gpio0 27 GPIO_FLAG_NONE>;
-		frequency-hz = <40000>;
+		pwm = <&display_backlight_pwm>;
 	};
 
 	spi0 {

Devices/elecrow-crowpanel-basic-28/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/elecrow-crowpanel-basic-35/CMakeLists.txt

@@ -2,5 +2,5 @@ file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    REQUIRES TactilityKernel driver
+    REQUIRES TactilityKernel
 )

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

@@ -7,7 +7,8 @@
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
 #include <bindings/ili9488.h>
 #include <bindings/xpt2046.h>
 
@@ -33,13 +34,20 @@
 		pin-scl = <&gpio0 21 GPIO_FLAG_NONE>;
 	};
 
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 27 GPIO_FLAG_NONE>;
+		period-ns = <1953125>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
+		compatible = "pwm-backlight";
 		// Off by default so display power-on won't show the screen from before the last power loss.
 		// The display backlight is turned on during the boot process.
 		status = "disabled";
-		pin-backlight = <&gpio0 27 GPIO_FLAG_NONE>;
-		frequency-hz = <512>;
+		pwm = <&display_backlight_pwm>;
 	};
 
 	spi0 {

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

@@ -8,7 +8,8 @@
 #include <tactility/bindings/esp32_spi.h>
 #include <tactility/bindings/esp32_uart.h>
 #include <tactility/bindings/esp32_sdspi.h>
-#include <tactility/bindings/esp32_ledc_backlight.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
 #include <bindings/rgb_display.h>
 #include <bindings/gt911.h>
 
@@ -63,10 +64,17 @@
 		};
 	};
 
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 2 GPIO_FLAG_NONE>;
+		period-ns = <1953125>;
+		ledc-timer = <0>;
+		ledc-channel = <0>;
+	};
+
 	display_backlight {
-		compatible = "espressif,esp32-ledc-backlight";
-		pin-backlight = <&gpio0 2 GPIO_FLAG_NONE>;
-		frequency-hz = <512>;
+		compatible = "pwm-backlight";
+		pwm = <&display_backlight_pwm>;
 	};
 
 	display0 {

Devices/generic-esp32/CMakeLists.txt

@@ -1,7 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility
+    REQUIRES TactilityKernel
 )

Devices/generic-esp32/Source/Configuration.cpp

@@ -1,3 +0,0 @@
-#include <Tactility/hal/Configuration.h>
-
-extern const tt::hal::Configuration hardwareConfiguration = {};

Devices/generic-esp32/device.properties

@@ -7,4 +7,6 @@ hardware.target=ESP32
 hardware.flashSize=8MB
 hardware.spiRam=false
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=Internal

Devices/generic-esp32/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/generic-esp32c6/CMakeLists.txt

@@ -1,7 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility
+    REQUIRES TactilityKernel
 )

Devices/generic-esp32c6/Source/Configuration.cpp

@@ -1,3 +0,0 @@
-#include <Tactility/hal/Configuration.h>
-
-extern const tt::hal::Configuration hardwareConfiguration = {};

Devices/generic-esp32c6/device.properties

@@ -7,4 +7,6 @@ hardware.target=ESP32C6
 hardware.flashSize=8MB
 hardware.spiRam=false
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=Internal

Devices/generic-esp32c6/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/generic-esp32p4/CMakeLists.txt

@@ -1,7 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility
+    REQUIRES TactilityKernel
 )

Devices/generic-esp32p4/Source/Configuration.cpp

@@ -1,3 +0,0 @@
-#include <Tactility/hal/Configuration.h>
-
-extern const tt::hal::Configuration hardwareConfiguration = {};

Devices/generic-esp32p4/device.properties

@@ -7,4 +7,6 @@ hardware.target=ESP32P4
 hardware.flashSize=8MB
 hardware.spiRam=false
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=Internal

Devices/generic-esp32p4/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/generic-esp32s3/CMakeLists.txt

@@ -1,7 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility
+    REQUIRES TactilityKernel
 )

Devices/generic-esp32s3/Source/Configuration.cpp

@@ -1,3 +0,0 @@
-#include <Tactility/hal/Configuration.h>
-
-extern const tt::hal::Configuration hardwareConfiguration = {};

Devices/generic-esp32s3/device.properties

@@ -7,4 +7,6 @@ hardware.target=ESP32S3
 hardware.flashSize=8MB
 hardware.spiRam=false
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=Internal

Devices/generic-esp32s3/source/module.cpp

@@ -3,12 +3,10 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 

Devices/guition-jc1060p470ciwy/CMakeLists.txt

@@ -1,8 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility esp_lvgl_port esp_lcd EspLcdCompat esp_lcd_jd9165 GT911 PwmBacklight driver vfs fatfs
-    PRIV_REQUIRES esp_adc EstimatedPower
+    REQUIRES TactilityKernel
 )

Devices/guition-jc1060p470ciwy/Source/Configuration.cpp

@@ -1,17 +0,0 @@
-#include "devices/Display.h"
-#include "devices/Power.h"
-
-#include <Tactility/hal/Configuration.h>
-
-using namespace tt::hal;
-
-static DeviceVector createDevices() {
-    return {
-        createDisplay(),
-        createPower()
-    };
-}
-
-extern const Configuration hardwareConfiguration = {
-    .createDevices = createDevices,
-};

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

@@ -1,66 +0,0 @@
-#include "Display.h"
-#include "Jd9165Display.h"
-
-#include <Gt911Touch.h>
-#include <PwmBacklight.h>
-#include <Tactility/Mutex.h>
-#include <tactility/check.h>
-#include <tactility/device.h>
-#include <tactility/log.h>
-
-constexpr auto LCD_PIN_RESET = GPIO_NUM_0;  // Match P4 EV board reset line
-constexpr auto LCD_PIN_BACKLIGHT = GPIO_NUM_23;
-constexpr auto LCD_HORIZONTAL_RESOLUTION = 1024;
-constexpr auto LCD_VERTICAL_RESOLUTION = 600;
-
-constexpr auto TOUCH_PIN_RESET = GPIO_NUM_NC;
-constexpr auto TOUCH_PIN_INTERRUPT = GPIO_NUM_NC;
-
-static std::shared_ptr<tt::hal::touch::TouchDevice> createTouch() {
-    auto* i2c = device_find_by_name("i2c_internal");
-    check(i2c);
-    auto configuration = std::make_unique<Gt911Touch::Configuration>(
-        i2c,
-        LCD_HORIZONTAL_RESOLUTION,
-        LCD_VERTICAL_RESOLUTION,
-        false,  // swapXY
-        false,  // mirrorX
-        false,  // mirrorY
-        TOUCH_PIN_RESET,
-        TOUCH_PIN_INTERRUPT
-    );
-
-    return std::make_shared<Gt911Touch>(std::move(configuration));
-}
-
-std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay() {
-    // Initialize PWM backlight
-    if (!driver::pwmbacklight::init(LCD_PIN_BACKLIGHT, 20000, LEDC_TIMER_1, LEDC_CHANNEL_0)) {
-        LOG_W("jc1060p470ciwy", "Failed to initialize backlight");
-    }
-
-    auto touch = createTouch();
-
-    auto configuration = std::make_shared<EspLcdConfiguration>(EspLcdConfiguration {
-        .horizontalResolution = LCD_HORIZONTAL_RESOLUTION,
-        .verticalResolution = LCD_VERTICAL_RESOLUTION,
-        .gapX = 0,
-        .gapY = 0,
-        .monochrome = false,
-        .swapXY = false,
-        .mirrorX = false,
-        .mirrorY = false,
-        .invertColor = false,
-        .bufferSize = 0, // 0 = default (1/10 of screen)
-        .touch = touch,
-        .backlightDutyFunction = driver::pwmbacklight::setBacklightDuty,
-        .resetPin = LCD_PIN_RESET,
-        .lvglColorFormat = LV_COLOR_FORMAT_RGB565,
-        .lvglSwapBytes = false,
-        .rgbElementOrder = LCD_RGB_ELEMENT_ORDER_RGB,
-        .bitsPerPixel = 16
-    });
-
-    const auto display = std::make_shared<Jd9165Display>(configuration);
-    return std::static_pointer_cast<tt::hal::display::DisplayDevice>(display);
-}

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

@@ -1,203 +0,0 @@
-#include "Jd9165Display.h"
-
-#include <tactility/log.h>
-
-#include <esp_lcd_jd9165.h>
-
-constexpr auto* TAG = "JD9165";
-
-// MIPI DSI PHY power configuration
-#define MIPI_DSI_PHY_PWR_LDO_CHAN 3  // LDO_VO3 connects to VDD_MIPI_DPHY
-#define MIPI_DSI_PHY_PWR_LDO_VOLTAGE_MV 2500
-
-// JD9165 initialization commands from ESP32-P4 Function EV Board
-// Delays set to match the reference sequence exactly.
-static const jd9165_lcd_init_cmd_t jd9165_init_cmds[] = {
-    {0x30, (uint8_t[]){0x00}, 1, 0},
-    {0xF7, (uint8_t[]){0x49,0x61,0x02,0x00}, 4, 0},
-    {0x30, (uint8_t[]){0x01}, 1, 0},
-    {0x04, (uint8_t[]){0x0C}, 1, 0},
-    {0x05, (uint8_t[]){0x00}, 1, 0},
-    {0x06, (uint8_t[]){0x00}, 1, 0},
-    {0x0B, (uint8_t[]){0x11}, 1, 0},
-    {0x17, (uint8_t[]){0x00}, 1, 0},
-    {0x20, (uint8_t[]){0x04}, 1, 0},
-    {0x1F, (uint8_t[]){0x05}, 1, 0},
-    {0x23, (uint8_t[]){0x00}, 1, 0},
-    {0x25, (uint8_t[]){0x19}, 1, 0},
-    {0x28, (uint8_t[]){0x18}, 1, 0},
-    {0x29, (uint8_t[]){0x04}, 1, 0},
-    {0x2A, (uint8_t[]){0x01}, 1, 0},
-    {0x2B, (uint8_t[]){0x04}, 1, 0},
-    {0x2C, (uint8_t[]){0x01}, 1, 0},
-    {0x30, (uint8_t[]){0x02}, 1, 0},
-    {0x01, (uint8_t[]){0x22}, 1, 0},
-    {0x03, (uint8_t[]){0x12}, 1, 0},
-    {0x04, (uint8_t[]){0x00}, 1, 0},
-    {0x05, (uint8_t[]){0x64}, 1, 0},
-    {0x0A, (uint8_t[]){0x08}, 1, 0},
-    {0x0B, (uint8_t[]){0x0A,0x1A,0x0B,0x0D,0x0D,0x11,0x10,0x06,0x08,0x1F,0x1D}, 11, 0},
-    {0x0C, (uint8_t[]){0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D}, 11, 0},
-    {0x0D, (uint8_t[]){0x16,0x1B,0x0B,0x0D,0x0D,0x11,0x10,0x07,0x09,0x1E,0x1C}, 11, 0},
-    {0x0E, (uint8_t[]){0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D}, 11, 0},
-    {0x0F, (uint8_t[]){0x16,0x1B,0x0D,0x0B,0x0D,0x11,0x10,0x1C,0x1E,0x09,0x07}, 11, 0},
-    {0x10, (uint8_t[]){0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D}, 11, 0},
-    {0x11, (uint8_t[]){0x0A,0x1A,0x0D,0x0B,0x0D,0x11,0x10,0x1D,0x1F,0x08,0x06}, 11, 0},
-    {0x12, (uint8_t[]){0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D,0x0D}, 11, 0},
-    {0x14, (uint8_t[]){0x00,0x00,0x11,0x11}, 4, 0},
-    {0x18, (uint8_t[]){0x99}, 1, 0},
-    {0x30, (uint8_t[]){0x06}, 1, 0},
-    {0x12, (uint8_t[]){0x36,0x2C,0x2E,0x3C,0x38,0x35,0x35,0x32,0x2E,0x1D,0x2B,0x21,0x16,0x29}, 14, 0},
-    {0x13, (uint8_t[]){0x36,0x2C,0x2E,0x3C,0x38,0x35,0x35,0x32,0x2E,0x1D,0x2B,0x21,0x16,0x29}, 14, 0},
-    {0x30, (uint8_t[]){0x0A}, 1, 0},
-    {0x02, (uint8_t[]){0x4F}, 1, 0},
-    {0x0B, (uint8_t[]){0x40}, 1, 0},
-    {0x12, (uint8_t[]){0x3E}, 1, 0},
-    {0x13, (uint8_t[]){0x78}, 1, 0},
-    {0x30, (uint8_t[]){0x0D}, 1, 0},
-    {0x0D, (uint8_t[]){0x04}, 1, 0},
-    {0x10, (uint8_t[]){0x0C}, 1, 0},
-    {0x11, (uint8_t[]){0x0C}, 1, 0},
-    {0x12, (uint8_t[]){0x0C}, 1, 0},
-    {0x13, (uint8_t[]){0x0C}, 1, 0},
-    {0x30, (uint8_t[]){0x00}, 1, 0},
-    {0X3A, (uint8_t[]){0x55}, 1, 0},
-    {0x11, (uint8_t[]){0x00}, 1, 120},
-    {0x29, (uint8_t[]){0x00}, 1, 20},
-};
-
-Jd9165Display::~Jd9165Display() {
-    // TODO: This should happen during ::stop(), but this isn't currently exposed
-    if (mipiDsiBus != nullptr) {
-        esp_lcd_del_dsi_bus(mipiDsiBus);
-        mipiDsiBus = nullptr;
-    }
-    if (ldoChannel != nullptr) {
-        esp_ldo_release_channel(ldoChannel);
-        ldoChannel = nullptr;
-    }
-}
-
-bool Jd9165Display::createMipiDsiBus() {
-    // Enable MIPI DSI PHY power (transition from "no power" to "shutdown" state)
-    esp_ldo_channel_config_t ldo_mipi_phy_config = {
-        .chan_id = MIPI_DSI_PHY_PWR_LDO_CHAN,
-        .voltage_mv = MIPI_DSI_PHY_PWR_LDO_VOLTAGE_MV,
-        .flags = {}
-    };
-    
-    if (esp_ldo_acquire_channel(&ldo_mipi_phy_config, &ldoChannel) != ESP_OK) {
-        LOG_E(TAG, "Failed to acquire LDO channel for MIPI DSI PHY");
-        return false;
-    }
-
-    LOG_I(TAG, "MIPI DSI PHY powered on");
-
-    // Create MIPI DSI bus
-    // TODO: use MIPI_DSI_PHY_CLK_SRC_DEFAULT() in future ESP-IDF 6.0.0 update with esp_lcd_jd9165 library version 2.x
-    const esp_lcd_dsi_bus_config_t bus_config = {
-        .bus_id = 0,
-        .num_data_lanes = 2,
-        .phy_clk_src = MIPI_DSI_PHY_CLK_SRC_DEFAULT,
-        .lane_bit_rate_mbps = 750
-    };
-
-    if (esp_lcd_new_dsi_bus(&bus_config, &mipiDsiBus) != ESP_OK) {
-        LOG_E(TAG, "Failed to create MIPI DSI bus");
-        return false;
-    }
-
-    LOG_I(TAG, "MIPI DSI bus created");
-    return true;
-}
-
-bool Jd9165Display::createIoHandle(esp_lcd_panel_io_handle_t& ioHandle) {
-    // Initialize MIPI DSI bus if not already done
-    if (mipiDsiBus == nullptr) {
-        if (!createMipiDsiBus()) {
-            return false;
-        }
-    }
-
-    // Use DBI interface to send LCD commands and parameters
-    esp_lcd_dbi_io_config_t dbi_config = JD9165_PANEL_IO_DBI_CONFIG();
-
-    if (esp_lcd_new_panel_io_dbi(mipiDsiBus, &dbi_config, &ioHandle) != ESP_OK) {
-        LOG_E(TAG, "Failed to create panel IO");
-        return false;
-    }
-
-    return true;
-}
-
-esp_lcd_panel_dev_config_t Jd9165Display::createPanelConfig(std::shared_ptr<EspLcdConfiguration> espLcdConfiguration, gpio_num_t resetPin) {
-    return {
-        .reset_gpio_num = resetPin,
-        .rgb_ele_order = espLcdConfiguration->rgbElementOrder,
-        .data_endian = LCD_RGB_DATA_ENDIAN_LITTLE,
-        .bits_per_pixel = static_cast<uint8_t>(espLcdConfiguration->bitsPerPixel),
-        .flags = {
-            .reset_active_high = 0
-        },
-        .vendor_config = nullptr  // Will be set in createPanelHandle
-    };
-}
-
-bool Jd9165Display::createPanelHandle(esp_lcd_panel_io_handle_t ioHandle, const esp_lcd_panel_dev_config_t& panelConfig, esp_lcd_panel_handle_t& panelHandle) {
-    // Create DPI panel configuration
-    // Override default timings
-    const esp_lcd_dpi_panel_config_t dpi_config = {
-        .virtual_channel = 0,
-        .dpi_clk_src = MIPI_DSI_DPI_CLK_SRC_DEFAULT,
-        .dpi_clock_freq_mhz = 50,
-        .pixel_format = LCD_COLOR_PIXEL_FORMAT_RGB565,
-        .in_color_format = LCD_COLOR_FMT_RGB565,
-        .out_color_format = LCD_COLOR_FMT_RGB565,
-        .num_fbs = 1,
-        .video_timing = {
-            .h_size = 1024,
-            .v_size = 600,
-            .hsync_pulse_width = 20,
-            .hsync_back_porch = 160,
-            .hsync_front_porch = 160,
-            .vsync_pulse_width = 2,
-            .vsync_back_porch = 21,
-            .vsync_front_porch = 12,
-        },
-        .flags = {
-            .use_dma2d = 1,
-            .disable_lp = 0
-        }
-    };
-
-    jd9165_vendor_config_t vendor_config = {
-        .init_cmds = jd9165_init_cmds,
-        .init_cmds_size = sizeof(jd9165_init_cmds) / sizeof(jd9165_lcd_init_cmd_t),
-        .mipi_config = {
-            .dsi_bus = mipiDsiBus,
-            .dpi_config = &dpi_config,
-        },
-    };
-
-    // Create a mutable copy of panelConfig to set vendor_config
-    esp_lcd_panel_dev_config_t mutable_panel_config = panelConfig;
-    mutable_panel_config.vendor_config = &vendor_config;
-
-    if (esp_lcd_new_panel_jd9165(ioHandle, &mutable_panel_config, &panelHandle) != ESP_OK) {
-        LOG_E(TAG, "Failed to create panel");
-        return false;
-    }
-
-    LOG_I(TAG, "JD9165 panel created successfully");
-    // Defer reset/init to base class applyConfiguration to avoid double initialization
-    return true;
-}
-
-lvgl_port_display_dsi_cfg_t Jd9165Display::getLvglPortDisplayDsiConfig(esp_lcd_panel_io_handle_t /*ioHandle*/, esp_lcd_panel_handle_t /*panelHandle*/) {
-    // Disable avoid_tearing to prevent stalls/blank flashes when other tasks (e.g. flash writes) block timing
-    return lvgl_port_display_dsi_cfg_t{
-        .flags = {
-            .avoid_tearing = 0,
-        },
-    };
-}

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

@@ -1,39 +0,0 @@
-#pragma once
-
-#include <EspLcdDisplayV2.h>
-#include <Tactility/RecursiveMutex.h>
-
-#include <esp_lcd_mipi_dsi.h>
-#include <esp_ldo_regulator.h>
-
-class Jd9165Display final : public EspLcdDisplayV2 {
-
-    esp_lcd_dsi_bus_handle_t mipiDsiBus = nullptr;
-    esp_ldo_channel_handle_t ldoChannel = nullptr;
-
-    bool createMipiDsiBus();
-
-protected:
-
-    bool createIoHandle(esp_lcd_panel_io_handle_t& ioHandle) override;
-
-    esp_lcd_panel_dev_config_t createPanelConfig(std::shared_ptr<EspLcdConfiguration> espLcdConfiguration, gpio_num_t resetPin) override;
-
-    bool createPanelHandle(esp_lcd_panel_io_handle_t ioHandle, const esp_lcd_panel_dev_config_t& panelConfig, esp_lcd_panel_handle_t& panelHandle) override;
-
-    bool useDsiPanel() const override { return true; }
-
-    lvgl_port_display_dsi_cfg_t getLvglPortDisplayDsiConfig(esp_lcd_panel_io_handle_t /*ioHandle*/, esp_lcd_panel_handle_t /*panelHandle*/) override;
-
-public:
-
-    Jd9165Display(
-        const std::shared_ptr<EspLcdConfiguration>& configuration
-    ) : EspLcdDisplayV2(configuration) {}
-
-    ~Jd9165Display() override;
-
-    std::string getName() const override { return "JD9165"; }
-
-    std::string getDescription() const override { return "JD9165 MIPI-DSI 1024x600 display"; }
-};

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

@@ -1,180 +0,0 @@
-#include "Power.h"
-
-#include <ChargeFromVoltage.h>
-#include <tactility/log.h>
-#include <esp_adc/adc_oneshot.h>
-#include <esp_adc/adc_cali.h>
-#include <esp_adc/adc_cali_scheme.h>
-
-using tt::hal::power::PowerDevice;
-
-constexpr auto* TAG = "JcPower";
-
-namespace {
-
-constexpr adc_unit_t ADC_UNIT = ADC_UNIT_2;
-constexpr adc_channel_t ADC_CHANNEL = ADC_CHANNEL_4; // matches ADC2 CH4 used in brookesia config
-constexpr adc_atten_t ADC_ATTEN = ADC_ATTEN_DB_12;
-constexpr int32_t UPPER_RESISTOR_OHM = 85'000;  // per brookesia settings
-constexpr int32_t LOWER_RESISTOR_OHM = 100'000; // per brookesia settings
-
-class JcPower final : public PowerDevice {
-public:
-    JcPower() : chargeEstimator(3.3f, 4.2f) {}
-    ~JcPower() override { deinit(); }
-
-    std::string getName() const override { return "JC Power"; }
-    std::string getDescription() const override { return "Battery voltage via ADC"; }
-
-    bool supportsMetric(MetricType type) const override {
-        switch (type) {
-            using enum MetricType;
-            case BatteryVoltage:
-            case ChargeLevel:
-                return true;
-            default:
-                return false;
-        }
-    }
-
-    bool getMetric(MetricType type, MetricData& data) override {
-        if (!ensureInit()) {
-            return false;
-        }
-
-        uint32_t batteryMv = 0;
-        if (!readBatteryMilliVolt(batteryMv)) {
-            return false;
-        }
-
-        switch (type) {
-            case MetricType::BatteryVoltage:
-                data.valueAsUint32 = batteryMv;
-                return true;
-            case MetricType::ChargeLevel:
-                data.valueAsUint8 = chargeEstimator.estimateCharge(batteryMv);
-                return true;
-            default:
-                return false;
-        }
-    }
-
-private:
-    bool ensureInit() {
-        if (initialized) {
-            return true;
-        }
-
-        adc_oneshot_unit_init_cfg_t init_cfg = {
-            .unit_id = ADC_UNIT,
-            .clk_src = ADC_RTC_CLK_SRC_DEFAULT,
-            .ulp_mode = ADC_ULP_MODE_DISABLE,
-        };
-        if (adc_oneshot_new_unit(&init_cfg, &adcHandle) != ESP_OK) {
-            LOG_E(TAG, "ADC unit init failed");
-            return false;
-        }
-
-        adc_oneshot_chan_cfg_t chan_cfg = {
-            .atten = ADC_ATTEN,
-            .bitwidth = ADC_BITWIDTH_DEFAULT,
-        };
-        if (adc_oneshot_config_channel(adcHandle, ADC_CHANNEL, &chan_cfg) != ESP_OK) {
-            LOG_E(TAG, "ADC channel config failed");
-            adc_oneshot_del_unit(adcHandle);
-            adcHandle = nullptr;
-            return false;
-        }
-
-        calibrated = tryInitCalibration();
-        initialized = true;
-        return true;
-    }
-
-    bool tryInitCalibration() {
-#if ADC_CALI_SCHEME_LINE_FITTING_SUPPORTED
-        adc_cali_line_fitting_config_t cali_config = {
-            .unit_id = ADC_UNIT,
-            .atten = ADC_ATTEN,
-            .bitwidth = ADC_BITWIDTH_DEFAULT,
-        };
-        if (adc_cali_create_scheme_line_fitting(&cali_config, &caliHandle) == ESP_OK) {
-            calScheme = CaliScheme::Line;
-            LOG_I(TAG, "ADC calibration (line fitting) enabled");
-            return true;
-        }
-#endif
-
-#if ADC_CALI_SCHEME_CURVE_FITTING_SUPPORTED
-        adc_cali_curve_fitting_config_t curve_cfg = {
-            .unit_id = ADC_UNIT,
-            .chan = ADC_CHANNEL,
-            .atten = ADC_ATTEN,
-            .bitwidth = ADC_BITWIDTH_DEFAULT,
-        };
-        if (adc_cali_create_scheme_curve_fitting(&curve_cfg, &caliHandle) == ESP_OK) {
-            calScheme = CaliScheme::Curve;
-            LOG_I(TAG, "ADC calibration (curve fitting) enabled");
-            return true;
-        }
-#endif
-
-        LOG_W(TAG, "ADC calibration not available, using raw scaling");
-        return false;
-    }
-
-    bool readBatteryMilliVolt(uint32_t& outMv) {
-        int raw = 0;
-        if (adc_oneshot_read(adcHandle, ADC_CHANNEL, &raw) != ESP_OK) {
-            LOG_E(TAG, "ADC read failed");
-            return false;
-        }
-
-        int mv = 0;
-        if (calibrated && adc_cali_raw_to_voltage(caliHandle, raw, &mv) == ESP_OK) {
-            // ok
-        } else {
-            // Fallback: approximate assuming 12-bit full scale 3.3V
-            mv = (raw * 3300) / 4095;
-        }
-
-        const int64_t numerator = static_cast<int64_t>(UPPER_RESISTOR_OHM + LOWER_RESISTOR_OHM) * mv;
-        const int64_t denominator = LOWER_RESISTOR_OHM;
-        outMv = static_cast<uint32_t>(numerator / denominator);
-        return true;
-    }
-
-    void deinit() {
-        if (adcHandle) {
-            adc_oneshot_del_unit(adcHandle);
-            adcHandle = nullptr;
-        }
-        if (caliHandle) {
-            if (calScheme == CaliScheme::Line) {
-#if ADC_CALI_SCHEME_LINE_FITTING_SUPPORTED
-                adc_cali_delete_scheme_line_fitting(caliHandle);
-#endif
-#if ADC_CALI_SCHEME_CURVE_FITTING_SUPPORTED
-            } else if (calScheme == CaliScheme::Curve) {
-                adc_cali_delete_scheme_curve_fitting(caliHandle);
-#endif
-            }
-            caliHandle = nullptr;
-            calibrated = false;
-        }
-    }
-
-    enum class CaliScheme { None, Line, Curve };
-
-    bool initialized = false;
-    bool calibrated = false;
-    CaliScheme calScheme = CaliScheme::None;
-    adc_oneshot_unit_handle_t adcHandle = nullptr;
-    adc_cali_handle_t caliHandle = nullptr;
-    ChargeFromVoltage chargeEstimator;
-};
-} // namespace
-
-std::shared_ptr<PowerDevice> createPower() {
-    return std::make_shared<JcPower>();
-}

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

@@ -1,7 +0,0 @@
-#pragma once
-
-#include <memory>
-#include <Tactility/hal/power/PowerDevice.h>
-
-// Battery measurement via ADC2 channel 4 with 85k/100k divider
-std::shared_ptr<tt::hal::power::PowerDevice> createPower();

Devices/guition-jc1060p470ciwy/device.properties

@@ -11,6 +11,8 @@ hardware.spiRamSpeed=200M
 hardware.esptoolFlashFreq=80M
 hardware.bluetooth=true
 
+dependencies.useDeprecatedHal=false
+
 storage.userDataLocation=SD
 
 display.size=7"

Devices/guition-jc1060p470ciwy/devicetree.yaml

@@ -1,3 +1,5 @@
 dependencies:
   - Platforms/platform-esp32
+  - Drivers/jd9165-module
+  - Drivers/gt911-module
 dts: guition,jc1060p470ciwy.dts

Devices/guition-jc1060p470ciwy/guition,jc1060p470ciwy.dts

@@ -7,13 +7,20 @@
 #include <tactility/bindings/esp32_i2s.h>
 #include <tactility/bindings/esp32_sdmmc.h>
 #include <tactility/bindings/esp32_wifi.h>
+#include <tactility/bindings/esp32_adc_oneshot.h>
+#include <tactility/bindings/battery_sense.h>
+#include <tactility/bindings/esp32_pwm_ledc.h>
+#include <tactility/bindings/pwm_backlight.h>
+#include <bindings/jd9165.h>
+#include <bindings/gt911.h>
 
 /**
  * For future reference:
  *  - ES8311 on I2C with PA PIN at GPIO 11
  *  - Built-in led at GPIO 26
  *  - Boot button at GPIO 21
- *  - LCD reset: GPIO 27
+ *  - LCD reset: GPIO 0 (matches P4 EV board reset line - deprecated HAL's old Display.cpp
+ *    comment claimed GPIO 27 here, but its actual LCD_PIN_RESET constant was GPIO_NUM_0)
  *  - LCD backlight: GPIO 23
  */
 / {
@@ -41,6 +48,13 @@
 		clock-frequency = <400000>;
 		pin-sda = <&gpio0 7 GPIO_FLAG_NONE>;
 		pin-scl = <&gpio0 8 GPIO_FLAG_NONE>;
+
+		touch0 {
+			compatible = "goodix,gt911";
+			reg = <0x5D>;
+			x-max = <1024>;
+			y-max = <600>;
+		};
 	};
 
 	i2s0 {
@@ -65,4 +79,116 @@
 		bus-width = <4>;
 		on-chip-ldo-chan = <4>;
 	};
+
+	adc0 {
+		compatible = "espressif,esp32-adc-oneshot";
+		unit-id = <ADC_UNIT_2>;
+		channels = <ADC_CHANNEL_4 ADC_ATTEN_DB_12 ADC_BITWIDTH_DEFAULT>;
+	};
+
+	// Matches the deprecated HAL's old Power.cpp: ADC2 CH4 behind an 85k/100k divider
+	// (multiplier = (85000+100000)/100000 = 1.850), reference-voltage-mv is the nominal
+	// full-scale value at ADC_ATTEN_DB_12 (battery-sense has no calibration path, unlike the
+	// old driver's adc_cali fallback).
+	battery-sense {
+		compatible = "battery-sense";
+		io-channel = <&adc0 0>;
+		reference-voltage-mv = <3300>;
+		multiplier = <1850>;
+	};
+
+	display_backlight_pwm {
+		compatible = "espressif,esp32-pwm-ledc";
+		pin = <&gpio0 23 GPIO_FLAG_NONE>;
+		period-ns = <50000>;
+		ledc-timer = <1>;
+		ledc-channel = <0>;
+	};
+
+	display_backlight {
+		compatible = "pwm-backlight";
+		pwm = <&display_backlight_pwm>;
+	};
+
+	display0 {
+		compatible = "jdi,jd9165";
+		horizontal-resolution = <1024>;
+		vertical-resolution = <600>;
+		pin-reset = <&gpio0 0 GPIO_FLAG_NONE>;
+		ldo-channel = <3>; // LDO_VO3 connects to VDD_MIPI_DPHY
+		ldo-voltage-mv = <2500>;
+		num-data-lanes = <2>;
+		lane-bit-rate-mbps = <750>;
+		dpi-clock-freq-mhz = <50>;
+		hsync-pulse-width = <20>;
+		hsync-back-porch = <160>;
+		hsync-front-porch = <160>;
+		vsync-pulse-width = <2>;
+		vsync-back-porch = <21>;
+		vsync-front-porch = <12>;
+		// Skips the wait-for-scanout in draw_bitmap(): prevents stalls/blank flashes when
+		// other tasks (e.g. flash writes) block timing, at the cost of tear-free rendering
+		// (matches the deprecated HAL's old Jd9165Display::getLvglPortDisplayDsiConfig(),
+		// which disabled esp_lvgl_port's avoid_tearing for the same reason).
+		allow-tearing;
+		backlight = <&display_backlight>;
+
+		// Vendor bring-up sequence from the ESP32-P4 Function EV Board reference, carried over
+		// unchanged from the deprecated HAL's old Jd9165Display.cpp. Framing is
+		// [cmd, data-len, delay-ms, data-len bytes of data...] - see jd9165-module's
+		// init-sequence binding property for the encoding.
+		init-sequence = [
+			0x30 1 0    0x00
+			0xF7 4 0    0x49 0x61 0x02 0x00
+			0x30 1 0    0x01
+			0x04 1 0    0x0C
+			0x05 1 0    0x00
+			0x06 1 0    0x00
+			0x0B 1 0    0x11
+			0x17 1 0    0x00
+			0x20 1 0    0x04
+			0x1F 1 0    0x05
+			0x23 1 0    0x00
+			0x25 1 0    0x19
+			0x28 1 0    0x18
+			0x29 1 0    0x04
+			0x2A 1 0    0x01
+			0x2B 1 0    0x04
+			0x2C 1 0    0x01
+			0x30 1 0    0x02
+			0x01 1 0    0x22
+			0x03 1 0    0x12
+			0x04 1 0    0x00
+			0x05 1 0    0x64
+			0x0A 1 0    0x08
+			0x0B 11 0   0x0A 0x1A 0x0B 0x0D 0x0D 0x11 0x10 0x06 0x08 0x1F 0x1D
+			0x0C 11 0   0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D
+			0x0D 11 0   0x16 0x1B 0x0B 0x0D 0x0D 0x11 0x10 0x07 0x09 0x1E 0x1C
+			0x0E 11 0   0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D
+			0x0F 11 0   0x16 0x1B 0x0D 0x0B 0x0D 0x11 0x10 0x1C 0x1E 0x09 0x07
+			0x10 11 0   0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D
+			0x11 11 0   0x0A 0x1A 0x0D 0x0B 0x0D 0x11 0x10 0x1D 0x1F 0x08 0x06
+			0x12 11 0   0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D 0x0D
+			0x14 4 0    0x00 0x00 0x11 0x11
+			0x18 1 0    0x99
+			0x30 1 0    0x06
+			0x12 14 0   0x36 0x2C 0x2E 0x3C 0x38 0x35 0x35 0x32 0x2E 0x1D 0x2B 0x21 0x16 0x29
+			0x13 14 0   0x36 0x2C 0x2E 0x3C 0x38 0x35 0x35 0x32 0x2E 0x1D 0x2B 0x21 0x16 0x29
+			0x30 1 0    0x0A
+			0x02 1 0    0x4F
+			0x0B 1 0    0x40
+			0x12 1 0    0x3E
+			0x13 1 0    0x78
+			0x30 1 0    0x0D
+			0x0D 1 0    0x04
+			0x10 1 0    0x0C
+			0x11 1 0    0x0C
+			0x12 1 0    0x0C
+			0x13 1 0    0x0C
+			0x30 1 0    0x00
+			0x3A 1 0    0x55
+			0x11 1 120  0x00
+			0x29 1 20   0x00
+		];
+	};
 };

Devices/guition-jc1060p470ciwy/source/module.cpp

@@ -3,16 +3,14 @@
 extern "C" {
 
 static error_t start() {
-    // Empty for now
     return ERROR_NONE;
 }
 
 static error_t stop() {
-    // Empty for now
     return ERROR_NONE;
 }
 
-struct Module guition_jc1060p470ciwy_module = {
+Module guition_jc1060p470ciwy_module = {
     .name = "guition-jc1060p470ciwy",
     .start = start,
     .stop = stop,

Devices/guition-jc2432w328c/CMakeLists.txt

@@ -1,7 +1,6 @@
-file(GLOB_RECURSE SOURCE_FILES Source/*.c*)
+file(GLOB_RECURSE SOURCE_FILES source/*.c*)
 
 idf_component_register(
     SRCS ${SOURCE_FILES}
-    INCLUDE_DIRS "Source"
-    REQUIRES Tactility esp_lvgl_port ST7789 CST816S PwmBacklight driver vfs fatfs
+    REQUIRES TactilityKernel driver
 )

Devices/guition-jc2432w328c/Source/Configuration.cpp

@@ -1,32 +0,0 @@
-#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
-};

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

@@ -1,43 +0,0 @@
-#include "Display.h"
-
-#include <Cst816Touch.h>
-#include <PwmBacklight.h>
-#include <St7789Display.h>
-
-static std::shared_ptr<tt::hal::touch::TouchDevice> createTouch() {
-    auto configuration = std::make_unique<Cst816sTouch::Configuration>(
-        I2C_NUM_0,
-        LCD_HORIZONTAL_RESOLUTION,
-        LCD_VERTICAL_RESOLUTION
-    );
-
-    return std::make_shared<Cst816sTouch>(std::move(configuration));
-}
-
-std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay() {
-    St7789Display::Configuration panel_configuration = {
-        .horizontalResolution = LCD_HORIZONTAL_RESOLUTION,
-        .verticalResolution = LCD_VERTICAL_RESOLUTION,
-        .gapX = 0,
-        .gapY = 0,
-        .swapXY = false,
-        .mirrorX = false,
-        .mirrorY = false,
-        .invertColor = false,
-        .bufferSize = LCD_BUFFER_SIZE,
-        .touch = createTouch(),
-        .backlightDutyFunction = driver::pwmbacklight::setBacklightDuty,
-        .resetPin = GPIO_NUM_NC,
-        .lvglSwapBytes = false
-    };
-
-    auto spi_configuration = std::make_shared<St7789Display::SpiConfiguration>(St7789Display::SpiConfiguration {
-        .spiHostDevice = LCD_SPI_HOST,
-        .csPin = LCD_PIN_CS,
-        .dcPin = LCD_PIN_DC,
-        .pixelClockFrequency = 62'500'000,
-        .transactionQueueDepth = 10
-    });
-
-    return std::make_shared<St7789Display>(panel_configuration, spi_configuration);
-}