GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ectool: add --dump / -d for RAM dump 

This moves the ram dump behind an argument, but
it's still called by default when no other arguments given.
To hold backward compatibility -i also prints out RAM.

Change-Id: I82648e8cf1eac455e9937bd3669a0e91a3ee87cf
Signed-off-by: Alexander Couzens <lynxis@fe80.eu>
Reviewed-on: http://review.coreboot.org/8381
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Peter Stuge <peter@stuge.se>
This commit is contained in:
Alexander Couzens 2015-02-07 04:10:12 +01:00 committed by Peter Stuge
parent 7ca6522412
commit 46ca3a5534
1 changed files with 22 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
util/ectool/ectool.c
View File

@ -46,19 +46,20 @@ void print_version(void)
void print_usage(const char *name) void print_usage(const char *name)
{ {
printf("usage: %s [-vh?Vi] [-w 0x<addr> -z 0x<data>]\n", name); printf("usage: %s [-vh?Vid] [-w 0x<addr> -z 0x<data>]\n", name);
printf("\n" printf("\n"
" -v | --version: print the version\n" " -v | --version: print the version\n"
" -h | --help: print this help\n\n" " -h | --help: print this help\n\n"
" -V | --verbose: print debug information\n" " -V | --verbose: print debug information\n"
" -i | --idx: print IDX RAM\n" " -d | --dump: print RAM\n"
" -i | --idx: print IDX RAM & RAM\n"
" -w <addr in hex> write to addr\n" " -w <addr in hex> write to addr\n"
" -z <data in hex> write to data\n" " -z <data in hex> write to data\n"
"\n"); "\n");
exit(1); exit(1);
} }
int verbose = 0, dump_idx = 0; int verbose = 0, dump_idx = 0, dump_ram = 0;
int main(int argc, char *argv[]) int main(int argc, char *argv[])
{ {
@ -74,7 +75,7 @@ int main(int argc, char *argv[])
{0, 0, 0, 0} {0, 0, 0, 0}
}; };
while ((opt = getopt_long(argc, argv, "vh?Viw:z:", while ((opt = getopt_long(argc, argv, "vh?Vidw:z:",
long_options, &option_index)) != EOF) { long_options, &option_index)) != EOF) {
switch (opt) { switch (opt) {
case 'v': case 'v':
@ -86,6 +87,7 @@ int main(int argc, char *argv[])
break; break;
case 'i': case 'i':
dump_idx = 1; dump_idx = 1;
dump_ram = 1;
break; break;
case 'w': case 'w':
write_addr = strtol(optarg , NULL, 16); write_addr = strtol(optarg , NULL, 16);
@ -93,6 +95,9 @@ int main(int argc, char *argv[])
case 'z': case 'z':
write_data = strtol(optarg , NULL, 16); write_data = strtol(optarg , NULL, 16);
break; break;
case 'd':
dump_ram = 1;
break;
case 'h': case 'h':
case '?': case '?':
default: default:
@ -113,13 +118,20 @@ int main(int argc, char *argv[])
ec_write(write_addr & 0xff, write_data & 0xff); ec_write(write_addr & 0xff, write_data & 0xff);
} }
printf("EC RAM:\n"); /* preserve default - dump_ram if nothing selected */
for (i = 0; i < 0x100; i++) { if (!dump_ram && !dump_idx) {
if ((i % 0x10) == 0) dump_ram = 1;
printf("\n%02x: ", i); }
printf("%02x ", ec_read(i));
if (dump_ram) {
printf("EC RAM:\n");
for (i = 0; i < 0x100; i++) {
if ((i % 0x10) == 0)
printf("\n%02x: ", i);
printf("%02x ", ec_read(i));
}
printf("\n\n");
} }
printf("\n\n");
if (dump_idx) { if (dump_idx) {
printf("EC IDX RAM:\n"); printf("EC IDX RAM:\n");