Skip to content
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

hecRegRemoteLock nicht auf 0 setzen #4118

Closed
wants to merge 2 commits into from
Closed

Conversation

AMiketta
Copy link

Um zu verhindern das ein eventuell gerade laufender Ladevorgang unterbrochen wird sollte das hecRegRemoteLock nie auf 0 (locked) gesetzt werden. Es ist jetzt auch schon mehrfach aufgetreten das wenn beim setzen ein Fehler auftritt die Box danach nie wieder in den 1 (unlocked) state zurückgesetzt wird, WakeUp scheint in diesem Fall nichtmehr aufgerufen zu werden (aber dazu kenn ich das Projekt zu wenig)

Um zu verhindern das ein eventuell gerade laufender Ladevorgang unterbrochen wird sollte das hecRegRemoteLock nie auf 0 (locked) gesetzt werden. Es ist jetzt auch schon mehrfach aufgetreten das wenn beim setzen ein Fehler auftritt die Box danach nie wieder in den 1 (unlocked) state zurückgesetzt wird, WakeUp scheint in diesem Fall nichtmehr aufgerufen zu werden (aber dazu kenn ich das Projekt zu wenig)
@andig andig requested a review from premultiply August 15, 2022 14:49
@andig
Copy link
Member

andig commented Aug 15, 2022

Wir setzen das auf 0 um bewusst einen Fehler herbei zu führen, der ein Wakeup auslöst. Ist denn Deine Variante in der Lage, ein schlafendes Auto aufzuwecken?

@andig
Copy link
Member

andig commented Aug 15, 2022

Es ist jetzt auch schon mehrfach aufgetreten das wenn beim setzen ein Fehler auftritt die Box danach nie wieder in den 1 (unlocked) state zurückgesetzt wird

Das können wir ändern, würde ich aber anders machen, also Rücksetzen unabhängig des Fehlers vorsehen. Nur warum sollte das bei Dir mehrfach Fehler auslösen? Probleme mit der Verkabelung?

@premultiply
Copy link
Member

Ersetzt durch #4117

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants