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

[Simon Glass]

Hi,

On 21 February 2017 at 16:16, Maxime Ripard
<maxime.ripard at free-electrons.com> wrote:
> 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?

I suppose whoever adds the next board gets to do the conversion?

This patch is a step forward. But how about adding a build error
whenever a non-whitelisted board defines CONFIG_SYS_EXTRA?

Regards,
Simon