From mboxrd@z Thu Jan 1 00:00:00 1970 From: linux@arm.linux.org.uk (Russell King - ARM Linux) Date: Tue, 23 Oct 2012 11:00:57 +0100 Subject: [PATCH V6 3/6] arm: cache-l2x0: add support for Aurora L2 cache ctrl In-Reply-To: <508666A6.9040701@free-electrons.com> References: <1348738523-28609-1-git-send-email-gregory.clement@free-electrons.com> <1348738523-28609-2-git-send-email-gregory.clement@free-electrons.com> <20121023090154.GP21164@n2100.arm.linux.org.uk> <508666A6.9040701@free-electrons.com> Message-ID: <20121023100057.GS21164@n2100.arm.linux.org.uk> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Oct 23, 2012 at 11:43:02AM +0200, Gregory CLEMENT wrote: > On 10/23/2012 11:01 AM, Russell King - ARM Linux wrote: > > On Thu, Sep 27, 2012 at 11:35:23AM +0200, Gregory CLEMENT wrote: > >> Aurora Cache Controller was designed to be compatible with the ARM L2 > >> Cache Controller. It comes with some difference or improvement such > >> as: > >> - no cache id part number available through hardware (need to get it > >> by the DT). > >> - always write through mode available. > >> - two flavors of the controller outer cache and system cache (meaning > >> maintenance operations on L1 are broadcasted to the L2 and L2 > >> performs the same operation). > >> - in outer cache mode, the cache maintenance operations are improved and > >> can be done on a range inside a page and are not limited to a cache > >> line. > > > > This adds new build warnings: > > > > arch/arm/mm/cache-l2x0.c:328:13: warning: 'aurora_inv_range' defined but not used > > arch/arm/mm/cache-l2x0.c:347:13: warning: 'aurora_clean_range' defined but not used > > arch/arm/mm/cache-l2x0.c:365:13: warning: 'aurora_flush_range' defined but not used > > > > to the realview build (non-DT). Please investigate, thanks. > > OK I found the problem: aurora cache support is new so it is only > enable with device tree support. I moved all this functions in the > under OF_CONFIG (see the patch below). Do you want I submit it on the > patch system, or can you get it directly from this email? Easier by the patch system, and much less likely to get buried and lost between now and when I do the update, thanks.