No Touchscreen + expander: Compilation error

esp-arduino-libs / ESP32_Display_Panel

Arduino library of driving display panel for the ESP SoCs

Apache License 2.0

59 stars 13 forks source link

No Touchscreen + expander: Compilation error #65

Open rolloo opened 6 days ago

rolloo commented 6 days ago

Hello, I am trying to build an example for Adafruit Qualia ESP32-S3 + Round RGB TTL TFT Display - 4" 720x720 - No Touchscreen

Compilation error: _'ESP_PANEL_TOUCH_BUSHOST' was not declared in this scope

_ESP_Panel_BoardCustom.h

#define ESP_PANEL_USE_TOUCH             (0) 
#define ESP_PANEL_USE_EXPANDER          (1)

Lzw655 commented 4 days ago

Hi @rolloo,

Sorry, there's a bug in the library. When setting up the expander's host, it should use the macro ESP_PANEL_EXPANDER_HOST_ID instead of ESP_PANEL_TOUCH_BUS_HOST.

The PR fixes this problem. Can you assist me in testing it?

rolloo commented 3 days ago

Hi @Lzw655

Compilation error: invalid conversion from 'int' to 'i2c_port_t' [-fpermissive]

  328 |     #define ESP_PANEL_EXPANDER_HOST_ID          (0)     // Typically set to 0
      |                                                 ~^~
      |                                                  |
      |                                                  int

Lzw655 commented 3 days ago

😂 Sorry, the macro should be ESP_PANEL_EXPANDER_HOST. Fixed in the new commit, please try again.