Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(display): add API to set raw buffer to display #5274

Merged
merged 6 commits into from
Jan 11, 2024

Conversation

XuNeo
Copy link
Collaborator

@XuNeo XuNeo commented Jan 11, 2024

Help us review this PR! Anyone can approve it or request changes.

Description of the feature or fix

The drivers have been changed back original logic. Fix #5270 Fix #5271
Related to #5204 lvgl/lv_binding_micropython#313
Discussion on #5273

lv_display_set_raw_buffers is kept.

Checkpoints

The drivers have been changed back original logic.
Fix lvgl#5270
Related to lvgl#5204 lvgl/lv_binding_micropython#313
Discussion on lvgl#5273

Signed-off-by: Xu Xingliang <xuxingliang@xiaomi.com>
Signed-off-by: Xu Xingliang <xuxingliang@xiaomi.com>
@kisvegabor kisvegabor mentioned this pull request Jan 11, 2024
22 tasks
@XuNeo
Copy link
Collaborator Author

XuNeo commented Jan 11, 2024

I have introduced new API lv_display_set_raw_buffers for better naming. lv_display_set_draw_buffers is kept, so @PGNetHun 's new following up can still work. And it does mean set lv_draw_buf_t as draw buffers.

@kisvegabor Please check if it's suitable.

The micropython CI failed and i'm looking into it.

src/display/lv_display.c Outdated Show resolved Hide resolved
src/display/lv_display_private.h Show resolved Hide resolved
src/display/lv_display.h Outdated Show resolved Hide resolved
@kisvegabor
Copy link
Member

For the MicroPython CI, probably this needs to be updated: https://github.com/lvgl/lvgl/blob/master/tests/micropy_test/micropy.py#L88

Signed-off-by: Xu Xingliang <xuxingliang@xiaomi.com>
Signed-off-by: Xu Xingliang <xuxingliang@xiaomi.com>
@XuNeo
Copy link
Collaborator Author

XuNeo commented Jan 11, 2024

CI should pass now. I missed one driver change in lv_linux_fbdev.c

Signed-off-by: Xu Xingliang <xuxingliang@xiaomi.com>
Signed-off-by: Xu Xingliang <xuxingliang@xiaomi.com>
@kisvegabor
Copy link
Member

Ah, LV_ASSERT_MSG doesn't handle formatted strings. Let's fix is once later.

Thank your this PR!

@kisvegabor kisvegabor merged commit 8735491 into lvgl:master Jan 11, 2024
15 checks passed
@PGNetHun
Copy link
Collaborator

Thx for this PR, I will update MicroPython display drivers to use it in the MicroPython - LVGL v9 update PR

lion2tomato pushed a commit to lion2tomato/lvgl that referenced this pull request Mar 19, 2024
VELAPLATFO-20122

The drivers have been changed back original logic. Fix lvgl#5270 Fix lvgl#5271
Related to lvgl#5204 lvgl/lv_binding_micropython#313
Discussion on lvgl#5273

Change-Id: I798af8753f50ba0eed9d6b7f00e184fb032bde1c
Signed-off-by: Xu Xingliang <xuxingliang@xiaomi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants