Hi Philippe, On Mon, Feb 19, 2024 at 03:14:11PM +0100, Philippe Mathieu-Daudé wrote: > Date: Mon, 19 Feb 2024 15:14:11 +0100 > From: Philippe Mathieu-Daudé <phi...@linaro.org> > Subject: [PATCH 5/5] hw/i386/q35: Include missing 'hw/acpi/acpi.h' header > X-Mailer: git-send-email 2.41.0 > > "hw/acpi/acpi.h" is implicitly included. > Included it
I'm guessing it's an imperative and shouldn't be in passive form. ;-) So, s/Included/Include/ > explicitly to avoid when refactoring headers: s/to avoid/to avoid the following error/ > > hw/i386/pc_q35.c:209:43: error: use of undeclared identifier > 'ACPI_PM_PROP_ACPI_PCIHP_BRIDGE' > ACPI_PM_PROP_ACPI_PCIHP_BRIDGE, > ^ > > Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> > --- > hw/i386/pc_q35.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c > index d346fa3b1d..a91f414922 100644 > --- a/hw/i386/pc_q35.c > +++ b/hw/i386/pc_q35.c > @@ -58,6 +58,7 @@ > #include "hw/hyperv/vmbus-bridge.h" > #include "hw/mem/nvdimm.h" > #include "hw/i386/acpi-build.h" > +#include "hw/acpi/acpi.h" > #include "target/i386/cpu.h" Here it is unordered, and similarly, does it need to be sorted alphabetically here? Regards, Zhao > > /* ICH9 AHCI has 6 ports */ > -- > 2.41.0 > >