On 15 February 2011 07:14, Santosh Shilimkar <santosh.shilimkar@xxxxxx> wrote:
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1140,7 +1140,7 @@ config ARM_ERRATA_742231
>
> Âconfig PL310_ERRATA_588369
> Â Â Â Âbool "Clean & Invalidate maintenance operations do not invalidate
> clean lines"
> - Â Â Â depends on CACHE_L2X0 && ARCH_OMAP4
> + Â Â Â depends on CACHE_L2X0 && CACHE_PL310
It can just depend on CACHE_PL310 as this depends on CACHE_L2X0.
> +config PL310_ERRATA_727915
> + Â Â Â bool "Background Clean & Invalidate by Way operation can cause
> data corruption"
> + Â Â Â depends on CACHE_L2X0 && CACHE_PL310
Same here.
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -45,7 +45,10 @@ config ARCH_OMAP4
> Â Â Â Âselect CPU_V7
> Â Â Â Âselect ARM_GIC
> Â Â Â Âselect LOCAL_TIMERS
> + Â Â Â select CACHE_L2X0
CACHE_L2X0 has a long dependency list. You could add ARCH_OMAP4 in
there or just change the other platforms to select a HAVE_CACHE_L2X0.
Ideally we would like this option to be selectable in config just in
case you want to debug some issues.
> --- a/arch/arm/mach-omap2/omap4-common.c
> +++ b/arch/arm/mach-omap2/omap4-common.c
> @@ -52,6 +52,12 @@ static void omap4_l2x0_disable(void)
> Â Â Â Âomap_smc1(0x102, 0x0);
> Â}
>
> +static void omap4_l2x0_set_debug(unsigned long val)
> +{
> + Â Â Â /* Program PL310 L2 Cache controller debug register */
> + Â Â Â omap_smc1(0x100, val);
> +}
This part together with the Kconfig changes for OMAP4 could be a
separate patch, OMAP-specific.
The rest seems fine.
--
Catalin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
