Add support to CST816 chips with different register for chip ID #2
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.
I have successfully used the library with a CST816D chips with no other modification than removing the test for the chip_type. For example the
Waveshare esp32 s3 touch lcd 2
uses it as well as a generic ESP32-C3 round display board (that is not in CP yet).When running, the library reports:
But it's the chip ID that's supposed to be
B6
.I was not able to find a datasheet containing that information, but the CST816_TouchLib Arduino library has these constants. Also in zephyr's support.
So since the rest of the registers used seem to work the same, this PR just tries to find a chip ID to validate that it's a valid chip.