-
-
Notifications
You must be signed in to change notification settings - Fork 641
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
Heidelberg: Fix locking #4117
Heidelberg: Fix locking #4117
Conversation
@@ -131,6 +131,19 @@ func (wb *HeidelbergEC) Status() (api.ChargeStatus, error) { | |||
case 9: | |||
return api.StatusE, nil | |||
case 10: | |||
// ensure RemoteLock is disabled after wake-up | |||
l, err := wb.conn.ReadInputRegisters(hecRegRemoteLock, 1) |
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.
Das brauchts doch nur beim Neustart und nicht ständig? Im Wakeup schaltest Du ja sonst immer frei- woher sollten da noch zusätzliche Locks kommen? Aber schadet vielleicht auch nichts...
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.
Geht ja nur darum dass Falls die WB den Unlock - aus welchem Grund auch immer - nicht wirklich mitbekommen hat, hier nochmal entsperrt wird.
Rein? |
Should fix #3596