Add 11 SmartSens chip ID detection cases#151
Merged
Conversation
Sourced from "思特威sensor的chipid整理-懒人必备" by 雨之小 (CSDN, 2025, CC-BY-SA 4.0): https://blog.csdn.net/fzktongyong/article/details/149630895 0x0031 -> SC031GS 0x0108 -> SC035GS 0x0132 -> SC132GS 0x17cb -> SC210IoT 0xcb34 -> SC230AI 0xcb6a -> SC231HAI 0x9c41 -> SC3336P 0x8e39 -> SC530AI 0xce50 -> SC5336 0xc170 -> SC831AI 0xeb2c -> SC2355 Not field-tested by submitter; values transcribed from third-party compilation. Paper analysis only. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds 11 missing SmartSens chip-ID → model-name mappings to
detect_smartsens_sensor()insrc/sensors.c. Previously these chip IDs returned viaSENSOR_ERR()and showed up in the field as unknown sensors.Source
All 11 mappings are transcribed from a public third-party compilation:
The author claims 33 SmartSens chip IDs total. After cross-referencing against the existing
detect_smartsens_sensor()switch, 22 of those 33 are already covered (with ipctool's coverage being substantially broader than the blog's — ~50 chip IDs total vs 33). The 11 below are the gaps.Cases added
0x00310x01080x01320x17cb0xcb340xcb140xcb6a0xcb5c → SC23310x9c410xcc41 → SC33360x9c42 → SC4336Ppattern (P-variant high byte 0xCx → 0x9x)0x8e390xce1f → SC501AI0xce500xce1f → SC501AI0xc1700xc143 → SC830AI0xeb2cDecisions deliberately NOT made
The blog also reveals three chip IDs shared between two model names:
0xcd2e0xce1f0xcc41I chose not to touch these. Commit b328d02 ("Fix SC3338 -> SC3336") established the project precedent of picking one canonical name per chip ID rather than surfacing aliases — these would all require maintainer judgment about which name to pick.
Apparent blog↔ipctool conflicts (not changed)
A few entries in the blog disagree with ipctool's existing switch:
0x2238— blog says SC2232, ipctool currently says SC2315E (with comment "aka SC4239Р and SC307E"). ipctool wins; the existing comment shows this was field-validated.0x2311— blog says SC2310, ipctool says SC2315.0x9d1e, ipctool says0xbd1e. Likely a typo in the blog (B↔9 hex confusion); ipctool's value is field-validated.No changes made for these.
Caveat
These chip-ID values are not field-tested by me — they are transcribed from a third-party compilation. If anyone has these sensors on actual hardware and can run `ipctool` against them to confirm or refute, that would be valuable. Reverting any individual case is trivially safe.
Test plan
caseblocks; no new symbols, includes, or APIs)ipctooloutput🤖 Generated with Claude Code