Irqinterlnalbatt am Beispiel des HM-SEC-SCO

[der-pw]

Ich versuche ja die neuen Klassen irq(internal/external)batt besser zu verstehen.
Jetzt fiel mir auf, dass am Beispiel des HM-SEC-SCO
ein AskSin Typ mit Irqinternalbatt definiert ist, aber keine Werte für "low" und "critical" angegeben sind.
In der BatterySensor.h gibt es aber keine Defaultwerte (zumindest hab ich nichts gefunden).
Wie wird das dort gehandhabt?

[jp112sdl]

Jetzt fiel mir auf, dass am Beispiel des HM-SEC-SCO
ein AskSin Typ mit Irqinternalbatt definiert ist, aber keine Werte für "low" und "critical" angegeben sind.

Hab ich wohl verpennt.

In der BatterySensor.h gibt es aber keine Defaultwerte (zumindest hab ich nichts gefunden).

Wird mit 0 initialisiert
m_Low(0), m_Critical(0)

[der-pw]

Hab ich wohl verpennt.

Ah okay, hab nun schon wirklich an mir gezweifelt.

Du hast also um den Teil
https://github.com/jp112sdl/Beispiel_As ... aL143-L157
zu viel entfernt? Plus die definierten Variablen.

Wird mit 0 initialisiert
m_Low(0), m_Critical(0)

Genau, das hab ich gefunden.

... ich muss mal 2-3 Monate unbezahlten Urlaub machen und ins C++ Bootcamp

[jp112sdl]

zu viel entfernt?

#define DEVICE_CONFIG CFG_STEPUP_OFF,22,19
Ich wollte das nicht im EEPROM haben weil ich nicht mit OTA Config arbeite.
Hätte dann die 22 und 19 für battery.low() und battery.critical() nehmen müssen

[der-pw]

Okay, verstehe so langsam.
#define DEVICE_CONFIG [...] wird in der device.h geregelt

Ist

Code: Alles auswählen

  hal.battery.init(seconds2ticks(60UL*60), sysclock);
  hal.battery.low(22);
  hal.battery.critical(19);

im void setup() dann quasi obsolet?

Sorry, wenn ich so blöd frage, aber "go to declaration" in VSCODE hat nur begrenzte Möglichkeiten (für mich), bzw. hangele ich mich dem Code entlang
und finde dann oftmals nicht weiter.

[jp112sdl]

Ist

Code: Alles auswählen

  hal.battery.low(22);
  hal.battery.critical(19);

im void setup() dann quasi obsolet?

Nein gerade das habe ich beim SCo ja vergessen und muss da wieder rein. ^^

War vorher im configChanged und holte die Werte aus der Device Config im EEPROM:

Code: Alles auswählen

    battery().low(getConfigByte(CFG_BAT_LOW_BYTE));
     battery().critical(getConfigByte(CFG_BAT_CRITICAL_BYTE));

[der-pw]

Nein gerade das habe ich beim SCo ja vergessen und muss da wieder rein. ^^

Sehr gut, dann habe ich es jetzt verstanden.

Dankeschön

[der-pw]

Vielleicht noch eine Frage, bevor ich ein Eis spendiere.

Welchen Unterschied macht es eigentlich, wenn die battery.[low/critical] schon direkt in der Klasse "Hal" verfügbar gemacht wird

Code: Alles auswählen

class Hal : public BaseHal {
public:
  void init (const HMID& id) {
    BaseHal::init(id);
    // measure battery every 1h
    battery.init(seconds2ticks(60UL*60),sysclock);
    battery.low(22);
    battery.critical(19);
  }
} hal;

gegenüber dem Konstrukt, dass hal.battery.[low/critical] erst in void setup() aufgerufen wird?

[jp112sdl]

Welchen Unterschied macht es eigentlich, wenn die battery.[low/critical] schon direkt in der Klasse "Hal" verfügbar gemacht wird

Keinen.

gegenüber dem Konstrukt, dass hal.battery.[low/critical] erst in void setup() aufgerufen wird?

Wird es so oder so.
sdev.init(hal); im setup() ruft ja das hal.init auf
Dann halt nur ein bisschen "früher"

[der-pw]

sdev.init(hal); im setup() ruft ja das hal.init auf

Stimmt!

Danke für die kleine Lehrstunde.