[U-Boot] [PATCH] Makefile: print warning if CONFIG_SYS_EXTRA_OPTIONS is used

Masahiro Yamada yamada.masahiro at socionext.com
Wed Feb 22 02:53:05 UTC 2017


2017-02-22 8:16 GMT+09:00 Maxime Ripard <maxime.ripard at free-electrons.com>:
> On Sat, Feb 18, 2017 at 03:24:53PM +0900, Masahiro Yamada wrote:
>> The prompt of this option is marked "DEPRECATED" in capital letters,
>> and its help clearly says:
>>   This option was prepared for the smooth migration from the old
>>   configuration to Kconfig. Since this option will be removed sometime,
>>   new boards should not use this option.
>>
>> However, we have had almost no progress on this.  In fact, this
>> option is often abused for fresh porting, and more and more new
>> defconfigs with it are coming in (for example, SUNXI boards).
>
> That's not entirely true. We've added more defconfig, but with the
> same set of options we've always had, and are shared by everyone
> (CONS_INDEX, RGMII).
>
> So this patch, in effect, gives the signal that it's better to have a
> proliferation of board-specific headers in include/configs, over using
> SYS_EXTRA until most of the generic functions are merged?

No.

I am saying to add
CONFIG_CONS_INDEX=2
CONFIG_RGMII=y

instead of

CONFIG_SYS_EXTRA_OPTIONS=CONS_INDEX=2,RGMII"


Why proliferation of board-specific headers?




-- 
Best Regards
Masahiro Yamada


More information about the U-Boot mailing list