arc: Improve warning when breakpoint has been overwritten

Signed-off-by: Anton Kolesov <[email protected]>

Author: anthony-kolesov

src/target/arc_dbg.c

@@ -198,8 +198,10 @@ static int arc_dbg_unset_breakpoint(struct target *target,
 				if (retval != ERROR_OK)
 					return retval;
 			} else {
-				LOG_WARNING("Software breakpoint @%" PRIx32
-					" has been overwritten outside of debugger.", breakpoint->address);
+				LOG_WARNING("Software breakpoint @0x%" PRIx32
+					" has been overwritten outside of debugger."
+					"Expected: 0x%08" PRIx32 ", got: 0x%08" PRIx32,
+					breakpoint->address, ARC32_SDBBP, current_instr);
 			}
 		} else if (breakpoint->length == 2) {
 			uint16_t current_instr;
@@ -216,8 +218,10 @@ static int arc_dbg_unset_breakpoint(struct target *target,
 				if (retval != ERROR_OK)
 					return retval;
 			} else {
-				LOG_WARNING("Software breakpoint @%" PRIx32
-					" has been overwritten outside of debugger.", breakpoint->address);
+				LOG_WARNING("Software breakpoint @0x%" PRIx32
+					" has been overwritten outside of debugger. "
+					"Expected: 0x%04" PRIx16 ", got: 0x%04" PRIx16,
+					breakpoint->address, ARC16_SDBBP, current_instr);
 			}
 		} else {
 			LOG_ERROR("Invalid breakpoint length: target supports only 2 or 4");