Skip to content

(feat) Add basic mimxrt support.#402

Open
andrewleech wants to merge 1 commit into
lvgl:masterfrom
andrewleech:mimx
Open

(feat) Add basic mimxrt support.#402
andrewleech wants to merge 1 commit into
lvgl:masterfrom
andrewleech:mimx

Conversation

@andrewleech

@andrewleech andrewleech commented Nov 13, 2025

Copy link
Copy Markdown
Contributor

I'm working on getting LVGL working cleanly on the NXP 1170 EVK with a LVGL v9 compatible driver in progress here: https://github.com/andrewleech/imxrt1170_display_driver

These minimal changes to the LVGL bindings files enables the build environment to enable port specific features more easily.


Summary by cubic

Add basic NXP i.MX RT (mimxrt) support with PXP acceleration and port-aware build rules. LVGL v9 now runs on the RT1170 EVK with PXP.

  • New Features
    • Detect the active port in micropython.mk; add a mimxrt block that sets LV_USE_PXP, LV_USE_DRAW_PXP, LV_USE_GPU_NXP_PXP, and LV_USE_GPU_NXP_PXP_AUTO_INIT via CFLAGS_USERMOD, and silences specific PXP driver warnings for a clean -Werror build.
    • Guard LV_DRAW_BUF_ALIGN, LV_USE_PXP, LV_USE_DRAW_PXP, and LV_USE_ROTATE_PXP with #ifndef in lv_conf.h so boards can override via build flags without redefinition warnings.
    • Add a submodules rule to initialize lvgl and pycparser from the port build with make submodules.

Written for commit 9a2fffa. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All reported issues were addressed across 2 files

Prompt for AI agents (all 1 issues)

Understand the root cause of the following 1 issues and fix them.


<file name="lv_conf.h">

<violation number="1" location="lv_conf.h:255">
`LV_USE_PXP` is still the controlling flag for this block, but the new guard defines `LV_USE_DRAW_PXP` instead, leaving `LV_USE_PXP` undefined so the feature can never be enabled.</violation>
</file>

Since this is your first cubic review, here's how it works:

  • cubic automatically reviews your code and comments on bugs and improvements
  • Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
  • Ask questions if you need clarification on any suggestion

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

Comment thread lv_conf.h Outdated
@andrewleech
andrewleech force-pushed the mimx branch 3 times, most recently from 17c31f7 to 09a4d3c Compare November 13, 2025 04:35
Enable NXP's PXP 2D accelerator for the mimxrt port by setting
LV_USE_PXP, LV_USE_DRAW_PXP, LV_USE_GPU_NXP_PXP and
LV_USE_GPU_NXP_PXP_AUTO_INIT via CFLAGS_USERMOD, since the mimxrt
port has no board-level makefile of its own to provide them.
Silence known PXP driver warnings that would otherwise fail a
-Werror build.

Guard LV_USE_PXP, LV_USE_DRAW_PXP, LV_USE_ROTATE_PXP and
LV_DRAW_BUF_ALIGN with #ifndef so a board can override any of
these from its own build flags without a macro redefinition
warning.
@andrewleech
andrewleech force-pushed the mimx branch 2 times, most recently from 4fa7830 to 9a2fffa Compare July 16, 2026 06:03
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.

1 participant