On Sat, May 16, 2015 at 11:08:36AM +0200, Matthias Brugger wrote: > 2015-05-16 9:58 GMT+02:00 Yingjoe Chen <yingjoe.chen@xxxxxxxxxxxx>: > > secondary_startup_arm is used as ARM mode secondary start up function > > when ther kernel is compiled in THUMB mode, however the label itself > > is still in .thumb mode. readelf shows: > > > > 160979: c020a581 120 FUNC GLOBAL DEFAULT 2 secondary_startup_arm > > > > Make sure the label is in ARM mode as well. > > > > Signed-off-by: Yingjoe Chen <yingjoe.chen@xxxxxxxxxxxx> > > --- > > arch/arm/kernel/head.S | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/arm/kernel/head.S b/arch/arm/kernel/head.S > > index 3637973..58ee8a2 100644 > > --- a/arch/arm/kernel/head.S > > +++ b/arch/arm/kernel/head.S > > @@ -346,8 +346,8 @@ __turn_mmu_on_loc: > > > > #if defined(CONFIG_SMP) > > .text > > -ENTRY(secondary_startup_arm) > > .arm > > +ENTRY(secondary_startup_arm) > > THUMB( adr r9, BSYM(1f) ) @ Kernel is entered in ARM. > > THUMB( bx r9 ) @ If this is a Thumb-2 kernel, > > THUMB( .thumb ) @ switch to Thumb now. > > This fixes and issue we had with SMP on mt6589. > > Tested-by: Matthias Brugger <matthias.bgg@xxxxxxxxx> Please ensure that this patch ends up in the patch system so it doesn't get forgotten about, thanks. -- FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up according to speedtest.net. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
