readRawData always returns true & setDisplayOnOff doesn't turn off the LCD

[dinisfc]

I'm planning on having a display idle with the LCD panel off and have the touch panel always waiting for touch, once touch is detected the LCD comes back on.

I'm trying to use readRawData to detect touch, regardless of location, strength or number of touch points. When using readRawData, it always returns true, even if there's no touch detected and for any value for the timeout, it returns instantly or almost instantly.

At the same time, I'm trying to turn on and off the LCD using setDisplayOnOff but it doesn't seem to do anything, I've tried using enable_on as bool and as an int but the LCD never turns off.

Before, using version 0.2.2, both displayOn and displayOff worked no problem, and readRawData worked too.
I can also confirm that the program is not getting stuck on any of those functions, for any possible reason, as the program continues running.

I'm sure there's better ways of doing this, but using readRawData seems the simplest since I basically only need a toggle of there being touch or not.

[Lzw655]

Hi @dinisfc,

I'm sorry you encountered this problem in the new version. Please refer to the steps to open the debug logs and send me the complete text for analysis. Thank you!

Edited:
For the new version of readRawData, it cannot be used to detect whether there is a touch point when INT pin is not used, its return value only represents whether the read operation was successful. Please use readPoints instead.

[dinisfc]

Using readPoints did work.
I turned on the debug logs and I saw no reference to setDisplayOnOff, while readPoints and the functions it uses all showed up.

[Lzw655]

Maybe setDisplayOnOff () is not triggered? Otherwise, there should be a log showing that this function was called.

[dinisfc]

The function that uses it is:

void scrIdle( void * pvParameters ){
  auto lcd = board->getLcd();
  auto touch = board->getTouch();
  vTaskDelay(10000/portTICK_PERIOD_MS);
  TouchPoint points[5];
  while(1){ 
    if(touch->readPoints(points,5,1000)>0){
      lcd->setDisplayOnOff(true);
      vTaskDelay(30000/portTICK_PERIOD_MS);
    }
    lcd->setDisplayOnOff(false);
    vTaskDelay(10/portTICK_PERIOD_MS);
  }
}

Which is a freeRTOS task initialiazed by xTaskCreatePinnedToCore(scrIdle, "screenIdle", 4096, NULL, 1, &screenIdle, 1);
board is initialized the same way as the example.
On core 1 is LVGL and this task, on core 0 is a task that reads from a air quality sensor and saves it to storage and computes some values.

After trying different versions of the function above, the timeout parameter of readPoints doesn't seem to actually change anything, unless I messed up again. Whatever value is on the third parameter, the timeout always ends instantly, which eventually causes this errors multiple times:

E (24803) lcd_panel.io.i2c: panel_io_i2c_tx_buffer(177): i2c transaction failed
E (25806) lcd_panel.io.i2c: panel_io_i2c_rx_buffer(136): i2c transaction failed
[E][Panel]esp_lcd_touch_gt911.c:0229: esp_lcd_touch_gt911_read_data(229): I2C read error!
[E][LvPort]esp_panel_touch.cpp:0302: Read data failed [ESP_ERR_TIMEOUT]
[E][LvPort]esp_panel_touch.cpp:0454: Read raw data failed

And, eventually, a full crash and reboot. This can be avoided by using the vTaskDelay outside of the if, but the touch detection doesn't work if the task is stopped on that delay.

[extremerotary]

@dinisfc , which LCD screen are you using? Is it one of the supported models? If so, check out the model.h file from ESP32_Display_Panel > src > board > supported > {make} > {model}.h
Then look for ESP_PANEL_BOARD_BACKLIGHT_TYPE. What kind is it?
For example, mine is the waveshare 7" touch. My backlight isn't PWM and is part of the expander, so I have to do:

auto backlight = board->getBacklight();
backlight->on();

Not sure if that'll help you, but that works for my screen.

[Lzw655]

The function that uses it is:

void scrIdle( void * pvParameters ){
  auto lcd = board->getLcd();
  auto touch = board->getTouch();
  vTaskDelay(10000/portTICK_PERIOD_MS);
  TouchPoint points[5];
  while(1){ 
    if(touch->readPoints(points,5,1000)>0){
      lcd->setDisplayOnOff(true);
      vTaskDelay(30000/portTICK_PERIOD_MS);
    }
    lcd->setDisplayOnOff(false);
    vTaskDelay(10/portTICK_PERIOD_MS);
  }
}

Which is a freeRTOS task initialiazed by xTaskCreatePinnedToCore(scrIdle, "screenIdle", 4096, NULL, 1, &screenIdle, 1); board is initialized the same way as the example. On core 1 is LVGL and this task, on core 0 is a task that reads from a air quality sensor and saves it to storage and computes some values.

After trying different versions of the function above, the timeout parameter of readPoints doesn't seem to actually change anything, unless I messed up again. Whatever value is on the third parameter, the timeout always ends instantly, which eventually causes this errors multiple times:

E (24803) lcd_panel.io.i2c: panel_io_i2c_tx_buffer(177): i2c transaction failed
E (25806) lcd_panel.io.i2c: panel_io_i2c_rx_buffer(136): i2c transaction failed
[E][Panel]esp_lcd_touch_gt911.c:0229: esp_lcd_touch_gt911_read_data(229): I2C read error!
[E][LvPort]esp_panel_touch.cpp:0302: Read data failed [ESP_ERR_TIMEOUT]
[E][LvPort]esp_panel_touch.cpp:0454: Read raw data failed

And, eventually, a full crash and reboot. This can be avoided by using the vTaskDelay outside of the if, but the touch detection doesn't work if the task is stopped on that delay.

Hi @dinisfc,

The timeout parameter of readPoints() is only valid if the INT pin is used because we can only be notified when the Interrupt signal is triggered.

[dinisfc]

@extremerotary I'm using the Waveshare ESP32-S3-Touch-LCD-4, there's no official support for it yet other than some files on the Waveshare website with outdated versions of the libraries, I'm using the custom board configurations from those files. I've been using the backlight off by default since, at least in this device, makes little difference. I've managed to turn the display on and off regardless of the state of the backlight before.

@Lzw655 Where should configInterruptActiveLevel be, assuming I start the display and LVGL with

    board->init();
#if LVGL_PORT_AVOID_TEARING_MODE
    auto lcd = board->getLCD();
    lcd->configFrameBufferNumber(LVGL_PORT_DISP_BUFFER_NUM);
#if ESP_PANEL_DRIVERS_BUS_ENABLE_RGB && CONFIG_IDF_TARGET_ESP32S3
    auto lcd_bus = lcd->getBus();
    if (lcd_bus->getBasicAttributes().type == ESP_PANEL_BUS_TYPE_RGB) {
        static_cast<BusRGB *>(lcd_bus)->configRGB_BounceBufferSize(lcd->getFrameWidth() * 10);
    }
#endif
#endif
    assert(board->begin());

Same as the example, since I get:

[E][Panel]esp_panel_touch.cpp:0143: Should be called before init()

[Lzw655]

Hi @extremerotary, The configInterruptActiveLevel() should be called between board->init(); and board->begin().