LVGL: Add support of LVGL9 into LVGL port

[espzav]

ESP-BSP Pull Request checklist

• Version of modified component bumped
• CI passing

Change description

Support of LVGL9 in ESP LVGL port (including support LVGL8 too).

LVGL9 supported parts:

• Automatic selection between LVGL8 and LVGL9
• LCD support
• Screen rotation
• Monochromatic screen
• LCD Touch
• Navigation buttons
• Encoder
• USB HID device

Other things not supported with LVGL9:

• Images
  - [ ] lv_obj_set_style_image_opa

TODO (Create Jira tickets)

• Enable SW rotation in LVGL9
• Support more color models (now only 16bit color RGB565 is supported)
• apply rotation from display configuration - esp_lvgl_port: apply rotation from display configuration (BSP-443) #278

[espzav]

@tore-espressif PTAL
All tested examples are here: #280
I didn't test it more in LVGL8, as we decided not support LVGL8 in BSP examples. Some simple examples are supported in LVGL8.
The main example in component is fully supported in both versions of LVGL.

• [] I think, that I should update IDF examples too.

[tore-espressif]

@espzav great job! I left few comments that might be worth considering

[espzav]

@tore-espressif Thank you for review. Fixed all except mail esp_lvgl_port.c file (as a comment).

• Divided header files too - PTAL

[espzav]

My last commit fixing usage of FPS monitor. I don't know, if it is right solution or workaround - I am discussing with Gabor: lvgl/lvgl#4926 (comment)

[Lzw655]

Hi @espzav, I also tried to port LVGL v9, and I've encountered some issues when using multiple CPU cores to render in parallel, and I'd like to share them with you:

• We cannot directly use the FreeRTOS from LVGL because the header file paths it contains are not suitable for ESP-IDF. Moreover, the use of taskENTER_CRITICAL() is not compatible. However, with some simple modifications, it can be adapted for use. I recommend getting official support for our system from LVGL or enabling LV_OS_CUSTOM and registering it in esp_lvgl_port.
• Ensure that the lv_init() function and lv_timer_handler() are in the same task; otherwise, the program will hang.

BTW, it indeed improves the performance by using multiple CPU cores. By running the v8 benchmark demo, the use of 2-unit increased the weighted FPS from 51 to 73 compared to a 1-unit.

[espzav]

My last commit fixing usage of FPS monitor. I don't know, if it is right solution or workaround - I am discussing with Gabor: lvgl/lvgl#4926 (comment)

Here is the fix in LVGL: lvgl/lvgl#5518

[espzav]

@Lzw655 Thank you for info, I will try it.

[tore-espressif]

@espzav few more details :)

[espzav]

@tore-espressif Thank you for your review! All fixed. Is it ok, can we merge it now?

[tore-espressif]

@tore-espressif Thank you for your review! All fixed. Is it ok, can we merge it now?

I can see that the CI is still using LVGL 8 https://github.com/espressif/esp-bsp/actions/runs/7707868489/job/21005817735?pr=268#step:4:186

Could you please add override_path for esp_lvgl_port to all BSPs? Eg here https://github.com/espressif/esp-bsp/blob/master/bsp/esp32_s3_korvo_2/idf_component.yml#L29

This way, the CI and local builds will automatically pick esp_lvgl_port from the current state of the repo, instead the latest released version.

After the CI passes with LVGL9 build, we should be good to merge :)

[tore-espressif]

@espzav Will we close this PR in favor of #280 ?