Re: [PATCH 1/6] monitor/hmp-cmds.c: Clean up hmp_dumpdtb printf

[Richard Henderson]

On 2/6/25 07:12, Peter Maydell wrote:

In hmp_dumpdtb(), we print a message when the command succeeds.  This
message is missing the trailing \n, so the HMP command prompt is
printed immediately after it.  We also weren't capitalizing 'DTB', or
quoting the filename in the message.  Fix these nits.

Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
---
  monitor/hmp-cmds.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/monitor/hmp-cmds.c b/monitor/hmp-cmds.c
index 0aa22e1ae27..ff87fd89e4d 100644
--- a/monitor/hmp-cmds.c
+++ b/monitor/hmp-cmds.c
@@ -431,6 +431,6 @@ void hmp_dumpdtb(Monitor *mon, const QDict *qdict)
          return;
      }

- monitor_printf(mon, "dtb dumped to %s", filename);

+    monitor_printf(mon, "DTB dumped to '%s'\n", filename);
  }
  #endif

Reviewed-by: Richard Henderson <richard.hender...@linaro.org>

r~