public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Timur Tabi <timur@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] 83xx: Put the version (and magic) after the HRCW.
Date: Fri, 08 Dec 2006 15:38:48 -0600	[thread overview]
Message-ID: <4579DB68.1010506@freescale.com> (raw)
In-Reply-To: <4579DAC1.3080804@smiths-aerospace.com>

Jerry Van Baren wrote:

> I suspect your 8349E-mITX is configured to "boothigh" (BMS = 1, also 
> known as HRCWH_FROM_0XFFF00100) so the u-boot image starts at 0xFFF00000 
> rather than at the start of flash (nominally 0xFE000000).  In this case 
> you should find the magic and version string starting at 0xFFF00040.  Of 
> course, in the .bin/.srec/whatever file, it should be at offset 0x40.

Yes, that's correct.

> That is one of the advantages of "bootlow" - the HRCW from u-boot is 
> placed at the start of flash and thus is used.  With "boothigh" , the 
> HRCW that is part of u-boot is in the "wrong place" and thus is not 
> used.  (Another advantage of "bootlow" is that you don't have to 
> dedicate 1MB of flash for the u-boot image.)

The original code from our BSP had only boothigh, so that's how I left it.  I 
guess I should test lowboot as an option.

> Looking at include/configs/MPC8349ITX.h, if PCI_64BIT is defined it does 
> a lowboot, otherwise it does a highboot (which seems odd to me, but 
> people tell me I'm odd and the world is normal).

I've noticed that too, and I'm trying to get answer as to why.

> The "lowboot" patch is specific to the MPC8360EMDS, although it could be 
> copy and pasted into the MPC8349ITX configuration easily enough.

Ok, if it gets accepted, I'll port it to the 8349ITX.


-- 
Timur Tabi
Linux Kernel Developer @ Freescale

  reply	other threads:[~2006-12-08 21:38 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-07  2:23 [U-Boot-Users] [PATCH] 83xx: Put the version (and magic) after the HRCW Jerry Van Baren
2006-12-07  2:55 ` Jerry Van Baren
2006-12-07  3:16 ` [U-Boot-Users] Re: " Sam Song
2006-12-07  3:45   ` [U-Boot-Users] " Jerry Van Baren
2006-12-08  2:49     ` Jerry Van Baren
2006-12-08 16:33       ` Timur Tabi
2006-12-08 16:50         ` Jerry Van Baren
2006-12-08 21:13           ` Timur Tabi
2006-12-08 21:36             ` Jerry Van Baren
2006-12-08 21:38               ` Timur Tabi [this message]
  -- strict thread matches above, loose matches on Subject: below --
2006-12-07  8:32 Joakim Tjernlund
2006-12-07 13:22 ` Jerry Van Baren
2006-12-07 17:05 Joakim Tjernlund

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=4579DB68.1010506@freescale.com \
    --to=timur@freescale.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