On 8 June 2013 16:46, Tomasz Figa <tomasz.figa@xxxxxxxxx> wrote: > On Thursday 06 of June 2013 16:31:18 Chander Kashyap wrote: >> Instead of having to specify the number for CPUs in Exynos5250 in >> platsmp.c file, let the number of CPUs be determined by having this >> information listed in Exynos5250 device tree file. >> >> Signed-off-by: Chander Kashyap <chander.kashyap@xxxxxxxxxx> >> --- >> arch/arm/boot/dts/exynos5250.dtsi | 14 ++++++++++++++ >> arch/arm/mach-exynos/platsmp.c | 10 +++++++--- >> 2 files changed, 21 insertions(+), 3 deletions(-) >> >> diff --git a/arch/arm/boot/dts/exynos5250.dtsi >> b/arch/arm/boot/dts/exynos5250.dtsi index e571d3b..7f6b1c7 100644 >> --- a/arch/arm/boot/dts/exynos5250.dtsi >> +++ b/arch/arm/boot/dts/exynos5250.dtsi >> @@ -50,6 +50,20 @@ >> pinctrl3 = &pinctrl_3; >> }; >> >> + cpus { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + cpu@0 { >> + compatible = "arm,cortex-a15"; >> + reg = <0>; >> + }; >> + cpu@1 { >> + compatible = "arm,cortex-a15"; >> + reg = <1>; >> + }; >> + }; >> + >> pd_gsc: gsc-power-domain@0x10044000 { >> compatible = "samsung,exynos4210-pd"; >> reg = <0x10044000 0x20>; >> diff --git a/arch/arm/mach-exynos/platsmp.c >> b/arch/arm/mach-exynos/platsmp.c index 1a4e4e5..85ea4ca 100644 >> --- a/arch/arm/mach-exynos/platsmp.c >> +++ b/arch/arm/mach-exynos/platsmp.c >> @@ -180,10 +180,14 @@ static void __init exynos_smp_init_cpus(void) >> void __iomem *scu_base = scu_base_addr(); >> unsigned int i, ncores; >> >> - if (soc_is_exynos5250()) >> - ncores = 2; >> - else >> + if (read_cpuid_part_number() == ARM_CPU_PART_CORTEX_A9) >> ncores = scu_base ? scu_get_core_count(scu_base) : 1; >> + else >> + /* >> + * CPU Nodes are passed thru DT and set_cpu_possible >> + * is set by "arm_dt_init_cpu_maps". >> + */ >> + return; >> >> /* sanity check */ >> if (ncores > nr_cpu_ids) { > > Looks good. Same thing should be done for remaining Exynos SoCs as well. Thanks for reviewing. > > Reviewed-by: Tomasz Figa <t.figa@xxxxxxxxxxx> > > Best regards, > Tomasz > -- with warm regards, Chander Kashyap -- 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