From: Ladislav Michl <ladis@linux-mips.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] TI: Rework SRAM definitions and maximums
Date: Tue, 6 Sep 2016 15:13:00 +0200 [thread overview]
Message-ID: <20160906131300.GA26433@localhost.localdomain> (raw)
In-Reply-To: <2443ec74-ee96-6f89-b5fb-1c3690a959fb@ti.com>
On Sat, Aug 27, 2016 at 05:29:44PM +0530, Lokesh Vutla wrote:
> On Friday 26 August 2016 11:00 PM, Tom Rini wrote:
> > On all TI platforms the ROM defines a "downloaded image" area at or near
> > the start of SRAM which is followed by a reserved area. As it is at
> > best bad form and at worst possibly harmful in corner cases to write in
> > this reserved area, we stop doing that by adding in the define
> > NON_SECURE_SRAM_IMG_END to say where the end of the downloaded image
> > area is and make SRAM_SCRATCH_SPACE_ADDR be one kilobyte before this.
> > At current we define the end of scratch space at 0x228 bytes past the
> > start of scratch space this this gives us a lot of room to grow. As
> > these scratch uses are non-optional today, all targets are modified to
> > respect this boundary.
> >
> > Tested on OMAP4 Pandaboard, OMAP3 Beagle xM
>
> Verified on AM437x-GP EVM.
>
> Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
> Acked-by: Lokesh Vutla <lokeshvutla@ti.com>
Verified on IGEPv2.
Tested-by: Ladislav Michl <ladis@linux-mips.org>
Also note, that this patch and CONFIG_SPL_SYS_MALLOC_SIMPLE is needed
for sucessfull build (with gcc-5.4.0), otherwise compilation fails with:
arm-v7a-linux-gnueabi-ld.bfd: u-boot-spl section `.data' will not fit in region `.sram'
arm-v7a-linux-gnueabi-ld.bfd: region `.sram' overflowed by 264 bytes
Just enabling CONFIG_SPL_SYS_MALLOC_SIMPLE without this patch makes
malloc in fat loading code fail and thus causes boot failure.
Best regards,
ladis
next prev parent reply other threads:[~2016-09-06 13:13 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-26 17:30 [U-Boot] [PATCH] TI: Rework SRAM definitions and maximums Tom Rini
2016-08-27 11:55 ` Adam Ford
2016-08-27 11:59 ` Lokesh Vutla
2016-09-06 13:13 ` Ladislav Michl [this message]
2016-09-07 17:58 ` [U-Boot] " Tom Rini
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=20160906131300.GA26433@localhost.localdomain \
--to=ladis@linux-mips.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