All of lore.kernel.org
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.aribaud@free.fr>
To: u-boot@lists.denx.de
Subject: [U-Boot] DNS323 (Orion5x) must double ORION5X_SZ_BOOTROM to access full flash
Date: Tue, 24 Aug 2010 08:07:07 +0200	[thread overview]
Message-ID: <4C73618B.5000607@free.fr> (raw)
In-Reply-To: <4C7356E0.500@free.fr>

Le 24/08/2010 07:21, Chris Moore a ?crit :
> Hi Rogan,
>
> Le 23/08/2010 18:06, Rogan Dawes a ?crit :
>> Doubling the value for ORION5X_SZ_BOOTROM allowed me to access the
>> additional sectors, but that makes me wonder what the reason for it is.
>>
>> I know that the flash chip is wired up strangely, but would that also
>> affect the window mappings? If that is the case, I just need to document
>> WHY the parameter is doubled, but if not, it would be good to understand
>> the real reason for the change.
>>
>
> I am (very rashly) stabbing in the dark here because I don't know much
> about U-Boot and I haven't followed your previous posts (hence this
> off-list reply).
>
> However if your device size is half the bus size (like an 8-bit device
> on a 16-bit bus) it would seem logical to have to double the window size.
>
> Cheers,
> Chris

Hi Chris, BTW. :)

Rogan,

Seeing as ORION5X_SZ_BOOTROM is only used as a byte address limit for 
window mapping, and device vs bus size govern the maximum amout in a 
single bus access but do not govern its addressing, I don't think device 
width is involved here.

I'd rather ask whether this could be a window alignment issue, i.e is 
the flash base address 8 MB aligned? Seems like it, because IIRC its 
base is FF800000, which is 8MB-aligned.

Can you still try the original u-boot? Does it allow reading the full 8 
MB? If so, can you do a 'md.l 0xF1020000 40' in it, and then in your own 
u-boot with ORION5X_SZ_BOOTROM set to 8MB then 16MB? These are the 
window mapping registers, and it will allow us to know what the CPU 
really thinks boot ROM looks like. Also dump F101046C, that's the boot 
device bus configuration, again in both U-boots.

Amicalement,
-- 
Albert.

  parent reply	other threads:[~2010-08-24  6:07 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-23 16:06 [U-Boot] DNS323 (Orion5x) must double ORION5X_SZ_BOOTROM to access full flash Rogan Dawes
     [not found] ` <4C7356E0.500@free.fr>
2010-08-24  6:07   ` Albert ARIBAUD [this message]
2010-08-24  7:47     ` Rogan Dawes
2010-08-24 11:34       ` Albert ARIBAUD
2010-08-24 12:17         ` Rogan Dawes
2010-08-24 12:52           ` Albert ARIBAUD
2010-08-25  4:38             ` Prafulla Wadaskar

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=4C73618B.5000607@free.fr \
    --to=albert.aribaud@free.fr \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.