Re: [PATCH] fix the -test3 input config damages

From: Geert Uytterhoeven
Date: Tue Aug 26 2003 - 03:36:59 EST


On Mon, 25 Aug 2003, Tom Rini wrote:
> On Mon, Aug 25, 2003 at 06:13:32PM +0200, Geert Uytterhoeven wrote:
> > On Fri, 22 Aug 2003, Tom Rini wrote:
> > > --- 1.18/drivers/video/console/Kconfig Wed Jul 16 10:39:32 2003
> > > +++ edited/drivers/video/console/Kconfig Fri Aug 22 13:27:21 2003
> > > @@ -5,7 +5,7 @@
> > > menu "Console display driver support"
> > >
> > > config VGA_CONSOLE
> > > - bool "VGA text console" if EMBEDDED || !X86
> > > + bool "VGA text console" if STANDARD && X86
> > > depends on !ARCH_ACORN && !ARCH_EBSA110 || !4xx && !8xx
> > > default y
> > > help
> >
> > Ugh, this makes VGA_CONSOLE default to yes if X86 is not set, right? Don't you
> > want
> >
> > bool "VGA text console" if !STANDARD || X86

Oops, and I meant !X86, of course.

> > ?
> >
> > Or do I need an update course on Kconfig syntax?
>
> No, I think that's a logic error on my part. What I intended was
> default to Y on (STANDARD && X86), otherwise ask. So it should have
> been:
> bool "VGA text console" if !(STANDARD && X86)

OK, that's equivalent to what I had in mind :-)

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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/