Subject: [PATCHv2] arm:socfpga: Enable SMP for socfpga

From: [email-masked] (Rob Herring)
Date: Wed, 24 Oct 2012 18:01:28 -0500

On 10/18/2012 12:32 PM, dinguyen at altera.com wrote:
> From: Dinh Nguyen <[email-masked]>
> 
> Enable SMP for the SOCFPGA platform.
> 
> Signed-off-by: Pavel Machek <[email-masked]>
> Signed-off-by: Dinh Nguyen <[email-masked]>
> ---
> v2:
> -Remove pen_release code
> -Remove code that was already done by v7_setup
> -Add bindings document for reset and system manager
> -Move socfpga_sysmgr_init from platsmp.c to socfpga.c, because
> we will need to use the reset and system manager for more than SMP.
> -Move core.h to mach-socfpga from mach-socfpga/include/mach

Just some lingering comments on the defconfig. Otherwise,

Reviewed-by: Rob Herring <[email-masked]>

> diff --git a/arch/arm/configs/socfpga_defconfig b/arch/arm/configs/socfpga_defconfig
> index 0ac1293..349ac22 100644
> --- a/arch/arm/configs/socfpga_defconfig
> +++ b/arch/arm/configs/socfpga_defconfig

I'm still not clear why multi_v7 config does not work for you?

> @@ -1,5 +1,5 @@
>  CONFIG_EXPERIMENTAL=y
> -CONFIG_SYSVIPC=y
> +CONFIG_NO_HZ=y
>  CONFIG_IKCONFIG=y
>  CONFIG_IKCONFIG_PROC=y
>  CONFIG_LOG_BUF_SHIFT=14
> @@ -16,10 +16,13 @@ CONFIG_MODULE_UNLOAD=y
>  # CONFIG_IOSCHED_DEADLINE is not set
>  # CONFIG_IOSCHED_CFQ is not set
>  CONFIG_ARCH_SOCFPGA=y
> -CONFIG_MACH_SOCFPGA_CYCLONE5=y
> -CONFIG_ARM_THUMBEE=y
> +# CONFIG_ARCH_VEXPRESS_CORTEX_A5_A9_ERRATA is not set
>  # CONFIG_CACHE_L2X0 is not set
>  CONFIG_HIGH_RES_TIMERS=y
> +CONFIG_SMP=y
> +CONFIG_ARM_ARCH_TIMER=y

You're an A9, right? You don't have arch timers.

> +CONFIG_HIGHMEM=y

How much RAM? You need more than 2GB with CONFIG_VMSPLIT_2G.

> +CONFIG_HIGHPTE=y
>  CONFIG_VMSPLIT_2G=y
>  CONFIG_NR_CPUS=2
>  CONFIG_AEABI=y