On Fri, 14 Aug 2015 22:51:54 +0800 Xiao Guangrong <guangrong.xiao@xxxxxxxxxxxxxxx> wrote: > Currently, the offset in OperationRegion is limited to 32 bit, extend it > to 64 bit so that we can switch SSDT to 64 bit in later patch > > Signed-off-by: Xiao Guangrong <guangrong.xiao@xxxxxxxxxxxxxxx> Reviewed-by: Igor Mammedov <imammedo@xxxxxxxxxx> > --- > hw/acpi/aml-build.c | 2 +- > include/hw/acpi/aml-build.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c > index 0d4b324..02f9e3d 100644 > --- a/hw/acpi/aml-build.c > +++ b/hw/acpi/aml-build.c > @@ -752,7 +752,7 @@ Aml *aml_package(uint8_t num_elements) > > /* ACPI 1.0b: 16.2.5.2 Named Objects Encoding: DefOpRegion */ > Aml *aml_operation_region(const char *name, AmlRegionSpace rs, > - uint32_t offset, uint32_t len) > + uint64_t offset, uint32_t len) > { > Aml *var = aml_alloc(); > build_append_byte(var->buf, 0x5B); /* ExtOpPrefix */ > diff --git a/include/hw/acpi/aml-build.h b/include/hw/acpi/aml-build.h > index e3afa13..996ac5b 100644 > --- a/include/hw/acpi/aml-build.h > +++ b/include/hw/acpi/aml-build.h > @@ -222,7 +222,7 @@ Aml *aml_interrupt(AmlConsumerAndProducer con_and_pro, > Aml *aml_io(AmlIODecode dec, uint16_t min_base, uint16_t max_base, > uint8_t aln, uint8_t len); > Aml *aml_operation_region(const char *name, AmlRegionSpace rs, > - uint32_t offset, uint32_t len); > + uint64_t offset, uint32_t len); > Aml *aml_irq_no_flags(uint8_t irq); > Aml *aml_named_field(const char *name, unsigned length); > Aml *aml_reserved_field(unsigned length); -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html