ili9341 driver backlight about lv_port_esp32

[kisvegabor]

Hi,

It should be specific to the display. Probably your display needs '0' on the backlight pin to enable it.

[C47D]

Hi,

I'm Carlos at the forum and found the same issue [1], maybe we can add some options so the users can define it's board inside a component.mk on the drv directory like so:

// Uncomment the line with your board:
CFLAGS+=-DESP32_BOARD=ESP32_WROVER_KIT
//CFLAGS+=-DESP32_BOARD=ESP32_30GPIO_KIT

This will make the following code set the correct level of the BCLK pin on the user board:

#if (ESP32_BOARD==ESP32_WROVER_KIT)
gpio_set_level(ILI9341_BCKL, 0)
#else // Other boards
gpio_set_level(ILI9341_BCKL, 1)
#endif

This can also help assign the right pins on this board or any board with hardwired LCDs.

What do you think?

[1] Lvgl in ESP-WROVER-KIT v4.1

[kisvegabor]

Hi Carlos,

Sounds good!
If there is no obsctacle we can use the Wrover Kit's pins as default. On Wroom32 modules, we can use any pins so we can be compatible with the Wroomkit by default.
Maybe the TP's SPI could be an issue because I saw that you need to use the SPI in half-duplex mode to achieve 40 MHz.

[C47D]

The Wrover kit doesn't have a touch screen, it's just the TFT display, I got the code ready for a pull request but I'm waiting for my other PR to get merged. The TFT on the Wrover kit is 340x240, where should I update the code for this?

[kisvegabor]

The Wrover kit doesn't have a touch screen, it's just the TFT display,

Great, then it won't be an issue.

I'm waiting for my other PR to get merged.

Just merged 🙂

The TFT on the Wrover kit is 340x240, where should I update the code for this?

It should be changed in lv_conf.h (LV_HOR/VER_RES_MAX).