Skip to content
This repository was archived by the owner on Jan 28, 2021. It is now read-only.

Added powerSaveMode #53

Closed
wants to merge 14 commits into from
Closed

Added powerSaveMode #53

wants to merge 14 commits into from

Conversation

PaulZC
Copy link
Collaborator

@PaulZC PaulZC commented Dec 19, 2019

Hi Nathan (@nseidle),
I think this should be my last PR for a while!
I have added powerSaveMode which uses the UBX-CFG-RXM message to put the GNSS into power save mode. You can put the receiver back into continuous mode by calling powerSaveMode(false).
There is, of course, an example too.
I've tested it on the ZOE-M8Q and the current drops from 25-28mA during acquisition to approx. 9mA when low power mode kicks in. (That's with a passive antenna. Using an active antenna will add 10's of mA to the total draw.)
Enjoy!
Merry Christmas,
Paul

PaulZC and others added 14 commits December 10, 2019 20:20
Added addGeofence, clearGeofences and getGeofenceState to allow up to four geofences to be defined. The geofence status can also be routed to one of the PIO pins and used as an interrupt etc..
Include NEO-M9N chip antenna and u.FL connector versions in README.md. Since there are no changes in code, we don't need to release another version tag. ;D
@nseidle
Copy link
Member

nseidle commented Dec 20, 2019

I'm generally ready to merge. Two topics to discuss:

  • The power save feature works well for my SAM-M8Q but fails on the ZED-F9P as it doesn't have support for the -CFG-RXM command (it uses UBX-RXM instead). Is there a better way to test and warn the user that a given command isn't supported? Best case is a Serial.print("This command is not supported on this hardware"). Next best thing is to notify user when a command causes a UBX-ACK fail (as was the case with ZED-F9P and NAV-CFG-RXM).

  • I'm making a bit of a mess of this PR. I'm attempting to remove/reject the files that are really parts of other PRs. It seems I'm supposed to ask you to fix the PR.

Please remove the dynamic model and geofence bits in this PR. If that's too much work, I could copy/paste your changes locally and apply the bits I want, then close this PR but you get none of the glory ;). Let me know if you want to attempt to fix this PR or if I should copy/paste.

In the future, please branch from master rather than your own branches. And I am far from a git guru so if you know of a way that I can merge just the power save bits, let me know.

@PaulZC
Copy link
Collaborator Author

PaulZC commented Dec 22, 2019

Closing this for now until I understand the root cause of the timeout failures from #52 (which also apply to this function). I'll re-submit from a fresh fork.
Bye for now!
Paul

@PaulZC PaulZC closed this Dec 22, 2019
@PaulZC PaulZC deleted the PaulZC_power_save_mode branch December 22, 2019 07:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants