Skip to content

libraries: add lvgl #2558

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 1 commit into
base: main
Choose a base branch
from
Open

libraries: add lvgl #2558

wants to merge 1 commit into from

Conversation

rbudai98
Copy link
Contributor

Add lvgl library to no-os

Pull Request Description

Please replace this with a detailed description and motivation of the changes.
You can tick the checkboxes below with an 'x' between square brackets or just check them after publishing the PR.
If this PR contains a breaking change, list dependent PRs and try to push all related PRs at the same time.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the Coding style guidelines
  • I have complied with the Submission Checklist
  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe etc), if applies

buha
buha previously approved these changes Apr 16, 2025
View reviewed changes
amiclaus
amiclaus previously approved these changes Apr 22, 2025
View reviewed changes
@buha
Copy link
Contributor

buha commented Apr 23, 2025

can you please solve the conflict ?

@rbudai98 rbudai98 dismissed stale reviews from amiclaus and buha via 83b54d3 April 23, 2025 17:25
@rbudai98 rbudai98 force-pushed the add_lvgl_submodule branch from 5a38ad6 to 83b54d3 Compare April 23, 2025 17:25
@rbudai98
Copy link
Contributor Author

v2:

  • rebased main to avoid conflicts

@rbudai98 rbudai98 force-pushed the add_lvgl_submodule branch from 83b54d3 to 6d236d3 Compare April 25, 2025 07:22
@rbudai98
Copy link
Contributor Author

v3:

  • included missing libraries/lvgl directory

@rbudai98
libraries: add lvgl
57b98a4 
Add lvgl library to no-os

Signed-off-by: Robert Budai <robert.budai@analog.com>
@rbudai98 rbudai98 force-pushed the add_lvgl_submodule branch from 6d236d3 to 57b98a4 Compare April 25, 2025 14:28
@rbudai98
Copy link
Contributor Author

v4:

  • added lv_conf.h check, it should be included in projects, similar to freeRTOS

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

@buha buha buha left review comments

@amiclaus amiclaus amiclaus left review comments

@dbogdan dbogdan Awaiting requested review from dbogdan dbogdan is a code owner

@rbolboac rbolboac Awaiting requested review from rbolboac rbolboac is a code owner

@SRaus SRaus Awaiting requested review from SRaus SRaus is a code owner

@AAndrisa AAndrisa Awaiting requested review from AAndrisa AAndrisa is a code owner

@CiprianRegus CiprianRegus Awaiting requested review from CiprianRegus CiprianRegus is a code owner

@danmois danmois Awaiting requested review from danmois danmois is a code owner

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.
3 participants
@rbudai98 @buha @amiclaus