From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] wandboard: add pxe support, set default boot command like highbank
Date: Wed, 7 Aug 2013 09:19:21 -0400 [thread overview]
Message-ID: <20130807131921.GP5164@bill-the-cat> (raw)
In-Reply-To: <20130806181125.0f7c510f@adria.ausil.us>
On Tue, Aug 06, 2013 at 06:11:25PM -0500, Dennis Gilmore wrote:
> On Tue, 6 Aug 2013 17:42:31 -0400
> Tom Rini <trini@ti.com> wrote:
>
> > On Tue, Aug 06, 2013 at 11:22:22AM -0500, Dennis Gilmore wrote:
> >
> > [snip]
> > > The only way I could see having us write a file to disk with the
> > > environment working is if all boards implement standard variable to
> > > define the memory locations and that is compiled into the u-boot
> > > binary.
> > >
> > > some variables that would need to be compiled in
> > >
> > > fdt_addr
> > > fdt_addr_r
> >
> > Why two?
> from cmd_pxe.c
[snip]
> u-boot by default would load a dtb to fdt_addr but a user could at
> least in the pxe/extlinux case load their own dtb if the want/need to.
>
> the only way a dtb would be optional is if fdtfile is not set
OK, so fdt_addr is for a memory-mapped location of the DT existing
somewhere already, basically.
> > > kernel_addr_r
> > > ramdisk_addr_r
> > > pxefile_addr_r
> > > scr_addr_r
> > > uenv_addr_r
> > >
> > > this should allow for for people to use boot.scr uEnv.txt or
> > > pxe/extlinux
> >
> > This is what I think we need to work towards. A board opting into
> > this standard must set CONFIG_CMD_A/B/C (or maybe we add a
> > CONFIG_SUPPORT_GENERIC_LINUX_DISTRO that does this in one of the
> > fallback files, whatever) and provide the following variables PLUS a,
> > and this needs some thinking I think, auto-boot tries to load said
> > file from ... ?
> >
> > We cannot provide a built-in environment that works for every distro
> > and case, we want the distro to tell us things it knows, and we'll
> > tell it what it can't easily know.
>
> we absolutely can, I would like for u-boot to load a dtb before doing
> anything. u-boot should know what devices can be booted from and likely
> an order of preference. i.e. removable media through to fixed, and
> finally pxe.
Looking at a couple of device trees, no, we can't. I don't see any
useful information like "this is an SD controller" for example (and all
of the mmc bindings that might provide a reliable clue are optional
ones). So, at the high level, if U-Boot relied on DTs in every driver,
we might be able to do what you're talking about. But we don't do that
today, and probably won't for a long time, if ever.
But we will know what devices are likely to exist and be bootable, at
build time. What we can do is try and load a file from everywhere we
expect might be someplace with this file in it. We could even set some
standard variables based on having found and loaded this file, so it can
assume that everything else exists in this same place (or the file
we have loaded knows better).
> i would like for u-boot to first try to
> load /boot/extlinux/extlinux.conf then /extlinux/extlinux.conf
> failing that try to load a /boot/uEnv.txt then /uEnv.txt and import that
> running with what is in it, then falling back to a /boot/boot.scr
> then /boot.scr then finally running dhcp, pxe get, and pxe boot.
I'd like to see most of that logic held in the file we load (so that
when something replaces extlinux.conf as the preferred method, it just
works still, or whatever) and run a command from. And fall back to
dhcp+pxe for the install case.
--
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/20130807/3d8348dc/attachment.pgp>
next prev parent reply other threads:[~2013-08-07 13:19 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-25 21:13 [U-Boot] [PATCH] wandboard: add pxe support, set default boot command like highbank dgilmore at fedoraproject.org
2013-07-26 13:52 ` Otavio Salvador
[not found] ` <20130726192012.1634241e@adria.ausil.us>
2013-07-29 18:47 ` Otavio Salvador
2013-07-31 23:01 ` Rob Herring
2013-07-31 23:27 ` Robert Nelson
2013-08-01 8:53 ` Stefano Babic
2013-08-01 17:06 ` Dennis Gilmore
2013-08-02 9:25 ` Stefano Babic
2013-08-04 20:00 ` Wolfgang Denk
2013-08-04 20:27 ` Tom Rini
2013-08-04 21:03 ` Wolfgang Denk
2013-08-04 21:47 ` Tom Rini
2013-08-04 22:11 ` Wolfgang Denk
2013-08-05 15:33 ` Tom Rini
2013-08-05 16:29 ` Otavio Salvador
2013-08-05 19:09 ` Tom Rini
2013-08-05 20:36 ` Wolfgang Denk
2013-08-05 20:44 ` Tom Rini
[not found] ` <20130804163442.31dca315@adria.ausil.us>
2013-08-04 22:26 ` Wolfgang Denk
2013-08-04 23:06 ` Dennis Gilmore
2013-08-01 17:19 ` Rob Herring
2013-08-02 9:14 ` Stefano Babic
2013-08-04 19:53 ` Wolfgang Denk
2013-08-04 20:14 ` Tom Rini
2013-08-05 2:21 ` Rob Herring
2013-08-05 5:29 ` Wolfgang Denk
2013-08-05 16:07 ` Tom Rini
2013-08-05 20:28 ` Wolfgang Denk
2013-08-05 20:42 ` Tom Rini
2013-08-05 22:16 ` Rob Herring
2013-08-06 11:37 ` Wolfgang Denk
2013-08-06 15:50 ` Stephen Warren
2013-08-06 16:18 ` Otavio Salvador
2013-08-06 16:54 ` Stephen Warren
2013-08-06 16:58 ` Otavio Salvador
2013-08-06 17:12 ` Stephen Warren
2013-08-06 17:17 ` Otavio Salvador
2013-08-06 21:37 ` Tom Rini
2013-08-06 21:44 ` Otavio Salvador
2013-08-06 21:45 ` Otavio Salvador
2013-08-06 16:22 ` Dennis Gilmore
2013-08-06 16:29 ` Otavio Salvador
2013-08-06 20:05 ` Stephen Warren
2013-08-06 21:42 ` Tom Rini
2013-08-06 23:11 ` Dennis Gilmore
2013-08-07 13:19 ` Tom Rini [this message]
2013-08-07 23:04 ` Dennis Gilmore
2013-08-08 15:26 ` Tom Rini
2013-08-10 20:50 ` Dennis Gilmore
2013-08-12 13:49 ` Tom Rini
2013-08-04 20:32 ` Tom Rini
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=20130807131921.GP5164@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox