summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlexander Couzens <lynxis@fe80.eu>2015-02-06 22:27:33 +0100
committerPeter Stuge <peter@stuge.se>2015-04-08 08:41:59 +0200
commit0edf419852501583985475ae7084e508dc66eb8f (patch)
tree8a8eae64beefa9ad9884c7fd6cabfee8e2f999ac
parent46ca3a55344b326553358b5119cb8a342a493ed4 (diff)
downloadcoreboot-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>
-rw-r--r--util/ectool/ec.c6
-rw-r--r--util/ectool/ec.h1
-rw-r--r--util/ectool/ectool.c17
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++) {