-
Notifications
You must be signed in to change notification settings - Fork 535
Hughsie/elantp forcetable #9800
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.
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
Conversation
88100c2 to
ea6dd64
Compare
There was a problem hiding this 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_WAITWDTflag 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 |
|
@jinglewu can you have a look at some of the copilot-found issues please? |
Co-authored-by: Copilot <[email protected]>
…the current iteration. Co-authored-by: Copilot <[email protected]>
Type of pull request: