target/xtensa: avoid IHI for writes to non-executable memory

[openocd.git] / src / rtt / tcl.c

diff --git a/src/rtt/tcl.c b/src/rtt/tcl.c
index 7cbdccf567577764809a98d918e0febe5bd9c237..2b8822fce808aaf7a64be076d9005c5ce2bc0c8d 100644 (file)
--- a/src/rtt/tcl.c
+++ b/src/rtt/tcl.c
@@ -17,7 +17,7 @@
 
 COMMAND_HANDLER(handle_rtt_setup_command)
 {
-struct rtt_source source;
+       struct rtt_source source;
 
        if (CMD_ARGC != 3)
                return ERROR_COMMAND_SYNTAX_ERROR;
@@ -150,17 +150,17 @@ COMMAND_HANDLER(handle_rtt_channels_command)
        return ERROR_OK;
 }
 
-static int jim_channel_list(Jim_Interp *interp, int argc,
-       Jim_Obj * const *argv)
+COMMAND_HANDLER(handle_channel_list)
 {
-       Jim_Obj *list;
-       Jim_Obj *channel_list;
        char channel_name[CHANNEL_NAME_SIZE];
        const struct rtt_control *ctrl;
        struct rtt_channel_info info;
 
+       if (CMD_ARGC != 0)
+               return ERROR_COMMAND_SYNTAX_ERROR;
+
        if (!rtt_found_cb()) {
-               Jim_SetResultFormatted(interp, "rtt: Control block not available");
+               command_print(CMD, "rtt: Control block not available");
                return ERROR_FAIL;
        }
 
@@ -169,81 +169,47 @@ static int jim_channel_list(Jim_Interp *interp, int argc,
        info.name = channel_name;
        info.name_length = sizeof(channel_name);
 
-       list = Jim_NewListObj(interp, NULL, 0);
-       channel_list = Jim_NewListObj(interp, NULL, 0);
+       command_print(CMD, "{");
 
        for (unsigned int i = 0; i < ctrl->num_up_channels; i++) {
-               int ret;
-               Jim_Obj *tmp;
-
-               ret = rtt_read_channel_info(i, RTT_CHANNEL_TYPE_UP, &info);
-
+               int ret = rtt_read_channel_info(i, RTT_CHANNEL_TYPE_UP, &info);
                if (ret != ERROR_OK)
                        return ret;
 
                if (!info.size)
                        continue;
 
-               tmp = Jim_NewListObj(interp, NULL, 0);
-
-               Jim_ListAppendElement(interp, tmp, Jim_NewStringObj(interp,
-                       "name", -1));
-               Jim_ListAppendElement(interp, tmp, Jim_NewStringObj(interp,
-                       info.name, -1));
-
-               Jim_ListAppendElement(interp, tmp, Jim_NewStringObj(interp,
-                       "size", -1));
-               Jim_ListAppendElement(interp, tmp, Jim_NewIntObj(interp,
-                       info.size));
-
-               Jim_ListAppendElement(interp, tmp, Jim_NewStringObj(interp,
-                       "flags", -1));
-               Jim_ListAppendElement(interp, tmp, Jim_NewIntObj(interp,
-                       info.flags));
-
-               Jim_ListAppendElement(interp, channel_list, tmp);
+               command_print(CMD,
+                       "    {\n"
+                       "        name  %s\n"
+                       "        size  0x%" PRIx32 "\n"
+                       "        flags 0x%" PRIx32 "\n"
+                       "    }",
+                       info.name, info.size, info.flags);
        }
 
-       Jim_ListAppendElement(interp, list, channel_list);
-
-       channel_list = Jim_NewListObj(interp, NULL, 0);
+       command_print(CMD, "}\n{");
 
        for (unsigned int i = 0; i < ctrl->num_down_channels; i++) {
-               int ret;
-               Jim_Obj *tmp;
-
-               ret = rtt_read_channel_info(i, RTT_CHANNEL_TYPE_DOWN, &info);
-
+               int ret = rtt_read_channel_info(i, RTT_CHANNEL_TYPE_DOWN, &info);
                if (ret != ERROR_OK)
                        return ret;
 
                if (!info.size)
                        continue;
 
-               tmp = Jim_NewListObj(interp, NULL, 0);
-
-               Jim_ListAppendElement(interp, tmp, Jim_NewStringObj(interp,
-                       "name", -1));
-               Jim_ListAppendElement(interp, tmp, Jim_NewStringObj(interp,
-                       info.name, -1));
-
-               Jim_ListAppendElement(interp, tmp, Jim_NewStringObj(interp,
-                       "size", -1));
-               Jim_ListAppendElement(interp, tmp, Jim_NewIntObj(interp,
-                       info.size));
-
-               Jim_ListAppendElement(interp, tmp, Jim_NewStringObj(interp,
-                       "flags", -1));
-               Jim_ListAppendElement(interp, tmp, Jim_NewIntObj(interp,
-                       info.flags));
-
-               Jim_ListAppendElement(interp, channel_list, tmp);
+               command_print(CMD,
+                       "    {\n"
+                       "        name  %s\n"
+                       "        size  0x%" PRIx32 "\n"
+                       "        flags 0x%" PRIx32 "\n"
+                       "    }",
+                       info.name, info.size, info.flags);
        }
 
-       Jim_ListAppendElement(interp, list, channel_list);
-       Jim_SetResult(interp, list);
+       command_print(CMD, "}");
 
-       return JIM_OK;
+       return ERROR_OK;
 }
 
 static const struct command_registration rtt_subcommand_handlers[] = {
@@ -284,7 +250,7 @@ static const struct command_registration rtt_subcommand_handlers[] = {
        },
        {
                .name = "channellist",
-               .jim_handler = jim_channel_list,
+               .handler = handle_channel_list,
                .mode = COMMAND_EXEC,
                .help = "list available channels",
                .usage = ""