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

Reply via email to