Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756522Ab3DWXqR (ORCPT ); Tue, 23 Apr 2013 19:46:17 -0400 Received: from co9ehsobe001.messaging.microsoft.com ([207.46.163.24]:47243 "EHLO co9outboundpool.messaging.microsoft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755970Ab3DWXqQ convert rfc822-to-8bit (ORCPT ); Tue, 23 Apr 2013 19:46:16 -0400 X-Forefront-Antispam-Report: CIP:70.37.183.190;KIP:(null);UIP:(null);IPV:NLI;H:mail.freescale.net;RD:none;EFVD:NLI X-SpamScore: -1 X-BigFish: VS-1(zcb8kzbb2dI98dI9371I1432Izz1f42h1fc6h1ee6h1de0h1fdah1202h1e76h1d1ah1d2ahzz8275bhz2dh2a8h668h839h944hd2bhf0ah1288h12a5h12a9h12bdh137ah139eh13b6h1441h1504h1537h162dh1631h16a6h1758h1898h18e1h1946h19b5h1ad9h1b0ah1155h) Date: Tue, 23 Apr 2013 18:46:10 -0500 From: Scott Wood Subject: Re: [PATCH v2 01/15] powerpc/85xx: cache operations for Freescale SoCs based on BOOK3E To: Zhao Chenhui CC: , In-Reply-To: <1366368468-29143-1-git-send-email-chenhui.zhao@freescale.com> (from chenhui.zhao@freescale.com on Fri Apr 19 05:47:34 2013) X-Mailer: Balsa 2.4.12 Message-ID: <1366760770.5825.17@snotra> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; delsp=Yes; format=Flowed Content-Disposition: inline Content-Transfer-Encoding: 8BIT X-OriginatorOrg: freescale.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2286 Lines: 57 On 04/19/2013 05:47:34 AM, Zhao Chenhui wrote: > These cache operations support Freescale SoCs based on BOOK3E. > Move L1 cache operations to fsl_booke_cache.S in order to maintain > easily. And, add cache operations for backside L2 cache and platform > cache. > > The backside L2 cache appears on e500mc and e5500 core. The platform > cache > supported by this patch is L2 Look-Aside Cache, which appears on SoCs > with e500v1/e500v2 core, such as MPC8572, P1020, etc. > > Signed-off-by: Zhao Chenhui > Signed-off-by: Li Yang > --- > arch/powerpc/include/asm/cacheflush.h | 8 ++ > arch/powerpc/kernel/Makefile | 1 + > arch/powerpc/kernel/fsl_booke_cache.S | 210 > +++++++++++++++++++++++++++++++++ > arch/powerpc/kernel/head_fsl_booke.S | 74 ------------ > 4 files changed, 219 insertions(+), 74 deletions(-) > create mode 100644 arch/powerpc/kernel/fsl_booke_cache.S > > diff --git a/arch/powerpc/include/asm/cacheflush.h > b/arch/powerpc/include/asm/cacheflush.h > index b843e35..bc3f937 100644 > --- a/arch/powerpc/include/asm/cacheflush.h > +++ b/arch/powerpc/include/asm/cacheflush.h > @@ -32,6 +32,14 @@ extern void flush_dcache_page(struct page *page); > > extern void __flush_disable_L1(void); > > +#ifdef CONFIG_FSL_SOC_BOOKE > +void flush_dcache_L1(void); > +void flush_backside_L2_cache(void); > +void disable_backside_L2_cache(void); > +void flush_disable_L2(void); > +void invalidate_enable_L2(void); > +#endif Don't ifdef prototypes unless there's a good reason, such as providing an inline alternative. Why do you have "flush_backside_L2_cache" and "disable_backside_L2_cache" as something different from "flush_disable_L2"? The latter should flush whatever L2 is present. Don't treat pre-corenet as the default. Why do we even need to distinguish L1 from L2 at all? Shouldn't the function that gets exposed just be "flush and disable data caches that are specific to this cpu"? What should happen on e6500? -Scott -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/