From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Fri, 15 Jul 2011 09:42:53 +0200 Subject: [U-Boot] [PATCH 1/2] integratorap: disable dcache In-Reply-To: References: <1310598267-9711-1-git-send-email-linus.walleij@linaro.org> <20110714113938.BB30D1593511@gemini.denx.de> Message-ID: <4E1FEF7D.5050007@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Linus, Linus Walleij wrote: > On Thu, Jul 14, 2011 at 1:39 PM, Wolfgang Denk wrote: > >> Sorry, but again I have to point out that I really dislike papering >> over existing bugs. If the disable D-cache in bootm does not work, >> _this_ should be analyzed and fixed instead of globally disabling he >> data cache. > > No problem, I didn't label it as RFC since what it does is actually fix a > regression. > > At one point U-boot was working on the Integrator and now > it doesn't, it seems commit 880eff5cfb9df6f0855f4e48affd349ca64692e9 > "ARM: cp15: setup mmu and enable dcache" is causing it. > > Heiko, Alessandro, do you have any hints on how to go about flushing > and disabling the D-cache in bootm for ARM920T? It doesn't seem to > work on this ARM CPU chip in its current form... Hmm.. trying bootm with actual u-boot on a davinci (arm926ejs) based board, there I couldn;t use the Davinci-EMAC (if I disable the dcache it works ...) Seems above commit introduced problems for other arm architectures, drivers, ... Hmm... if we globally always enable the dcache, we have a lot of more drivers to fix (I think) ... bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany