On Mon, 13 Jul 2020 08:17:41 -0400, Michael S. Tsirkin wrote:
Sounds good. And how about also adding: .impl.min_access_size = 4, ?
Yes, this works too - what does that do? static const MemoryRegionOps acpi_pm_tmr_ops = { .read = acpi_pm_tmr_read, .write = acpi_pm_tmr_write, .valid.min_access_size = 2, .valid.max_access_size = 4, .impl.min_access_size = 4, .impl.max_access_size = 4, .endianness = DEVICE_LITTLE_ENDIAN, }; Regards. -- Simon John