Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1)

Unified Diff: tools/battor_agent/battor_agent.h

Issue 1991403002: [battor agent] Fix the init sequence so it retries inits instead of resets. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleaned up and ready for a 2nd review Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tools/battor_agent/battor_agent.cc » ('j') | tools/battor_agent/battor_agent.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/battor_agent/battor_agent.h
diff --git a/tools/battor_agent/battor_agent.h b/tools/battor_agent/battor_agent.h
index 3c6c77d170e1eabeb90e68feb871b8654fe7a8cb..92d087b4dbd659e86b7e2c1a714150415f1f672a 100644
--- a/tools/battor_agent/battor_agent.h
+++ b/tools/battor_agent/battor_agent.h
@@ -94,7 +94,6 @@ class BattOrAgent : public BattOrConnection::Listener,
REQUEST_CONNECTION,
// Actions required for starting tracing.
- SEND_RESET,
SEND_INIT,
READ_INIT_ACK,
SEND_SET_GAIN,
@@ -176,6 +175,9 @@ class BattOrAgent : public BattOrConnection::Listener,
// The number of times that we've attempted to read the last message.
uint8_t num_read_attempts_;
+ // The number of times we've attempted to init the BattOr.
charliea (OOO until 10-5) 2016/06/10 17:05:33 Logically, it might make sense to move this above
aschulman 2016/06/10 18:27:25 Done.
+ uint8_t num_init_attempts_;
+
// The timeout that's run when an action times out.
base::CancelableClosure timeout_callback_;
« no previous file with comments | « no previous file | tools/battor_agent/battor_agent.cc » ('j') | tools/battor_agent/battor_agent.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698