public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jerry Van Baren <gerald.vanbaren@ge.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] MPC83xx HRCW
Date: Fri, 28 Mar 2008 07:55:39 -0400	[thread overview]
Message-ID: <47ECDCBB.7050508@ge.com> (raw)
In-Reply-To: <47ECD295.3090707@matrix-vision.de>

Andre Schwarz wrote:
> In "cpu/mpc83xx/start.S" there is a comment :
> 
> /*
>  * The Hard Reset Configuration Word (HRCW) table is in the first 64
>  * (0x40) bytes of flash.  It has 8 bytes, but each byte is repeated 8
>  * times so the processor can fetch it out of flash whether the flash
>  * is 8, 16, 32, or 64 bits wide (hardware trickery).
>  */
> 
> This does _not_ hold true for all configurations. Flash is simply one of 
> many options ...
> Maybe it's true for the Freescale boards.
> 
> Other sources of the HRCW can be hard-coded strapping pins or an I2C EEPROM.
> 
> Why is there a need to define the HRCW ?
> 
> regards,
> Andre Schwarz

Hi Andre,

The HRCW in flash (could be other memory or a FPGA register) is a 
processor feature which a board may or may not use.  I am not familiar 
with the whole 83xx family, but I presume the feature is part of the 
whole family.

As you point out, there are other ways of configuring the processor on 
power up, and it is board-specific which way is used on the particular 
board.

For the boards that support the HRCW, obviously the definition in the 
first 64 bytes of flash is necessary.  For other boards, it is 
unnecessary.  FWIIW, the Freescale eval boards that I have experience 
with allow the HRCW to come from flash, i2c, or an FPGA (BCSR).

To date, having a potentially unused HRCW definition in memory has not 
been an issue - people either use it or ignore it.  If it is an issue, 
you could use conditionals to disable it.  I'm sure the 83xx custodian 
(Kim Phillips) would consider patches to do that.  ;-)  Note that there 
is a possibility that some of the code assumes the presence of a HRCW, 
so you would have to inspect and/or regression test as part of a 
conditionalization patch.

On the other hand, the HRCW is occupying 64 bytes of memory that is 
marked "reserved" by Freescale/PowerPC, so it doesn't "cost" anything to 
have an unused HRCW programmed there.  (We also put the u-boot version 
string in the reserved space which is a violation of the "reserved" 
marking, but it is convenient to put it in a well known place at the 
start of the memory map and, besides, we've gotten away with it for 
years ;-).

Best regards,
gvb

  reply	other threads:[~2008-03-28 11:55 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-28 11:12 [U-Boot-Users] MPC83xx HRCW Andre Schwarz
2008-03-28 11:55 ` Jerry Van Baren [this message]
     [not found]   ` <47ECE17A.9060906@matrix-vision.de>
2008-03-28 12:24     ` Jerry Van Baren
2008-03-28 12:34       ` Andre Schwarz
2008-03-28 16:11 ` David Hawkins
2008-03-28 21:10   ` André Schwarz
2008-03-28 21:53     ` David Hawkins
2008-03-28 22:13       ` André Schwarz

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=47ECDCBB.7050508@ge.com \
    --to=gerald.vanbaren@ge.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