From: Albert ARIBAUD <albert.aribaud@free.fr>
To: u-boot@lists.denx.de
Subject: [U-Boot] Fixing IXP42x boards - some general questions
Date: Thu, 23 Dec 2010 12:28:35 +0100 [thread overview]
Message-ID: <4D133263.4020402@free.fr> (raw)
In-Reply-To: <4D1326C6.10905@discworld.dascon.de>
Le 23/12/2010 11:39, Michael Schwingen a ?crit :
> Am 12/23/2010 11:20 AM, schrieb Albert ARIBAUD:
>> Hi Michael,
>>
>> Answering as the brand new ARM custodian :) :
>>
>> Le 23/12/2010 10:33, Michael Schwingen a ?crit :
>>
>>> Startup code. Is the following correct?
>>> - code starts from flash, with TEXT_BASE = start of flash, ie. the code
>>> is linked to flash addresses.
>> Correct. The goal of ELF relocation is to allow moving the code from
>> FLASH to any place in RAM (actually the highest possible location) by
>> correctly relocating it without developers having to fix or code
>> anything manually.
> Thanks. That means the code in the IXP startup code that copies flash to
> RAM (before calling board_init_f, and long before relocation code does a
> second copy) is really not needed, and can be removed if the branch to
> the reset code is replaced by an absolute jump to the real
> (not-aliased-to zero) flash location.
As for the additional copy, yes, it is uneeded -- look at some
ARM-based archs or boards which support ELF relocation (such as arm926ejs).
As for the jump, 'b' instruction is relative, and there is no need to
make an absolute jump -- unless you boot in a weird mode where FLASH
mirrors all over the memory space (and over RAM) and you need to jump to
the 'real', not 'mirrored', FLASH?
>>> What about interrupts? Use them or avoid them?
>> I say in any case don't use them before running from RAM; and if you can
>> avoid them in u-boot without incurring a huge performance penalty, I
>> would suggest avoiding them altogether.
> Fine with me. I got the non-interrupt code running, and will simply
> leave the (broken) interrupt code as is if that is OK.
Maybe it would be better to remove it. If someone really feels the need
to revive it, they can copy back it from a previous commit where it
still exists.
> cu
> Michael
Amicalement,
--
Albert.
next prev parent reply other threads:[~2010-12-23 11:28 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-23 9:33 [U-Boot] Fixing IXP42x boards - some general questions Michael Schwingen
2010-12-23 10:10 ` Wolfgang Denk
2010-12-23 10:55 ` Marek Vasut
2010-12-23 15:13 ` Michael Schwingen
2010-12-23 12:23 ` Michael Schwingen
2010-12-23 10:20 ` Albert ARIBAUD
2010-12-23 10:39 ` Michael Schwingen
2010-12-23 11:28 ` Albert ARIBAUD [this message]
2010-12-23 11:57 ` Michael Schwingen
2010-12-23 18:25 ` Michael Schwingen
2010-12-23 18:47 ` Albert ARIBAUD
2010-12-23 20:17 ` Michael Schwingen
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=4D133263.4020402@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox