[riot-notifications] [RIOT-OS/RIOT] pkg/lvgl: set LVGL thread priority higher than main (#15428)
Jean Pierre Dudey
notifications at github.com
Wed Nov 11 09:56:23 CET 2020
The RIOT community cares a lot about code quality.
Therefore, before describing what your contribution is about, we would like
you to make sure that your modifications are compliant with the RIOT
coding conventions, see https://github.com/RIOT-OS/RIOT/wiki/Coding-conventions.
### Contribution description
Using a lower priority than of the main thread would make the LVGL thread to preempt the main thread, for example,
if you do a `xtimer_msleep(200);` on the main thread giving any chance for the LVGL thread to preempt it
while working with LVGL on the main thread would make the CPU crash.
To reproduce the bug just create various widgets, and fill them with data from a sensor that "waits" for data to be
Aditionally fixed the RGB and inversion parameters for the `esp32-wrover-kit` that were wrong (I'm sorry for it), didn't notice that it was wrong on the `driver_ili9341` example since the RIOT logo looked very well, but the background should have been black (was white due to the inversion).
### Testing procedure
- `make -C tests/pkg_lvgl BOARD=esp32-wrover-kit` or with any board/configuration that can be used to test LVGL.
### Issues/PRs references
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* pkg/lvgl: set LVGL thread priority higher than main
* boards/esp32-wrover-kit: don't invert & use RGB for ILI9341
-- File Changes --
M boards/esp32-wrover-kit/include/board.h (4)
M pkg/lvgl/Makefile.include (2)
-- Patch Links --
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications