Add QMP6988 environmental telemetry support#10279
Draft
mokjpn wants to merge 2 commits into
Draft
Conversation
Author
|
This firmware PR depends on the protobuf change here: split the enum addition into the protobufs repository as requested. Once that protobuf PR is merged, I will update this firmware PR to align the generated protobuf output cleanly. |
Author
|
Marking this as draft until meshtastic/protobufs#899 is merged, since the firmware side still needs to be aligned with the generated protobuf output after that change lands. |
2 tasks
Author
|
Update: the protobuf dependency has been replaced with meshtastic/protobufs#921, based on protobufs develop. The previous meshtastic/protobufs#899 was closed because it was based on master and would not match the current firmware protobuf baseline. |
This was referenced May 24, 2026
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
This PR adds environmental telemetry support for the QMP6988 pressure sensor.
Changes included:
0x70and0x56QMP6988Sensortelemetry driverEnvironmentTelemetryNotes
This PR depends on the protobuf enum addition here:
During hardware testing, some QMP6988 devices did not accept the reset command over this hardware path, but calibration readout and measurements still worked correctly. The driver therefore treats reset as optional and continues initialization when the sensor is otherwise responsive.
This PR is intended to add QMP6988 support only. It does not address unrelated SHT3x/SHT30 hardware address conflicts on specific boards.
Testing
Tested on:
Observed result:
Build tested:
pio run -e m5stack-unitc6l🤝 Attestations
Other:
pio run -e m5stack-unitc6l