[PATCH]: 02a6274ad3 target: Unify the format of bp command list

[gerrit]

This is an automated email from Gerrit.

"Marek Vrbka <marek.vr...@codasip.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7861

-- gerrit

commit 02a6274ad386892392d71548178e36faa1a2fa5d
Author: Marek Vrbka <marek.vr...@codasip.com>
Date:   Wed Aug 23 15:20:47 2023 +0200

    target: Unify the format of bp command list
    
    This patch unifies the format of breakpoints when using the "bp"
    tcl command as well as tell the user what each field means.
    
    Change-Id: Ic1335eda1c58072a334aed9cf0011431c8ec86a4
    Signed-off-by: Marek Vrbka <marek.vr...@codasip.com>

diff --git a/src/target/target.c b/src/target/target.c
index 96f4ae7d3e..6702855eb3 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -3926,24 +3926,24 @@ static int handle_bp_command_list(struct 
command_invocation *cmd)
                if (breakpoint->type == BKPT_SOFT) {
                        char *buf = buf_to_hex_str(breakpoint->orig_instr,
                                        breakpoint->length);
-                       command_print(cmd, "IVA breakpoint: " TARGET_ADDR_FMT 
", 0x%x, 0x%s",
+                       command_print(cmd, "Software breakpoint(IVA): addr=" 
TARGET_ADDR_FMT ", len=0x%x, orig_instr=0x%s",
                                        breakpoint->address,
                                        breakpoint->length,
                                        buf);
                        free(buf);
                } else {
                        if ((breakpoint->address == 0) && (breakpoint->asid != 
0))
-                               command_print(cmd, "Context breakpoint: 0x%8.8" 
PRIx32 ", 0x%x, %u",
+                               command_print(cmd, "Context breakpoint: 
asid=0x%8.8" PRIx32 ", len=0x%x, num=%u",
                                                        breakpoint->asid,
                                                        breakpoint->length, 
breakpoint->number);
                        else if ((breakpoint->address != 0) && 
(breakpoint->asid != 0)) {
-                               command_print(cmd, "Hybrid breakpoint(IVA): " 
TARGET_ADDR_FMT ", 0x%x, %u",
+                               command_print(cmd, "Hybrid breakpoint(IVA): 
addr=" TARGET_ADDR_FMT ", len=0x%x, num=%u",
                                                        breakpoint->address,
                                                        breakpoint->length, 
breakpoint->number);
                                command_print(cmd, "\t|--->linked with 
ContextID: 0x%8.8" PRIx32,
                                                        breakpoint->asid);
                        } else
-                               command_print(cmd, "Breakpoint(IVA): " 
TARGET_ADDR_FMT ", 0x%x, %u",
+                               command_print(cmd, "Hardware breakpoint(IVA): 
addr=" TARGET_ADDR_FMT ", len=0x%x, num=%u",
                                                        breakpoint->address,
                                                        breakpoint->length, 
breakpoint->number);
                }

--