On Tuesday, August 23, 2011 05:19:39 Lukasz Majewski wrote:
> On Mon, 22 Aug 2011 11:57:57 -0700 Anton Staaf wrote:
> > drivers/mmc/mmc.c: ext_csd in mmc_change_freq is allocated on the stac
> > drivers/mmc/mmc.c: scr and switch_status in sd_change_freq are
> > allocated on the stack.
> > drivers/mmc/mmc.c: ext_csd in mmc_startup is allocated on the stack.
> 
> This allocations are already fixed:
> 
> http://patchwork.ozlabs.org/patch/110300/
> http://patchwork.ozlabs.org/patch/109790/
> 
> If any doubts/comments/ideas, please let me know :-)

hmm, i wish we had a memalign_alloca().  and all this copy & pasting of 
get_dcache_line_size() makes me unhappy as we're encoding too-low-of-a-level 
logic into funcs.

what about adding a new func like:
#define dma_buffer_alloca(size)

and it would take care of allocating a big enough aligned buffer on the stack.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to