Nagabhushana,
On Mon, Aug 29, 2011 at 6:56 AM, Netagunte, Nagabhushana nagabhushana.netagunte@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@ti.com wrote:
If you have no further comments, can you please ACK patches?
for what it's worth:
Acked-by: Ben Gardiner bengardiner@nanometrics.ca
Best Regards, Ben Gardiner
--- Nanometrics Inc. +1 (613) 592-6776 x239 http://www.nanometrics.ca