From mboxrd@z Thu Jan  1 00:00:00 1970
From: Dirk Behme <dirk.behme@de.bosch.com>
Date: Fri, 20 Jul 2012 07:35:30 +0200
Subject: [U-Boot] [PATCH V2] i.MX: fsl_esdhc: allow use with
	cache	enabled.
In-Reply-To: <201207200432.46649.marex@denx.de>
References: <http://lists.denx.de/pipermail/u-boot/2012-March/#119312>
	<201206121923.30555.marex@denx.de> <4FFD1D58.5000904@de.bosch.com>
	<201207200432.46649.marex@denx.de>
Message-ID: <5008EE22.6040003@de.bosch.com>
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

On 20.07.2012 04:32, Marek Vasut wrote:
> Dear Dirk Behme,
> 
> [...]
> 
>>>> I'm not likely to have any time until after FTF next week.
>>>>
>>>> Any chance that a PPC maintainer can chime in here?
>>> I think WD applied the cache stub patch already. Can you try now please?
>> Ping ;)
>>
>> Thanks
>>
>> Dirk
> 
> I just noticed this with latest mainline:
> Configuring for MPC8569MDS_ATM - Board: MPC8569MDS, Options: ATM
> make: *** [u-boot] Error 1
> powerpc-linux-size: './u-boot': No such file
> drivers/mmc/libmmc.o: In function `esdhc_setup_data':
> /home/marex/testing/u-boot.git/drivers/mmc/fsl_esdhc.c:193: undefined reference 
> to `flush_dcache_range'
> drivers/mmc/libmmc.o: In function `check_and_invalidate_dcache_range':
> /home/marex/testing/u-boot.git/drivers/mmc/fsl_esdhc.c:263: undefined reference 
> to `invalidate_dcache_range'
> make: *** [u-boot] Error 1
> Configuring for MPC8569MDS_NAND - Board: MPC8569MDS, Options: NAND
> make: *** [u-boot] Error 1
> powerpc-linux-size: './u-boot': No such file
> drivers/mmc/libmmc.o: In function `esdhc_setup_data':
> /home/marex/testing/u-boot.git/drivers/mmc/fsl_esdhc.c:193: undefined reference 
> to `flush_dcache_range'
> drivers/mmc/libmmc.o: In function `check_and_invalidate_dcache_range':
> /home/marex/testing/u-boot.git/drivers/mmc/fsl_esdhc.c:263: undefined reference 
> to `invalidate_dcache_range'
> make: *** [u-boot] Error 1
> 
> Can this be caused by this patch?

Yes, I think so.

Reading the logs, to my understanding the conclusion in

http://lists.denx.de/pipermail/u-boot/2012-May/124036.html

was to "add an empty flush_dcache_range() to the MPC85xx, maybe as weak 
function".

Then

http://lists.denx.de/pipermail/u-boot/2012-June/126097.html

mentions "WD applied the cache stub patch already".

So do we have this "cache stub patch" somewhere which fixes the issue 
mentioned above? I'm not sure I've seen it ...

Best regards

Dirk