Re: [PATCH] X86_NUMAQ build fix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 2006-04-22 at 03:50 +0900, OGAWA Hirofumi wrote:
> Dave Hansen <[email protected]> writes:
> >>  pci-y                          := fixup.o
> >> -pci-$(CONFIG_ACPI)             += acpi.o
> >>  pci-y                          += legacy.o irq.o
> >>  
> >>  pci-$(CONFIG_X86_VISWS)                := visws.o fixup.o
> >>  pci-$(CONFIG_X86_NUMAQ)                := numa.o irq.o
> >>  
> >> +pci-$(CONFIG_ACPI)             += acpi.o
> >
> > Am I reading this wrong, or does this just move the option down a bit?
> > Did you need to change the link order?  Why?
> 
> No, this is not link order. Note that CONFIG_X86_VISWS/CONFIG_X86_NUMAQ
> uses ":=", not "+=".  In case of ACPI=y", it breaks build.

Ahh, so NUMAQ and VISWS don't seem to allow or want ACPI code.
Especially for the NUMAQ, I don't think we should even mess with this
too much.  Just fix it in the build system and directly disallow NUMAQ
&& ACPI. 

We already have this:

menu "ACPI (Advanced Configuration and Power Interface) Support"
        depends on !X86_VISWS

Just add the NUMAQ in there, too.

-- Dave

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux