Daheimladen: fix autostart after charger reboot

[premultiply]

Fixes #30934

[VolkerK62]

sorry für die falschen (heartbeat) Fährte.
Bei den Pro Modellen ist failsafe-timeout = 0. Dann wird heartbeat doch gar nicht aufgerufen, oder?

[premultiply]

Nein, dann nicht.

[VolkerK62]

Könnte man diese Ladestromprüfung in der Status Funktion unterbringen?

[VolkerK62]

Dann macht das im heartbeat keine Sinn.

[VolkerK62]

Wäre stattdessen diese Ergänzung eine Option?

func (wb *DaheimLaden) getCurrent() (uint16, error) {
    b, err := wb.conn.ReadHoldingRegisters(dlRegCurrentLimit, 1)
    if err != nil {
        return 0, err
    }

    curr := binary.BigEndian.Uint16(b)

    // change 0 to 1 to avoid autostart after charger reboot
    if curr == 0 {
        _ = wb.setCurrent(1)
        curr = 1
    }

    return curr, nil
}

[premultiply]

Die Herausforderung ist, dass es eigentlich nur Getter oder Setter gibt.
Durch das Lesen und ggf. Schreiben in einer Funktion würden wir diese Trennung aufheben.

Ich denke nochmal drüber nach.

[VolkerK62]

Ok. Nachdem ich mich über Getter und Setter schlau gemacht habe, verstehe ich prinzipiell den Einwand.
Allerdings ist es doch klar abgegrenzt und hat keine Nebeneffekte.