rzeldent / esp32-smartdisplay

PlatformIO library LVGL Drivers for Sunton Smart display boards (CYD Cheap Yellow Display). This library supports these boards without any effort. ESP32-2432S024N/R/C, ESP32-2432S028R/C, ESP32-3248S035R/C, ESP32_8048S070N/C
https://github.com/rzeldent/platformio-espressif32-sunton
GNU General Public License v3.0
394 stars 69 forks source link

Issue with new clone of esp32_smartdisplay and lvgl_ili9341.c #75

Closed lazzaroevan closed 8 months ago

lazzaroevan commented 9 months ago

I was compiling the demo fine until all of a sudden this error popped up, and now it wont compile and none of my previous projects are compiling either.

.pio/libdeps/esp32-2432S028Rv2/esp32_smartdisplay/src/lvgl_ili9341.c: In function 'lvgl_lcd_init': .pio/libdeps/esp32-2432S028Rv2/esp32_smartdisplay/src/lvgl_ili9341.c:49:180: error: expected ',' or ';' before '}' token const esp_lcd_panel_dev_config_t panel_dev_config = {.reset_gpio_num = -1, .color_space = ESP_LCD_COLOR_SPACE_BGR, .bits_per_pixel = 16, .vendor_config = ILI9341_VENDOR_CONFIG}; ^ Compiling .pio\build\esp32-2432S028Rv2\lib63e\FFat\FFat.cpp.o *** [.pio\build\esp32-2432S028Rv2\libb84\esp32_smartdisplay\lvgl_ili9341.c.o] Error 1

Do you have any Idea what happened?

rzeldent commented 9 months ago

The board definitions contained structures with the definition. Not all compilers could handle this (mac could not compile) These structures are now initialized in the code but the pins and timings in the boards definition.

To fix this update the smartdisplay library and the boards submodule to the latest main.

Hope this helps.

lazzaroevan commented 9 months ago

|-- esp32_smartdisplay @ 2.0.2+sha.cb0cf33 Even with a newly built fresh clone the demo is still not working and throwing the same error, do you know what could be causing this? I am using the new esp32_2432S028v2 board that you just updated for.

rzeldent commented 9 months ago

Yes, something i'm workin on. The develop branch of the boards has this fixed already.

In the file esp32-2432S028Rv2.json the vendor config has a } to many at the end

lazzaroevan commented 9 months ago

Gotcha, Thank you

rzeldent commented 9 months ago

Fix also merged to main. Was still looking for another way to fix this. This does not work well in CI builds...

rzeldent commented 8 months ago

Fixed with single quotes in defines