On Sun, 22 Jan 2023 18:07:21 +0100
Bernhard Beschow <shen...@gmail.com> wrote:
> Cosmetic change emphasizing
... how 3/7 makes it more confusing
> that always the actual address of the gpe0
> block is returned.
>
> Signed-off-by: Bernhard Beschow <shen...@gmail.com>
> ---
> hw/acpi/ich9.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c
> index f8af238974..40a20e01ea 100644
> --- a/hw/acpi/ich9.c
> +++ b/hw/acpi/ich9.c
> @@ -348,7 +348,9 @@ static void ich9_pm_get_gpe0_blk(Object *obj, Visitor *v,
> const char *name,
> void *opaque, Error **errp)
> {
> ICH9LPCPMRegs *pm = opaque;
> - uint64_t value = pm->io.addr + ICH9_PMIO_GPE0_STS;
> + uint64_t value = pm->io.addr + pm->io_gpe.addr;
do not poke inside of MemoryRegion, right API to use here
if you really want to get it from memory region would be
memory_region_find && MemoryRegionSection::offset_within_address_space
magic
I'd just drop 3/7 and this patch
> +
> + assert(&pm->io == pm->io_gpe.container);
>
> visit_type_uint64(v, name, &value, errp);
> }
