# Bitte um Review: Neuer Battery Code, Ausschalten bei Akku leer und MAX17055

**URL:** <https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730>\
**Category:** Software\
**Created:** [21. November 2021 um 21:53 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730 "2021-11-21T21:53:15Z")\
**Posts on this page:** 20\
**Page:** 3

<div class="post-metadata">

**Author:** ![SZenglein](https://forum.espuino.de/user_avatar/forum.espuino.de/szenglein/32/297_2.png) [@SZenglein](https://forum.espuino.de/u/SZenglein)\
**Post date:** [7. Dezember 2021 um 11:29 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/41 "2021-12-07T11:29:34Z")

</div>

> [@JHB](#):
>
> Da kann man aber je nach gewünschter Genauigkeit Monate drin versenken 😃

Genau, ich hab das Problem gekonnt ignoriert und einfach den Widerstand irgendwie verbunden. 😁

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [8. Dezember 2021 um 10:11 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/42 "2021-12-08T10:11:07Z")

</div>

Ich habe den PR gestern mal lokal in mein Repository integriert und auf einer Complete mit FEPO-Akku getestet (ohne MAX17055).

Mindestens zwei Bugs habe ich da:  
a) Obwohl die Spannung deutlich zu hoch gemessen wird (glaub es waren 4,5 V rum), leuchtet die LED-Anzeige rot (critical).  
b) Nach dem Einschalten kommt aus dem Lautsprecher kein Ton. Ich muss erst einen Kopfhörer einstecken (der geht dann) und wieder rausziehen. Klingt für mich nach einer falschen Reihenfolge von was auch immer.

War dann zu spät gestern, um das weiter zu analysieren.

---

<div class="post-metadata">

**Author:** ![SZenglein](https://forum.espuino.de/user_avatar/forum.espuino.de/szenglein/32/297_2.png) [@SZenglein](https://forum.espuino.de/u/SZenglein)\
**Post date:** [8. Dezember 2021 um 13:07 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/43 "2021-12-08T13:07:55Z")

</div>

a) Also du meinst die Spannungsmessung ist bei mir falsch? Der Code ist ja eigentlich komplett aus der alten Version übernommen, nur anders verpackt. Kannst du nochmal kontrollieren ob auch wirklich die settings richtig sind? Die dummy-implementierung gibt nämlich eigentlich 4.2V als Spannung zurück, aber auch isCritical → False, das passt also irgendwie nicht zusammen.  
Also in der settings musste ich ja ein paar Sachen ändern, vielleicht ist da im merge auch was falsch gelaufen. Wird BATTERY\_MEASURE\_ENABLE richtig definiert?

b) Oh ja, mh. Ich habe etwas an der Reihenfolge gedreht. Zum Beispiel aktiviere ich erst die Audioausgabe, damit die einen definierten Wert hat und dann schalte ich den Verstärker erst an. Hat bei mir ein Knacken beseitigt. Dürfte zwar nicht daran liegen, aber tausch doch mal Power\_PeripheralOn() mit AudioPlayer\_Init(). Sonst ist es vielleicht ein Problem mit der invertierten Logik, wobei die Power.h eigentlich denkbar einfach ist.

Weiteres:

- Passt die Änderung mit I2C so? Habe kein anderes I2C Device am Bus. Also Port-Expander wird weiterhin erkannt?
- Könntest du mit einem „normalen“ Akku testen? Nur um auszuschließen dass es nicht ein bestehendes Problem mit LiFePO4 ist, an der Spannungsmessung hab ich nämlich nicht wirklich was geändert.

Wenn die Probleme nicht gleich ersichtlich sind, mache ich am besten nochmal kleinere Pull-requests, die einzeln getestet werden können, um nicht zu viele Änderungen auf einmal zu haben.

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [8. Dezember 2021 um 14:22 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/44 "2021-12-08T14:22:06Z")

</div>

a) Das muss ich generell nochmal überprüfen. Hier hatte ich vorher immer mit deutlich zu niedriger Spannung zu kämpfen und habe deswegen ein Offset draufgehauen: [ESPuino/settings-complete.h at 0476fb05f85c67feeaf7e4d9e320b7233866b766 · biologist79/ESPuino · GitHub](https://github.com/biologist79/ESPuino/blob/0476fb05f85c67feeaf7e4d9e320b7233866b766/src/settings-complete.h#L97).  
Das ist halt der Prototyp von den Tonuino-Jungs, da habe ich keinen Schaltungsplan. Ich habe die Platine aber auch nicht mit LiPo getestet, das will ich jetzt endlich mal machen.

b) Also es ist so, dass es zwei Ansätze gibt. Bei einem hängt der SD-Pin des MAX98357a quasi offen und wird, wenn man eine Kopfhörerplatine hat, auf LOW gezogen, wenn ein Kopfhörer eingesteckt ist. Den ESP32 muss dann vom Einstecken eigentlich nix mitkriegen, aber wünschenswert ist es doch, weil man dann die max. Lautstärke auf dem Kopfhörer begrenzen kann.  
Die zweite Alternative, und von dieser rede ich, ist die Enable-Pins der DACs für Lautsprecher und Kopfhörer nicht direkt miteinander zu verbinden, sondern sie stattdessen auf einen Port-Expander aufzulegen. D.h. es wird immer vom ESP32 gesteuert, welcher DAC gerade aktiv ist. Schön an dieser Lösung ist auch, dass man problemlos eine Einschaltverzögerung realisieren kann. ESPuino schaut dann also, ob welchen logischen Pegel `HP_DETECT` ([ESPuino/settings-complete.h at 0476fb05f85c67feeaf7e4d9e320b7233866b766 · biologist79/ESPuino · GitHub](https://github.com/biologist79/ESPuino/blob/0476fb05f85c67feeaf7e4d9e320b7233866b766/src/settings-complete.h#L90)) hat und steuert damit GPIO\_HP\_EN und GPIO\_PA\_EN wechselseitig ([ESPuino/settings-complete.h at 0476fb05f85c67feeaf7e4d9e320b7233866b766 · biologist79/ESPuino · GitHub](https://github.com/biologist79/ESPuino/blob/0476fb05f85c67feeaf7e4d9e320b7233866b766/src/settings-complete.h#L54) und [ESPuino/settings-complete.h at 0476fb05f85c67feeaf7e4d9e320b7233866b766 · biologist79/ESPuino · GitHub](https://github.com/biologist79/ESPuino/blob/0476fb05f85c67feeaf7e4d9e320b7233866b766/src/settings-complete.h#L55)).

Ja, also i2c beim Port-Expander funktioniert, nur wie gesagt passt offenbar mit GPIO\_PA\_EN was nicht, da der Lautsprecher nicht angeht. Das tut er erst, nachdem ich einen Kopfhörer (mindestens mal kurz) eingesteckt hatte.

---

<div class="post-metadata">

**Author:** ![SZenglein](https://forum.espuino.de/user_avatar/forum.espuino.de/szenglein/32/297_2.png) [@SZenglein](https://forum.espuino.de/u/SZenglein)\
**Post date:** [8. Dezember 2021 um 14:27 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/45 "2021-12-08T14:27:31Z")

</div>

Kannst du mal probieren die zwei Zeilen zu vertauschen? Kann mir gut vorstellen, dass HP\_DETECT einfach undefined ist wenn alles aus ist und dadurch etwas falsch läuft.

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [8. Dezember 2021 um 14:36 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/46 "2021-12-08T14:36:44Z")

</div>

> [@SZenglein](#):
>
> Kannst du mal probieren die zwei Zeilen zu vertauschen?

Welche? PA und HP \_EN?

---

<div class="post-metadata">

**Author:** ![SZenglein](https://forum.espuino.de/user_avatar/forum.espuino.de/szenglein/32/297_2.png) [@SZenglein](https://forum.espuino.de/u/SZenglein)\
**Post date:** [8. Dezember 2021 um 14:38 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/47 "2021-12-08T14:38:03Z")

</div>

> [@SZenglein](#):
>
> Power\_PeripheralOn() mit AudioPlayer\_Init()

Power\_PeripheralOn() mit AudioPlayer\_Init() in main().

Und bitte nochmal pullen, gab einen kleinen Fehler wenn PN5180 ohne LPCD aktiviert ist, dann wurde init() nicht aufgerufen.

---

<div class="post-metadata">

**Author:** ![SZenglein](https://forum.espuino.de/user_avatar/forum.espuino.de/szenglein/32/297_2.png) [@SZenglein](https://forum.espuino.de/u/SZenglein)\
**Post date:** [8. Dezember 2021 um 14:57 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/48 "2021-12-08T14:57:49Z")

</div>

AudioPlayer.cpp:  
`pinMode(HP_DETECT, INPUT_PULLUP);`

settings-complete.h  
`#define DETECT_HP_ON_HIGH`

Passt meiner Ansicht nach nicht zusammen. Klar, wenn jetzt der Pin undefiniert ist wird durch den Pullup der Kopfhörer „erkannt“. Der port expander kann nicht beliebig pullup/pulldown setzen, oder? Eigentlich müsste hier je nach DETECT\_HP\_ON\_HIGH PULLUP oder PULLDOWN verwendet werden.  
Ich würde ja fast sagen eigentlich müsste ein externer Widerstand an den pin und es ist ein Hardware-Fehler 😁.

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [8. Dezember 2021 um 15:20 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/49 "2021-12-08T15:20:15Z")

</div>

> [@SZenglein](#):
>
> Der port expander kann nicht beliebig pullup/pulldown setzen, oder?

Der Port-Expander hat immer einen „weak“ PullUp-Widerstand, wenn er auf Input konfiguriert ist. Und per Default ist ein Port-Expander immer auf Input konfiguriert; wenn man Output auf einem Pin will, dann muss man ihm das explizit „sagen“.

> <https://github.com/biologist79/ESPuino/blob/0476fb05f85c67feeaf7e4d9e320b7233866b766/src/Port.cpp#L203>

> <https://github.com/biologist79/ESPuino/blob/0476fb05f85c67feeaf7e4d9e320b7233866b766/src/Port.cpp#L215>

> [@SZenglein](#):
>
> pinMode(HP\_DETECT, INPUT\_PULLUP);

Eigentlich muss man die Zeile (zum Verständnis) rausnehmen, wenn man GPIOs hat, die nicht in der Range von 0 bis 39 liegen. Wenn man da einen ungültigen GPIO reinsteckt, was bei 10x ja zB der Fall wäre, dann passiert da einfach gar nix.

Aber nicht auszuschließen, dass hier irgendwas noch doppelt ist. Was in meinem Fall auf jeden Fall greift ist das hier:

> <https://github.com/biologist79/ESPuino/blob/0476fb05f85c67feeaf7e4d9e320b7233866b766/src/AudioPlayer.cpp#L168>

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [8. Dezember 2021 um 15:48 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/50 "2021-12-08T15:48:59Z")

</div>

> [@SZenglein](#):
>
> Power\_PeripheralOn() mit AudioPlayer\_Init() in main().

Also das alleine hat nix geändert.  
PN5180 habe ich an diesem Board nicht. Tja, muss ich wohl debuggen. Sau stark, dass ich die Commits ohne neuen Branch eingebunden habe, jetzt kann ich nicht einfach hin und herspringen (außer ich mache es rückgäng und nachträglich alles nochmal).

Wie auch immer: Das kann ich leider eh nicht fertigstellen, bevor keine P-Mosfets da sind, weil das muss auch noch für meine Platine mit Port-Expander getestet werden. Keine Ahnung was gerade mit der Briefpost abgeht, aber kommt alles erheblich verspätet.

---

<div class="post-metadata">

**Author:** ![SZenglein](https://forum.espuino.de/user_avatar/forum.espuino.de/szenglein/32/297_2.png) [@SZenglein](https://forum.espuino.de/u/SZenglein)\
**Post date:** [8. Dezember 2021 um 16:08 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/51 "2021-12-08T16:08:40Z")

</div>

Ah ok, ursprünglich hat AudioPlayer\_Init wesentlich weiter unten gestanden. Unter anderem nach Port\_Init(). Das könnte natürlich sehr gut der Fehler sein.  
Es ist etwas schwierig zu sehen, welches Modul von einem anderen Modul abhängt und nach diesem ausgeführt werden muss. Das Problem gab es ja auch schon bei LPCD, wo die Reihenfolge nicht gepasst hat. Wenn es da eine Möglichkeit gäbe die Reihenfolge zu definieren, wäre für die Zukunft sicher auch gut.

Generell fände ich es gut, wenn der Verstärker möglichst zeitnah zum Anschalten auch definierte Eingänge bekommt. Wenn man nicht in Software stumm schaltet kommt es sonst manchmal zu einem unschönen Knacken beim Einschalten.

Kurzgesagt: Vermutlich muss Audio\_Init() wieder später initialisiert werden.

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [8. Dezember 2021 um 16:52 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/52 "2021-12-08T16:52:11Z")

</div>

> [@SZenglein](#):
>
> Ah ok, ursprünglich hat AudioPlayer\_Init wesentlich weiter unten gestanden. Unter anderem nach Port\_Init(). Das könnte natürlich sehr gut der Fehler sein.

Ja, ok. Ich hatte es mir so genau noch nicht angeschaut aber das klingt sehr danach, dass das der Fehler sein könnte. Eija, ich schaue es mir mal an. Aber vermutlich heute nicht mehr, da ich noch Puppenbett bauen muss 😂.

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [20. Dezember 2021 um 11:22 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/53 "2021-12-20T11:22:07Z")

</div>

> [@biologist](#):
>
> Bei Ali habe ich bereits im September 40 JST PH 2fach bestellt. Die liegen schon seit 1.11 beim Zoll (laut Tracking) und ich habe keine Ahnung warum. Sollte doch alles bezahlt sein. Habe danach noch einiges bestellt und das ist längst da.

Nachtrag: Das kam tatsächlich heute noch an. Dachte ja schon, dass es verschollen ist. War jetzt vom 25.9 bis heute unterwegs 😂. Flaschenpost vielleicht? 🙂.

Aber schön: Damit kann ich weiterhin sagen, dass noch nie was „verschütt“ gegangen ist, was ich bei Ali bestellt habe.

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [22. Dezember 2021 um 18:45 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/54 "2021-12-22T18:45:56Z")

</div>

Zum Thema LiFePO4 noch ein [interessantes Video von Andreas Spiess](https://www.youtube.com/watch?v=eiyBavjQ1Rk).

@SZenglein Ich habe das Review nicht vergessen. Ist nur gerade so ein bisschen die Luft raus. Ich kümmere mich darum.

---

<div class="post-metadata">

**Author:** ![SZenglein](https://forum.espuino.de/user_avatar/forum.espuino.de/szenglein/32/297_2.png) [@SZenglein](https://forum.espuino.de/u/SZenglein)\
**Post date:** [22. Dezember 2021 um 23:51 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/55 "2021-12-22T23:51:28Z")

</div>

Ja ist von meiner Seite kein Problem. Kann auch total verstehen wenn dir der pull request etwas zu umfangreich ist. Die Änderungen haben sich einfach ein wenig ineinander verknotet 😃

Wenn du den pull lieber in kleinere Häppchen unterteilt haben willst sag einfach Bescheid.

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [18. Januar 2022 um 16:20 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/56 "2022-01-18T16:20:44Z")

</div>

Ich habe das Thema nicht vergessen und habe es lokal inzwischen nochmal gebrancht. Beim Drücken des Buttons des Drehencoders wird auf meinem Port-Expander-Lolin D32pro-PCB „rot“ angezeigt. Das passt so nicht; muss ich nochmal schauen, woran das liegt. Dann hast du ein paar neue NVS-Variablen eingeführt; da ist ein Copy’n’Paste-Fehler drin (das habe ich bei mir schon gefixt).

Port\_Init() habe ich mal nach oben geschoben - das behebt das Problem, dass ich weiter oben hier schon beschrieben habe, dass man auf dem Lautsprecher nix hört nach dem Einschalten. Nur habe ich mit dem Lolin D32 pro in o.g. PCB so ein elendiges Problem, in das ich anderweitig auch schon gelaufen bin. Und zwar wird der Power-GPIO ja zuerst deklariert und im Anschluss (in deinem Code später) dann HIGH drauf geschickt. Ohne Kopfhörerplatine startet der D32 pro verzögert und mit Kopfhörerplatine gar nicht. Ich habe zwei D32 pro hier, wobei der Erste irgendwie einen „Schlag“ hat. Jedenfalls mit dem hatte ich im Code, der aktuell online ist, auch so ein Problem. Das habe ich in den Griff bekommen, wenn ich zwischen der Deklaration und dem Draufschalten von High kein delay() hatte:

> <https://github.com/biologist79/ESPuino/blob/1f799a03152777ea4aa7de7d30219c2d233ac16e/src/System.cpp#L40>

Mir ist leider völlig unklar an was das liegt. Äußern tut sich das Ganze so, dass man die geschalteten 3.3 V messen kann, sie jedoch immer wieder einbrechen. Man hört das auch ganz ganz leise. Also irgendwie versucht er zu starten aber es klappt nicht.

Vielleicht hat irgendwer ja nen Tipp dazu. Ich hatte das so noch bei keinem anderen ESP32.

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [20. Januar 2022 um 23:37 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/57 "2022-01-20T23:37:36Z")

</div>

@SZenglein Was meinst du mit folgendem Kommentar?

> <https://github.com/SZenglein/ESPuino/blob/bcdfa3322234b2ea22991ec2a9986c1430c561c1/src/BatteryCommon.cpp#L35>

Was spricht denn gegen System\_RequestSleep() an dieser Stelle?. Das setzt System\_GoToSleep = true und damit wird System\_DeepSleepManager() durchlaufen und der ganze Shutdown-Prozess wird ausgeführt.

Und noch eine Frage: Für was steht eigentlich „SOC“?

---

<div class="post-metadata">

**Author:** ![SZenglein](https://forum.espuino.de/user_avatar/forum.espuino.de/szenglein/32/297_2.png) [@SZenglein](https://forum.espuino.de/u/SZenglein)\
**Post date:** [21. Januar 2022 um 00:45 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/58 "2022-01-21T00:45:42Z")

</div>

Der gleiche Grund aus dem im Rfid code auch manuell esp\_deep\_sleep\_start() aufgerufen wird. Die RequestSleep Funktion greift erst nachdem die ganze Initialisierung durch ist, also Wifi, Neopixel, etc. Es ist etwas problematisch wenn bei sehr geringer Batterie dafür Energie verschwendet wird, aber viel unschöner ist dass der ESPuino nur angeht um dann sofort wieder auszugehen.

Kurzgesagt: Weil ich an der Stelle _sofort_ wieder runterfahren will.

SOC steht für State of Charge (der Batterie). Ist eben die Lingo von dem chip.

Ich fühle mich ein wenig schlecht weil du so viel Arbeit in diesen Pull steckst, aber das ist ja letztendlich deine Entscheidung.  
Ich kann die nächsten Wochen dazu leider gar nichts beitragen, aber danach kann ich mir das ganze nochmal in Ruhe anschauen…

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [21. Januar 2022 um 07:55 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/59 "2022-01-21T07:55:34Z")

</div>

> [@SZenglein](#):
>
> Ich fühle mich ein wenig schlecht weil du so viel Arbeit in diesen Pull steckst, aber das ist ja letztendlich deine Entscheidung.

Nee, keine Angst. Ist jetzt nicht so, dass ich da seit Wochen stundenlang pro Tag dran hänge. Die Hardware-Probleme, die ich zuvor beschrieben habe mit [diesem PCB](https://forum.espuino.de/t/lolin-d32-pro-mit-sd-mmc-pn5180-max-fuenf-buttons-und-port-expander-smd/638), habe ich gestern mit einem zusätzlichen Kondensator gefixt bekommen. @tueddy Du hattest mir das ja auch mal in einer PM geschrieben, dass man das vielleicht machen sollte. Also mit 100 uF geht’s auf jeden Fall.

> [@SZenglein](#):
>
> SOC steht für State of Charge (der Batterie). Ist eben die Lingo von dem chip.

Da läuft halt auch die generische Spannungsmessung drüber. Daher wollte ich das einfach gerne verstehen.

Das mit der Spannungsmessung klappt doch, weiß nicht, was ich da beim letzten Mal falsch gemacht habe. Was mir nicht so gefällt, das ist die Umwidmung der Farben bei der Spannungsanzeige auf dem Neopixel. Orange ist jetzt critical und greift erst bei einer vergleichsweisen tiefen Spannung. Ich wollte das aber eher so als „halb voll“. Da werde ich nochmal ne Runde drüber nachdenken, wie wir da vorgehen.

Ansonsten muss statt „…Impl“ für mich da was Anderes hin. Ich bin immer sehr für sprechende Funktions/Methodennamen zu haben.

Wie auch immer: Wenn ich mit der Arbeit durch bin, dann werde ich zum Testen einen eigenen Branch aufmachen und die vielen Commits (von mir sind auch welche dabei) mittels eines Squash-Merge in einen einzigen packen.

---

<div class="post-metadata">

**Author:** ![biologist](https://forum.espuino.de/user_avatar/forum.espuino.de/biologist/32/66_2.png) [@biologist](https://forum.espuino.de/u/biologist)\
**Post date:** [23. Februar 2022 um 14:48 UTC](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730/60 "2022-02-23T14:48:01Z")

</div>

@SZenglein Du hattest ja kürzlich die Power.cpp eingeführt. Geändert hat sich dabei, dass die Initialisierung des Power-Pins und das anschließende Beschalten nicht mehr zwei Anweisungen hintereinander sind. @compactflash verwendet ja mit dem LTC2954 einen Powerswitch und hat seit dieser Aufspaltung offenbar ein Problem mit seiner Complete.  
@compactflash Vielleicht willst du nochmal kurz was dazu schreiben, was das Problem ist.

[Vorherige Seite](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730.md?page=2)

[Nächste Seite](https://forum.espuino.de/t/bitte-um-review-neuer-battery-code-ausschalten-bei-akku-leer-und-max17055/730.md?page=4)
