On Fri, Nov 22, 2013 at 8:29 AM, Takashi Iwai <tiwai@xxxxxxx> wrote: >> config HISAX_TELESPCI >> bool "Teles PCI" >> - depends on PCI && (BROKEN || !(SPARC || PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN))) >> + depends on PCI && (BROKEN || !(SPARC || PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN) || (ARM && !CPU_LITTLE_ENDIAN))) > > Isn't it a bit better to exclude CPU_LITTLE_ENDIAN globally? > > depends on PCI && CPU_LITTLE_ENDIAN && (BROKEN || !(SPARC || PPC || PARISC || M68K || MIPS || FRV || XTENSA || ARM)) CPU_LITTLE_ENDIAN is not defined on all little endian platforms, only on those that can support both big and little endian. But I guess it wouldn't hurt to move CPU_{BIG,LITTLE}_ENDIAN to a common Kconfig file. > Or maybe just > > depends on PCI && (X86 || BROKEN) > > (Alpha? Can anyone test? :) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html