All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Murphy <dmurphy@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH 1/3] add file with a default boot environment based heavily on Stephen Warrens recent tegra work.
Date: Wed, 19 Feb 2014 14:03:00 -0600	[thread overview]
Message-ID: <53050DF4.3060706@ti.com> (raw)
In-Reply-To: <53050837.3050102@wwwdotorg.org>

On 02/19/2014 01:38 PM, Stephen Warren wrote:
> On 02/19/2014 12:32 PM, Dan Murphy wrote:
>> On 02/19/2014 01:04 PM, Stephen Warren wrote:
>>> On 02/19/2014 11:59 AM, Dan Murphy wrote:
>>>> On 02/19/2014 12:57 PM, Stephen Warren wrote:
>>>>> On 02/19/2014 11:52 AM, Dan Murphy wrote:
> ...
>>>>>> If no config file exists should we not try to default to a known good default tested case?
>>>>> I believe always loading a script/config-file is the simplest and most
>>>>> flexible approach, for a *distro* *oriented* boot process.
>>>>>
>>>>> Now, specific U-Boot board configs can always add extra stuff on the end
>>>>> (or start?) of bootcmd in order to do some custom fallbacks or
>>>>> backwards-compatibility if they want, but I'm certainly not planning on
>>>>> doing anything like that for Tegra or Raspberry Pi, for example.
>>>> Yeah I am not seeing how the board config can do that if there is no provisions in the common file.
>>> Presumably all it needs is an extra hook/variable that is added to the
>>> start/end of bootcmd. Should be pretty easy to add in a future patch
>>> rev, or followon patch.
>> <snip>
>>
>> I am not sure it is that simple.  Once you are in the BOOTCMD macro's if you end up back at the board file macros
>> you kinda have to repeat the load steps again just to get the args or specifics set.
>>
>> When the loading of the ENV file fails but the loading of the other images succeeds maybe something as simple as
>>
>> "run board_cfg"  Which can be a fall back to a board file specific configuration macro for whatever you want.
>>
>> If you don't need it then it is NULL
> If the value of bootcmd was customizable, then you could easily add e.g.
> "nand_custom" as the last entry in it, and that could do whatever was
> appropriate (it would translate to run bootcmd_nand_custom, which the
> board would define). That'd work very consistently with all the other
> options.

Well is this not what the BB patch does is calls upon the common boot_cmd and if that
fails then comes back to the board config to run nand_boot?

This does not really solve any of my issue of not having (or realizing I need) a script for boot on the boot partition.
No default case here to rely on as an in tree solution.

Dan

> Eventually, I'd love to have a BIOS-like (runtime) config menu, where
> any variable named bootcmd_* would show up in the list, and provide an
> interactive way to re-order and enable/disable all the options, i.e. an
> interactive menu-driven editor for the value of $bootcmd. If custom
> options get implemented the same way as the standard options, then the
> custom options would integrate very well into that scheme.


-- 
------------------
Dan Murphy

  reply	other threads:[~2014-02-19 20:03 UTC|newest]

Thread overview: 81+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-17 17:56 [U-Boot] RFC unified boot environment Dennis Gilmore
2014-02-17 17:56 ` [U-Boot] [RFC PATCH 1/3] add file with a default boot environment based heavily on Stephen Warrens recent tegra work Dennis Gilmore
2014-02-19 13:42   ` Tom Rini
2014-02-19 13:57     ` Dennis Gilmore
2014-02-19 15:54   ` Marek Vasut
2014-02-19 17:28     ` Stephen Warren
2014-02-19 17:30       ` Marek Vasut
2014-02-19 17:41         ` Stephen Warren
2014-02-19 17:44           ` Marek Vasut
2014-02-19 17:40   ` Stephen Warren
2014-02-22  8:20     ` Dennis Gilmore
2014-02-24 18:40       ` Stephen Warren
2014-02-24 20:07         ` Tom Rini
2014-02-19 18:44   ` Dan Murphy
2014-02-19 18:48     ` Stephen Warren
2014-02-19 18:52       ` Dan Murphy
2014-02-19 18:57         ` Stephen Warren
2014-02-19 18:59           ` Dan Murphy
2014-02-19 19:04             ` Stephen Warren
2014-02-19 19:10               ` Tom Rini
2014-02-19 19:16                 ` Stephen Warren
2014-02-19 19:36                   ` Tom Rini
2014-02-19 19:43                     ` Stephen Warren
2014-02-19 19:57                       ` Tom Rini
2014-02-19 20:10                       ` Dennis Gilmore
2014-02-19 19:32               ` Dan Murphy
2014-02-19 19:38                 ` Stephen Warren
2014-02-19 20:03                   ` Dan Murphy [this message]
2014-02-19 19:02           ` Eric Nelson
2014-02-19 19:05             ` Dan Murphy
2014-02-19 19:16               ` Tom Rini
2014-02-19 19:24                 ` Dan Murphy
2014-02-19 19:29                   ` Stephen Warren
2014-02-19 19:37                     ` Dan Murphy
2014-02-19 19:43                       ` Tom Rini
2014-02-19 19:41                     ` Tom Rini
2014-02-19 21:20                       ` Denys Dmytriyenko
2014-02-20 12:31                       ` Otavio Salvador
2014-02-20 13:46                         ` Tom Rini
2014-02-22 12:56                           ` Otavio Salvador
2014-02-17 17:56 ` [U-Boot] [RFC PATCH 2/3] move the beaglebones over to the generic configs Dennis Gilmore
2014-02-19 13:52   ` Tom Rini
2014-02-19 17:46   ` Stephen Warren
2014-02-19 19:57   ` Dan Murphy
2014-02-19 19:58     ` Dan Murphy
2014-02-19 20:05     ` Stephen Warren
2014-02-19 20:20       ` Dan Murphy
2014-02-19 20:22         ` Stephen Warren
2014-02-19 20:31           ` Dan Murphy
2014-02-19 20:38             ` Stephen Warren
2014-02-19 20:58               ` Dan Murphy
2014-02-19 21:07             ` Dennis Gilmore
2014-02-17 17:56 ` [U-Boot] [RFC PATCH 3/3] move wandboard over to use the generic distro configuratin and environment Dennis Gilmore
2014-02-19 11:52   ` Otavio Salvador
2014-02-19 17:50   ` Stephen Warren
2014-02-18 10:18 ` [U-Boot] RFC unified boot environment Stefano Babic
2014-02-18 16:09   ` Dennis Gilmore
2014-02-19 13:33     ` Tom Rini
2014-03-20 22:12 ` [U-Boot] [PATCH 0/6] " Dennis Gilmore
2014-03-20 22:12   ` [U-Boot] [PATCH 1/6] add README.distro file Dennis Gilmore
2014-03-21 18:48     ` Tom Rini
2014-03-25 20:40       ` Stephen Warren
2014-03-25 20:24     ` Stephen Warren
2014-03-28 15:42       ` Tom Rini
2014-03-28 16:11         ` Stephen Warren
2014-03-28 16:25           ` Tom Rini
2014-03-20 22:12   ` [U-Boot] [PATCH 2/6] add header with a generic set of boot commands defined Dennis Gilmore
2014-03-21 18:37     ` Marek Vasut
2014-03-21 18:53       ` Tom Rini
2014-03-21 21:00         ` Marek Vasut
2014-03-21 18:48     ` Tom Rini
2014-03-25 20:38       ` Stephen Warren
2014-03-25 20:36     ` Stephen Warren
2014-03-20 22:12   ` [U-Boot] [PATCH 3/6] move wandboard over to use the generic distro configuation and environment Dennis Gilmore
2014-03-20 22:12   ` [U-Boot] [PATCH 4/6] move beagleboard " Dennis Gilmore
2014-03-21 18:48     ` Tom Rini
2014-03-20 22:13   ` [U-Boot] [PATCH 5/6] move pandaboard " Dennis Gilmore
2014-03-21 18:49     ` Tom Rini
2014-03-20 22:13   ` [U-Boot] [PATCH 6/6] pxe: additionaly check for fdt_file env variable Dennis Gilmore
2014-03-21 18:49     ` Tom Rini
2014-03-25 20:45     ` Stephen Warren

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=53050DF4.3060706@ti.com \
    --to=dmurphy@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.