From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V3 2/3] arm/davinci: spl - boot device selection
Date: Wed, 11 Jul 2012 05:15:49 -0700 [thread overview]
Message-ID: <4FFD6E75.7070706@ti.com> (raw)
In-Reply-To: <20120711120837.GA28077@Hardy>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 07/11/2012 05:08 AM, Sughosh Ganu wrote:
> On Wed Jul 11, 2012 at 04:43:19AM -0700, Tom Rini wrote:
>> On 07/11/2012 04:05 AM, Sughosh Ganu wrote:
>
> <snip>
>
>>>> I'm fine saying that we should wrap the call around an #if,
>>>> but I would expect it to be set in the common case and only
>>>> not set in custom production boards.
>>>
>>> Correct, so all that is needed for SD/MMC loading can still be
>>> kept separate from, say nand load in the spl.c file, isn't it.
>>> So if sd/mmc spl load needs heap to be set up, can't it be done
>>> only for that case.
>>
>> Yes, but we don't want to have N locations that set up the heap.
>> We want one location that will, if the board has defined a heap
>> area, we set it up. Or to put it another way, calling
>> mem_malloc_init isn't part of the mmc boot-flow, it's part of the
>> general boot flow.
>
> Fair enough. My only concern was just that if some feature is not
> needed for a board, it should be possible to exclude it out.
To be clear, absolutely agreed.
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQIcBAEBAgAGBQJP/W50AAoJENk4IS6UOR1W6VEQAJ2C+IlU/irdxIBQJkWKdGoe
Rwr1bdQpnUIA4ILEBuD7jwiDj/Q6znfzUQf0XXZuRg+ShYu10y/2HBazn/alfrMj
FyLqm7YnjxDoWpfbSzR5EajdUOD7UgdDbZURwyte/NQos2WeS89IZTrlIT5sxfYV
L7DT0fDdK4pfh7+gArr99Ge/eE3ddm9kVdoxFbvYsr0BQs8xLnjh7t1DZ2rk0P7Y
AabwKIiM1o7xNn5GPIb7lraO4A1/m7U1/if2ad+I/qtEsAnsTC+Ds7VPvnxSdK/w
ga/r/OcYzpQLHna0dBQ4qXqnl5hKDARw3ZX2EYiqxRECRDLMDo2OWJio2F7yEqID
nYhhVKFNcSIC6mcIz1DyPGd/q3QFziCRFfFq0xrfF6K2rNByfXK4xHdld33EFvM1
V7Kfu8XSXeswGU41T+I6hHZyaCFWeNzDBTF/JtfXoZ0IoGgb045JLc6i14KAmwpO
AVCLhQy64h4O7BfvrzYK5u93RmI0PtjdXwkrBMpqAcICwsiPJ0mAU2azYQNbhYVk
NJM5xecgobni0HQek1f/UgzwllwDusjrjo5Hv+Di/D5wYBYTOWxPKZGEHOWfcuf6
O/io1QVQc3FMdrU3cCf5kB/tDAXDOpENhStCkkCgbFVf6u2hVz4vdSxV7oZ5Gmkp
bpFoazwtHzXpzE5B6eIe
=kJeB
-----END PGP SIGNATURE-----
next prev parent reply other threads:[~2012-07-11 12:15 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-09 18:53 [U-Boot] [PATCH V3 1/3] MMC: u-boot-spl may be compiled without partition support Mikhail Kshevetskiy
2012-07-09 18:53 ` [U-Boot] [PATCH V3 2/3] arm/davinci: spl - boot device selection Mikhail Kshevetskiy
2012-07-09 19:41 ` Tom Rini
2012-07-09 19:57 ` Mikhail Kshevetskiy
2012-07-09 20:08 ` Tom Rini
2012-07-09 20:10 ` Mikhail Kshevetskiy
2012-07-10 18:39 ` Sughosh Ganu
2012-07-10 19:20 ` Mikhail Kshevetskiy
2012-07-11 6:38 ` Sughosh Ganu
2012-07-11 8:19 ` Tom Rini
2012-07-11 10:40 ` Sughosh Ganu
2012-07-11 10:46 ` Tom Rini
2012-07-11 11:05 ` Sughosh Ganu
2012-07-11 11:43 ` Tom Rini
2012-07-11 12:08 ` Sughosh Ganu
2012-07-11 12:15 ` Tom Rini [this message]
2012-07-09 18:53 ` [U-Boot] [PATCH V3 3/3] arm/davinci: spl - add compressed u-boot image support Mikhail Kshevetskiy
2012-07-10 18:49 ` Sughosh Ganu
2012-07-10 19:29 ` Mikhail Kshevetskiy
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=4FFD6E75.7070706@ti.com \
--to=trini@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