From: Bill Pringlemeir <bpringlemeir@nbsps.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arm: vf610: fix boot from SD-card
Date: Thu, 08 Jan 2015 13:29:44 -0500 [thread overview]
Message-ID: <87egr55dnr.fsf@nbsps.com> (raw)
In-Reply-To: <1420737505-32119-1-git-send-email-stefan@agner.ch> (Stefan Agner's message of "Thu, 8 Jan 2015 18:18:25 +0100")
On 8 Jan 2015, stefan at agner.ch wrote:
> Boot from SD-card (and probably also from NAND) was broken since
> commit d6d07a9bec ("arm: vf610: add NAND support for vf610twr").
It was broke before that when fsl_qspi was added? Or at least that also
added a lot of code. It maybe dependant on the compiler (and hence
binary size).
> It looks like the increased size of U-Boot lead to a situation where
> the boot ROM overwrote its own stack/heap while loading U-Boot from
> the SD-card to the SRAM.
I didn't think of this. That is a possibility.
> However, U-Boot worked fine when loaded through USB serial loader
> directly into SRAM. It looks like loading from SD-card uses other
> stack/heap location then the serial loader (or maybe no stack or heap
> at all).
I have tried various u-boot images with git-bisect. For instance when
the QSPI was added, I had issues. It seems that an image near 220k was
a threshold. The offset in the On-chip SRAM is 0x8000 (from the 'ld'
file) and the imximage prints '3f008000'. So, I think that the offset
takes the breaking point to where we cross from OC-SRAM0 to OC-SRAM1; I
guess a stack maybe located here but it must be fairly small. I thought
that maybe the HAB has a mapping of device memory start/size and didn't
like it when we crossed the boundary.
> This fix moves U-Boot to gfxRAM, which is 512kB in size and is not
> used by the boot ROM nor the SD-card loader of it.
Whatever the issue is, this fixes it for me as well. I would guess that
NAND boot is also not possible or probably anything but serial-download
modes.
Thanks,
Bill Pringlemeir.
next prev parent reply other threads:[~2015-01-08 18:29 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-08 17:18 [U-Boot] [PATCH] arm: vf610: fix boot from SD-card Stefan Agner
2015-01-08 17:32 ` Bill Pringlemeir
2015-01-09 14:13 ` Stefan Agner
2015-01-09 14:29 ` Otavio Salvador
2015-01-09 14:51 ` Stefano Babic
2015-01-08 18:29 ` Bill Pringlemeir [this message]
2015-01-09 15:28 ` [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=87egr55dnr.fsf@nbsps.com \
--to=bpringlemeir@nbsps.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.