> > +/* FIXME: copy & paste */
> > +static void acpi_dsdt_add_power_button(Aml *scope)
> > +{
> > +    Aml *dev = aml_device(ACPI_POWER_BUTTON_DEVICE);
> > +    aml_append(dev, aml_name_decl("_HID", aml_string("PNP0C0C")));
> > +    aml_append(dev, aml_name_decl("_UID", aml_int(0)));
> > +    aml_append(scope, dev);
> > +}
> 
> could be unified with ARM's version

Yep.  Suggestions for a good place?  hw/acpi/aml-build.c ?

> > +    acpi_align_size(tables_blob, ACPI_BUILD_TABLE_SIZE);
> > +    acpi_align_size(tables->linker->cmd_blob, ACPI_BUILD_ALIGN_SIZE);
> I'd drop these as that was mostly to counter various migration issues on 
> legacy

Dropped.

take care,
  Gerd


Reply via email to