From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] config: enable CONFIG_API in distro config
Date: Fri, 18 Apr 2014 11:23:35 -0600 [thread overview]
Message-ID: <53515F97.1060909@wwwdotorg.org> (raw)
In-Reply-To: <20140418153558.GD5698@bill-the-cat>
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
>
> How do you guys want to handle this? I can apply now and they can be
> fixed up later, or I can wait for fixup patches to come out and apply
> them all at once. Thanks!
I've sent patches that solve all the build problems on Tegra with
CONFIG_API enabled.
That said, I still conceptually object to config_distro_defaults.h
enabling API support in order to support grub; I believe the distros
need to get together and nail down a *single* boot mechanism to
standardize upon, rather than having Fedora support BootloaderSpec,
Ubuntu/Linaro(?) support Grub, something else support LILO, something
else support UEFI, etc. Without this, we'll force every U-Boot binary to
be bloated with all kinds of redundant code. Having a single standard
mechanism was always the point of config_distro_defaults.h in my mind.
BTW, does Grub work well for you? I did try it many months ago, and found:
a) It was excruciatingly slow, to the point of not being useful.
b) The binary had hard-coded memory layout, and hence couldn't be
generic across multiple boards, since SoCs put their RAM in different
places, boards have differing amounts of RAM, etc. This kinda defeats
the whole purpose, since distros would need to install board-specific
Grub binaries. Perhaps this could be solved similarly to the kernel's
CONFIG_ARM_PATCH_PHYS_VIRT?
If we really need Grub support, wouldn't it be better to have U-Boot
parse and execute grub.cfg, just like it does extlinux.cfg?
next prev parent reply other threads:[~2014-04-18 17:23 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
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 [this message]
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=53515F97.1060909@wwwdotorg.org \
--to=swarren@wwwdotorg.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox