[U-Boot] [PATCH 2/3] am33xx: Correct NON_SECURE_SRAM_START/END

Tom Rini trini at ti.com
Sat Jun 1 15:23:27 CEST 2013


On Fri, May 31, 2013 at 10:56:52PM +0200, Peter Korsgaard wrote:
> >>>>> "Tom" == Tom Rini <trini at ti.com> writes:
> 
>  Tom> Prior to Sricharan's cleanup of the boot parameter saving code, we
>  Tom> did not make use of NON_SECURE_SRAM_START on am33xx, so it wasn't a
>  Tom> problem that the address was pointing to the middle of our running SPL.
>  Tom> Correct to point to the base location of the download image area.
>  Tom> Increase CONFIG_SPL_TEXT_BASE to account for this scratch area being
>  Tom> used.  As part of correcting these tests, make use of the fact that
>  Tom> we've always been placing our stack outside of the download image area
>  Tom> (which is fine, once the downloaded image is run, ROM is gone) so
>  Tom> correct the max size test to be the ROM defined top of the download area
>  Tom> to where we link/load at.
[snip]
>  Tom> + * We limit our size to the ROM-defined dowloaded image area, and use the
> 
> s/dowloaded/downloaded/

And copy/pasted to the other configs, oops, fixed in all 3.  If there's
no other feedback, I'll refrain from a v2 posting and just include the
typo correction when I merge it middle of next week, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130601/68bdad50/attachment.pgp>


More information about the U-Boot mailing list