[U-Boot] [PATCH] env_mmc: fix buffer allocation for armv7
Tom Rini
trini at ti.com
Fri Oct 11 20:27:24 CEST 2013
On Tue, Oct 08, 2013 at 08:17:01PM +0200, Wolfgang Denk wrote:
> Dear Tom,
>
> In message <20131008134456.GB15917 at bill-the-cat> you wrote:
> >
> > > Well, if we have DDR such that we can use it for the malloc arena, we
> > > also should use it for the stack. Or is there a good reason for not
> > > doing this? It would solve all these issues at the root...
> >
> > Making SPL more complex for everyone? We would need to do a fairly
> > good-sized re-jigger of SPL to setup and swap around stack pointers like
> > we do in full U-Boot.
>
> Hm, I'm not convinced. As proposed, we make the code bigger, less
> efficient, more error prone and more ugly for everyone, not only for
> SPL users. Aslo, this might not be the only place where buffers or
> such may be kept on the stack. I hope you don't want to change all
> these?
>
> Really, if we have the resources, we should use them. If RAM is
> abailable, it should also be used for the stack. Just using it for
> malloc() is neither fish nor fowl.
I'll ceed the point and re-work things on my series then. Markus, your
patch is good as-is and I shall pick it up shortly.
--
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/20131011/1b527155/attachment.pgp>
More information about the U-Boot
mailing list