Re: [PATCH v3 18/25] hw/watchdog/aspeed: Use memory_region_get_address()

[Cédric Le Goater]

On 10/28/25 19:12, Philippe Mathieu-Daudé wrote:

MemoryRegion::addr is private data of MemoryRegion, use
memory_region_get_address() to access it.

Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
---
  hw/watchdog/wdt_aspeed.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/watchdog/wdt_aspeed.c b/hw/watchdog/wdt_aspeed.c
index 30226435efc..f842d8e973a 100644
--- a/hw/watchdog/wdt_aspeed.c
+++ b/hw/watchdog/wdt_aspeed.c
@@ -274,7 +274,7 @@ static void aspeed_wdt_timer_expired(void *dev)
      }

qemu_log_mask(CPU_LOG_RESET, "Watchdog timer %" HWADDR_PRIx " expired.\n",

-                  s->iomem.addr);
+                  memory_region_get_address(&s->iomem));
      watchdog_perform_action();
      timer_del(s->timer);
  }

Reviewed-by: Cédric Le Goater <c...@redhat.com>

Thanks,

C.