From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aneesh V Date: Tue, 17 May 2011 15:29:49 +0530 Subject: [U-Boot] [PATCH v3 04/10] armv7: replace CONFIG_L2_OFF with CONFIG_SYS_NO_L2CACHE In-Reply-To: <20110515185316.2F8B61491B06@gemini.denx.de> References: <1299589658-30896-1-git-send-email-aneesh@ti.com> <1305202276-27784-5-git-send-email-aneesh@ti.com> <20110515185316.2F8B61491B06@gemini.denx.de> Message-ID: <4DD24715.4010105@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Wolfgang, On Monday 16 May 2011 12:23 AM, Wolfgang Denk wrote: > Dear Aneesh V, > > In message<1305202276-27784-5-git-send-email-aneesh@ti.com> you wrote: >> replace all occurences of CONFIG_L2_OFF with a more appropriate >> CONFIG_SYS_NO_L2CACHE >> >> CONFIG_SYS_NO_L2CACHE has been chosen to be in line with >> CONFIG_SYS_NO_ICACHE and CONFIG_SYS_NO_DCACHE > > sorry, but these are ugly mis-nomers. The situation is not that there > is no cache present (which "NO_*CACHE" suggestes), but that the cache > is (kept) turned off. > > CONFIG_SYS_L2CACHE_OFF is a _much_ better name than CONFIG_SYS_NO_L2CACHE. > > Please fix all these names. Ok. Including the existing CONFIG_SYS_NO_DCACHE and CONFIG_SYS_NO_ICACHE too, right? best regards, Aneesh