[U-Boot] [PATCH v3 1/7] arm: dra7xx: move CONFIG_DRA7XX to Kconfig

Tom Rini
2017-02-20T12:09:04Z

On Mon, Feb 20, 2017 at 09:18:47AM +0200, Uri Mashiach wrote:
>
>
> On 02/19/2017 05:15 PM, Tom Rini wrote:
> >On Sun, Feb 19, 2017 at 03:27:13PM +0200, Uri Mashiach wrote:
> >
> >>The symbol CONFIG_DRA7XX is needed for Kconfig conditions.
> >>
> >>Cc: Lokesh Vutla <lokeshvutla@ti.com>
> >>Signed-off-by: Uri Mashiach <uri.mashiach@compulab.co.il>
> >>---
> >>V3 -> V2: new commit
> >>
> >> board/compulab/cl-som-am57x/Kconfig | 4 ++++
> >> board/ti/am57xx/Kconfig | 4 ++++
> >> board/ti/dra7xx/Kconfig | 4 ++++
> >> include/configs/am57xx_evm.h | 2 --
> >> include/configs/cl-som-am57x.h | 2 --
> >> include/configs/dra7xx_evm.h | 2 --
> >> scripts/config_whitelist.txt | 1 -
> >> 7 files changed, 12 insertions(+), 7 deletions(-)
> >
> >We need to introduce the symbol in arch/arm/mach-omap2/omap5/Kconfig and
> >then select it under the appropriate boards. Thanks!
>
> Maybe instead of selecting it under the appropriate board it can be
> defaulted to yes in the board's Kconfig?

No. The correct thing is to select it (if it's not something that can
be opted out of, for example what type of SoC this is) or imply'd if
it's something that really should be enabled, but the user can opt out
of. Doing a default yes in the board Kconfig was a stop-gap until
'imply' came around. Thanks!

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170220/47575020/attachment.sig>