Re: [RFC] Templates and tweaks (for size performance and more)

Tom Rini (trini@kernel.crashing.org)
Thu, 7 Nov 2002 17:37:21 -0700


On Fri, Nov 08, 2002 at 12:29:05AM +0000, Russell King wrote:
> On Thu, Nov 07, 2002 at 03:06:29PM -0700, Tom Rini wrote:
> > But having that as one line in arch/arm/Kconfig looks any better?
>
> Actually its supposed to depend on CONFIG_SA1111, not that random
> collection of other symbols. The following just happened to be a nice
> way to specify it under CML1:
>
> if [ "$CONFIG_ASSABET_NEPONSET" = "y" -o \
> "$CONFIG_SA1100_ACCELENT" = "y" -o \
> "$CONFIG_SA1100_ADSBITSY" = "y" -o \
> "$CONFIG_SA1100_BADGE4" = "y" -o \
> "$CONFIG_SA1100_CONSUS" = "y" -o \
> "$CONFIG_SA1100_GRAPHICSMASTER" = "y" -o \
> "$CONFIG_SA1100_JORNADA720" = "y" -o \
> "$CONFIG_SA1100_PFS168" = "y" -o \
> "$CONFIG_SA1100_PT_SYSTEM3" = "y" -o \
> "$CONFIG_SA1100_XP860" = "y" ]; then
> define_bool CONFIG_SA1111 y
> define_int CONFIG_FORCE_MAX_ZONEORDER 9
> fi
>
> The conversion should've been:
>
> config FORCE_MAX_ZONEORDER
> int
> depends on SA1111
> default "9"

Ah. I should have guessed :) I've seen a few similar ones on PPC.
I'll update the patch.

> Even so, my original point remains about the dependency between config
> symbols concentrating in one "tweaks" header file leading to the situation
> where you change one symbol and everything rebuilds.

Yes, this is an annoyance. I'm trying to figure out how to adapt
split-include to work with the tweak headers, and I don't think it will
be too painful. It'll just require something in the middle to take
<asm/tweaks.h> and spit out something akin to <linux/autoconf.h>.

So aside from that, any other comments about the idea or implementation?

-- 
Tom Rini (TR1265)
http://gate.crashing.org/~trini/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/