public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 4/4] ARM: tegra: increase CONFIG_SYS_TEXT_BASE
Date: Thu, 18 Oct 2012 14:42:09 -0600	[thread overview]
Message-ID: <508069A1.1030908@wwwdotorg.org> (raw)
In-Reply-To: <20121018163114.GP27770@bill-the-cat>

On 10/18/2012 10:31 AM, Tom Rini wrote:
> On Wed, Oct 17, 2012 at 09:20:31PM -0600, Stephen Warren wrote:
>> On 10/17/2012 06:05 PM, Simon Glass wrote:
>>> On Tue, Oct 16, 2012 at 3:43 PM, Stephen Warren
>>> <swarren@wwwdotorg.org> wrote:
>>>> On 10/16/2012 04:09 PM, Lucas Stach wrote:
...
>>> To ask the opposite question, is it worth increasing by a whole
>>> 16KB so that the base address of U-Boot is a more aligned
>>> number?
>> 
>> That would bloat the binary by about 12KB more than it needs to
>> be. I don't believe there's any particular need for the main
>> U-Boot to be built for any particular address, and we can just
>> continue to bump up this value as/when the SPL grows.
> 
> Well, lets stop and think for a minute more.  Are we likely to add
> new features to SPL on Tegra (direct OS booting, support in one
> binary for both SPL-from-flash and SPL-from-something-else) ?

I don't think so. The only purpose of the SPL on Tegra is to run from
SDRAM on the AVP CPU, set up clocks for the main Cortex-A9 core, and
to cause the A9 to start executing the concatenated main U-Boot image.
The SPL always runs from SDRAM.

(As background, the boot ROM sets up SDRAM on Tegra, and copies the
concatenated SPL+U-Boot binaries into SDRAM from whatever boot device,
so the typical reasons for using SPL don't exist on Tegra).

Allen Martin was thinking about getting the SPL to run from IRAM
rather than SDRAM, and I think only execute on the AVP CPU (e.g. for
use as a slimmed-down flashing tool downloaded via the boot ROM's USB
recovery mechanism). However, I think that would end up being an
entirely separate SPL build (since we'd need both, not cut over), so
the size requirements would not impact in any way the SPL-in-SDRAM
that we have today.

  reply	other threads:[~2012-10-18 20:42 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-16 21:50 [U-Boot] [PATCH 1/4] ARM: fix u-boot.lds for -ffunction-sections/-fdata-sections Stephen Warren
2012-10-16 21:50 ` [U-Boot] [PATCH 2/4] ARM: tegra: combine duplicate Makefile rules Stephen Warren
2012-10-18  0:01   ` Simon Glass
2012-10-16 21:50 ` [U-Boot] [PATCH 3/4] ARM: tegra: check for SPL size overflow in makefile Stephen Warren
2012-10-18  0:03   ` Simon Glass
2012-10-18  3:18     ` Stephen Warren
2012-10-18 16:27   ` Tom Rini
2012-10-18 20:45     ` Stephen Warren
2012-10-18 20:50       ` Tom Rini
2012-10-16 21:50 ` [U-Boot] [PATCH 4/4] ARM: tegra: increase CONFIG_SYS_TEXT_BASE Stephen Warren
2012-10-16 22:09   ` Lucas Stach
2012-10-16 22:43     ` Stephen Warren
2012-10-18  0:05       ` Simon Glass
2012-10-18  3:20         ` Stephen Warren
2012-10-18 16:31           ` Tom Rini
2012-10-18 20:42             ` Stephen Warren [this message]
2012-10-17 23:58 ` [U-Boot] [PATCH 1/4] ARM: fix u-boot.lds for -ffunction-sections/-fdata-sections Simon Glass
2012-10-18  3:17   ` Stephen Warren
2012-10-18 20:36     ` Albert ARIBAUD
2012-10-18 20:58       ` Stephen Warren
2012-10-18 21:17         ` Stephen Warren
2012-10-20 10:31           ` Albert ARIBAUD

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=508069A1.1030908@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --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