public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2] ORIGEN: Enhance origen config to be more flexible on boot.
Date: Tue, 25 Nov 2014 09:20:48 +0100	[thread overview]
Message-ID: <1416903648.14110.18.camel@collabora.co.uk> (raw)
In-Reply-To: <5473DB5D.1080307@samsung.com>

On Tue, 2014-11-25 at 10:29 +0900, Minkyu Kang wrote:
> On 08/10/14 22:04, Guillaume GARDET wrote:
> > V2: rebased on latest GIT version
> > 
> > V1: This patch enhances the boot of origen board by adding support to ext2, bootz, initrd, bootenv loading and boot script.
> > It still keeps the previous mmc load command if boot script fails.
> > 
> > Signed-off-by: Guillaume GARDET <guillaume.gardet@free.fr>
> > Cc: Minkyu Kang <mk7.kang@samsung.com>
> > 
> > ---
> >  include/configs/origen.h | 35 ++++++++++++++++++++++++++++++++++-
> >  1 file changed, 34 insertions(+), 1 deletion(-)
> > 
> 
> applied to u-boot-samsung.

Would it not be better to standardize on the bootcmd from
config_distro_bootcmd.h, like Ian Campell did for exynos5 based boards?
_Especially_ when moving from current trivial bootcmds to a complex
ones. 

Not having standard bootcommands in the default u-boot settings, is
always a massive PITA when starting to work with new boards. Would be
great if that could slowly be fixed :) 

-- 
Sjoerd Simons <sjoerd.simons@collabora.co.uk>
Collabora Ltd.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 6170 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141125/2c2b5309/attachment.bin>

      reply	other threads:[~2014-11-25  8:20 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-19 13:32 [U-Boot] [PATCH] ORIGEN: Enhance origen config to be more flexible on boot Guillaume GARDET
2014-09-29  6:35 ` Guillaume Gardet
2014-10-08 10:52   ` Minkyu Kang
2014-10-08 13:04     ` [U-Boot] [PATCH V2] " Guillaume GARDET
2014-11-25  1:29       ` Minkyu Kang
2014-11-25  8:20         ` Sjoerd Simons [this message]

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=1416903648.14110.18.camel@collabora.co.uk \
    --to=sjoerd.simons@collabora.co.uk \
    --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