Commit 56f7e42c47 for qemu.org
commit 56f7e42c474fa9f9f409f37c0882be4149d8c427
Author: Philippe Mathieu-Daudé <philmd@linaro.org>
Date: Wed Dec 10 11:48:13 2025 +0100
hw/timer/hpet: Mark implementation as being little-endian
The HPET component is only built / used by X86 targets, which
are only built in little endianness. Thus we only ever built
as little endian, never testing the big-endian possibility of
the DEVICE_NATIVE_ENDIAN definition. Simplify by only keeping
the little endian variant.
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Zhao Liu <zhao1.liu@intel.com>
Message-Id: <20251224134644.85582-6-philmd@linaro.org>
diff --git a/hw/timer/hpet.c b/hw/timer/hpet.c
index 070b40e7f6..0d7b8e0c7a 100644
--- a/hw/timer/hpet.c
+++ b/hw/timer/hpet.c
@@ -648,7 +648,7 @@ static const MemoryRegionOps hpet_ram_ops = {
.min_access_size = 4,
.max_access_size = 8,
},
- .endianness = DEVICE_NATIVE_ENDIAN,
+ .endianness = DEVICE_LITTLE_ENDIAN,
};
static void hpet_reset(DeviceState *d)
diff --git a/rust/hw/timer/hpet/src/device.rs b/rust/hw/timer/hpet/src/device.rs
index 3166688e4e..4757bf5fd5 100644
--- a/rust/hw/timer/hpet/src/device.rs
+++ b/rust/hw/timer/hpet/src/device.rs
@@ -763,7 +763,7 @@ unsafe fn init(mut this: ParentInit<Self>) {
MemoryRegionOpsBuilder::<HPETState>::new()
.read(&HPETState::read)
.write(&HPETState::write)
- .native_endian()
+ .little_endian()
.valid_sizes(4, 8)
.impl_sizes(4, 8)
.build();