Skip to content

Add Goodix 27c6:5117 support#32

Closed
gulp wants to merge 171 commits intogoodix-fp-linux-dev:0x00002a/libfprint-sigfmfrom
gulp:goodix-5117-support
Closed

Add Goodix 27c6:5117 support#32
gulp wants to merge 171 commits intogoodix-fp-linux-dev:0x00002a/libfprint-sigfmfrom
gulp:goodix-5117-support

Conversation

@gulp
Copy link

@gulp gulp commented Jan 9, 2026

Summary

  • Add 5117 device ID to goodix511 driver id_table
  • Update firmware version to GF_ST411SEC_APP_12109 (5117 variant)
  • Enable SIGFM algorithm for low-resolution sensor matching

Tested on

  • Huawei MateBook 13 2021
  • Arch Linux (Omarchy)
  • Sensor: Goodix 27c6:5117 (80x88 pixels)

Details

The 5117 sensor is similar to 5110 but reports different firmware. Without SIGFM, the NBIS minutiae detector fails with "No minutiae found" due to the low resolution.

With these changes, enrollment and verification work correctly.

0x00002a and others added 28 commits November 10, 2022 22:59
Signed-off-by: Alexander Meiler <rootd@users.noreply.github.com>
Signed-off-by: Alexander Meiler <rootd@users.noreply.github.com>
scan ssm wasn't completing and hence wasn't getting cleaned up
I changed my email, rather have the new one set in stone :p
- Add 5117 device ID to id_table
- Update firmware version to GF_ST411SEC_APP_12109
- Enable SIGFM algorithm for low-res sensor matching
- Fix score_threshold field name for sigfm branch

Tested on Huawei MateBook 13 2021 with Omarchy (Arch Linux).
Copy link
Member

@egormanga egormanga left a comment

Choose a reason for hiding this comment

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

Just a first glance

{ .vid = 0x1c7a, .pid = 0x0575 },
{ .vid = 0x1c7a, .pid = 0x0576 },
{ .vid = 0x27c6, .pid = 0x5042 },
{ .vid = 0x27c6, .pid = 0x5110 },
Copy link
Member

Choose a reason for hiding this comment

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

Why is this change needed?

Copy link
Author

Choose a reason for hiding this comment

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

@egormanga This deletion was unintentional - it happened during troubleshooting to support the 0x5117 device. I don't actually need to remove 0x5110 support.

The 0x5110 device should definitely remain supported. I'll close this PR.

Copy link
Member

Choose a reason for hiding this comment

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

Do we really need this?

@gulp gulp closed this Jan 9, 2026
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.

8 participants