ectool: fix battery command
This patch fixes the ec_readmem function pointer, which is currently
set to fake_readmem unconditionally.
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
BUG=b/122568808
BRANCH=none
TEST=Verify 'ectool battery' runs successfully on Nami.
Change-Id: I2d57d0ca7103d9b9a1e44e685bc966d158a16a47
Reviewed-on: https://chromium-review.googlesource.com/c/1405068
Commit-Queue: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
diff --git a/util/comm-host.c b/util/comm-host.c
index e9e7fb1..f8be252 100644
--- a/util/comm-host.c
+++ b/util/comm-host.c
@@ -84,6 +84,9 @@
int comm_init_alt(int interfaces, const char *device_name)
{
+ /* Default memmap access */
+ ec_readmem = fake_readmem;
+
if ((interfaces & COMM_SERVO) && comm_init_servo_spi &&
!comm_init_servo_spi(device_name))
return 0;
@@ -110,9 +113,6 @@
int allow_large_buffer;
struct ec_response_get_protocol_info info;
- /* Default memmap access */
- ec_readmem = fake_readmem;
-
allow_large_buffer = kernel_version_ge(3, 14, 0);
if (allow_large_buffer < 0) {
fprintf(stderr, "Unable to check linux version\n");
