`LV_HOR_RES_MAX` and `LV_VER_RES_MAX`

How-to
1

I (think I) see now that in LVGL v8 the LV_HOR_RES_MAX and LV_VER_RES_MAX on ESP32 are not set from Kconfig, which they were on v7. I’m sure there’s a reason, but what is the preferred method of setting these now?

2

They no longer exist in v8.

3

They are configured in disp_drv now.
check this blog for more info

4

Ah, thanks. Duh…

5

Eh, but that does mean stuff like below in ‘lvgl_esp32_drivers’ also needs to change. no?

#if defined (CONFIG_CUSTOM_DISPLAY_BUFFER_SIZE)
#define DISP_BUF_SIZE   CONFIG_CUSTOM_DISPLAY_BUFFER_BYTES
#else
#if defined (CONFIG_LV_TFT_DISPLAY_CONTROLLER_ST7789)
#define DISP_BUF_SIZE  (LV_HOR_RES_MAX * 40)
#elif defined CONFIG_LV_TFT_DISPLAY_CONTROLLER_ST7735S
#define DISP_BUF_SIZE  (LV_HOR_RES_MAX * 40)
#elif defined CONFIG_LV_TFT_DISPLAY_CONTROLLER_ST7796S
#define DISP_BUF_SIZE  (LV_HOR_RES_MAX * 40)
#elif defined CONFIG_LV_TFT_DISPLAY_CONTROLLER_HX8357
#define DISP_BUF_SIZE  (LV_HOR_RES_MAX * 40)
#elif defined CONFIG_LV_TFT_DISPLAY_CONTROLLER_SH1107
#define DISP_BUF_SIZE  (LV_HOR_RES_MAX * LV_VER_RES_MAX)
#elif defined CONFIG_LV_TFT_DISPLAY_CONTROLLER_ILI9481
#define DISP_BUF_SIZE  (LV_HOR_RES_MAX * 40)
#elif defined CONFIG_LV_TFT_DISPLAY_CONTROLLER_ILI9486
#define DISP_BUF_SIZE  (LV_HOR_RES_MAX * 40)
#elif defined CONFIG_LV_TFT_DISPLAY_CONTROLLER_ILI9488
#define DISP_BUF_SIZE  (LV_HOR_RES_MAX * 40)
#elif defined CONFIG_LV_TFT_DISPLAY_CONTROLLER_ILI9341
#define DISP_BUF_SIZE  (LV_HOR_RES_MAX * 40)
#elif defined CONFIG_LV_TFT_DISPLAY_CONTROLLER_SSD1306

(lvgl_helpers.h, line 36, among other places)

6

We’re currently working on having support for v8 on the lv_port_esp32 repo. :smiley_cat:

7

Gents, 3 months later any update on progress of V8 support for ESP32?

1 Like
8

Dirty changes should be minor, I think. I can’t suggest anything right now since I’m busy, but I think less than 15 lines of change will bring you pass the compilation.
Clean changes is another story, I believe.

9

Glad I found this. I added:
#define LV_HOR_RES_MAX 320
To ili9341.h.
Is that a correct way to solve this? (This is the LCD driver I need.)