Skip to content

Conversation

@uLipe
Copy link
Member

@uLipe uLipe commented May 29, 2025

Add zephyr infrastructure to make the DMA2D to work on it, the idea takes inspiration on how PXP does that, the idea is to start from zephyr-lvgl side, and then eventually move it to the lvgl upstream.

@uLipe uLipe requested review from faxe1008 and pdgendt May 29, 2025 18:43
@uLipe uLipe self-assigned this May 29, 2025
@uLipe uLipe force-pushed the feature/dma2d_zephyr branch from fa62cdc to 62b182a Compare May 29, 2025 18:44
@uLipe
Copy link
Member Author

uLipe commented May 29, 2025

It seems the CI errors are not related to this PR

@uLipe uLipe force-pushed the feature/dma2d_zephyr branch from 62b182a to 125b2eb Compare May 30, 2025 16:45
@uLipe uLipe requested a review from faxe1008 May 30, 2025 16:50
acceleration of the DMA2D peripheral

Signed-off-by: Felipe Neves <[email protected]>
@uLipe uLipe force-pushed the feature/dma2d_zephyr branch from 125b2eb to 3b84511 Compare May 30, 2025 16:51
Copy link
Collaborator

@faxe1008 faxe1008 left a comment

Choose a reason for hiding this comment

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

Change it's self looks ok now. We have two possibilities:

  • Change the destination of this PR to a temporary branch based on master. We can not merge this to master since with the next sync there will be merge conflicts otherwise
  • Upstream this to LVGL master and get this in with the next sync

In any case would be lovely to have a PR (even if its only a draft) inside of the zephyr main repo to verify that this PR builds and also make testing easier. I myself do not own a board with that graphics engine but maybe @CharlesDias can take a look (he added the initial gluecode to zephyr)

@uLipe
Copy link
Member Author

uLipe commented May 31, 2025

@faxe1008 I like the second approach, I already tested it successfully on a H7 board that I have, so I can PR it to zephyr master branch as draft and source for testing

Meanwhile I can move this change to the lvgl upstream, then after 9.3 release it can be synced and the draft PR of zephyr gets up to date.

@uLipe
Copy link
Member Author

uLipe commented Jun 2, 2025

@faxe1008, I will closing this PR, since there is a better way to enable the DMA2D on Zephyr side without touching at least for now on LVGL files.

Maybe this thing can be addresed on zephyr side only.

@uLipe uLipe closed this Jun 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants