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 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.