From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] config: enable CONFIG_API in distro config
Date: Fri, 18 Apr 2014 11:55:36 -0400 [thread overview]
Message-ID: <20140418155536.GF5698@bill-the-cat> (raw)
In-Reply-To: <5351486C.1030601@wwwdotorg.org>
On Fri, Apr 18, 2014 at 09:44:44AM -0600, Stephen Warren wrote:
> On 04/18/2014 09:35 AM, Tom Rini wrote:
> > On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote:
> >
> >> From: Rob Herring <robh@kernel.org>
> >>
> >> CONFIG_API is needed for u-boot apps such as grub2, so enable it for
> >> distro config.
> >>
> >> Cc: Dennis Gilmore <dennis@ausil.us>
> >> Signed-off-by: Rob Herring <robh@kernel.org>
> >
> > This breaks a number of boards that have opted in to the distro config.
> > The full list is:
> > whistler trimslice beaver tec paz00 ventana cardhu harmony dalmore
> > plutux medcom-wide rpi_b venice2 colibri_t20_iris tec-ng seaboard
>
> Hmm. That's all or most Tegra boards. What's the break?
>
> That said, I wonder why we need CONFIG_API in the distro config; the
> purpose of that config is to allow the "pxe" and "sysboot" commands not
> to work. If we start supporting all kinds of other random boot
> mechanisms, we'll just end up with a fragmented mess, rather than a
> unified boot mechanism.
It's to add grub2, as Rob says :) But the breakage is over things like
CONFIG_SYS_MAX_MMC_DEVICE, and a few other things (that's just the
common one, but I still saw some breakage when I set that to 2 for tegra
stuff).
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140418/7ac3e76c/attachment.pgp>
next prev parent reply other threads:[~2014-04-18 15:55 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-10 20:55 [U-Boot] [PATCH] config: enable CONFIG_API in distro config Rob Herring
2014-04-18 15:35 ` [U-Boot] " Tom Rini
2014-04-18 15:44 ` Stephen Warren
2014-04-18 15:55 ` Tom Rini [this message]
2014-04-18 15:58 ` Stephen Warren
2014-04-18 15:58 ` Rob Herring
2014-04-18 15:44 ` Tom Warren
2014-04-18 15:56 ` Tom Rini
2014-04-18 16:42 ` Rob Herring
2014-04-18 17:23 ` Stephen Warren
2014-04-18 17:43 ` Simon Glass
2014-04-19 14:01 ` Ian Campbell
[not found] ` <20140419104218.702c3706@adria.ausil.us>
2014-04-20 16:55 ` Simon Glass
2014-04-21 14:49 ` Dennis Gilmore
2014-04-21 17:57 ` Simon Glass
2014-04-21 18:05 ` Stephen Warren
2014-04-18 22:54 ` Rob Herring
2014-04-19 1:05 ` Leif Lindholm
2014-04-19 14:04 ` Ian Campbell
2014-04-19 14:34 ` Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140418155536.GF5698@bill-the-cat \
--to=trini@ti.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.