[U-Boot] [PATCH v3 04/10] armv7: replace CONFIG_L2_OFF with CONFIG_SYS_NO_L2CACHE

Wolfgang Denk
2011-05-15T18:53:16Z

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.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Not one hundred percent efficient, of course ... but nothing ever is.
-- Kirk, "Metamorphosis", stardate 3219.8