On Tue, Aug 23, 2011 at 2:19 AM, Lukasz Majewski <l.majew...@samsung.com> wrote:
> Hi Anton,
>
> On Mon, 22 Aug 2011 11:57:57 -0700
> Anton Staaf <robot...@google.com> 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/
Yup, should have said that you'd already taken care of some of these.
I'll send a patch now for another unaligned buffer in the mmc code
that isn't in your patch set.
Thanks,
Anton
> If any doubts/comments/ideas, please let me know :-)
>
>> But that's probably not right. I should probably send them now so we
>> can see a number of examples of the problem and get a better idea of
>> how to fix it.
>
> Discussion is always welcome.
>
> --
> Best regards,
>
> Lukasz Majewski
>
> Samsung Poland R&D Center
> Platform Group
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
