This probably won't work...

Shadowtrance · Shadowtrance · commit 143badba3e6c · 2025-12-02T04:48:11.000+10:00
diff --git a/Boards/cyd-jc3248w535c/Source/Configuration.cpp b/Boards/cyd-jc3248w535c/Source/Configuration.cpp
@@ -7,7 +7,7 @@
 
 using namespace tt::hal;
 
-#define SPI_TRANSFER_SIZE_LIMIT ((320) * 16 / 8)
+#define SPI_TRANSFER_SIZE_LIMIT 320 * 16 / 8
 
 static DeviceVector createDevices() {
     return {
diff --git a/Boards/cyd-jc3248w535c/Source/devices/Display.cpp b/Boards/cyd-jc3248w535c/Source/devices/Display.cpp
@@ -4,7 +4,7 @@
 #include <Axs15231bDisplay.h>
 #include <Axs15231bTouch.h>
 
-#define JC3248W535C_LCD_DRAW_BUFFER_SIZE ((320) * (480))
+#define JC3248W535C_LCD_DRAW_BUFFER_SIZE 320 * 480
 
 static std::shared_ptr<tt::hal::touch::TouchDevice> createTouch() {
     auto configuration = std::make_unique<Axs15231bTouch::Configuration>(
@@ -27,6 +27,7 @@ std::shared_ptr<tt::hal::display::DisplayDevice> createDisplay() {
         GPIO_NUM_45,
         GPIO_NUM_NC,
         GPIO_NUM_NC,
+        GPIO_NUM_38,
         320,
         480,
         touch,
diff --git a/Drivers/Axs15231b/Source/Axs15231bDisplay.cpp b/Drivers/Axs15231b/Source/Axs15231bDisplay.cpp
@@ -8,6 +8,18 @@
 
 constexpr const char* TAG = "AXS15231B";
 
+typedef struct
+{
+    SemaphoreHandle_t te_v_sync_sem;
+    SemaphoreHandle_t te_catch_sem;
+    uint32_t time_Tvdl;
+    uint32_t time_Tvdh;
+    uint32_t te_timestamp;
+    portMUX_TYPE lock;
+} lcd_tear_t;
+
+lcd_tear_t *tear_ctx = nullptr;
+
 static const axs15231b_lcd_init_cmd_t lcd_init_cmds[] = {
     {0xBB, (uint8_t []){0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x5A, 0xA5}, 8, 0},
     {0xA0, (uint8_t[]){0xC0, 0x10, 0x00, 0x02, 0x00, 0x00, 0x04, 0x3F, 0x20, 0x05, 0x3F, 0x3F, 0x00, 0x00, 0x00, 0x00, 0x00}, 17, 0},
@@ -43,6 +55,58 @@ static const axs15231b_lcd_init_cmd_t lcd_init_cmds[] = {
     {0x2C, (uint8_t []){0x00, 0x00, 0x00, 0x00}, 4, 0}
 };
 
+static void display_sync_cb(lv_event_t *e)
+{
+    lcd_tear_t *tear_handle = (lcd_tear_t *)lv_event_get_user_data(e);
+    
+    if (tear_handle == nullptr || tear_handle->te_v_sync_sem == nullptr)
+    {
+        return;
+    }
+
+    if (tear_handle->te_catch_sem)
+    {
+        xSemaphoreGive(tear_handle->te_catch_sem);
+    }
+
+    xSemaphoreTake(tear_handle->te_v_sync_sem, pdMS_TO_TICKS(100));
+}
+
+static void display_sync_task(void *arg)
+{
+    assert(arg);
+    lcd_tear_t *tear_handle = (lcd_tear_t *)arg;
+
+    while (true)
+    {
+        if (pdPASS != xSemaphoreTake(tear_handle->te_catch_sem, pdMS_TO_TICKS(tear_handle->time_Tvdl)))
+        {
+            xSemaphoreTake(tear_handle->te_v_sync_sem, 0);
+        }
+    }
+    vTaskDelete(nullptr);
+}
+
+static void IRAM_ATTR display_tear_interrupt(void *arg)
+{
+    assert(arg);
+    lcd_tear_t *tear_handle = (lcd_tear_t *)arg;
+    BaseType_t xHigherPriorityTaskAwoken = pdFALSE;
+
+    if (tear_handle->te_v_sync_sem)
+    {
+        portENTER_CRITICAL_ISR(&tear_handle->lock);
+        tear_handle->te_timestamp = esp_log_timestamp();
+        portEXIT_CRITICAL_ISR(&tear_handle->lock);
+        xSemaphoreGiveFromISR(tear_handle->te_v_sync_sem, &xHigherPriorityTaskAwoken);
+
+        if (xHigherPriorityTaskAwoken)
+        {
+            portYIELD_FROM_ISR();
+        }
+    }
+}
+
 bool Axs15231bDisplay::createIoHandle(esp_lcd_panel_io_handle_t& outHandle) {
     const esp_lcd_panel_io_spi_config_t panel_io_config = {
         .cs_gpio_num = configuration->csPin,
@@ -72,6 +136,9 @@ bool Axs15231bDisplay::createIoHandle(esp_lcd_panel_io_handle_t& outHandle) {
 }
 
 bool Axs15231bDisplay::createPanelHandle(esp_lcd_panel_io_handle_t ioHandle, esp_lcd_panel_handle_t& panelHandle) {
+    SemaphoreHandle_t te_catch_sem = nullptr;
+    SemaphoreHandle_t te_v_sync_sem = nullptr;
+
     const axs15231b_vendor_config_t vendor_config = {
         .init_cmds = lcd_init_cmds,
         .init_cmds_size = sizeof(lcd_init_cmds) / sizeof(lcd_init_cmds[0]),
@@ -127,6 +194,45 @@ bool Axs15231bDisplay::createPanelHandle(esp_lcd_panel_io_handle_t ioHandle, esp
         return false;
     }
 
+    // TE sync initialization
+    uint32_t time_Tvdl = 13;
+    uint32_t time_Tvdh = 3;
+
+    if (configuration->te_gpio_num > 0)
+    {
+        tear_ctx = malloc(sizeof(lcd_tear_t));
+        //ESP_GOTO_ON_FALSE(tear_ctx != NULL, ESP_ERR_NO_MEM, err, TAG, "Not enough memory for tear_ctx allocation!");
+
+        te_v_sync_sem = xSemaphoreCreateCounting(1, 0);
+        //ESP_GOTO_ON_FALSE(te_v_sync_sem, ESP_ERR_NO_MEM, err, TAG, "Not enough memory for te_v_sync_sem Semaphore.");
+        tear_ctx->te_v_sync_sem = te_v_sync_sem;
+
+        te_catch_sem = xSemaphoreCreateCounting(1, 0);
+        //ESP_GOTO_ON_FALSE(te_catch_sem, ESP_ERR_NO_MEM, err, TAG, "Not enough memory for te_catch_sem Semaphore.");
+        tear_ctx->te_catch_sem = te_catch_sem;
+
+        tear_ctx->time_Tvdl = time_Tvdl;
+        tear_ctx->time_Tvdh = time_Tvdh;
+        tear_ctx->lock.owner = portMUX_FREE_VAL;
+        tear_ctx->lock.count = 0;
+
+        const gpio_config_t te_detect_cfg = {
+            .intr_type = GPIO_INTR_NEGEDGE,
+            .mode = GPIO_MODE_INPUT,
+            .pin_bit_mask = BIT64(configuration->te_gpio_num),
+            .pull_down_en = GPIO_PULLDOWN_DISABLE,
+            .pull_up_en = GPIO_PULLUP_ENABLE};
+
+        ESP_ERROR_CHECK(gpio_config(&te_detect_cfg));
+        gpio_install_isr_service(0);
+        ESP_ERROR_CHECK(gpio_isr_handler_add(configuration->te_gpio_num, display_tear_interrupt, tear_ctx));
+
+        BaseType_t res = xTaskCreate(display_sync_task, "Tear Task", 2048, tear_ctx, 4, NULL);
+        //ESP_GOTO_ON_FALSE(res == pdPASS, ESP_FAIL, err, TAG, "Create Sync Task Fail!");
+    }
+
+    lv_display_add_event_cb(lvgl_disp, display_sync_cb, LV_EVENT_FLUSH_START, tear_ctx);
+
     return true;
 }
 
@@ -137,7 +243,7 @@ lvgl_port_display_cfg_t Axs15231bDisplay::getLvglPortDisplayConfig(esp_lcd_panel
         .control_handle = nullptr,
         .buffer_size = configuration->bufferSize,
         .double_buffer = true,
-        .trans_size = 0,
+        .trans_size = configuration->horizontalResolution * configuration->verticalResolution / 4,
         .hres = configuration->horizontalResolution,
         .vres = configuration->verticalResolution,
         .monochrome = false,
@@ -148,7 +254,7 @@ lvgl_port_display_cfg_t Axs15231bDisplay::getLvglPortDisplayConfig(esp_lcd_panel
         },
         .color_format = LV_COLOR_FORMAT_RGB565,
         .flags = {
-            .buff_dma = true,
+            .buff_dma = false,
             .buff_spiram = true,
             .sw_rotate = true,
             .swap_bytes = true,
diff --git a/Drivers/Axs15231b/Source/Axs15231bDisplay.h b/Drivers/Axs15231b/Source/Axs15231bDisplay.h
@@ -24,6 +24,7 @@ class Axs15231bDisplay final : public EspLcdDisplay {
             gpio_num_t csPin,
             gpio_num_t dcPin,
             gpio_num_t resetPin,
+            gpio_num_t tePin,
             unsigned int horizontalResolution,
             unsigned int verticalResolution,
             std::shared_ptr<tt::hal::touch::TouchDevice> touch,
@@ -37,6 +38,7 @@ class Axs15231bDisplay final : public EspLcdDisplay {
             csPin(csPin),
             dcPin(dcPin),
             resetPin(resetPin),
+            tePin(tePin),
             horizontalResolution(horizontalResolution),
             verticalResolution(verticalResolution),
             swapXY(swapXY),
@@ -56,6 +58,7 @@ class Axs15231bDisplay final : public EspLcdDisplay {
         gpio_num_t csPin;
         gpio_num_t dcPin;
         gpio_num_t resetPin;
+        gpio_num_t tePin;
         unsigned int pixelClockFrequency = 40'000'000; // Hertz
         size_t transactionQueueDepth = 10;
         unsigned int horizontalResolution;
