No wake up on receiving LoRa packet on companion firmware #1076
+32
−16
Conversation
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
* dev: * Sensor: DISCOVER_REQ, prefix_only support * fix: repeater and room server telemetry requests now return all telemetry for _READ & _WRITE ACL permissions. * added StrHelper::fromHex() sensor: copy control data code from repeater
* dev: * ver 1.10.0 Update library.json to latest libs and version set max contacts to 350 and channels to 40 for esp32c3, s3 and c6 SensorManager: remove setSettingByKey
* dev: (22 commits) tweaks get/set adcMultiplier logic add board.setAdcMultiplier to room server and sensor Repeater: add adc.multiplier setting rename adverts rename faketec to promicro fix: move bme680 detection before bme280 Modify CMD_GET_STATS with sub-types for core, radio, and packet statistics. Consolidated to a single RESP_CODE_STATS with a second byte to identify response structure. Updated documentation and examples to reflect the new command structure and response parsing. xiao_s3: relocate serial pins on repeater_bridge_rs232 xiao_s3: use environment sensor manager and add sensor role Keepteen LT1: remove terminal_chat and sensor targets initial support: Keepteen LT1 Allow SF smaller than 7 to be saved Fix RAK4631 GPS UART pin macros Remove unused statistics formatting methods and associated header includes from MyMesh class. Whoops. Update statistics handling to use binary frames instead of JSON formatting for consistency with other companion commands. Added documentation of frame structure with code examples. Reverted MyMesh constructor for simplicity. Updated formatStatsReply method to use new member variables for statistics handling. Removed excess variable creation Add statistics commands and response handling in MyMesh Fix for display not coming on after poweron tweak getBattMilliVolts to report battery more accurately change ADC_MULTIPLIER to better reflect battery voltage ...
|
I made some tests with these changes. Power consumption: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
In fact, it puts the device into a complete sleep in companion firmware. :)
The problem is that the current implementation allows the device to wake up with any LoRa packet, even one that isn't related to the node. Therefore, any activity in the mesh network causes the device to turn on, and then it simply drains the battery, since the device remains on and active.
The presented changes remove wakeup on LoRa activity for companion firmware.
The discussion here #388 (comment)
and #1049