Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@hughsie
Copy link
Member

@hughsie hughsie commented Jan 16, 2026

Type of pull request:

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for Elan touchpad firmware updates with improved forcetable handling, support for new IC types (0x14 and 0x15), and a new IAP password for version 7.

Changes:

  • Added support for IC types 0x14 and 0x15 with appropriate password handling and forcetable address logic
  • Implemented early firmware write termination when ETP_FW_IAP_END_WAITWDT flag is detected
  • Relaxed firmware validation to allow 0xFFFF as a valid value instead of treating values > 0x7FFF as invalid

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

File Description
plugins/elantp/fu-elantp-common.h Added new IAP password constant for version 7 and new status flag for early termination
plugins/elantp/fu-elantp-firmware.c Relaxed validation bounds to accept 0xFFFF values for IAP and module addresses
plugins/elantp/fu-elantp-hid-device.c Extended device support for IC types 0x14/0x15, improved forcetable handling, and implemented early write termination
plugins/elantp/fu-elantp-i2c-device.c Implemented early write termination when watchdog timer flag is set

@hughsie
Copy link
Member Author

hughsie commented Jan 16, 2026

@jinglewu can you have a look at some of the copilot-found issues please?

@jinglewu jinglewu marked this pull request as ready for review January 16, 2026 14:45
@jinglewu jinglewu self-requested a review as a code owner January 16, 2026 14:45
@jinglewu jinglewu enabled auto-merge (squash) January 19, 2026 01:47
@jinglewu jinglewu merged commit d2a8bae into main Jan 19, 2026
19 checks passed
@jinglewu jinglewu deleted the hughsie/elantp-forcetable branch January 19, 2026 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants