use COMMAND_HELPER for command helper functions
Define the numerous helpers that inherit command handler parameters using the COMMAND_HELPER macro.
This commit is contained in:
parent
cfc4d5c6b7
commit
63a26b421b
|
@ -462,9 +462,7 @@ COMMAND_HANDLER(parport_handle_write_on_exit_command)
|
||||||
return ERROR_OK;
|
return ERROR_OK;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int
|
COMMAND_HANDLER(parport_handle_parport_toggling_time_command)
|
||||||
parport_handle_parport_toggling_time_command(struct command_context_s *cmd_ctx,
|
|
||||||
char *cmd, char **args, int argc)
|
|
||||||
{
|
{
|
||||||
if (argc == 1) {
|
if (argc == 1) {
|
||||||
uint32_t ns;
|
uint32_t ns;
|
||||||
|
|
|
@ -551,8 +551,7 @@ int server_register_commands(command_context_t *context)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
int server_port_command(struct command_context_s *cmd_ctx,
|
SERVER_PORT_COMMAND()
|
||||||
char *cmd, char **args, int argc, unsigned short *out)
|
|
||||||
{
|
{
|
||||||
switch (argc) {
|
switch (argc) {
|
||||||
case 0:
|
case 0:
|
||||||
|
|
|
@ -81,8 +81,16 @@ int server_loop(command_context_t *command_context);
|
||||||
|
|
||||||
int server_register_commands(command_context_t *context);
|
int server_register_commands(command_context_t *context);
|
||||||
|
|
||||||
int server_port_command(struct command_context_s *cmd_ctx,
|
/**
|
||||||
char *cmd, char **args, int argc, unsigned short *port);
|
* Defines an extended command handler function declaration to enable
|
||||||
|
* access to (and manipulation of) the server port number.
|
||||||
|
* Call server_port like a normal COMMAND_HANDLER with an extra @a out parameter
|
||||||
|
* to receive the specified port number.
|
||||||
|
*/
|
||||||
|
#define SERVER_PORT_COMMAND() \
|
||||||
|
COMMAND_HELPER(server_port_command, unsigned short *out)
|
||||||
|
|
||||||
|
SERVER_PORT_COMMAND();
|
||||||
|
|
||||||
extern int server_use_pipes;
|
extern int server_use_pipes;
|
||||||
|
|
||||||
|
|
|
@ -1991,8 +1991,7 @@ static int arm11_build_reg_cache(target_t *target)
|
||||||
return ERROR_OK;
|
return ERROR_OK;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int arm11_handle_bool(struct command_context_s *cmd_ctx,
|
static COMMAND_HELPER(arm11_handle_bool, bool *var, char *name)
|
||||||
char *cmd, char **args, int argc, bool * var, char * name)
|
|
||||||
{
|
{
|
||||||
if (argc == 0)
|
if (argc == 0)
|
||||||
{
|
{
|
||||||
|
@ -2143,7 +2142,7 @@ static int arm11_mcr(target_t *target, int cpnum,
|
||||||
return arm11_mrc_inner(target, cpnum, op1, op2, CRn, CRm, &value, false);
|
return arm11_mrc_inner(target, cpnum, op1, op2, CRn, CRm, &value, false);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int arm11_handle_etm_read_write(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, bool read)
|
static COMMAND_HELPER(arm11_handle_etm_read_write, bool read)
|
||||||
{
|
{
|
||||||
if (argc != (read ? 2 : 3))
|
if (argc != (read ? 2 : 3))
|
||||||
{
|
{
|
||||||
|
|
|
@ -1176,9 +1176,8 @@ static int etmv1_analyze_trace(etm_context_t *ctx, struct command_context_s *cmd
|
||||||
return ERROR_OK;
|
return ERROR_OK;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int handle_etm_tracemode_command_update(
|
static COMMAND_HELPER(handle_etm_tracemode_command_update,
|
||||||
struct command_context_s *cmd_ctx,
|
etmv1_tracemode_t *mode)
|
||||||
char **args, etmv1_tracemode_t *mode)
|
|
||||||
{
|
{
|
||||||
etmv1_tracemode_t tracemode;
|
etmv1_tracemode_t tracemode;
|
||||||
|
|
||||||
|
@ -1272,7 +1271,7 @@ COMMAND_HANDLER(handle_etm_tracemode_command)
|
||||||
case 0:
|
case 0:
|
||||||
break;
|
break;
|
||||||
case 4:
|
case 4:
|
||||||
handle_etm_tracemode_command_update(cmd_ctx, args, &tracemode);
|
handle_etm_tracemode_command_update(cmd_ctx, cmd, args, argc, &tracemode);
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
command_print(cmd_ctx, "usage: configure trace mode "
|
command_print(cmd_ctx, "usage: configure trace mode "
|
||||||
|
|
|
@ -2363,8 +2363,7 @@ COMMAND_HANDLER(handle_mw_command)
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int parse_load_image_command_args(struct command_context_s *cmd_ctx,
|
static COMMAND_HELPER(parse_load_image_command_args, image_t *image,
|
||||||
char **args, int argc, image_t *image,
|
|
||||||
uint32_t *min_address, uint32_t *max_address)
|
uint32_t *min_address, uint32_t *max_address)
|
||||||
{
|
{
|
||||||
if (argc < 1 || argc > 5)
|
if (argc < 1 || argc > 5)
|
||||||
|
@ -2411,7 +2410,7 @@ COMMAND_HANDLER(handle_load_image_command)
|
||||||
int i;
|
int i;
|
||||||
image_t image;
|
image_t image;
|
||||||
|
|
||||||
int retval = parse_load_image_command_args(cmd_ctx, args, argc,
|
int retval = parse_load_image_command_args(cmd_ctx, cmd, args, argc,
|
||||||
&image, &min_address, &max_address);
|
&image, &min_address, &max_address);
|
||||||
if (ERROR_OK != retval)
|
if (ERROR_OK != retval)
|
||||||
return retval;
|
return retval;
|
||||||
|
@ -2555,7 +2554,7 @@ COMMAND_HANDLER(handle_dump_image_command)
|
||||||
return retval;
|
return retval;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int handle_verify_image_command_internal(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, int verify)
|
static COMMAND_HELPER(handle_verify_image_command_internal, int verify)
|
||||||
{
|
{
|
||||||
uint8_t *buffer;
|
uint8_t *buffer;
|
||||||
uint32_t buf_cnt;
|
uint32_t buf_cnt;
|
||||||
|
@ -4548,7 +4547,7 @@ COMMAND_HANDLER(handle_fast_load_image_command)
|
||||||
|
|
||||||
image_t image;
|
image_t image;
|
||||||
|
|
||||||
int retval = parse_load_image_command_args(cmd_ctx, args, argc,
|
int retval = parse_load_image_command_args(cmd_ctx, cmd, args, argc,
|
||||||
&image, &min_address, &max_address);
|
&image, &min_address, &max_address);
|
||||||
if (ERROR_OK != retval)
|
if (ERROR_OK != retval)
|
||||||
return retval;
|
return retval;
|
||||||
|
|
Loading…
Reference in New Issue