Re: [PATCH] 2.6.19-rc1: Fix build breakage with CONFIG_PPC32

From: Srinivasa Ds
Date: Wed Oct 18 2006 - 07:45:23 EST


Paul Mackerras wrote:
Srinivasa Ds writes:

arch/powerpc/platforms/built-in.o: In function `flush_disable_caches':
(.text+0x96d4): undefined reference to `low_cpu_die'
======================================================
low_cpu_die() is defined under CONFIG_PM || CONFIG_CPU_FREQ_PMAC options ,but while calling this function ,no care has been to taken to check these options. So please apply this fix,which solves the problem.

Nack. The correct fix is to adjust the ifdefs in sleep.S to make
low_cpu_die available. Otherwise it won't be possible to off-line
CPUs properly.
So resending the patch, by adjusting ifdefs in sleep.S
Signed-off-by: Srinivasa DS <srinivasa@xxxxxxxxxx>

Paul.

---
arch/powerpc/platforms/powermac/sleep.S | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux-2.6.19-rc2/arch/powerpc/platforms/powermac/sleep.S
===================================================================
--- linux-2.6.19-rc2.orig/arch/powerpc/platforms/powermac/sleep.S
+++ linux-2.6.19-rc2/arch/powerpc/platforms/powermac/sleep.S
@@ -45,7 +45,7 @@
.section .text
.align 5

-#if defined(CONFIG_PM) || defined(CONFIG_CPU_FREQ_PMAC)
+#if defined(CONFIG_PM) || defined(CONFIG_CPU_FREQ_PMAC) || (defined(CONFIG_HOTPLUG_CPU) && defined(CONFIG_PPC32))

/* This gets called by via-pmu.c late during the sleep process.
* The PMU was already send the sleep command and will shut us down