#endif
/* This may actually crash... we do it last */
- if (interp && interp->cookie_stderr){
+ if (interp && interp->cookie_stderr) {
Jim_fprintf(interp, interp->cookie_stderr, JIM_NL "JIM INTERPRETER PANIC: ");
Jim_vfprintf(interp, interp->cookie_stderr, fmt, ap);
Jim_fprintf(interp, interp->cookie_stderr, JIM_NL JIM_NL);
buf = jim_vasprintf(fmt, ap);
va_end(ap);
- if (buf){
+ if (buf) {
Jim_AppendString(interp, objPtr, buf, -1);
jim_vasprintf_done(buf);
}
haveprec = 0;
prec = -1; /* not found yet */
next_fmt:
- if (fmtLen <= 0){
+ if (fmtLen <= 0) {
break;
}
- switch (*fmt){
+ switch (*fmt) {
/* terminals */
case 'b': /* binary - not all printfs() do this */
case 's': /* string */
case '8':
case '9':
accum = 0;
- while (isdigit(*fmt) && (fmtLen > 0)){
+ while (isdigit(*fmt) && (fmtLen > 0)) {
accum = (accum * 10) + (*fmt - '0');
fmt++; fmtLen--;
}
- if (inprec){
+ if (inprec) {
haveprec = 1;
prec = accum;
} else {
/* suck up the next item as an integer */
fmt++; fmtLen--;
objc--;
- if (objc <= 0){
+ if (objc <= 0) {
goto not_enough_args;
}
- if (Jim_GetWide(interp,objv[0],&wideValue)== JIM_ERR){
+ if (Jim_GetWide(interp,objv[0],&wideValue)== JIM_ERR) {
Jim_FreeNewObj(interp, resObjPtr);
return NULL;
}
- if (inprec){
+ if (inprec) {
haveprec = 1;
prec = wideValue;
- if (prec < 0){
+ if (prec < 0) {
/* man 3 printf says */
/* if prec is negative, it is zero */
prec = 0;
}
} else {
width = wideValue;
- if (width < 0){
+ if (width < 0) {
ljust = 1;
width = -width;
}
*/
cp = fmt_str;
*cp++ = '%';
- if (altfm){
+ if (altfm) {
*cp++ = '#';
}
- if (forceplus){
+ if (forceplus) {
*cp++ = '+';
- } else if (spad){
+ } else if (spad) {
/* PLUS overrides */
*cp++ = ' ';
}
- if (ljust){
+ if (ljust) {
*cp++ = '-';
}
- if (zpad){
+ if (zpad) {
*cp++ = '0';
}
- if (width > 0){
+ if (width > 0) {
sprintf(cp, "%d", width);
/* skip ahead */
cp = strchr(cp,0);
}
/* did we find a period? */
- if (inprec){
+ if (inprec) {
/* then add it */
*cp++ = '.';
/* did something occur after the period? */
- if (haveprec){
+ if (haveprec) {
sprintf(cp, "%d", prec);
}
cp = strchr(cp,0);
case 'E':
*cp++ = *fmt;
*cp = 0;
- if (Jim_GetDouble(interp, objv[0], &doubleValue) == JIM_ERR){
+ if (Jim_GetDouble(interp, objv[0], &doubleValue) == JIM_ERR) {
Jim_FreeNewObj(interp, resObjPtr);
return NULL;
}
case 'x':
case 'X':
/* jim widevaluse are 64bit */
- if (sizeof(jim_wide) == sizeof(long long)){
+ if (sizeof(jim_wide) == sizeof(long long)) {
*cp++ = 'l';
*cp++ = 'l';
} else {
int e;
e = Jim_Nvp_name2value_obj(interp, nvp_table, objPtr, &n);
- if (e == JIM_ERR){
+ if (e == JIM_ERR) {
return e;
}
/* Success? found? */
- if (n->name){
+ if (n->name) {
/* remove const */
*result = (Jim_Nvp *)n;
return JIM_OK;
Jim_IncrRefCount(scriptObjPtr);
- if (filename){
+ if (filename) {
JimSetSourceInfo(interp, scriptObjPtr, filename, lineno);
}
int Jim_vfprintf(Jim_Interp *interp, void *cookie, const char *fmt, va_list ap)
{
- if ((interp == NULL) || (interp->cb_vfprintf == NULL)){
+ if ((interp == NULL) || (interp->cb_vfprintf == NULL)) {
errno = ENOTSUP;
return -1;
}
size_t Jim_fwrite(Jim_Interp *interp, const void *ptr, size_t size, size_t n, void *cookie)
{
- if ((interp == NULL) || (interp->cb_fwrite == NULL)){
+ if ((interp == NULL) || (interp->cb_fwrite == NULL)) {
errno = ENOTSUP;
return 0;
}
size_t Jim_fread(Jim_Interp *interp, void *ptr, size_t size, size_t n, void *cookie)
{
- if ((interp == NULL) || (interp->cb_fread == NULL)){
+ if ((interp == NULL) || (interp->cb_fread == NULL)) {
errno = ENOTSUP;
return 0;
}
int Jim_fflush(Jim_Interp *interp, void *cookie)
{
- if ((interp == NULL) || (interp->cb_fflush == NULL)){
+ if ((interp == NULL) || (interp->cb_fflush == NULL)) {
/* pretend all is well */
return 0;
}
char* Jim_fgets(Jim_Interp *interp, char *s, int size, void *cookie)
{
- if ((interp == NULL) || (interp->cb_fgets == NULL)){
+ if ((interp == NULL) || (interp->cb_fgets == NULL)) {
errno = ENOTSUP;
return NULL;
}
Jim_Nvp *
Jim_Nvp_name2value_simple(const Jim_Nvp *p, const char *name)
{
- while (p->name){
- if (0 == strcmp(name, p->name)){
+ while (p->name) {
+ if (0 == strcmp(name, p->name)) {
break;
}
p++;
Jim_Nvp *
Jim_Nvp_name2value_nocase_simple(const Jim_Nvp *p, const char *name)
{
- while (p->name){
- if (0 == strcasecmp(name, p->name)){
+ while (p->name) {
+ if (0 == strcasecmp(name, p->name)) {
break;
}
p++;
p = Jim_Nvp_name2value_simple(_p, name);
/* result */
- if (result){
+ if (result) {
*result = (Jim_Nvp *)(p);
}
/* found? */
- if (p->name){
+ if (p->name) {
return JIM_OK;
} else {
return JIM_ERR;
p = Jim_Nvp_name2value_nocase_simple(_p, name);
- if (puthere){
+ if (puthere) {
*puthere = (Jim_Nvp *)(p);
}
/* found */
- if (p->name){
+ if (p->name) {
return JIM_OK;
} else {
return JIM_ERR;
jim_wide w;
e = Jim_GetWide(interp, o, &w);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return e;
}
Jim_Nvp *
Jim_Nvp_value2name_simple(const Jim_Nvp *p, int value)
{
- while (p->name){
- if (value == p->value){
+ while (p->name) {
+ if (value == p->value) {
break;
}
p++;
p = Jim_Nvp_value2name_simple(_p, value);
- if (result){
+ if (result) {
*result = (Jim_Nvp *)(p);
}
- if (p->name){
+ if (p->name) {
return JIM_OK;
} else {
return JIM_ERR;
int x;
Jim_fprintf(p->interp, p->interp->cookie_stderr, "---args---\n");
- for (x = 0 ; x < p->argc ; x++){
+ for (x = 0 ; x < p->argc ; x++) {
Jim_fprintf(p->interp, p->interp->cookie_stderr,
"%2d) %s\n",
x,
Jim_Obj *o;
o = NULL; // failure
- if (goi->argc){
+ if (goi->argc) {
// success
o = goi->argv[0];
goi->argc -= 1;
goi->argv += 1;
}
- if (puthere){
+ if (puthere) {
*puthere = o;
}
- if (o != NULL){
+ if (o != NULL) {
return JIM_OK;
} else {
return JIM_ERR;
r = Jim_GetOpt_Obj(goi, &o);
- if (r == JIM_OK){
+ if (r == JIM_OK) {
cp = Jim_GetString(o, len);
- if (puthere){
+ if (puthere) {
/* remove const */
*puthere = (char *)(cp);
}
Jim_Obj *o;
double _safe;
- if (puthere == NULL){
+ if (puthere == NULL) {
puthere = &_safe;
}
r = Jim_GetOpt_Obj(goi, &o);
- if (r == JIM_OK){
+ if (r == JIM_OK) {
r = Jim_GetDouble(goi->interp, o, puthere);
- if (r != JIM_OK){
+ if (r != JIM_OK) {
Jim_SetResult_sprintf(goi->interp,
"not a number: %s",
Jim_GetString(o, NULL));
Jim_Obj *o;
jim_wide _safe;
- if (puthere == NULL){
+ if (puthere == NULL) {
puthere = &_safe;
}
r = Jim_GetOpt_Obj(goi, &o);
- if (r == JIM_OK){
+ if (r == JIM_OK) {
r = Jim_GetWide(goi->interp, o, puthere);
}
return r;
Jim_Obj *o;
int e;
- if (puthere == NULL){
+ if (puthere == NULL) {
puthere = &_safe;
}
e = Jim_GetOpt_Obj(goi, &o);
- if (e == JIM_OK){
+ if (e == JIM_OK) {
e = Jim_Nvp_name2value_obj(goi->interp,
nvp,
o,
const Jim_Nvp *nvptable,
int hadprefix)
{
- if (hadprefix){
+ if (hadprefix) {
Jim_SetResult_NvpUnknown(goi->interp,
goi->argv[-2],
goi->argv[-1],
Jim_Obj *o;
int e;
- if (puthere == NULL){
+ if (puthere == NULL) {
puthere = &_safe;
}
e = Jim_GetOpt_Obj(goi, &o);
- if (e == JIM_OK){
+ if (e == JIM_OK) {
e = Jim_GetEnum(goi->interp,
o,
lookup,
va_start(ap,fmt);
buf = jim_vasprintf(fmt, ap);
va_end(ap);
- if (buf){
+ if (buf) {
Jim_SetResultString(interp, buf, -1);
jim_vasprintf_done(buf);
}
Jim_Obj *param_value,
const Jim_Nvp *nvp)
{
- if (param_name){
+ if (param_name) {
Jim_SetResult_sprintf(interp,
"%s: Unknown: %s, try one of: ",
Jim_GetString(param_name, NULL),
"Unknown param: %s, try one of: ",
Jim_GetString(param_value, NULL));
}
- while (nvp->name){
+ while (nvp->name) {
const char *a;
const char *b;
- if ((nvp + 1)->name){
+ if ((nvp + 1)->name) {
a = nvp->name;
b = ", ";
} else {
{
int x;
- if (debug_string_obj){
+ if (debug_string_obj) {
Jim_FreeObj(interp, debug_string_obj);
}
debug_string_obj = Jim_NewEmptyStringObj(interp);
- for (x = 0 ; x < argc ; x++){
+ for (x = 0 ; x < argc ; x++) {
Jim_AppendStrings(interp,
debug_string_obj,
Jim_GetString(argv[x], NULL),
/*
* we expect CHIP + TAP + OPTIONS
* */
- if (goi->argc < 3){
+ if (goi->argc < 3) {
Jim_SetResult_sprintf(goi->interp, "Missing CHIP TAP OPTIONS ....");
return JIM_ERR;
}
/* clear them as we find them */
reqbits = (NTREQ_IRLEN | NTREQ_IRCAPTURE | NTREQ_IRMASK);
- while (goi->argc){
+ while (goi->argc) {
e = Jim_GetOpt_Nvp(goi, opts, &n);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_GetOpt_NvpUnknown(goi, opts, 0);
return e;
}
LOG_DEBUG("Processing option: %s", n->name);
- switch (n->value){
+ switch (n->value) {
case NTAP_OPT_ENABLED:
pTap->disabled_after_reset = false;
break;
case NTAP_OPT_IRMASK:
case NTAP_OPT_IRCAPTURE:
e = Jim_GetOpt_Wide(goi, &w);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_SetResult_sprintf(goi->interp, "option: %s bad parameter", n->name);
return e;
}
- switch (n->value){
+ switch (n->value) {
case NTAP_OPT_IRLEN:
if (w > (jim_wide) (8 * sizeof(pTap->ir_capture_value)))
LOG_WARNING("huge IR length %d", (int) w);
Jim_GetOpt_Setup(&goi, interp, argc-1, argv + 1);
e = Jim_GetOpt_Nvp(&goi, jtag_cmds, &n);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_GetOpt_NvpUnknown(&goi, jtag_cmds, 0);
return e;
}
Jim_SetEmptyResult(goi.interp);
- switch (n->value){
+ switch (n->value) {
case JTAG_CMD_INTERFACE:
/* return the name of the interface */
/* TCL code might need to know the exact type... */
/* FUTURE: we allow this as a means to "set" the interface. */
- if (goi.argc != 0){
+ if (goi.argc != 0) {
Jim_WrongNumArgs(goi.interp, 1, goi.argv-1, "(no params)");
return JIM_ERR;
}
Jim_SetResultString(goi.interp, jtag_interface->name, -1);
return JIM_OK;
case JTAG_CMD_INIT_RESET:
- if (goi.argc != 0){
+ if (goi.argc != 0) {
Jim_WrongNumArgs(goi.interp, 1, goi.argv-1, "(no params)");
return JIM_ERR;
}
e = jtag_init_reset(context);
- if (e != ERROR_OK){
+ if (e != ERROR_OK) {
Jim_SetResult_sprintf(goi.interp, "error: %d", e);
return JIM_ERR;
}
case JTAG_CMD_TAPISENABLED:
case JTAG_CMD_TAPENABLE:
case JTAG_CMD_TAPDISABLE:
- if (goi.argc != 1){
+ if (goi.argc != 1) {
Jim_SetResultString(goi.interp, "Too many parameters",-1);
return JIM_ERR;
}
break;
case JTAG_CMD_CGET:
- if (goi.argc < 2){
+ if (goi.argc < 2) {
Jim_WrongNumArgs(goi.interp, 0, NULL, "?tap-name? -option ...");
return JIM_ERR;
}
Jim_GetOpt_Obj(&goi, &o);
t = jtag_tap_by_jim_obj(goi.interp, o);
- if (t == NULL){
+ if (t == NULL) {
return JIM_ERR;
}
break;
case JTAG_CMD_CONFIGURE:
- if (goi.argc < 3){
+ if (goi.argc < 3) {
Jim_WrongNumArgs(goi.interp, 0, NULL, "?tap-name? -option ?VALUE? ...");
return JIM_ERR;
}
Jim_GetOpt_Obj(&goi, &o);
t = jtag_tap_by_jim_obj(goi.interp, o);
- if (t == NULL){
+ if (t == NULL) {
return JIM_ERR;
}
* argv[ 3] = not actually used by anything but in the docs
*/
- if (argc < 4){
+ if (argc < 4) {
command_print(cmd_ctx, "OLD DEPRECATED SYNTAX: Please use the NEW syntax");
return ERROR_OK;
}
Jim_GetString(newargs[9], NULL));
e = jim_jtag_command(interp, 10, newargs);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
command_print(cmd_ctx, "%s", Jim_GetString(Jim_GetResult(interp), NULL));
}
return e;
command_print(cmd_ctx, " TapName | Enabled | IdCode Expected IrLen IrCap IrMask Instr ");
command_print(cmd_ctx, "---|--------------------|---------|------------|------------|------|------|------|---------");
- while (tap){
+ while (tap) {
uint32_t expected, expected_mask, cur_instr, ii;
expected = buf_get_u32(tap->expected, 0, tap->ir_length);
expected_mask = buf_get_u32(tap->expected_mask, 0, tap->ir_length);
*/
endstate = TAP_IDLE;
- if (argc >= 4){
+ if (argc >= 4) {
/* have at least one pair of numbers. */
/* is last pair the magic text? */
- if (0 == strcmp("-endstate", args[ argc - 2 ])){
+ if (0 == strcmp("-endstate", args[ argc - 2 ])) {
const char *cpA;
const char *cpS;
cpA = args[ argc-1 ];
- for (endstate = 0 ; endstate < TAP_NUM_STATES ; endstate++){
+ for (endstate = 0 ; endstate < TAP_NUM_STATES ; endstate++) {
cpS = tap_state_name(endstate);
- if (0 == strcmp(cpA, cpS)){
+ if (0 == strcmp(cpA, cpS)) {
break;
}
}
- if (endstate >= TAP_NUM_STATES){
+ if (endstate >= TAP_NUM_STATES) {
return ERROR_COMMAND_SYNTAX_ERROR;
} else {
if (!scan_is_safe(endstate))
e = Jim_GetLong(interp, args[i], &bits);
/* If valid - try next arg */
- if (e == JIM_OK){
+ if (e == JIM_OK) {
continue;
}
/* Not valid.. are we at the end? */
- if (((i + 2) != argc)){
+ if (((i + 2) != argc)) {
/* nope, then error */
return e;
}
/* get arg as a string. */
cp = Jim_GetString(args[i], NULL);
/* is it the magic? */
- if (0 == strcmp("-endstate", cp)){
+ if (0 == strcmp("-endstate", cp)) {
/* is the statename valid? */
cp = Jim_GetString(args[i + 1], NULL);
/* see if it is a valid state name */
endstate = tap_state_by_name(cp);
- if (endstate < 0){
+ if (endstate < 0) {
/* update the error message */
Jim_SetResult_sprintf(interp,"endstate: %s invalid", cp);
} else {
}
/* Still an error? */
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return e; /* too bad */
}
} /* validate args */
tap = jtag_tap_by_jim_obj(interp, args[1]);
- if (tap == NULL){
+ if (tap == NULL) {
return JIM_ERR;
}
const Jim_Nvp *n;
n = Jim_Nvp_value2name_simple(nvp_error_target, err);
- if (n->name == NULL){
+ if (n->name == NULL) {
return "unknown";
} else {
return n->name;
x = -1;
t = all_targets;
- while (t){
- if (x < t->target_number){
+ while (t) {
+ if (x < t->target_number) {
x = (t->target_number) + 1;
}
t = t->next;
/* number is 0 based */
x = -1;
t = all_targets;
- while (t){
- if (x < t->target_number){
+ while (t) {
+ if (x < t->target_number) {
x = t->target_number;
}
t = t->next;
{
target_t *target = all_targets;
- while (target){
- if (target->target_number == num){
+ while (target) {
+ if (target->target_number == num) {
return target;
}
target = target->next;
int retval;
Jim_Nvp *n;
n = Jim_Nvp_value2name_simple(nvp_reset_modes, reset_mode);
- if (n->name == NULL){
+ if (n->name == NULL) {
LOG_ERROR("invalid reset mode");
return ERROR_FAIL;
}
{
const Jim_Nvp *n;
n = Jim_Nvp_name2value_simple(nvp_reset_modes, args[0]);
- if ((n->name == NULL) || (n->value == RESET_UNKNOWN)){
+ if ((n->name == NULL) || (n->value == RESET_UNKNOWN)) {
return ERROR_COMMAND_SYNTAX_ERROR;
}
reset_mode = n->value;
target_t *target;
context = Jim_GetAssocData(interp, "context");
- if (context == NULL){
+ if (context == NULL) {
LOG_ERROR("array2mem: no command context");
return JIM_ERR;
}
target = get_current_target(context);
- if (target == NULL){
+ if (target == NULL) {
LOG_ERROR("array2mem: no current target");
return JIM_ERR;
}
Jim_Nvp_value2name_simple(nvp_target_event, e)->name);
target = all_targets;
- while (target){
+ while (target) {
target_handle_event(target, e);
target = target->next;
}
teap = target->event_action;
done = 0;
- while (teap){
- if (teap->event == e){
+ while (teap) {
+ if (teap->event == e) {
done = 1;
LOG_DEBUG("target: (%d) %s (%s) event: %d (%s) action: %s\n",
target->target_number,
}
teap = teap->next;
}
- if (!done){
+ if (!done) {
LOG_DEBUG("event: %d %s - no action",
e,
Jim_Nvp_value2name_simple(nvp_target_event, e)->name);
int e;
/* parse config or cget options ... */
- while (goi->argc > 0){
+ while (goi->argc > 0) {
Jim_SetEmptyResult(goi->interp);
/* Jim_GetOpt_Debug(goi); */
- if (target->type->target_jim_configure){
+ if (target->type->target_jim_configure) {
/* target defines a configure function */
/* target gets first dibs on parameters */
e = (*(target->type->target_jim_configure))(target, goi);
- if (e == JIM_OK){
+ if (e == JIM_OK) {
/* more? */
continue;
}
- if (e == JIM_ERR){
+ if (e == JIM_ERR) {
/* An error */
return e;
}
/* otherwise we 'continue' below */
}
e = Jim_GetOpt_Nvp(goi, nvp_config_opts, &n);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_GetOpt_NvpUnknown(goi, nvp_config_opts, 0);
return e;
}
- switch (n->value){
+ switch (n->value) {
case TCFG_TYPE:
/* not setable */
- if (goi->isconfigure){
+ if (goi->isconfigure) {
Jim_SetResult_sprintf(goi->interp, "not setable: %s", n->name);
return JIM_ERR;
} else {
no_params:
- if (goi->argc != 0){
+ if (goi->argc != 0) {
Jim_WrongNumArgs(goi->interp, goi->argc, goi->argv, "NO PARAMS");
return JIM_ERR;
}
/* loop for more */
break;
case TCFG_EVENT:
- if (goi->argc == 0){
+ if (goi->argc == 0) {
Jim_WrongNumArgs(goi->interp, goi->argc, goi->argv, "-event ?event-name? ...");
return JIM_ERR;
}
e = Jim_GetOpt_Nvp(goi, nvp_target_event, &n);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_GetOpt_NvpUnknown(goi, nvp_target_event, 1);
return e;
}
- if (goi->isconfigure){
- if (goi->argc != 1){
+ if (goi->isconfigure) {
+ if (goi->argc != 1) {
Jim_WrongNumArgs(goi->interp, goi->argc, goi->argv, "-event ?event-name? ?EVENT-BODY?");
return JIM_ERR;
}
} else {
- if (goi->argc != 0){
+ if (goi->argc != 0) {
Jim_WrongNumArgs(goi->interp, goi->argc, goi->argv, "-event ?event-name?");
return JIM_ERR;
}
teap = target->event_action;
/* replace existing? */
- while (teap){
- if (teap->event == (enum target_event)n->value){
+ while (teap) {
+ if (teap->event == (enum target_event)n->value) {
break;
}
teap = teap->next;
}
- if (goi->isconfigure){
- if (teap == NULL){
+ if (goi->isconfigure) {
+ if (teap == NULL) {
/* create new */
teap = calloc(1, sizeof(*teap));
}
teap->event = n->value;
Jim_GetOpt_Obj(goi, &o);
- if (teap->body){
+ if (teap->body) {
Jim_DecrRefCount(interp, teap->body);
}
teap->body = Jim_DuplicateObj(goi->interp, o);
Jim_SetEmptyResult(goi->interp);
} else {
/* get */
- if (teap == NULL){
+ if (teap == NULL) {
Jim_SetEmptyResult(goi->interp);
} else {
Jim_SetResult(goi->interp, Jim_DuplicateObj(goi->interp, teap->body));
break;
case TCFG_WORK_AREA_VIRT:
- if (goi->isconfigure){
+ if (goi->isconfigure) {
target_free_all_working_areas(target);
e = Jim_GetOpt_Wide(goi, &w);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return e;
}
target->working_area_virt = w;
} else {
- if (goi->argc != 0){
+ if (goi->argc != 0) {
goto no_params;
}
}
break;
case TCFG_WORK_AREA_PHYS:
- if (goi->isconfigure){
+ if (goi->isconfigure) {
target_free_all_working_areas(target);
e = Jim_GetOpt_Wide(goi, &w);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return e;
}
target->working_area_phys = w;
} else {
- if (goi->argc != 0){
+ if (goi->argc != 0) {
goto no_params;
}
}
break;
case TCFG_WORK_AREA_SIZE:
- if (goi->isconfigure){
+ if (goi->isconfigure) {
target_free_all_working_areas(target);
e = Jim_GetOpt_Wide(goi, &w);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return e;
}
target->working_area_size = w;
} else {
- if (goi->argc != 0){
+ if (goi->argc != 0) {
goto no_params;
}
}
break;
case TCFG_WORK_AREA_BACKUP:
- if (goi->isconfigure){
+ if (goi->isconfigure) {
target_free_all_working_areas(target);
e = Jim_GetOpt_Wide(goi, &w);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return e;
}
/* make this exactly 1 or 0 */
target->backup_working_area = (!!w);
} else {
- if (goi->argc != 0){
+ if (goi->argc != 0) {
goto no_params;
}
}
break;
case TCFG_ENDIAN:
- if (goi->isconfigure){
+ if (goi->isconfigure) {
e = Jim_GetOpt_Nvp(goi, nvp_target_endian, &n);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_GetOpt_NvpUnknown(goi, nvp_target_endian, 1);
return e;
}
target->endianness = n->value;
} else {
- if (goi->argc != 0){
+ if (goi->argc != 0) {
goto no_params;
}
}
n = Jim_Nvp_value2name_simple(nvp_target_endian, target->endianness);
- if (n->name == NULL){
+ if (n->name == NULL) {
target->endianness = TARGET_LITTLE_ENDIAN;
n = Jim_Nvp_value2name_simple(nvp_target_endian, target->endianness);
}
break;
case TCFG_VARIANT:
- if (goi->isconfigure){
- if (goi->argc < 1){
+ if (goi->isconfigure) {
+ if (goi->argc < 1) {
Jim_SetResult_sprintf(goi->interp,
"%s ?STRING?",
n->name);
return JIM_ERR;
}
- if (target->variant){
+ if (target->variant) {
free((void *)(target->variant));
}
e = Jim_GetOpt_String(goi, &cp, NULL);
target->variant = strdup(cp);
} else {
- if (goi->argc != 0){
+ if (goi->argc != 0) {
goto no_params;
}
}
/* loop for more */
break;
case TCFG_CHAIN_POSITION:
- if (goi->isconfigure){
+ if (goi->isconfigure) {
Jim_Obj *o;
jtag_tap_t *tap;
target_free_all_working_areas(target);
e = Jim_GetOpt_Obj(goi, &o);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return e;
}
tap = jtag_tap_by_jim_obj(goi->interp, o);
- if (tap == NULL){
+ if (tap == NULL) {
return JIM_ERR;
}
/* make this exactly 1 or 0 */
target->tap = tap;
} else {
- if (goi->argc != 0){
+ if (goi->argc != 0) {
goto no_params;
}
}
/* commands here are in an NVP table */
e = Jim_GetOpt_Nvp(&goi, target_options, &n);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_GetOpt_NvpUnknown(&goi, target_options, 0);
return e;
}
/* Assume blank result */
Jim_SetEmptyResult(goi.interp);
- switch (n->value){
+ switch (n->value) {
case TS_CMD_CONFIGURE:
- if (goi.argc < 2){
+ if (goi.argc < 2) {
Jim_WrongNumArgs(goi.interp, goi.argc, goi.argv, "missing: -option VALUE ...");
return JIM_ERR;
}
return target_configure(&goi, target);
case TS_CMD_CGET:
// some things take params
- if (goi.argc < 1){
+ if (goi.argc < 1) {
Jim_WrongNumArgs(goi.interp, 0, goi.argv, "missing: ?-option?");
return JIM_ERR;
}
* argv[3] = optional count.
*/
- if ((goi.argc == 2) || (goi.argc == 3)){
+ if ((goi.argc == 2) || (goi.argc == 3)) {
/* all is well */
} else {
mwx_error:
}
e = Jim_GetOpt_Wide(&goi, &a);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
goto mwx_error;
}
e = Jim_GetOpt_Wide(&goi, &b);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
goto mwx_error;
}
if (goi.argc == 3) {
e = Jim_GetOpt_Wide(&goi, &c);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
goto mwx_error;
}
} else {
c = 1;
}
- switch (n->value){
+ switch (n->value) {
case TS_CMD_MWW:
target_buffer_set_u32(target, target_buf, b);
b = 4;
b = 1;
break;
}
- for (x = 0 ; x < c ; x++){
+ for (x = 0 ; x < c ; x++) {
e = target_write_memory(target, a, b, 1, target_buf);
- if (e != ERROR_OK){
+ if (e != ERROR_OK) {
Jim_SetResult_sprintf(interp, "Error writing @ 0x%08x: %d\n", (int)(a), e);
return JIM_ERR;
}
* argv[1] = address
* argv[2] = optional count
*/
- if ((goi.argc == 2) || (goi.argc == 3)){
+ if ((goi.argc == 2) || (goi.argc == 3)) {
Jim_SetResult_sprintf(goi.interp, "expected: %s ADDR [COUNT]", n->name);
return JIM_ERR;
}
e = Jim_GetOpt_Wide(&goi, &a);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return JIM_ERR;
}
- if (goi.argc){
+ if (goi.argc) {
e = Jim_GetOpt_Wide(&goi, &c);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return JIM_ERR;
}
} else {
c = 1;
}
b = 1; /* shut up gcc */
- switch (n->value){
+ switch (n->value) {
case TS_CMD_MDW:
b = 4;
break;
/* convert to "bytes" */
c = c * b;
/* count is now in 'BYTES' */
- while (c > 0){
+ while (c > 0) {
y = c;
- if (y > 16){
+ if (y > 16) {
y = 16;
}
e = target_read_memory(target, a, b, y / b, target_buf);
- if (e != ERROR_OK){
+ if (e != ERROR_OK) {
Jim_SetResult_sprintf(interp, "error reading target @ 0x%08lx", (int)(a));
return JIM_ERR;
}
Jim_fprintf(interp, interp->cookie_stdout, "0x%08x ", (int)(a));
- switch (b){
+ switch (b) {
case 4:
- for (x = 0 ; (x < 16) && (x < y) ; x += 4){
+ for (x = 0 ; (x < 16) && (x < y) ; x += 4) {
z = target_buffer_get_u32(target, &(target_buf[ x * 4 ]));
Jim_fprintf(interp, interp->cookie_stdout, "%08x ", (int)(z));
}
- for (; (x < 16) ; x += 4){
+ for (; (x < 16) ; x += 4) {
Jim_fprintf(interp, interp->cookie_stdout, " ");
}
break;
case 2:
- for (x = 0 ; (x < 16) && (x < y) ; x += 2){
+ for (x = 0 ; (x < 16) && (x < y) ; x += 2) {
z = target_buffer_get_u16(target, &(target_buf[ x * 2 ]));
Jim_fprintf(interp, interp->cookie_stdout, "%04x ", (int)(z));
}
- for (; (x < 16) ; x += 2){
+ for (; (x < 16) ; x += 2) {
Jim_fprintf(interp, interp->cookie_stdout, " ");
}
break;
case 1:
default:
- for (x = 0 ; (x < 16) && (x < y) ; x += 1){
+ for (x = 0 ; (x < 16) && (x < y) ; x += 1) {
z = target_buffer_get_u8(target, &(target_buf[ x * 4 ]));
Jim_fprintf(interp, interp->cookie_stdout, "%02x ", (int)(z));
}
- for (; (x < 16) ; x += 1){
+ for (; (x < 16) ; x += 1) {
Jim_fprintf(interp, interp->cookie_stdout, " ");
}
break;
}
/* ascii-ify the bytes */
- for (x = 0 ; x < y ; x++){
+ for (x = 0 ; x < y ; x++) {
if ((target_buf[x] >= 0x20) &&
- (target_buf[x] <= 0x7e)){
+ (target_buf[x] <= 0x7e)) {
/* good */
} else {
/* smack it */
}
}
/* space pad */
- while (x < 16){
+ while (x < 16) {
target_buf[x] = ' ';
x++;
}
return target_array2mem(goi.interp, target, goi.argc, goi.argv);
break;
case TS_CMD_EXAMINE:
- if (goi.argc){
+ if (goi.argc) {
Jim_WrongNumArgs(goi.interp, 2, argv, "[no parameters]");
return JIM_ERR;
}
if (!target->tap->enabled)
goto err_tap_disabled;
e = target->type->examine(target);
- if (e != ERROR_OK){
+ if (e != ERROR_OK) {
Jim_SetResult_sprintf(interp, "examine-fails: %d", e);
return JIM_ERR;
}
return JIM_OK;
case TS_CMD_POLL:
- if (goi.argc){
+ if (goi.argc) {
Jim_WrongNumArgs(goi.interp, 2, argv, "[no parameters]");
return JIM_ERR;
}
if (!target->tap->enabled)
goto err_tap_disabled;
- if (!(target_was_examined(target))){
+ if (!(target_was_examined(target))) {
e = ERROR_TARGET_NOT_EXAMINED;
} else {
e = target->type->poll(target);
}
- if (e != ERROR_OK){
+ if (e != ERROR_OK) {
Jim_SetResult_sprintf(interp, "poll-fails: %d", e);
return JIM_ERR;
} else {
}
break;
case TS_CMD_RESET:
- if (goi.argc != 2){
+ if (goi.argc != 2) {
Jim_WrongNumArgs(interp, 2, argv, "t | f|assert | deassert BOOL");
return JIM_ERR;
}
e = Jim_GetOpt_Nvp(&goi, nvp_assert, &n);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_GetOpt_NvpUnknown(&goi, nvp_assert, 1);
return e;
}
/* the halt or not param */
e = Jim_GetOpt_Wide(&goi, &a);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return e;
}
if (!target->tap->enabled)
target_free_all_working_areas_restore(target, 0);
/* do the assert */
- if (n->value == NVP_ASSERT){
+ if (n->value == NVP_ASSERT) {
target->type->assert_reset(target);
} else {
target->type->deassert_reset(target);
}
return JIM_OK;
case TS_CMD_HALT:
- if (goi.argc){
+ if (goi.argc) {
Jim_WrongNumArgs(goi.interp, 0, argv, "halt [no parameters]");
return JIM_ERR;
}
return JIM_OK;
case TS_CMD_WAITSTATE:
/* params: <name> statename timeoutmsecs */
- if (goi.argc != 2){
+ if (goi.argc != 2) {
Jim_SetResult_sprintf(goi.interp, "%s STATENAME TIMEOUTMSECS", n->name);
return JIM_ERR;
}
e = Jim_GetOpt_Nvp(&goi, nvp_target_state, &n);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_GetOpt_NvpUnknown(&goi, nvp_target_state,1);
return e;
}
e = Jim_GetOpt_Wide(&goi, &a);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return e;
}
if (!target->tap->enabled)
goto err_tap_disabled;
e = target_wait_state(target, n->value, a);
- if (e != ERROR_OK){
+ if (e != ERROR_OK) {
Jim_SetResult_sprintf(goi.interp,
"target: %s wait %s fails (%d) %s",
target->cmd_name,
target->cmd_name);
command_print(cmd_ctx, "%-25s | Body", "Event");
command_print(cmd_ctx, "------------------------- | ----------------------------------------");
- while (teap){
+ while (teap) {
command_print(cmd_ctx,
"%-25s | %s",
Jim_Nvp_value2name_simple(nvp_target_event, teap->event)->name,
return JIM_OK;
}
case TS_CMD_CURSTATE:
- if (goi.argc != 0){
+ if (goi.argc != 0) {
Jim_WrongNumArgs(goi.interp, 0, argv, "[no parameters]");
return JIM_ERR;
}
Jim_Nvp_value2name_simple(nvp_target_state,target->state)->name,-1);
return JIM_OK;
case TS_CMD_INVOKE_EVENT:
- if (goi.argc != 1){
+ if (goi.argc != 1) {
Jim_SetResult_sprintf(goi.interp, "%s ?EVENTNAME?",n->name);
return JIM_ERR;
}
e = Jim_GetOpt_Nvp(&goi, nvp_target_event, &n);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
Jim_GetOpt_NvpUnknown(&goi, nvp_target_event, 1);
return e;
}
struct command_context_s *cmd_ctx;
cmd_ctx = Jim_GetAssocData(goi->interp, "context");
- if (goi->argc < 3){
+ if (goi->argc < 3) {
Jim_WrongNumArgs(goi->interp, 1, goi->argv, "?name? ?type? ..options...");
return JIM_ERR;
}
Jim_GetOpt_Obj(goi, &new_cmd);
/* does this command exist? */
cmd = Jim_GetCommand(goi->interp, new_cmd, JIM_ERRMSG);
- if (cmd){
+ if (cmd) {
cp = Jim_GetString(new_cmd, NULL);
Jim_SetResult_sprintf(goi->interp, "Command/target: %s Exists", cp);
return JIM_ERR;
e = Jim_GetOpt_String(goi, &cp2, NULL);
cp = cp2;
/* now does target type exist */
- for (x = 0 ; target_types[x] ; x++){
- if (0 == strcmp(cp, target_types[x]->name)){
+ for (x = 0 ; target_types[x] ; x++) {
+ if (0 == strcmp(cp, target_types[x]->name)) {
/* found */
break;
}
}
- if (target_types[x] == NULL){
+ if (target_types[x] == NULL) {
Jim_SetResult_sprintf(goi->interp, "Unknown target type %s, try one of ", cp);
- for (x = 0 ; target_types[x] ; x++){
- if (target_types[x + 1]){
+ for (x = 0 ; target_types[x] ; x++) {
+ if (target_types[x + 1]) {
Jim_AppendStrings(goi->interp,
Jim_GetResult(goi->interp),
target_types[x]->name,
e = JIM_ERR;
}
- if (e != JIM_OK){
+ if (e != JIM_OK) {
free(target->type);
free(target);
return e;
}
- if (target->endianness == TARGET_ENDIAN_UNKNOWN){
+ if (target->endianness == TARGET_ENDIAN_UNKNOWN) {
/* default endian to little if not specified */
target->endianness = TARGET_LITTLE_ENDIAN;
}
target->variant = strdup("");
/* create the target specific commands */
- if (target->type->register_commands){
+ if (target->type->register_commands) {
(*(target->type->register_commands))(cmd_ctx);
}
- if (target->type->target_create){
+ if (target->type->target_create) {
(*(target->type->target_create))(target, goi->interp);
}
{
target_t **tpp;
tpp = &(all_targets);
- while (*tpp){
+ while (*tpp) {
tpp = &((*tpp)->next);
}
*tpp = target;
Jim_GetOpt_Setup(&goi, interp, argc-1, argv + 1);
- if (goi.argc == 0){
+ if (goi.argc == 0) {
Jim_WrongNumArgs(interp, 1, argv, "missing: command ...");
return JIM_ERR;
}
/* Jim_GetOpt_Debug(&goi); */
r = Jim_GetOpt_Enum(&goi, target_cmds, &x);
- if (r != JIM_OK){
+ if (r != JIM_OK) {
return r;
}
- switch (x){
+ switch (x) {
default:
Jim_Panic(goi.interp,"Why am I here?");
return JIM_ERR;
case TG_CMD_CURRENT:
- if (goi.argc != 0){
+ if (goi.argc != 0) {
Jim_WrongNumArgs(goi.interp, 1, goi.argv, "Too many parameters");
return JIM_ERR;
}
Jim_SetResultString(goi.interp, get_current_target(cmd_ctx)->cmd_name, -1);
return JIM_OK;
case TG_CMD_TYPES:
- if (goi.argc != 0){
+ if (goi.argc != 0) {
Jim_WrongNumArgs(goi.interp, 1, goi.argv, "Too many parameters");
return JIM_ERR;
}
Jim_SetResult(goi.interp, Jim_NewListObj(goi.interp, NULL, 0));
- for (x = 0 ; target_types[x] ; x++){
+ for (x = 0 ; target_types[x] ; x++) {
Jim_ListAppendElement(goi.interp,
Jim_GetResult(goi.interp),
Jim_NewStringObj(goi.interp, target_types[x]->name, -1));
}
return JIM_OK;
case TG_CMD_NAMES:
- if (goi.argc != 0){
+ if (goi.argc != 0) {
Jim_WrongNumArgs(goi.interp, 1, goi.argv, "Too many parameters");
return JIM_ERR;
}
Jim_SetResult(goi.interp, Jim_NewListObj(goi.interp, NULL, 0));
target = all_targets;
- while (target){
+ while (target) {
Jim_ListAppendElement(goi.interp,
Jim_GetResult(goi.interp),
Jim_NewStringObj(goi.interp, target->cmd_name, -1));
}
return JIM_OK;
case TG_CMD_CREATE:
- if (goi.argc < 3){
+ if (goi.argc < 3) {
Jim_WrongNumArgs(goi.interp, goi.argc, goi.argv, "?name ... config options ...");
return JIM_ERR;
}
return target_create(&goi);
break;
case TG_CMD_NUMBER:
- if (goi.argc != 1){
+ if (goi.argc != 1) {
Jim_SetResult_sprintf(goi.interp, "expected: target number ?NUMBER?");
return JIM_ERR;
}
e = Jim_GetOpt_Wide(&goi, &w);
- if (e != JIM_OK){
+ if (e != JIM_OK) {
return JIM_ERR;
}
{
target_t *t;
t = get_target_by_num(w);
- if (t == NULL){
+ if (t == NULL) {
Jim_SetResult_sprintf(goi.interp,"Target: number %d does not exist", (int)(w));
return JIM_ERR;
}
return JIM_OK;
}
case TG_CMD_COUNT:
- if (goi.argc != 0){
+ if (goi.argc != 0) {
Jim_WrongNumArgs(goi.interp, 0, goi.argv, "<no parameters>");
return JIM_ERR;
}
projects / openocd / commitdiff