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

[Tom Rini]

On Wed, Feb 22, 2017 at 03:26:01PM -0800, Maxime Ripard wrote:
> On Tue, Feb 21, 2017 at 08:58:55PM -0700, Simon Glass wrote:
> > 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?
> 
> That seems a bit unfair to ask someone that barely got into U-Boot to
> enable his new shiny sunxi board to convert the old Freescale's PPC
> defconfig in the process.

Well, yes and no.  Some of the options that need migrating (say
CONFIG_NAND) are a PITA, and no, we aren't asking for that.

Drilling down to the sunxi case for a moment, the SYS_EXTRA_OPTIONS that
get set are almost entirely sunxi centric (CONFIG_RGMII is used in some
atmel cases) so converting those is I feel an appropriate ask.

Getting back up to the high level question, if someone makes an effort
to write the Kconfig logic for an option and tries to get the depends
logic correct (after looking at how all the various cases of it go), I'm
happy to pick it up and do the last part of migrating all of the configs
and size-checking for correctness.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170223/cbe28424/attachment.sig>