public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] mmc changes in v2014.04-rc3 cause problems when using mmc in SPL
@ 2014-04-08 22:05 Hans de Goede
  2014-04-08 23:07 ` Tom Rini
  0 siblings, 1 reply; 3+ messages in thread
From: Hans de Goede @ 2014-04-08 22:05 UTC (permalink / raw)
  To: u-boot

Hi,

While merging v2014.04-rc3 into our sunxi u-boot branch, I hit a problem
where mmc would no longer work in the SPL.

The problem is that the new mmc_create call forces use of HEAP memory,
while in the SPL there may be no HEAP.

For now I've created a small 2048 bytes HEAP in our SPL, but if possible
I would like to get rid of the need for a HEAP though, as we only have
24K of RAM for our SPL.

Regards,

Hans

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] mmc changes in v2014.04-rc3 cause problems when using mmc in SPL
  2014-04-08 22:05 [U-Boot] mmc changes in v2014.04-rc3 cause problems when using mmc in SPL Hans de Goede
@ 2014-04-08 23:07 ` Tom Rini
  2014-04-13 13:15   ` Hans de Goede
  0 siblings, 1 reply; 3+ messages in thread
From: Tom Rini @ 2014-04-08 23:07 UTC (permalink / raw)
  To: u-boot

On Wed, Apr 09, 2014 at 12:05:04AM +0200, Hans de Goede wrote:

> Hi,
> 
> While merging v2014.04-rc3 into our sunxi u-boot branch, I hit a problem
> where mmc would no longer work in the SPL.
> 
> The problem is that the new mmc_create call forces use of HEAP memory,
> while in the SPL there may be no HEAP.
> 
> For now I've created a small 2048 bytes HEAP in our SPL, but if possible
> I would like to get rid of the need for a HEAP though, as we only have
> 24K of RAM for our SPL.

24K should still be enough to have a small malloc pool and the rest of
SPL, especially since it sounds like you already didn't have FAT support
in there (which has required malloc for ages and is why Pantelis didn't
spot this as a problem I think).  How close to the limit are we today in
the sunxi platforms?  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/20140408/c664ad1b/attachment.pgp>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] mmc changes in v2014.04-rc3 cause problems when using mmc in SPL
  2014-04-08 23:07 ` Tom Rini
@ 2014-04-13 13:15   ` Hans de Goede
  0 siblings, 0 replies; 3+ messages in thread
From: Hans de Goede @ 2014-04-13 13:15 UTC (permalink / raw)
  To: u-boot

Hi,

On 04/09/2014 01:07 AM, Tom Rini wrote:
> On Wed, Apr 09, 2014 at 12:05:04AM +0200, Hans de Goede wrote:
> 
>> Hi,
>>
>> While merging v2014.04-rc3 into our sunxi u-boot branch, I hit a problem
>> where mmc would no longer work in the SPL.
>>
>> The problem is that the new mmc_create call forces use of HEAP memory,
>> while in the SPL there may be no HEAP.
>>
>> For now I've created a small 2048 bytes HEAP in our SPL, but if possible
>> I would like to get rid of the need for a HEAP though, as we only have
>> 24K of RAM for our SPL.
> 
> 24K should still be enough to have a small malloc pool and the rest of
> SPL, especially since it sounds like you already didn't have FAT support
> in there (which has required malloc for ages and is why Pantelis didn't
> spot this as a problem I think).  How close to the limit are we today in
> the sunxi platforms?

We're close to the limit, but in the mean time I've realized that we don't
need the HEAP until after we've brought up the DRAM (and the base SPL code
does not init the HEAP until after DRAM init, and once we've DRAM we no
longer have any space constraints. So things are fine as is. Sorry for the
noise.

Regards,

Hans

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-04-13 13:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-08 22:05 [U-Boot] mmc changes in v2014.04-rc3 cause problems when using mmc in SPL Hans de Goede
2014-04-08 23:07 ` Tom Rini
2014-04-13 13:15   ` Hans de Goede

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox