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

Russell King (rmk@arm.linux.org.uk)
Fri, 8 Nov 2002 00:29:05 +0000


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"

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.

-- 
Russell King (rmk@arm.linux.org.uk)                The developer of ARM Linux
             http://www.arm.linux.org.uk/personal/aboutme.html

- 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/