Skip to content

EMB-486: Goodix touchscreen driver - #223

Merged
Frederic Meeuwissen (Frederic98) merged 3 commits into
master/imx8from
EMB-486_goodix_touch_driver
Jul 22, 2026
Merged

EMB-486: Goodix touchscreen driver#223
Frederic Meeuwissen (Frederic98) merged 3 commits into
master/imx8from
EMB-486_goodix_touch_driver

Conversation

@Frederic98

@Frederic98 Frederic Meeuwissen (Frederic98) commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Description

Support the new NGP displays with Goodix GT911 touch controller.
This works in parallel with the existing F4 screens. The kernel will figure out which chip is available and load the corresponding driver.

A patch is applied to the linux submodule: https://github.com/Ultimaker/linux/tree/EMB-486_Polling_goodix_touchscreen.
However, there is no active "master" branch for iMX8. So, it will point to a new feature branch. Maybe we should create a master branch?

How has this been tested

On NGP, attached new screen - touch works.
On NGP, attach old screen - touch still works.

Ready for Review Checklist

To help with deciding if this PR is RFR, use this checklist.

The author confirms that:

  • the author has self-reviewed this work and is highly confident about the quality
  • this work satisfies all acceptance criteria that are stated in the linked ticket
  • this work has been tested on all product families and the process and results are documented in the above section
  • The description above is concise yet complete
  • the reviewer has been offered a walkthrough (if needed)
  • the code is cleaned from any rubbish (e.g. meaningless comments, log-spamming, etc...)
  • remaining #TODO comments mention a Jira ticket number
  • all CI checks are passing
  • all commits are (re)structured to be meaningful and clearly arranged, and all are prepended with the ticket number for traceability

- Include goodix driver in kernel (not as a module)
- Add the chip in a new device tree node
- Remove ulticontroller 4.1 (NGP is now back on the standard 4.0 version)
- Add polling support from upstream linux
@Frederic98
Frederic Meeuwissen (Frederic98) merged commit bfdebe8 into master/imx8 Jul 22, 2026
4 checks passed
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