spi_lcd_touch example: wrong colors with ILI9341 display (IDFGH-8811)

[abusca]

Answers checklist.

• I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
• I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
• I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.

v5.0-rc1

Operating System used.

macOS

How did you build your project?

Eclipse IDE

If you are using Windows, please specify command line type.

No response

Development Kit.

ESP32-2432S028

Power Supply used.

USB

What is the expected behavior?

I expected to see the dash board with correct colors. For example, the arc and tick lines at the end of the scale should be red.

What is the actual behavior?

Some colors are not correct. For example, the arc and tick lines at the end of the scale are shown yellow.

Steps to reproduce.

I took spi_lcd_touch example and I only modified LCD pinout defines with the ones of ESP32-2432S028 board.

Debug Logs.

No response

More Information.

No response

[suda-morris]

please try to turn on or off the CONFIG_LV_COLOR_16_SWAP option

[abusca]

CONFIG_LV_COLOR_16_SWAP is enabled by default. If I turn it off, I see even weirder colors.

CONFIG_LV_COLOR_16_SWAP ON:

CONFIG_LV_COLOR_16_SWAP OFF:

[espzav]

@abusca The first image looks good. I have it same:

[espzav]

@abusca The example is universal for two LCD controllers and the default CONFIG_LV_COLOR_16_SWAP is disabled for another LCD controller. If you enable it, it is ok for ILI9341.

[abusca]

@abusca The first image looks good. I have it same:

Shouldn't those lines and arc be in red?

esp-idf/examples/peripherals/lcd/spi_lcd_touch/main/lvgl_demo_ui.c

Line 60 in 3f2f35b

indic = lv_meter_add_scale_lines(meter, scale, lv_palette_main(LV_PALETTE_RED), lv_palette_main(LV_PALETTE_RED), false, 0);

[espzav]

@abusca I tried another project with image and you are right, there is some problem with colors. I am sorry, I didn't see before. I will fix it soon.

[espzav]

Hi @abusca, the component ILI9341 was fixed, but the example has still bad rgb_endian. The merge request of the example is waiting for approve. You should use this .rgb_endian = LCD_RGB_ENDIAN_BGR, in the example.
The CONFIG_LV_COLOR_16_SWAP should be enabled (as default in example).

This is the new fixed result:

[abusca]

Thank you @espzav

[espzav]

@abusca You are welcome.

[Alvin1Zhang]

Thanks for sharing the updates, fix on master branch is available at 2e3c72a. Thanks.