diff mbox

Fix shmobile build with CONFIG_SMP=n

Message ID 20160719115703.GJ1041@n2100.armlinux.org.uk (mailing list archive)
State New, archived
Headers show

Commit Message

Russell King (Oracle) July 19, 2016, 11:57 a.m. UTC
Shmobile currently fails to build when CONFIG_SMP is disabled:

arch/arm/mach-shmobile/built-in.o: In function `shmobile_smp_init_fallback_ops':
pm-rcar-gen2.c:(.init.text+0x744): undefined reference to `platform_can_secondary_boot'
make[1]: *** [vmlinux] Error 1

Fix this by providing a stub for this function.

Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
---
Untested, please test.

 arch/arm/mach-shmobile/common.h  | 7 +++++++
 arch/arm/mach-shmobile/platsmp.c | 2 ++
 2 files changed, 9 insertions(+)

Comments

Geert Uytterhoeven July 19, 2016, 12:02 p.m. UTC | #1
Hi Russell,

On Tue, Jul 19, 2016 at 1:57 PM, Russell King - ARM Linux
<linux@armlinux.org.uk> wrote:
> Shmobile currently fails to build when CONFIG_SMP is disabled:
>
> arch/arm/mach-shmobile/built-in.o: In function `shmobile_smp_init_fallback_ops':
> pm-rcar-gen2.c:(.init.text+0x744): undefined reference to `platform_can_secondary_boot'
> make[1]: *** [vmlinux] Error 1
>
> Fix this by providing a stub for this function.
>
> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>

Arnd send a smaller one https://patchwork.kernel.org/patch/9207413/,
which seems have fallen through the cracks.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Simon Horman July 20, 2016, 11:14 p.m. UTC | #2
On Tue, Jul 19, 2016 at 02:02:53PM +0200, Geert Uytterhoeven wrote:
> Hi Russell,
> 
> On Tue, Jul 19, 2016 at 1:57 PM, Russell King - ARM Linux
> <linux@armlinux.org.uk> wrote:
> > Shmobile currently fails to build when CONFIG_SMP is disabled:
> >
> > arch/arm/mach-shmobile/built-in.o: In function `shmobile_smp_init_fallback_ops':
> > pm-rcar-gen2.c:(.init.text+0x744): undefined reference to `platform_can_secondary_boot'
> > make[1]: *** [vmlinux] Error 1
> >
> > Fix this by providing a stub for this function.
> >
> > Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
> 
> Arnd send a smaller one https://patchwork.kernel.org/patch/9207413/,
> which seems have fallen through the cracks.

Sorry for missing that. I have queued up Arnd's patch as a fix for v4.8
with Acked-by Geert and a Reported-by Russel tags.
Olof Johansson Aug. 2, 2016, 7:35 p.m. UTC | #3
Hi,

On Thu, Jul 21, 2016 at 08:14:17AM +0900, Simon Horman wrote:
> On Tue, Jul 19, 2016 at 02:02:53PM +0200, Geert Uytterhoeven wrote:
> > Hi Russell,
> > 
> > On Tue, Jul 19, 2016 at 1:57 PM, Russell King - ARM Linux
> > <linux@armlinux.org.uk> wrote:
> > > Shmobile currently fails to build when CONFIG_SMP is disabled:
> > >
> > > arch/arm/mach-shmobile/built-in.o: In function `shmobile_smp_init_fallback_ops':
> > > pm-rcar-gen2.c:(.init.text+0x744): undefined reference to `platform_can_secondary_boot'
> > > make[1]: *** [vmlinux] Error 1
> > >
> > > Fix this by providing a stub for this function.
> > >
> > > Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
> > 
> > Arnd send a smaller one https://patchwork.kernel.org/patch/9207413/,
> > which seems have fallen through the cracks.
> 
> Sorry for missing that. I have queued up Arnd's patch as a fix for v4.8
> with Acked-by Geert and a Reported-by Russel tags.

Since I haven't seen this patch yet, I've applied it (Arnd's version) directly
to our fixes branch.


-Olof
Simon Horman Aug. 8, 2016, 11:01 a.m. UTC | #4
On Tue, Aug 02, 2016 at 12:35:24PM -0700, Olof Johansson wrote:
> Hi,
> 
> On Thu, Jul 21, 2016 at 08:14:17AM +0900, Simon Horman wrote:
> > On Tue, Jul 19, 2016 at 02:02:53PM +0200, Geert Uytterhoeven wrote:
> > > Hi Russell,
> > > 
> > > On Tue, Jul 19, 2016 at 1:57 PM, Russell King - ARM Linux
> > > <linux@armlinux.org.uk> wrote:
> > > > Shmobile currently fails to build when CONFIG_SMP is disabled:
> > > >
> > > > arch/arm/mach-shmobile/built-in.o: In function `shmobile_smp_init_fallback_ops':
> > > > pm-rcar-gen2.c:(.init.text+0x744): undefined reference to `platform_can_secondary_boot'
> > > > make[1]: *** [vmlinux] Error 1
> > > >
> > > > Fix this by providing a stub for this function.
> > > >
> > > > Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
> > > 
> > > Arnd send a smaller one https://patchwork.kernel.org/patch/9207413/,
> > > which seems have fallen through the cracks.
> > 
> > Sorry for missing that. I have queued up Arnd's patch as a fix for v4.8
> > with Acked-by Geert and a Reported-by Russel tags.
> 
> Since I haven't seen this patch yet, I've applied it (Arnd's version) directly
> to our fixes branch.

Thanks. The progression through my tree got delayed by me taking
a holiday last week. I'll drop it from my tree as you have picked it up.
diff mbox

Patch

diff --git a/arch/arm/mach-shmobile/common.h b/arch/arm/mach-shmobile/common.h
index 1a8f7b3ab449..6badb3e41cfa 100644
--- a/arch/arm/mach-shmobile/common.h
+++ b/arch/arm/mach-shmobile/common.h
@@ -10,7 +10,14 @@  extern void shmobile_smp_sleep(void);
 extern void shmobile_smp_hook(unsigned int cpu, unsigned long fn,
 			      unsigned long arg);
 extern bool shmobile_smp_cpu_can_disable(unsigned int cpu);
+#ifdef CONFIG_SMP
 extern bool shmobile_smp_init_fallback_ops(void);
+#else
+static inline bool shmobile_smp_init_fallback_ops(void)
+{
+	return false;
+}
+#endif
 extern void shmobile_boot_scu(void);
 extern void shmobile_smp_scu_prepare_cpus(phys_addr_t scu_base_phys,
 					  unsigned int max_cpus);
diff --git a/arch/arm/mach-shmobile/platsmp.c b/arch/arm/mach-shmobile/platsmp.c
index f3dba6f356e2..44d91f5624a3 100644
--- a/arch/arm/mach-shmobile/platsmp.c
+++ b/arch/arm/mach-shmobile/platsmp.c
@@ -37,8 +37,10 @@  bool shmobile_smp_cpu_can_disable(unsigned int cpu)
 }
 #endif
 
+#ifdef CONFIG_SMP
 bool __init shmobile_smp_init_fallback_ops(void)
 {
 	/* fallback on PSCI/smp_ops if no other DT based method is detected */
 	return platform_can_secondary_boot() ? true : false;
 }
+#endif