linux-kernel - Re: linux-next: Tree for July 22 (arm-davinci Kconfig warnings)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAPcSpEJZ47FQ3e84Mrf=U8GYw1W3Nx2DErgeSpvyuexC9fhsgA@mail.gmail.com>
Date:	Mon, 25 Jul 2011 09:25:58 -0400
From:	Ben Gardiner <bengardiner@...ometrics.ca>
To:	Arnaud Lacombe <lacombar@...il.com>
Cc:	Stephen Rothwell <sfr@...b.auug.org.au>,
	linux-next@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>,
	Sekhar Nori <nsekhar@...com>, Sam Ravnborg <sam@...nborg.org>
Subject: Re: linux-next: Tree for July 22 (arm-davinci Kconfig warnings)

On Sun, Jul 24, 2011 at 5:34 PM, Arnaud Lacombe <lacombar@...il.com> wrote:
> Hi,

Hi Arnaud,

> The arm-davinci tree has 2 Kconfig warning, but also present in v3.0:
>
> arch/arm/mach-davinci/Kconfig:195:warning: config symbol defined without type
> arch/arm/mach-davinci/Kconfig:198:warning: config symbol defined without type

I hadn't noticed those warnings, sorry.

> This concerns GPIO_PCA953X and KEYBOARD_GPIO_POLLED. In both case,
> symbols are re-defined later on in `drivers/gpio/Kconfig' respectively
> in `arch/arm/mach-davinci/Kconfig' and
> `drivers/input/keyboard/Kconfig'.
>
> Is there any reason not to select those from MACH_DAVINCI_DA850_EVM
> definition, ala:

The reason for the GPIO_PCA953X and KEYBOARD_GPIO_POLLED fragments is
to get the symbols selected by default but allow those who
re-configure the kernel to select them instead as modules or even
disable them.

> diff --git a/arch/arm/mach-davinci/Kconfig b/arch/arm/mach-davinci/Kconfig
> index c0deaca..cec22e2 100644
> --- a/arch/arm/mach-davinci/Kconfig
> +++ b/arch/arm/mach-davinci/Kconfig
> @@ -163,6 +163,8 @@ config MACH_DAVINCI_DA850_EVM
>        bool "TI DA850/OMAP-L138/AM18x Reference Platform"
>        default ARCH_DAVINCI_DA850
>        depends on ARCH_DAVINCI_DA850
> +       select GPIO_PCA953X
> +       select KEYBOARD_GPIO_POLLED
>        help
>          Say Y here to select the TI DA850/OMAP-L138/AM18x Evaluation Module.
>
> @@ -192,12 +194,6 @@ config DA850_UI_RMII
>
>  endchoice
>
> -config GPIO_PCA953X
> -       default MACH_DAVINCI_DA850_EVM
> -
> -config KEYBOARD_GPIO_POLLED
> -       default MACH_DAVINCI_DA850_EVM
> -
>  config MACH_TNETV107X
>        bool "TI TNETV107X Reference Platform"
>        default ARCH_DAVINCI_TNETV107X

My understanding of this diff is that both GPIO_PCA953X and
KEYBOARD_GPIO_POLLED will be selected unconditionally as built-ins for
any MACH_DAVINCI_DA850_EVM. As opposed to having the symbols default
to 'y' and potentially be changed to 'm' or 'n' by someone
reconfiguring.

I took the idea of using these KConfig fragments from Sam Ravnborg
(added to CC); he suggested the method [1] in a response to a patch by
Stephen Rothwell (already on CC) which proposed a extension to the
'select' syntax [2]. If the select syntax extension or other
equivalent means is ready then arch/arm/mach-davinci can be converted.

Best Regards,
Ben Gardiner

[1] http://article.gmane.org/gmane.linux.ports.ppc.embedded/37132
[2] http://article.gmane.org/gmane.linux.ports.ppc.embedded/36839

---
Nanometrics Inc.
http://www.nanometrics.ca
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/