[U-Boot] [PATCH v4] powerpc/t1023rdb: Add T1023 RDB board support

[Masahiro Yamada]

Hi.

Sorry for my late reply.

2015-04-24 0:38 GMT+09:00 York Sun <yorksun at freescale.com>:
> Resend with corrected email address.
>
> +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 Yamada.
>

As you notice, many macros are missing from Kconfig,
so we can not ban SYS_EXTRA_OPTIONS yet.



-- 
Best Regards
Masahiro Yamada