From: Scott Wood <scottwood@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 07/13] davinci: Use correct #ifdef around gdata/bdata
Date: Tue, 28 Feb 2012 16:03:38 -0600 [thread overview]
Message-ID: <4F4D4F3A.6010902@freescale.com> (raw)
In-Reply-To: <4F4D4D3F.5030005@aribaud.net>
On 02/28/2012 03:55 PM, Albert ARIBAUD wrote:
> Le 27/02/2012 13:02, Sughosh Ganu a ?crit :
>
>>> When I asked Heiko Schocher a few month ago why he defined putc and
>>> puts in arch/arm/cpu/arm926ejs/davinci/spl.c he replied that he could
>>> not use LIBCOMMON due to size limitations for the SPL. So I guess that
>>> this board will not be able to use the generic relocation functions,
>>> unless the SPL is smaller than 16kB, right? Simon's patchset will
>>> break this board then, right?
>>
>> That is exactly what i reported in one of the threads in response to
>> addition of libcommon and libgeneric to the hawkboard's spl. In
>> fact, this might cause problems on quite a few boards with spl size
>> restrictions. I am not sure, whether the generic relocation feature
>> should be turned on by default on all boards or should be a config
>> option -- at least for the spl builds. Another option would be to
>> move it to a place where it is not needed to compile in the entire
>> libcommon/libgeneric support that is not needed for the generic
>> relocation code. I think that would help us keep the generic
>> relocation without the size bloat that we see right now.
>>
>> http://lists.denx.de/pipermail/u-boot/2012-February/118567.html
>
> Sorry for appearing dumb, but can someone explain to me how SPL relates
> to relocation in the first place? I thought SPL was meant to be a
> preloader for the full(er) U-boot, small enough to be loaded by some
> SoCs' ROM code and possibly even to fit in SRAM. Why does it need
> relocation? And if it does not, how come it is affected by a rework of
> the relocation feature? I really would like a heads-up on this.
SPL may need relocation to vacate a buffer that will be used for further
I/O, such as on Freescale PPC (NAND boot execution begins in the NAND
controller's SRAM), or possibly to allow the memory map to be
transformed to what the final image is going to expect, etc.
Even in cases where the SPL text itself doesn't really need to move, you
may need some other things that typically happen along with relocation,
such as moving the stack to a larger memory after that memory is
initialized, zeroing BSS, etc.
-Scott
next prev parent reply other threads:[~2012-02-28 22:03 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-21 1:32 [U-Boot] [PATCH v4 0/13] Introduce generic relocation feature Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 01/13] Define CONFIG_SYS_SKIP_RELOC for all archs Simon Glass
2012-03-06 5:04 ` Mike Frysinger
2012-03-06 6:27 ` Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 02/13] Add generic relocation feature Simon Glass
2012-02-21 19:36 ` Albert ARIBAUD
2012-02-21 20:03 ` Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 03/13] arm: Export and promote relocation symbols Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 04/13] arm: Add start_call_board_init_r() to each start.S Simon Glass
2012-02-21 19:32 ` Albert ARIBAUD
2012-02-21 20:02 ` Simon Glass
2012-02-23 13:07 ` Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 05/13] Move memcpy(), memset() into new lib/membasic.c Simon Glass
2012-03-06 5:05 ` Mike Frysinger
2012-03-06 6:34 ` Simon Glass
2012-03-06 16:20 ` Mike Frysinger
2012-03-07 5:43 ` Simon Glass
2012-03-07 5:50 ` Graeme Russ
2012-03-07 17:48 ` Simon Glass
2012-03-11 7:35 ` Mike Frysinger
2012-02-21 1:32 ` [U-Boot] [PATCH v4 06/13] arm: Add explicit __image_copy_end symbol for ARM926EJ-S Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 07/13] davinci: Use correct #ifdef around gdata/bdata Simon Glass
2012-02-21 15:24 ` Tom Rini
2012-02-23 17:25 ` Sughosh Ganu
2012-02-26 17:56 ` Simon Glass
2012-02-27 10:16 ` Sughosh Ganu
2012-02-27 10:39 ` Christian Riesch
2012-02-27 10:56 ` Sughosh Ganu
2012-02-27 11:37 ` Christian Riesch
2012-02-27 12:02 ` Sughosh Ganu
2012-02-28 21:55 ` Albert ARIBAUD
2012-02-28 22:03 ` Scott Wood [this message]
2012-03-03 20:22 ` Simon Glass
2012-03-03 20:29 ` Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 08/13] tx25: Modify to work with generic relocation Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 09/13] hawkboard: " Simon Glass
2012-02-21 15:25 ` Tom Rini
2012-02-23 8:43 ` Sughosh Ganu
2012-02-23 15:11 ` Sughosh Ganu
2012-02-26 18:09 ` Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 10/13] mx31pdk: " Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 11/13] smdk6400: " Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 12/13] arm: Move over to " Simon Glass
2012-02-21 1:32 ` [U-Boot] [PATCH v4 13/13] arm: Remove unused code in start.S Simon Glass
2012-02-21 19:40 ` [U-Boot] [PATCH v4 0/13] Introduce generic relocation feature Albert ARIBAUD
2012-02-21 20:07 ` Simon Glass
2012-02-21 20:10 ` Graeme Russ
2012-02-21 20:16 ` Simon Glass
[not found] ` <4F4D4511.6020708@aribaud.net>
2012-03-03 16:01 ` Simon Glass
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=4F4D4F3A.6010902@freescale.com \
--to=scottwood@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