From: Aneesh V <aneesh@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Master branch broken for omap4_panda (SPL_BUILD)
Date: Tue, 04 Oct 2011 21:07:14 +0530 [thread overview]
Message-ID: <4E8B282A.8040301@ti.com> (raw)
In-Reply-To: <20111004145157.3DD6318E5B38@gemini.denx.de>
Dear Wolfgang,
On Tuesday 04 October 2011 08:21 PM, Wolfgang Denk wrote:
> Dear Aneesh V,
>
> In message<4E8B1508.7020706@ti.com> you wrote:
>>
>> Yes, I tried this just now and I am also facing this issue. I can fix
>> the build issue using the following change. But that doesn't solve the
>> problem. We do not have heap in SPL. So, this is a real serious problem
>> for any SPL that has FAT support. I will try to setup a heap in SDRAM
>> as soon as I get some time to work on this.
>
> All file system code has been designed and implemented to run AFTER
> relocation. Access to a FAT file system is not supposed to happen
> from SPL code.
SPL does call relocate_code() but defeats it by passing the relocation
target same as the SPL_TEXT_BASE.
OMAP has traditionally supported booting from the FAT partition of an
external SD card. The ROM code can pick up the SPL from the FAT
partition and SPL should pick up u-boot from there too. Some of our
boards like OMAP4 Panda have only an SD slot for non-volatile memory
and no flash memory installed. Of course we could use the SD card in
raw mode. But the FAT mode is much more convenient for many users. If
FAT is not supported in SPL, it will be a major regression compared to
x-loader and a show-stopper for us.
I have submitted a patch to fix the problem reported in this thread and
FAT boot is working fine again.
However, I am worried about the increasing footprint of SPL. The main
contributors are MMC and FAT. x-loader, with all it's short-comings,
does the same job with a smaller footprint. NAND has a custom driver
for SPL while MMC and FAT doesn't have. I don't think it's practical
either. However, I wonder if these drivers could be scaled down in size
for SPL with some '#ifdef CONFIG_SPL_BUILD's. I wish if somebody could
look into this.
best regards,
Aneesh
next prev parent reply other threads:[~2011-10-04 15:37 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-03 18:50 [U-Boot] Master branch broken for omap4_panda (SPL_BUILD) Aguirre, Sergio
2011-10-04 14:15 ` Aneesh V
2011-10-04 14:51 ` Wolfgang Denk
2011-10-04 15:37 ` Aneesh V [this message]
2011-10-04 18:05 ` Wolfgang Denk
2011-10-05 9:42 ` Aneesh V
2011-10-07 1:40 ` Tom Rini
2011-10-07 5:27 ` Wolfgang Denk
2011-10-04 15:11 ` [U-Boot] [PATCH] omap: spl: fix build break due to changes in FAT Aneesh V
2011-10-04 15:38 ` Wolfgang Denk
2011-10-04 16:00 ` Aneesh V
2011-10-04 16:17 ` Scott Wood
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=4E8B282A.8040301@ti.com \
--to=aneesh@ti.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