[coreboot] [PATCH]Clean up ACPI
stepan at coresystems.de
Mon Feb 8 16:42:20 CET 2010
On 2/8/10 4:11 PM, Patrick Georgi wrote:
> with newconfig out of the way, I decided to clean up ACPI.
> This implies lots of file movements, so the patch I attach is of
> somewhat limited use (even if I were to add the deletions in full it
> doesn't apply cleanly)
> - All files use the .asl suffix now (acpi source language)
> - dsdt, ssdt*, sspr* belong in the source directories (eg.
> src/mainboard/kontron/986lcd-m), all files included by them belong in an
> acpi/ subdirectory.
> - iasl is handled in a generic rule in the top level Makefile. Non-dsdt
> files get their AmlCode array renamed to AmlCode_$filename (without suffix)
> - pci were renamed to ssdt for the above to work
> - ACPI code is preprocessed with cpp. We already did that for Kontron,
> and it has the nice property that you can share certain header files
> with C code, to deduplicate some data (eg. LAPIC base address).
> This doesn't negatively affect code that wasn't made for it, as only
> #define statement could create conflicts
> - I changed all ACPI Include() statements to #include preprocessor
> commands, so once this feature is more commonly used, cpp gets a chance
> to fix up all defines
> - No more need for iasl rules in Makefile.incs, just add dsdt.o, ssdt3.o
> etc. to obj-y and make will figure out the right thing to do
> All boards built with abuilt.
> Signed-off-by: Patrick Georgi <patrick.georgi at coresystems.de>
Great! ACPI in coreboot is really developing from a mess to a framework.
Acked-by: Stefan Reinauer <stepan at coresystems.de>
For the future it would be nice if files like
usb.asl/sata.asl/ide.asl/globutil.asl of the AMD RS690/SB600 based
boards could live in southbridge/amd/sb600/acpi
Maybe someone can step up and fix this?
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
More information about the coreboot