From e19fe9ad09e28df1976257052afe6edf765aa778 Mon Sep 17 00:00:00 2001 From: David Brownell Date: Thu, 7 Jan 2010 15:22:41 -0800 Subject: [PATCH] ARM ETM/ETB/trace: help/usage updates Provide helptext which was sometimes missing; update some of it to be more accurate. Usage syntax messages have the same EBNF as the User's Guide; no angle brackets in either place. Don't use "&function"; functions are like arrays, their address is their name. Shrink some overlong lines, remove some empties. Signed-off-by: David Brownell --- src/target/etb.c | 12 ++++++--- src/target/etm.c | 54 ++++++++++++++++++++++++++--------------- src/target/etm_dummy.c | 4 +-- src/target/oocd_trace.c | 6 ++--- 4 files changed, 47 insertions(+), 29 deletions(-) diff --git a/src/target/etb.c b/src/target/etb.c index fb2dd609..18258f6a 100644 --- a/src/target/etb.c +++ b/src/target/etb.c @@ -447,17 +447,21 @@ COMMAND_HANDLER(handle_etb_trigger_percent_command) static const struct command_registration etb_config_command_handlers[] = { { + /* NOTE: with ADIv5, ETBs are accessed using DAP operations, + * possibly over SWD, not through separate TAPs... + */ .name = "config", - .handler = &handle_etb_config_command, + .handler = handle_etb_config_command, .mode = COMMAND_CONFIG, + .help = "Associate ETB with target and JTAG TAP.", .usage = "target tap", }, { .name = "trigger_percent", - .handler = &handle_etb_trigger_percent_command, + .handler = handle_etb_trigger_percent_command, .mode = COMMAND_EXEC, - .help = "percent of trace buffer to be filled " - "after the trigger occurs", + .help = "Set percent of trace buffer to be filled " + "after the trigger occurs (2..100).", .usage = "[percent]", }, COMMAND_REGISTRATION_DONE diff --git a/src/target/etm.c b/src/target/etm.c index d22bc407..3126efc2 100644 --- a/src/target/etm.c +++ b/src/target/etm.c @@ -1259,12 +1259,16 @@ COMMAND_HANDLER(handle_etm_tracemode_command) case 0: break; case 4: - CALL_COMMAND_HANDLER(handle_etm_tracemode_command_update, &tracemode); + CALL_COMMAND_HANDLER(handle_etm_tracemode_command_update, + &tracemode); break; default: - command_print(CMD_CTX, "usage: configure trace mode " - " " - " "); + command_print(CMD_CTX, "usage: tracemode " + "('none'|'data'|'address'|'all') " + "context_id_bits " + "('enable'|'disable') " + "('enable'|'disable')" + ); return ERROR_FAIL; } @@ -2112,11 +2116,16 @@ COMMAND_HANDLER(handle_etm_analyze_command) static const struct command_registration etm_config_command_handlers[] = { { + /* NOTE: with ADIv5, ETMs are accessed by DAP operations, + * possibly over SWD, not JTAG scanchain 6 of 'target'. + * + * Also, these parameters don't match ETM v3+ modules... + */ .name = "config", - .handler = &handle_etm_config_command, + .handler = handle_etm_config_command, .mode = COMMAND_CONFIG, - .usage = " " - " ", + .help = "Set up ETM output port.", + .usage = "target port_width port_mode clocking capture_driver", }, COMMAND_REGISTRATION_DONE }; @@ -2132,33 +2141,36 @@ const struct command_registration etm_command_handlers[] = { static const struct command_registration etm_exec_command_handlers[] = { { - .name = "tracemode", handle_etm_tracemode_command, + .name = "tracemode", + .handler = handle_etm_tracemode_command, .mode = COMMAND_EXEC, .help = "configure/display trace mode", - .usage = " " - " ", + .usage = "('none'|'data'|'address'|'all') " + "context_id_bits " + "['enable'|'disable'] " + "['enable'|'disable']", }, { .name = "info", - .handler = &handle_etm_info_command, + .handler = handle_etm_info_command, .mode = COMMAND_EXEC, .help = "display info about the current target's ETM", }, { .name = "status", - .handler = &handle_etm_status_command, + .handler = handle_etm_status_command, .mode = COMMAND_EXEC, .help = "display current target's ETM status", }, { .name = "start", - .handler = &handle_etm_start_command, + .handler = handle_etm_start_command, .mode = COMMAND_EXEC, .help = "start ETM trace collection", }, { .name = "stop", - .handler = &handle_etm_stop_command, + .handler = handle_etm_stop_command, .mode = COMMAND_EXEC, .help = "stop ETM trace collection", }, @@ -2167,7 +2179,7 @@ static const struct command_registration etm_exec_command_handlers[] = { .handler = handle_etm_trigger_debug_command, .mode = COMMAND_EXEC, .help = "enable/disable debug entry on trigger", - .usage = "(enable | disable)", + .usage = "['enable'|'disable']", }, { .name = "analyze", @@ -2177,19 +2189,21 @@ static const struct command_registration etm_exec_command_handlers[] = { }, { .name = "image", - .handler = &handle_etm_image_command, + .handler = handle_etm_image_command, .mode = COMMAND_EXEC, - .help = "load image from [base address]", + .help = "load image from file with optional offset", + .usage = "filename [offset]", }, { .name = "dump", - .handler = &handle_etm_dump_command, + .handler = handle_etm_dump_command, .mode = COMMAND_EXEC, - .help = "dump captured trace data ", + .help = "dump captured trace data to file", + .usage = "filename", }, { .name = "load", - .handler = &handle_etm_load_command, + .handler = handle_etm_load_command, .mode = COMMAND_EXEC, .help = "load trace data for analysis ", }, diff --git a/src/target/etm_dummy.c b/src/target/etm_dummy.c index 19a078f1..f9c6fe73 100644 --- a/src/target/etm_dummy.c +++ b/src/target/etm_dummy.c @@ -61,9 +61,9 @@ COMMAND_HANDLER(handle_etm_dummy_config_command) static const struct command_registration etm_dummy_config_command_handlers[] = { { .name = "config", - .handler = &handle_etm_dummy_config_command, + .handler = handle_etm_dummy_config_command, .mode = COMMAND_CONFIG, - .usage = "", + .usage = "target", }, COMMAND_REGISTRATION_DONE }; diff --git a/src/target/oocd_trace.c b/src/target/oocd_trace.c index 2533c40d..3b435719 100644 --- a/src/target/oocd_trace.c +++ b/src/target/oocd_trace.c @@ -398,13 +398,13 @@ COMMAND_HANDLER(handle_oocd_trace_resync_command) static const struct command_registration oocd_trace_all_command_handlers[] = { { .name = "config", - .handler = &handle_oocd_trace_config_command, + .handler = handle_oocd_trace_config_command, .mode = COMMAND_CONFIG, - .usage = "", + .usage = "target", }, { .name = "status", - .handler = &handle_oocd_trace_status_command, + .handler = handle_oocd_trace_status_command, .mode = COMMAND_EXEC, .help = "display OpenOCD + trace status", }, -- 2.39.5