Re: [RESEND PATCH 2/4] ARM: msm: Re-organize platsmp to make it extensible

2013-08-20 Thread David Rientjes
On Mon, 12 Aug 2013, Mark Rutland wrote: static void __init msm_smp_prepare_cpus(unsigned int max_cpus) { + int cpu, map; + unsigned int flags = 0; + + for_each_present_cpu(cpu) { + map = cpu_logical_map(cpu); + if (map ARRAY_SIZE(cold_boot_flags)) {

Re: [RESEND PATCH 2/4] ARM: msm: Re-organize platsmp to make it extensible

2013-08-16 Thread Mark Rutland
On Wed, Aug 14, 2013 at 09:55:31PM +0100, Rohit Vaswani wrote: On 8/12/2013 8:50 AM, Mark Rutland wrote: Hi, Apologies for the long delay for review on this. I really like the direction this is going, but I have some qualms with the implementation. Thanks for your review, but a few

Re: [RESEND PATCH 2/4] ARM: msm: Re-organize platsmp to make it extensible

2013-08-14 Thread Rohit Vaswani
On 8/12/2013 8:50 AM, Mark Rutland wrote: Hi, Apologies for the long delay for review on this. I really like the direction this is going, but I have some qualms with the implementation. Thanks for your review, but a few direct recommendations would help in making the implementation right.

Re: [RESEND PATCH 2/4] ARM: msm: Re-organize platsmp to make it extensible

2013-08-12 Thread Mark Rutland
Hi, Apologies for the long delay for review on this. I really like the direction this is going, but I have some qualms with the implementation. On Fri, Aug 02, 2013 at 03:15:23AM +0100, Rohit Vaswani wrote: This makes it easy to add SMP support for new targets by adding cpus property and the

[RESEND PATCH 2/4] ARM: msm: Re-organize platsmp to make it extensible

2013-08-01 Thread Rohit Vaswani
This makes it easy to add SMP support for new targets by adding cpus property and the release sequence. We add the enable-method property for the cpus property to specify which release sequence to use. While at it, add the 8660 cpus bindings to make SMP work. Signed-off-by: Rohit Vaswani