diff --git a/main/display/display.cpp b/main/display/display.cpp index 943522d..679d506 100644 --- a/main/display/display.cpp +++ b/main/display/display.cpp @@ -118,12 +118,23 @@ void DisplayHandler::_epd_init(void) { epd_write_cmd(0x04); // Power ON vTaskDelay(pdMS_TO_TICKS(100)); // Wait for power on - // Check BUSY pin - ESP_LOGI("DisplayHandler", "Waiting for EPD to be ready..."); + // Check BUSY pin with detailed logging + ESP_LOGI("DisplayHandler", "Waiting for EPD to be ready after power on..."); + ESP_LOGI("DisplayHandler", "BUSY pin level after power on: %d (0=BUSY, 1=FREE)", gpio_get_level(PIN_BUSY)); + + int busy_timeout = 0; while (gpio_get_level(PIN_BUSY) == BUSY_ACTIVE_LEVEL) { // BUSY is active LOW vTaskDelay(pdMS_TO_TICKS(10)); + busy_timeout++; + if (busy_timeout > 500) { // 5 second timeout + ESP_LOGE("DisplayHandler", "EPD power on timeout! BUSY pin stuck at 0"); + break; + } + if (busy_timeout % 50 == 0) { // Log every 500ms + ESP_LOGW("DisplayHandler", "Still waiting for EPD power on, timeout: %d/500", busy_timeout); + } } - ESP_LOGI("DisplayHandler", "EPD is ready."); + ESP_LOGI("DisplayHandler", "EPD power on complete after %d * 10ms, BUSY pin: %d", busy_timeout, gpio_get_level(PIN_BUSY)); const uint8_t booster_data[] = { 0x27, 0x27, 0x18, 0x17 }; epd_write_cmd_with_data(0x06, booster_data, 4); // Booster Soft Start vTaskDelay(pdMS_TO_TICKS(10)); diff --git a/main/display/eink_display_handler.cpp b/main/display/eink_display_handler.cpp index a3f97cb..310b339 100644 --- a/main/display/eink_display_handler.cpp +++ b/main/display/eink_display_handler.cpp @@ -52,10 +52,10 @@ void EInkDisplayHandler::init() { io_conf.intr_type = GPIO_INTR_DISABLE; gpio_config(&io_conf); - // Configure BUSY pin as input + // Configure BUSY pin as input (no pull-up like sample code) io_conf.pin_bit_mask = (1ULL << PIN_BUSY); io_conf.mode = GPIO_MODE_INPUT; - io_conf.pull_up_en = GPIO_PULLUP_ENABLE; + io_conf.pull_up_en = GPIO_PULLUP_DISABLE; gpio_config(&io_conf); // Initialize SPI bus @@ -198,9 +198,12 @@ void EInkDisplayHandler::_lvgl_flush_cb(lv_display_t* disp, const lv_area_t* are return; } - // Check if display is busy + // Check if display is busy with detailed logging + int busy_level = gpio_get_level(PIN_BUSY); + ESP_LOGI(TAG, "Flush callback: BUSY pin = %d, is_busy() = %d", busy_level, handler->is_busy()); + if (handler->is_busy()) { - ESP_LOGW(TAG, "Display busy, skipping flush"); + ESP_LOGW(TAG, "Display busy (BUSY pin = 0), skipping flush"); lv_display_flush_ready(disp); return; } @@ -323,7 +326,7 @@ void EInkDisplayHandler::_perform_full_refresh(const uint8_t* framebuffer) { gpio_set_level(PIN_DC, 1); // Data mode ESP_LOGI(TAG, "Starting SPI data transmission for old data (0x10)..."); - + // Use simpler polling transmission instead of queued to avoid complexity static uint8_t zero_byte = 0x00; // Static to persist esp_err_t ret; @@ -347,8 +350,8 @@ void EInkDisplayHandler::_perform_full_refresh(const uint8_t* framebuffer) { return; } gpio_set_level(PIN_DC, 1); // Re-set data mode after yield - ESP_LOGI(TAG, "Transmitted %zu/%zu bytes (%.1f%%)", i+1, DISPLAY_BUFFER_SIZE, - (float)(i+1) * 100.0f / DISPLAY_BUFFER_SIZE); + ESP_LOGI(TAG, "Transmitted %zu/%zu bytes (%.1f%%)", i + 1, DISPLAY_BUFFER_SIZE, + (float)(i + 1) * 100.0f / DISPLAY_BUFFER_SIZE); } } ESP_LOGI(TAG, "Completed SPI data transmission for old data"); @@ -364,11 +367,11 @@ void EInkDisplayHandler::_perform_full_refresh(const uint8_t* framebuffer) { gpio_set_level(PIN_DC, 1); // Data mode ESP_LOGI(TAG, "Starting SPI data transmission for new data (0x13)..."); - + // Use polling transmission for simplicity and reliability for (size_t i = 0; i < DISPLAY_BUFFER_SIZE; i++) { uint8_t data_byte = framebuffer[i]; // Write data directly (no inversion) - + spi_transaction_t t = {}; t.length = 8; t.tx_buffer = &data_byte; @@ -388,17 +391,19 @@ void EInkDisplayHandler::_perform_full_refresh(const uint8_t* framebuffer) { return; } gpio_set_level(PIN_DC, 1); // Re-set data mode after yield - ESP_LOGI(TAG, "Transmitted %zu/%zu bytes (%.1f%%)", i+1, DISPLAY_BUFFER_SIZE, - (float)(i+1) * 100.0f / DISPLAY_BUFFER_SIZE); + ESP_LOGI(TAG, "Transmitted %zu/%zu bytes (%.1f%%)", i + 1, DISPLAY_BUFFER_SIZE, + (float)(i + 1) * 100.0f / DISPLAY_BUFFER_SIZE); } } - + ESP_LOGI(TAG, "Completed SPI data transmission for new data"); xSemaphoreGive(_spi_mutex); // Step 3: Trigger display refresh (DRF) epd_write_cmd(0x12); + // Critical delay - sample code says "!!!The delay here is necessary, 200uS at least!!!" vTaskDelay(pdMS_TO_TICKS(10)); + ESP_LOGI(TAG, "Display refresh triggered, BUSY pin: %d", gpio_get_level(PIN_BUSY)); // Wait for refresh to complete _wait_for_busy(); @@ -440,7 +445,7 @@ void EInkDisplayHandler::_perform_partial_refresh(const uint8_t* framebuffer) { gpio_set_level(PIN_DC, 1); // Data mode ESP_LOGI(TAG, "Starting SPI data transmission for partial refresh..."); - + // Send data in chunks with task yields to prevent blocking static uint8_t tx_byte; const size_t CHUNK_SIZE = 1000; // Send 1000 bytes between yields @@ -460,8 +465,8 @@ void EInkDisplayHandler::_perform_partial_refresh(const uint8_t* framebuffer) { return; } gpio_set_level(PIN_DC, 1); // Re-set data mode after yield - ESP_LOGI(TAG, "Partial refresh progress: %zu/%zu bytes (%.1f%%)", i+1, DISPLAY_BUFFER_SIZE, - (float)(i+1) * 100.0f / DISPLAY_BUFFER_SIZE); + ESP_LOGI(TAG, "Partial refresh progress: %zu/%zu bytes (%.1f%%)", i + 1, DISPLAY_BUFFER_SIZE, + (float)(i + 1) * 100.0f / DISPLAY_BUFFER_SIZE); } } ESP_LOGI(TAG, "Completed SPI data transmission for partial refresh"); @@ -469,7 +474,9 @@ void EInkDisplayHandler::_perform_partial_refresh(const uint8_t* framebuffer) { // Step 5: Trigger partial display refresh (DRF) epd_write_cmd(0x12); + // Critical delay - sample code says "!!!The delay here is necessary, 200uS at least!!!" vTaskDelay(pdMS_TO_TICKS(10)); + ESP_LOGI(TAG, "Partial refresh triggered, BUSY pin: %d", gpio_get_level(PIN_BUSY)); // Wait for refresh to complete _wait_for_busy(); @@ -482,8 +489,15 @@ void EInkDisplayHandler::_perform_partial_refresh(const uint8_t* framebuffer) { void EInkDisplayHandler::_wait_for_busy() { ESP_LOGI(TAG, "Waiting for display ready (BUSY pin)..."); - ESP_LOGI(TAG, "Initial BUSY pin level: %d", gpio_get_level(PIN_BUSY)); - + int initial_level = gpio_get_level(PIN_BUSY); + ESP_LOGI(TAG, "Initial BUSY pin level: %d (0=BUSY, 1=FREE)", initial_level); + + // If already free, no need to wait + if (initial_level == BUSY_INACTIVE_LEVEL) { + ESP_LOGI(TAG, "Display already ready (BUSY pin = 1)"); + return; + } + int timeout = 0; while (gpio_get_level(PIN_BUSY) == BUSY_ACTIVE_LEVEL) { // 0=BUSY, 1=FREE vTaskDelay(pdMS_TO_TICKS(100)); @@ -491,17 +505,17 @@ void EInkDisplayHandler::_wait_for_busy() { if (timeout > 100) { // 10 second timeout ESP_LOGE(TAG, "Display BUSY timeout! Pin level: %d", gpio_get_level(PIN_BUSY)); ESP_LOGW(TAG, "Attempting hardware reset..."); - + // Hardware reset sequence gpio_set_level(PIN_RST, 0); vTaskDelay(pdMS_TO_TICKS(10)); gpio_set_level(PIN_RST, 1); vTaskDelay(pdMS_TO_TICKS(100)); - + // Re-initialize display ESP_LOGI(TAG, "Re-initializing display after reset..."); _epd_init(); - + // Check if reset worked int reset_timeout = 0; while (gpio_get_level(PIN_BUSY) == BUSY_ACTIVE_LEVEL) { @@ -512,17 +526,17 @@ void EInkDisplayHandler::_wait_for_busy() { break; } } - + if (gpio_get_level(PIN_BUSY) != BUSY_ACTIVE_LEVEL) { ESP_LOGI(TAG, "Display reset successful after %d tenths of a second", reset_timeout); } break; } - + // Log every 2 seconds to track progress if (timeout % 20 == 0) { - ESP_LOGW(TAG, "Still waiting for BUSY pin, timeout: %d/100, level: %d", - timeout, gpio_get_level(PIN_BUSY)); + ESP_LOGW(TAG, "Still waiting for BUSY pin, timeout: %d/100, level: %d", + timeout, gpio_get_level(PIN_BUSY)); } } ESP_LOGI(TAG, "Display ready after %d tenths of a second", timeout);