Skip to content

Update/micropython v1.24.1 usermod #367

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: update/micropython_v1.24.1_usermod
Choose a base branch
from
Open

Update/micropython v1.24.1 usermod #367

wants to merge 5 commits into from

Conversation

diskman88
Copy link

No description provided.

@diskman88
Changes to be committed:
2b8533e 
	modified:   .gitmodules
	deleted:    lvgl
	deleted:    pycparser
@diskman88
把lvgl pycparser从子模块变为文件夹:
c021cd3
@diskman88
add lv_binding_micropython
485ad09
@diskman88
fix lvgl:
2cac845 
	deleted:    lvgl/.devcontainer/__lv_conf.h__
@diskman88
重新加入中文字体:
cd0c330 
	modified:   lv_conf.h
	modified:   lvgl/src/font/lv_font.h
	new file:   lvgl/src/font/lv_font_siyuan_heiti_medium_16.c
	modified:   lvgl/src/font/lv_symbol_def.h
PGNetHun
PGNetHun requested changes Mar 15, 2025
View reviewed changes
Copy link
Collaborator

@PGNetHun PGNetHun left a comment

Choose a reason for hiding this comment

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

Rejected because LVGL submodule was deleted, and instead LVGL source code is added directly.
That makes future LVGL updates more difficult.

@PGNetHun
Copy link
Collaborator

PGNetHun commented Mar 15, 2025 •
edited
Loading

Hello @diskman88 ,

lv_micropython is updated with MicroPython v1.24.1 code changes, and LVGL is used as user module.
See PRs:

Thank you that you wanted to help to upgade to latest stable MicroPython version.
Could you please checkout latest lv_micropython, and test it that it works as you expect?
If there are any problems, please create github issue (or PR if you know also how to fix).

If everything is OK, could you please close this PR too?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PGNetHun PGNetHun PGNetHun requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@diskman88 @PGNetHun