From mboxrd@z Thu Jan  1 00:00:00 1970
From: Lukasz Majewski <l.majewski@samsung.com>
Date: Tue, 23 Aug 2011 11:19:39 +0200
Subject: [U-Boot] [PATCH] mmc:dcache: Cache line size aligned internal
 MMC buffers
In-Reply-To: <CAF6FioUJTTfruGZUfxD_yXLg6Z7oCc+=WP8r_354thh8-_C3VQ@mail.gmail.com>
References: <1313745913-28672-1-git-send-email-l.majewski@samsung.com>
	<201108221317.36786.vapier@gentoo.org>
	<CAF6FioUS-GkRBwWbJJBM9PJBvvgrqvSwHhWZp5Ktf1Xp_pJC+A@mail.gmail.com>
	<201108221431.02180.vapier@gentoo.org>
	<CAF6FioUJTTfruGZUfxD_yXLg6Z7oCc+=WP8r_354thh8-_C3VQ@mail.gmail.com>
Message-ID: <20110823111939.2385daf5@lmajewski.digital.local>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Anton,

On Mon, 22 Aug 2011 11:57:57 -0700
Anton Staaf <robotboy@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/

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