Hello, On Tuesday, November 17, 2009 9:10 AM Marc Zyngier wrote: > On Tue, 17 Nov 2009 08:47:54 +0100 > Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> wrote: > > > From: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > > > > Samsung S5PC1xx SoCs are Coretex8 based, so use 64 bytes of L1 cache > > line instead of the default 32 bytes. > > > > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > > Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > > --- > > arch/arm/mm/Kconfig | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/arch/arm/mm/Kconfig b/arch/arm/mm/Kconfig > > index 9264d81..7b7d4c3 100644 > > --- a/arch/arm/mm/Kconfig > > +++ b/arch/arm/mm/Kconfig > > @@ -774,5 +774,5 @@ config CACHE_XSC3L2 > > > > config ARM_L1_CACHE_SHIFT > > int > > - default 6 if ARCH_OMAP3 > > + default 6 if ARCH_OMAP3 || ARCH_S5PC1XX > > default 5 > > If this feature is related to the S5PC1xx being a CortexA8, why not > depend on CPU_V7 rather that adding each and every new CortexA8 > implementation to this list? There can be ARM v7 CPUs with different size of L1 line cache - Cortex-R4 is an example of such: http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.ddi0363d/Caccifbd.html (32 bytes of L1 cache line size). This config option should depend on particular ARM core implementation, otherwise it will be hard cover all possibilities. Best regards -- Marek Szyprowski Samsung Poland R&D Center -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html