Re: [PATCH update9] drivers: add LCD support

[Andrew Morton]

On Wed, 20 Dec 2006 15:10:00 +0100
Miguel Ojeda Sandonis <maxextreme@xxxxxxxxx> wrote:

> --- a/drivers/auxdisplay/Kconfig
> +++ b/drivers/auxdisplay/Kconfig
> @@ -65,6 +65,7 @@ config KS0108_DELAY
>  config CFAG12864B
>         tristate "CFAG12864B LCD"
>         depends on X86
> +       depends on FB
>         depends on KS0108
>         default n
>         ---help---
> @@ -74,6 +75,8 @@ config CFAG12864B
>           For help about how to wire your LCD to the parallel port,
>           check Documentation/auxdisplay/cfag12864b
> 
> +         Depends on the x86 arch and the framebuffer support.

heh, that was clever.

scripts/kconfig/conf -m arch/i386/Kconfig
drivers/auxdisplay/Kconfig:81:warning: multi-line strings not supported
drivers/auxdisplay/Kconfig:78: unknown option "Depends"
drivers/auxdisplay/Kconfig:80: unknown option "The"
drivers/auxdisplay/Kconfig:81: unknown option "It"
drivers/auxdisplay/Kconfig:82: unknown option "of"
drivers/auxdisplay/Kconfig:84: unknown option "To"
drivers/auxdisplay/Kconfig:85: unknown option "the"
drivers/auxdisplay/Kconfig:87: unknown option "If"
make[1]: *** [allmodconfig] Error 1

you broke Roman's parser.


Oh, your patch used spaces where tabs should have been.  Please don't do
that.

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