From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 20 Feb 2017 07:09:04 -0500 Subject: [U-Boot] [PATCH v3 1/7] arm: dra7xx: move CONFIG_DRA7XX to Kconfig In-Reply-To: <4c1eeb84-c3d5-e720-870d-7900a285d0cd@compulab.co.il> References: <1487510839-17359-1-git-send-email-uri.mashiach@compulab.co.il> <1487510839-17359-2-git-send-email-uri.mashiach@compulab.co.il> <20170219151555.GB27120@bill-the-cat> <4c1eeb84-c3d5-e720-870d-7900a285d0cd@compulab.co.il> Message-ID: <20170220120904.GJ27120@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > >>Signed-off-by: Uri Mashiach > >>--- > >>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: