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

Update master branch to LVGL v9 version #311

Merged
merged 71 commits into from
Jan 23, 2024
Merged

Conversation

PGNetHun
Copy link
Collaborator

@PGNetHun PGNetHun commented Jan 8, 2024

  • Update LVGL to v9-rc version, commit: lvgl/lvgl@09cb87c
  • Fix gen_mpy.py binding script
  • Amend MicroPython files and examples to the new LVGL namings (for example: img -> image, btn -> button)
  • Update README.md: rename Micropython -> MicroPython
  • Remove lv.COLOR_FORMAT.NATIVE_REVERSED, and use instead lv_draw_sw_rgb565_swap() function for some displays
  • Remove lv.COLOR_DEPTH from display drivers and examples
  • Modify ili9XXX.py: change class attributes to instance attributes (-> eliminate "singleton" class attributes)
  • Modify ili9XXX.py: set explicit default color formats instead of lv.COLOR_FORMAT.NATIVE
  • Modify MicroPython display drivers (ili9XXX.py and st77xx.py) and examples to use new display draw buffer approach (related PR: refactor(display): use draw_buf_t for display lvgl#5204)
  • Update examples to align to LVGL changes
  • Disable LVGL log (by default)
  • Merge master changes into PR branch

@PGNetHun
Copy link
Collaborator Author

PGNetHun commented Jan 12, 2024

@kisvegabor , @XuNeo , @kdschlosser ,
Could you please review & approve this big PR?
I don't want to add any more commits to it, so it is ready to merge to master branch.
After this, every new PRs should target master.
(and prepare the work to upgrade MicroPython to v1.22)

PS: this PR is tested on M5Core2 ESP32 device, and it works

Copy link
Member

@kisvegabor kisvegabor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not super confident, but as far as I can tell it's ok 🙂

@PGNetHun
Copy link
Collaborator Author

PGNetHun commented Jan 18, 2024

Reminder: disable LV_USE_ARABIC_PERSIAN_CHARS
see: lvgl/lvgl#5386

UPDATE: it is already disabled

examples/Dynamic_loading_font_example.py Outdated Show resolved Hide resolved
examples/example1.py Outdated Show resolved Hide resolved
@PGNetHun
Copy link
Collaborator Author

PGNetHun commented Jan 18, 2024

To have a linear and clean git history on master branch, use "Squash and merge"

@kisvegabor
Copy link
Member

@PGNetHun feel free to merge it when it feels appropriate.

Copy link
Contributor

@XuNeo XuNeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can only review part of the code. It looks good to me.

@PGNetHun PGNetHun changed the title Update master branch to LVGL v9 prerelease version Update master branch to LVGL v9 version Jan 23, 2024
@PGNetHun PGNetHun merged commit 5e80637 into master Jan 23, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v9 Required for LVGL v9 compatability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants