hw/microblaze: Restrict MemoryRegionOps are implemented as 32-bit
authorPhilippe Mathieu-Daudé <philmd@linaro.org>
Wed, 25 Sep 2024 21:14:08 +0000 (23:14 +0200)
committerPhilippe Mathieu-Daudé <philmd@linaro.org>
Mon, 13 Jan 2025 16:16:03 +0000 (17:16 +0100)
All these MemoryRegionOps read() and write() handlers are
implemented expecting 32-bit accesses. Clarify that setting
.impl.min/max_access_size fields.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Anton Johansson <anjo@rev.ng>
Message-Id: <20241105130431.22564-8-philmd@linaro.org>

hw/intc/xilinx_intc.c
hw/net/xilinx_ethlite.c
hw/timer/xilinx_timer.c

index d99cf567aeb65d296ae4f33c4f6dff17958c727a..6930f83907acaec030d929659fbf8eb082402cd3 100644 (file)
@@ -144,6 +144,10 @@ static const MemoryRegionOps pic_ops = {
     .read = pic_read,
     .write = pic_write,
     .endianness = DEVICE_NATIVE_ENDIAN,
+    .impl = {
+        .min_access_size = 4,
+        .max_access_size = 4,
+    },
     .valid = {
         .min_access_size = 4,
         .max_access_size = 4
index 4c0c7fcae3ea8b81be452f8fb6c75452f5f5ec3c..88ab331acce9bc60203965ac961d0589883ce181 100644 (file)
@@ -166,6 +166,10 @@ static const MemoryRegionOps eth_ops = {
     .read = eth_read,
     .write = eth_write,
     .endianness = DEVICE_NATIVE_ENDIAN,
+    .impl = {
+        .min_access_size = 4,
+        .max_access_size = 4,
+    },
     .valid = {
         .min_access_size = 4,
         .max_access_size = 4
index 4955fe1b01bd14f08d8521667616a170935f3f1f..6595cf5f51742455b7970ed8e4d5f1a15c206d98 100644 (file)
@@ -193,6 +193,10 @@ static const MemoryRegionOps timer_ops = {
     .read = timer_read,
     .write = timer_write,
     .endianness = DEVICE_NATIVE_ENDIAN,
+    .impl = {
+        .min_access_size = 4,
+        .max_access_size = 4,
+    },
     .valid = {
         .min_access_size = 4,
         .max_access_size = 4