pciehp borkage.

[Dave Jones <davej@redhat.com>]

My head hurts..

drivers/pci/pcie/Kconfig ..

config HOTPLUG_PCI_PCIE
    tristate "PCI Express Hotplug driver"
    depends on HOTPLUG_PCI && PCIEPORTBUS && (BROKEN || ACPI)



but drivers/pci/hotplug/Makefile has..

pciehp-objs     :=  pciehp_core.o   \
                pciehp_ctrl.o   \
                pciehp_pci.o    \
                pciehp_hpc.o

So it gets built regardless of the option, which leaves ppc (among others)
totally busted..

In file included from include/acpi/platform/acenv.h:140,
                 from include/acpi/acpi.h:54,
                 from drivers/pci/hotplug/pciehp_hpc.c:41:
include/acpi/platform/aclinux.h:59:22: error: asm/acpi.h: No such file or directory
In file included from include/acpi/acpi.h:55,
                 from drivers/pci/hotplug/pciehp_hpc.c:41:
include/acpi/actypes.h:129: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'UINT64'
include/acpi/actypes.h:130: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'INT64'
make[3]: *** [drivers/pci/hotplug/pciehp_hpc.o] Error 1
make[2]: *** [drivers/pci/hotplug] Error 2
make[1]: *** [drivers/pci] Error 2


Should that Makefile be more along the lines of..

pciehp-$(CONFIG_PCI_PCIE)	:=  pciehp_core.o   \
                pciehp_ctrl.o   \
                pciehp_pci.o    \
                pciehp_hpc.o

perhaps ?

		Dave

-- 
http://www.codemonkey.org.uk