Build failure with 2.6.24-mm1
Greg KH
greg at kroah.com
Tue Feb 5 03:19:23 EST 2008
On Mon, Feb 04, 2008 at 02:19:17AM -0800, Andrew Morton wrote:
> On Mon, 4 Feb 2008 15:37:43 +0530 Balbir Singh <balbir at linux.vnet.ibm.com> wrote:
>
> > * Balbir Singh <balbir at linux.vnet.ibm.com> [2008-02-04 15:35:09]:
> >
> > I just saw the following build failure on a power machine.
> >
> > In file included from include/acpi/platform/acenv.h:140,
> > from include/acpi/acpi.h:54,
> > from include/acpi/acpi_bus.h:31,
> > from drivers/pci/pcie/aspm.c:20:
> > include/acpi/platform/aclinux.h:59:22: error: asm/acpi.h: No such file or directory
> > In file included from include/acpi/platform/aclinux.h:120,
> > from include/acpi/platform/acenv.h:140,
> > from include/acpi/acpi.h:54,
> > from include/acpi/acpi_bus.h:31,
> > from drivers/pci/pcie/aspm.c:20:
> > include/acpi/actypes.h:130: error: expected '=', ',', ';', 'asm' or
> > '__attribute__' before 'UINT64'
> > include/acpi/actypes.h:131: error: expected '=', ',', ';', 'asm' or
> > '__attribute__' before 'INT64'
> > include/acpi/actypes.h:753: error: expected ')' before '*' token
> > include/acpi/actypes.h:756: error: expected ')' before '*' token
> > In file included from include/acpi/acpi.h:61,
> > from include/acpi/acpi_bus.h:31,
> > from drivers/pci/pcie/aspm.c:20:
> > include/acpi/acpiosxf.h:179: error: expected declaration specifiers or '...'
> > before 'acpi_osd_handler'
> > include/acpi/acpiosxf.h:183: error: expected declaration specifiers or '...'
> > before 'acpi_osd_handler'
> > include/acpi/acpiosxf.h:192: error: expected declaration specifiers or '...'
> > before 'acpi_osd_exec_callback'
> > make[3]: *** [drivers/pci/pcie/aspm.o] Error 1
> > make[2]: *** [drivers/pci/pcie] Error 2
> > make[2]: *** Waiting for unfinished jobs....
> > CC drivers/ps3/ps3-vuart.o
> > CC net/netlink/attr.o
> > make[1]: *** [drivers/pci] Error 2
> > make[1]: *** Waiting for unfinished jobs..
> >
> > The following config option is responsible for the build failure
> >
> > config PCIEASPM
> > bool "PCI Express ASPM support(Experimental)"
> > depends on PCI && EXPERIMENTAL && PCIEPORTBUS
> > default y
> > help
> > This enables PCI Express ASPM (Active State Power Management) and
> > Clock Power Management. ASPM supports state L0/L0s/L1.
> >
> > When in doubt, say N.
> >
> > Here's a probable fix for the problem.
> >
> >
> > Make the build of drivers/pci/pcie/aspm.c depend on ACPI.
> >
> > NOTE, the patch has not been tested. The dependency on ACPI might be wrong,
> > but setting it to default "y" caused the build on my powerpc box to break.
> >
> > Signed-off-by: Balbir Singh <balbir at linux.vnet.ibm.com>
> > ---
> >
> > drivers/pci/pcie/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff -puN drivers/pci/pcie/Kconfig~fix-mm-ppc-build drivers/pci/pcie/Kconfig
> > --- linux-2.6.24/drivers/pci/pcie/Kconfig~fix-mm-ppc-build 2008-02-04 15:30:29.000000000 +0530
> > +++ linux-2.6.24-balbir/drivers/pci/pcie/Kconfig 2008-02-04 15:33:45.000000000 +0530
> > @@ -32,7 +32,7 @@ source "drivers/pci/pcie/aer/Kconfig"
> > #
> > config PCIEASPM
> > bool "PCI Express ASPM support(Experimental)"
> > - depends on PCI && EXPERIMENTAL && PCIEPORTBUS
> > + depends on PCI && EXPERIMENTAL && PCIEPORTBUS && ACPI
> > default y
> > help
> > This enables PCI Express ASPM (Active State Power Management) and
>
> Thanks. I think Greg is going to revert PCIEASPM altogether?
Greg did, hopefully Linus will pull the changes soon...
thanks,
greg k-h
More information about the Linuxppc-dev
mailing list