[PATCH 1/2] ARM: mach-imx: convert logical CPU numbers to physical numbers

[Will Deacon]

On Fri, Nov 11, 2011 at 01:07:02PM +0000, Shawn Guo wrote:
> On Fri, Nov 11, 2011 at 10:56:07AM +0000, Will Deacon wrote:
> > On Fri, Nov 11, 2011 at 10:08:00AM +0000, Will Deacon wrote:
> > > On Fri, Nov 11, 2011 at 09:29:25AM +0000, Sascha Hauer wrote:
> > > > On Tue, Nov 08, 2011 at 02:34:53PM +0000, Will Deacon wrote:
> > > > > This patch uses the new cpu_logical_map() macro for converting logical
> > > > > CPU numbers into physical numbers when releasing CPUs during the SMP
> > > > > boot and CPU hotplug paths.
> > > > 
> > > > Shawn, is this one ok?
> > > 
> > > I reckon I need to post a v2 because this probably breaks when !SMP.
> > 
> > Actually, I take that back. Looks like the function is only used by
> > hotplug.c and platsmp.c, both of which depend on CONFIG_SMP.
> > 
> But the build of src.c does not depend on CONFIG_SMP.  So when building
> with no CONFIG_SMP, we get:
> 
>   CC      arch/arm/mach-imx/src.o
> arch/arm/mach-imx/src.c: In function ‘imx_enable_cpu’:
> arch/arm/mach-imx/src.c:31:2: error: implicit declaration of function ‘cpu_logical_map’
> make[2]: *** [arch/arm/mach-imx/src.o] Error 1
> make[1]: *** [arch/arm/mach-imx] Error 2

Ok, then maybe we should #ifdef CONFIG_SMP that function entirely. If that
sounds ok, I can post a v2.

Will