public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Greg Ungerer <gerg@moreton.com.au>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] add OpenGear CM4008 board support
Date: Mon, 23 May 2005 11:32:05 +1000	[thread overview]
Message-ID: <42913295.6080308@moreton.com.au> (raw)
In-Reply-To: <20050520144554.E25FDC1512@atlas.denx.de>

Hi Wolfgang,

Wolfgang Denk wrote:
> Dear Greg,
> 
> in message <428DF0F8.5090404@moreton.com.au> you wrote:
> 
>>The gofsk command is something I added to support loading
>>snapgear/uClinux-dist style raw filesystem+kernel images. 
> 
> I don't understand why a separate command is  needed  for  that;  you
> need  to store the start address of the image anyway, so you can also
> store it's size and use this to compute any other  offset;  or  store
> the kernel address, too.

Not quite. The start address is fixed (as in fixed position in flash
normally). Size is completely dependant on the image - so you need
to compute that at run time. You don't want to store the length
independantly.


>>The image itself is a simple concatenation of a root filesystem
>>and a kernel. Typically it is used with a CRAMfs root filesystem.
>>The primary motivator for not having any header on this image is
>>that it can be dumped directly into an MTD flash partition, and
>>it can be directly mounted as the root filesystem. For known
> 
> 
> This is normal. We don't have headers for standard filesystem  images
> like cramfs, ext2 or jffs2 for exactly this reason.

Yes, but that is not the point here. The is a complete system image
with no heads (system being kernel and root filesystem).


>>filesystem types (like CRAMfs and ROMfs) the loader simply looks
>>over the filesystem to get to the kernel for boot time starting.
> 
> Why do you need a separate command for this?

The other possibility seemed to be for a complicated sequence
of commands to extract the size - though I didn't try to actually
do this.

But I still wanted to construct the armtags for boot. I couldn't
see that this was currently possible if not used a strictly tagged
ARM u-boot image.


>>Patch attached is my current implementation of this.
>>
>>Is this something that you would allow into the main tree?
> 
> No, I don't think so. I don't see what it gives you what  you  cannot
> get  with the existing commands and clever usage of a few environment
> variables. Also, in general I find it is not a good idea  to  combine
> filesystem  and  OS  kernel  into  ane  image.

As a general rule I have always found the exact opposite. Although
conveient for debugging I find sepearate kernel and filesystems a
limitation in real field setups. A single flash partition with no
additional layout restrictions is a much more flexible arrangment.
Doesn't limit one or the others size or location.


>  Usually  you buy more
> restrictions than advantages this way. If you feel you  must  do  it,
> you  could extend the capabilities of mutli-file images to match your
> needs.
> 
> But the main question is: what can you do with this setup that cannot
> be done with the existing code, too?

I couldn't see how to load a raw image (no header) and still get the
ARM tags setup for booting. ofcousre this was with 1.1.1 when I 
originally did this. Is it possible now?

Regards
Greg

  reply	other threads:[~2005-05-23  1:32 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-17 13:53 [U-Boot-Users] [PATCH] add OpenGear CM4008 board support Greg Ungerer
2005-05-17 14:25 ` Wolfgang Denk
2005-05-19 14:30   ` Greg Ungerer
2005-05-19 22:47     ` Wolfgang Denk
2005-05-20 14:09       ` Greg Ungerer
2006-03-12  0:24         ` Wolfgang Denk
2005-05-20 14:15       ` Greg Ungerer
2005-05-20 14:45         ` Wolfgang Denk
2005-05-23  1:32           ` Greg Ungerer [this message]
2005-05-23  8:48             ` Wolfgang Denk
2005-05-24  6:42               ` Greg Ungerer
2005-05-24  9:00                 ` Wolfgang Denk
2005-05-24 10:23                   ` Greg Ungerer
2005-05-24 11:15                     ` Wolfgang Denk
2005-05-24 13:12                       ` Greg Ungerer
2005-05-24 13:52                         ` Wolfgang Denk
2005-05-25  0:35                           ` Greg Ungerer

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=42913295.6080308@moreton.com.au \
    --to=gerg@moreton.com.au \
    --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