From mboxrd@z Thu Jan 1 00:00:00 1970 From: York Sun Date: Thu, 23 Apr 2015 08:34:43 -0700 Subject: [U-Boot] [PATCH v4] powerpc/t1023rdb: Add T1023 RDB board support In-Reply-To: References: <1427442514-46634-1-git-send-email-Shengzhou.Liu@freescale.com> <55300BED.8040503@freescale.com> Message-ID: <55391113.9090805@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de +Masahiro Yamada On 04/23/2015 01:27 AM, Liu Shengzhou-B36685 wrote: > >> -----Original Message----- >> From: Sun York-R58495 >> Sent: Friday, April 17, 2015 3:22 AM >> To: Liu Shengzhou-B36685; u-boot at lists.denx.de >> Subject: Re: [PATCH v4] powerpc/t1023rdb: Add T1023 RDB board support >> >> On 03/27/2015 12:48 AM, Shengzhou Liu wrote: >>> T1023RDB is a Freescale Reference Design Board that hosts the T1023 SoC. >>> >>> +++ b/configs/T1023RDB_NAND_defconfig >>> @@ -0,0 +1,5 @@ >>> +CONFIG_SPL=y >>> +CONFIG_SYS_EXTRA_OPTIONS="PPC_T1023,T1023RDB,RAMBOOT_PBL,SPL_FSL_PBL,NAND" >>> ++S:CONFIG_PPC=y >>> ++S:CONFIG_MPC85xx=y >>> ++S:CONFIG_TARGET_T102XRDB=y >> >> Please do not use CONFIG_SYS_EXTRA_OPTIONS for new boards. >> >> York > > I noted "config SYS_EXTRA_OPTIONS string "Extra Options (DEPRECATED)" in Kconfig. > But we still have to use the CONFIG_SYS_EXTRA_OPTIONS in current configuration > Infrastructure, unless a newer approach is available in future. > > Because there are some something like 'SPL_FSL_PBL', 'NAND', 'SPIFLASH''SDCARD' which must be > defined dynamically during build u-boot for various boot-mode in those *_defconfig files > instead of in static include/config.h I was thinking to use "select" in Kconfig. But as you said, there are some macros not standardized. Maybe it is too early to enforce that. Let's hear from Masahiro. York