public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marek.vasut@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V6] BOOT: Add "bootz" command to boot Linux zImage on ARM
Date: Fri, 16 Mar 2012 01:53:34 +0100	[thread overview]
Message-ID: <201203160153.34986.marek.vasut@gmail.com> (raw)
In-Reply-To: <4F626711.8030808@gmail.com>

Dear Rob Herring,

> On 03/15/2012 04:10 PM, Rob Herring wrote:
> > On 03/15/2012 03:29 PM, Marek Vasut wrote:
> >> Dear Rob Herring,
> >> 
> >>> On 03/15/2012 02:05 PM, Marek Vasut wrote:
> >>>> Dear Rob Herring,
> >>>> 
> >>>>> On 03/15/2012 02:52 AM, Marek Vasut wrote:
> >>>>>> From: Marek Vasut <marek.vasut@gmail.com>
> >>>>>> 
> >>>>>> This command boots Linux zImage from where the zImage is loaded to.
> >>>>>> Passing initrd and fdt is supported.
> >>>>> 
> >>>>> This is great! However, the initrd is still in the u-boot image
> >>>>> format, right?
> >>>>> 
> >>>>> If a raw initrd is supported, then we could get rid of flash-kernel
> >>>>> on disk-based debian/ubuntu systems. The file size is known from the
> >>>>> filesystem, but then needs to get to the bootz command somehow.
> >>>> 
> >>>> Yes. Initrd still needs a bit of work, ideas?
> >> 
> >> You lost some people in the Cc ;-)
> > 
> > They weren't in the original email header, only in the commit log as
> > Cc's.
> > 
> >>> How about:
> >>> 
> >>> bootz <kern addr> [<fdt addr> [<initrd addr> [<initrd size>]]]
> >> 
> >> What about bootz <kern addr> <initrd addr>[:<initrd size>] <fdt addr>
> >> 
> >> example: bootz 0x12000000 0x13000000:0x40000 0x14000000
> > 
> > Sure. That works for me.
> > 
> >>> I changed the order so the size can be optional without resorting to
> >>> using a "-". Then no size means u-boot image format.
> >>> 
> >>> File load commands would need to set loadsize env var.
> >> 
> >> If you load kernel after initrd, that's screw things up.
> > 
> > Yes, but you already have to be aware of some commands which will change
> > env vars. This has to be script-able to be useful. The only other way I
> > see is adding a command to get the size of a file. Perhaps the ls
> > commands when given a single file can do that.
> 
> Turns out there is already a filesize variable that gets set on file
> load commands.
> 
> And RTFM didn't help as that one is not documented... ;)

Rob, I just send in support for RAW initrd. Can you give it a go?

Best regards,
Marek Vasut

  reply	other threads:[~2012-03-16  0:53 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <Message-Id: <1331696876-20903-1-git-send-email-marex@denx.de>
2012-03-15  7:52 ` [U-Boot] [PATCH V6] BOOT: Add "bootz" command to boot Linux zImage on ARM Marek Vasut
2012-03-15 18:51   ` Rob Herring
2012-03-15 19:05     ` Marek Vasut
2012-03-15 19:48       ` Rob Herring
2012-03-15 20:29         ` Marek Vasut
2012-03-15 21:10           ` Rob Herring
2012-03-15 22:02             ` Rob Herring
2012-03-16  0:53               ` Marek Vasut [this message]
2012-03-16  7:16               ` Wolfgang Denk
2012-03-16 18:20                 ` Rob Herring
2012-03-15 20:39         ` Wolfgang Denk
2012-03-30 20:59   ` Wolfgang Denk
2012-03-31 19:10   ` Mike Frysinger
2012-03-31 19:27     ` Marek Vasut
2012-04-01 11:25       ` Graeme Russ
2012-04-01 14:02         ` Marek Vasut
     [not found] <Message-Id: <1331689093-19211-1-git-send-email-marex@denx.de>
2012-03-14  3:47 ` [U-Boot] [PATCH V5] " Marek Vasut
2012-03-14  5:07   ` Mike Frysinger
2012-03-14  5:38     ` Marek Vasut
2012-03-14 18:35       ` Mike Frysinger
2012-03-14 19:27         ` Marek Vasut
2012-03-15 16:09       ` Wolfgang Denk
2012-03-15 17:08         ` Marek Vasut
2012-03-15 17:36         ` Mike Frysinger
2012-03-15 17:40           ` Marek Vasut
2012-03-14 16:45   ` Stephen Warren
2012-03-14 17:23     ` Marek Vasut
     [not found] <Message-Id: <1331588061-21546-1-git-send-email-marex@denx.de>
2012-03-12 21:34 ` [U-Boot] [PATCH V3] BOOT: Add "bootz" command to boot Linux zImage Marek Vasut
2012-03-12 22:03   ` Wolfgang Denk
2012-03-12 22:08     ` Marek Vasut
2012-03-12 22:42       ` Wolfgang Denk
2012-03-13  0:13         ` Marek Vasut
2012-03-13  0:22           ` Graeme Russ
2012-03-13  4:30             ` Wolfgang Denk
2012-03-13  4:46               ` Graeme Russ
2012-03-15 15:03                 ` Wolfgang Denk
2012-03-15 17:08                   ` Marek Vasut
2012-03-13  4:47               ` Marek Vasut
2012-03-15 15:06                 ` Wolfgang Denk
2012-03-15 17:18                   ` Marek Vasut
2012-03-13  4:25           ` Wolfgang Denk
2012-03-13  4:50             ` Marek Vasut
2012-03-13  5:32               ` Graeme Russ
2012-03-13  2:03   ` Mike Frysinger
2012-03-14  1:38 ` [U-Boot] [PATCH V4] BOOT: Add "bootz" command to boot Linux zImage on ARM Marek Vasut
2012-03-14  2:03   ` Mike Frysinger
2012-03-14  2:40     ` Marek Vasut
2012-03-14  3:07       ` Mike Frysinger

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=201203160153.34986.marek.vasut@gmail.com \
    --to=marek.vasut@gmail.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