-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
saul: initial import of saul_bat_voltage module #21018
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code looks good to me. I think @chrysn has plenty of particle boards, maybe also the one that still needs testing?
Was only soft-ACK as none of the requested tests seem to have been run.
db0ad27
to
291858d
Compare
I just remembered, that I also have a |
291858d
to
745aca6
Compare
I am removing the |
Moved to #21022 |
I only have particle-xenon, but it works fine there. It reads a voltage of 4.458V, no matter whether the battery is connected or not (probably b/c I can't use serial w/o also powering the device). All my boards are xenon, but my understanding is that the differences between the particle boards are quite minimal. |
745aca6
to
d892c2c
Compare
Successfully tested on |
The |
Anyone has an Argon lying around? Otherwise, can we just assume that it works, as they use the common interface? |
Compromise: Let's merge this as is, but document that we did not test on Argon specifically, but are optimistic that it will just work due to the similarity? A one-line comment would be totally sufficient. |
Where should this doc go then? To the header file of |
9f8d4dc
to
df79e1a
Compare
df79e1a
to
eddd93e
Compare
Done for the board(s). There is also the |
Let's prioritize the release. |
Contribution description
This exposes the battery measuring capabilities for feather-like boards (see, e.g., https://learn.adafruit.com/introducing-the-adafruit-nrf52840-feather/power-management-2#measuring-battery-3122383) into SAUL so that the output value is the voltage and provides support for
47 boards:feather-m0, as well as its derivates,, see saul: initial import of saul_bat_voltage module #21018 (comment)feather-m0-lora
andfeather-m0-wifi
, ( 🟨 tested, but not working as expected1 )Testing procedure
Flash
example/saul
to one (or ideally all) of the supported boards and use the CLI to read from theBAT
registry entry. Here is a handy check list so reviewers might distribute the job among themselves. The bold ones are particularly important as I was unable to test them properly myself.feather-m0, feather-m0-lora, or feather-m0-wifiIssues/PRs references
Used for testing the power consumption of #20996.
Footnotes
not sure if the board I used for testing is broken. It used to work 5 years ago, here is the app where I took the config from. Sadly, I did not find the LARP prop I integrated the board I used for that into. ↩