Re: [PATCH: NEW ARCHITECTURE FOR 2.5.3] support for NCR voyager

[James Bottomley <James.Bottomley@HansenPartnership.com>]

It's historical creep.

Initially the question about being Voyager was asked immediately below 
CONFIG_MCA, which was then at the top level.  Then that got wrapped by if != 
VISW, so it ends up looking like this.

How about:

if [ "$CONFIG_VISWS" != "y" ]; then
    bool 'MCA support' CONFIG_MCA
else
    define_bool CONFIG_MCA n
fi

if [ "$CONFIG_MCA" = "y" ]; then
    bool '   Support for the NCR Voyager Architecture' CONFIG_VOYAGER
    define_bool CONFIG_X86_TSC n
fi

Actually, this also exposes a bug, the last statement should be:

if [ "$CONFIG_MCA" = "y" ]; then
    bool '   Support for the NCR Voyager Architecture' CONFIG_VOYAGER
    if [ "$CONFIG_VOYAGER" = "y"]; then
        define_bool CONFIG_X86_TSC n
    fi
fi

Since MCA machines may use the pentium TSC but voyager may not.

James