On 9/22/25 5:43 AM, Alex Bennée wrote:
Philippe Mathieu-Daudé <[email protected]> writes:

On 22/9/25 11:36, Alex Bennée wrote:
From: Pierrick Bouvier <[email protected]>
Reviewed-by: Richard Henderson <[email protected]>
Signed-off-by: Pierrick Bouvier <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Alex Bennée <[email protected]>
---
   semihosting/arm-compat-semi.c | 22 +++++++++++-----------
   1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/semihosting/arm-compat-semi.c
b/semihosting/arm-compat-semi.c
index 6725dcf4ba7..29cdab66f73 100644
--- a/semihosting/arm-compat-semi.c
+++ b/semihosting/arm-compat-semi.c
@@ -123,7 +123,7 @@ static GuestFD console_out_gf;
    */
     typedef struct LayoutInfo {
-    target_ulong rambase;
+    vaddr rambase;

This is what the subject describes. Most of the other changes aren't.

I was going to say vaddr is functionally a uint64_t but we actually made
it a uintptr_t in February. Shall I update to:

   semihosting/arm-compat-semi: replace target_ulong

   Replace with vaddr or uint64_t where appropriate

?

Looks good to me, thanks.
Pierrick

Reply via email to