diff options
author | Alexander Couzens <lynxis@fe80.eu> | 2015-02-06 22:27:33 +0100 |
---|---|---|
committer | Peter Stuge <peter@stuge.se> | 2015-04-08 08:41:59 +0200 |
commit | 0edf419852501583985475ae7084e508dc66eb8f (patch) | |
tree | 8a8eae64beefa9ad9884c7fd6cabfee8e2f999ac /util | |
parent | 46ca3a55344b326553358b5119cb8a342a493ed4 (diff) | |
download | coreboot-0edf419852501583985475ae7084e508dc66eb8f.tar.xz |
ectool: add query function to ectool
`ectool -q` - Query the EC IRQ byte.
Should return 0x00 otherwise the IRQ handler is usally broken or disabled.
Tested-on: Lenovo X201t
Change-Id: I0b8c2dbcf38d2eab89d0fbea05795759c4517f6d
Signed-off-by: Alexander Couzens <lynxis@fe80.eu>
Reviewed-on: http://review.coreboot.org/8382
Tested-by: build bot (Jenkins)
Reviewed-by: Nicolas Reinecke <nr@das-labor.org>
Reviewed-by: Peter Stuge <peter@stuge.se>
Diffstat (limited to 'util')
-rw-r--r-- | util/ectool/ec.c | 6 | ||||
-rw-r--r-- | util/ectool/ec.h | 1 | ||||
-rw-r--r-- | util/ectool/ectool.c | 17 |
3 files changed, 20 insertions, 4 deletions
diff --git a/util/ectool/ec.c b/util/ectool/ec.c index 0f5b891d54..323a43fe73 100644 --- a/util/ectool/ec.c +++ b/util/ectool/ec.c @@ -147,3 +147,9 @@ uint8_t ec_idx_read(uint16_t addr) return inb(lpc_idx + 3); } + +uint8_t ec_query(void) +{ + send_ec_command(QR_EC); + return recv_ec_data(); +} diff --git a/util/ectool/ec.h b/util/ectool/ec.h index a7b10e4324..f25a3d92cb 100644 --- a/util/ectool/ec.h +++ b/util/ectool/ec.h @@ -50,4 +50,5 @@ int ec_write(uint8_t addr, uint8_t data); uint8_t ec_ext_read(uint16_t addr); int ec_ext_write(uint16_t addr, uint8_t data); uint8_t ec_idx_read(uint16_t addr); +uint8_t ec_query(void); #endif diff --git a/util/ectool/ectool.c b/util/ectool/ectool.c index 5609c9bba9..2479633aaa 100644 --- a/util/ectool/ectool.c +++ b/util/ectool/ectool.c @@ -46,20 +46,21 @@ void print_version(void) void print_usage(const char *name) { - printf("usage: %s [-vh?Vid] [-w 0x<addr> -z 0x<data>]\n", name); + printf("usage: %s [-vh?Vidq] [-w 0x<addr> -z 0x<data>]\n", name); printf("\n" " -v | --version: print the version\n" " -h | --help: print this help\n\n" " -V | --verbose: print debug information\n" " -d | --dump: print RAM\n" " -i | --idx: print IDX RAM & RAM\n" + " -q | --query: print query byte\n" " -w <addr in hex> write to addr\n" " -z <data in hex> write to data\n" "\n"); exit(1); } -int verbose = 0, dump_idx = 0, dump_ram = 0; +int verbose = 0, dump_idx = 0, dump_ram = 0, dump_query = 0; int main(int argc, char *argv[]) { @@ -72,10 +73,11 @@ int main(int argc, char *argv[]) {"help", 0, 0, 'h'}, {"verbose", 0, 0, 'V'}, {"idx", 0, 0, 'i'}, + {"query", 0, 0, 'q'}, {0, 0, 0, 0} }; - while ((opt = getopt_long(argc, argv, "vh?Vidw:z:", + while ((opt = getopt_long(argc, argv, "vh?Vidqw:z:", long_options, &option_index)) != EOF) { switch (opt) { case 'v': @@ -98,6 +100,9 @@ int main(int argc, char *argv[]) case 'd': dump_ram = 1; break; + case 'q': + dump_query = 1; + break; case 'h': case '?': default: @@ -119,7 +124,7 @@ int main(int argc, char *argv[]) } /* preserve default - dump_ram if nothing selected */ - if (!dump_ram && !dump_idx) { + if (!dump_ram && !dump_idx && !dump_query) { dump_ram = 1; } @@ -133,6 +138,10 @@ int main(int argc, char *argv[]) printf("\n\n"); } + if (dump_query) { + printf("EC QUERY %02x\n", ec_query()); + } + if (dump_idx) { printf("EC IDX RAM:\n"); for (i = 0; i < 0x10000; i++) { |