[U-Boot] [PATCH v3 1/8] da850: indicate cache usage disable in config file

[Ben Gardiner]

Nagabhushana,

On Mon, Aug 29, 2011 at 6:56 AM, Netagunte, Nagabhushana
<nagabhushana.netagunte at ti.com> wrote:
>
> Gentlemen,
>
> We will address cache coherency issues soon after these patches.

Ok -- that's great news; we greatly anticipate U-boot improvements for
da850 from TI.

> Earlier also, chache was disabled. Only due to new cache management
> Framework which was added recently, it is explicitly needed to be
> Indicated to turn off cache.

My experience with the ARM-relocation changes on da8xx (which enabled
caches as I understood it) was very positive. I'm sorry I missed the
patch that disabled cache for davinci;

git log --decorate u-boot/master -- board/davinci/
arch/arm/cpu/arm926ejs/davinci/ include/configs/davinci_*
arch/arm/include/asm/arch-davinci doesn't turn up anything related to
caches so the patch (re: 'Earlier also, chache was disabled') is still
in mailing lists?

> Since fixing the cache coherency issues with EMAC will take some time,
> I want this patch to go in mainline so that issue doesn't crop up for
> People who use u-boot.

I certainly don't want anything less than the best user experience for
da850 u-boot.

It's possible that I have not been using caches in u-boot as I thought
I was -- considering I did not experience corrupted frames from the
EMAC.

On Wed, Aug 31, 2011 at 1:39 AM, Netagunte, Nagabhushana
<nagabhushana.netagunte at ti.com> wrote:
> If you have no further comments, can you please ACK patches?

for what it's worth:

Acked-by: Ben Gardiner <bengardiner at nanometrics.ca>


Best Regards,
Ben Gardiner

---
Nanometrics Inc.
+1 (613) 592-6776 x239
http://www.nanometrics.ca