[U-Boot] [PATCH] sunxi: Select CONFIG_CMD_NET and CONFIG_CMD_SETEXPR by default

Tom Rini trini at konsulko.com
Thu Jun 11 17:12:58 CEST 2015


On Thu, Jun 11, 2015 at 09:14:33AM -0500, Joe Hershberger wrote:
> Hi Tom,
> 
> On Wed, Jun 3, 2015 at 5:12 PM, Tom Rini <trini at konsulko.com> wrote:
> > On Wed, Jun 03, 2015 at 08:12:16PM +0200, Hans de Goede wrote:
> >
> >> Select CONFIG_CMD_NET and CONFIG_CMD_SETEXPR by default rather then
> >> needing to have this in every sunxi defconfig file.
> >>
> >> This also fixes the Merrii_A80_Optimus defconfig no longer building.
> >>
> >> Cc: Maxin B. John <maxin.john at enea.com>
> >> Reported-by: Maxin B. John <maxin.john at enea.com>
> >> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> >
> > Joe? Masahiro?  It feels like something has gone wrong with the
> > conversion here.  Or do people need to get used to the defconfig files
> > being a non-trivial size?  Or do we need some more default y if ...
> > lines around things?  Or a few of the above?  Thanks!
> 
> I'm surprised you pulled this in for -rc2. I guess we can always revert it.

OK, so where did we end up again exactly?  Ah yes, (a) we can't
unexpectedly break peoples build.  So I took this for now.  (b) I'm
being stubborn here and saying that if it was in
include/config_cmd_default.h as a #define it needs to get converted to
Kconfig as a "default y" (with text so that it's a question and not a
silently enforced option).  That doesn't preclude boards from setting it
to N in their defconfig and savedefconfig capturing it.  That's in fact
what I want.  So I'm expecting (or hunting for time to make) a patch
that updates CMD_NET/etc to match that behavour.

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


More information about the U-Boot mailing list