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

[Tom Rini <trini@kernel.crashing.org>]

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/