[PATCH] cell: fix ras.c compilation

[PATCH] cell: fix ras.c compilation

[Alexey Dobriyan]

arch/powerpc/platforms/cell/ras.c:299: error: implicit declaration of function 'crash_shutdown_register'

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
---

 arch/powerpc/platforms/cell/ras.c |    1 +
 1 file changed, 1 insertion(+)

--- a/arch/powerpc/platforms/cell/ras.c
+++ b/arch/powerpc/platforms/cell/ras.c
@@ -16,6 +16,7 @@
 #include <linux/kexec.h>
 #include <linux/crash_dump.h>
 
+#include <asm/kexec.h>
 #include <asm/reg.h>
 #include <asm/io.h>
 #include <asm/prom.h>

Re: [PATCH] cell: fix ras.c compilation

[Geert Uytterhoeven]

[-- Attachment #1: Type: TEXT/PLAIN, Size: 944 bytes --]

On Sun, 2 Nov 2008, Alexey Dobriyan wrote:
> arch/powerpc/platforms/cell/ras.c:299: error: implicit declaration of function 'crash_shutdown_register'
> 
> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> ---
> 
>  arch/powerpc/platforms/cell/ras.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> --- a/arch/powerpc/platforms/cell/ras.c
> +++ b/arch/powerpc/platforms/cell/ras.c
> @@ -16,6 +16,7 @@
>  #include <linux/kexec.h>
>  #include <linux/crash_dump.h>
>  
> +#include <asm/kexec.h>

#include <linux/kexec.h>?

With kind regards,

Geert Uytterhoeven
Software Architect

Sony Techsoft Centre Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium

Phone:    +32 (0)2 700 8453
Fax:      +32 (0)2 700 8622
E-mail:   Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/

A division of Sony Europe (Belgium) N.V.
VAT BE 0413.825.160 · RPR Brussels
Fortis · BIC GEBABEBB · IBAN BE41293037680010

Re: [PATCH] cell: fix ras.c compilation

[Michael Ellerman]

[-- Attachment #1: Type: text/plain, Size: 1087 bytes --]

On Mon, 2008-11-03 at 09:20 +0100, Geert Uytterhoeven wrote:
> On Sun, 2 Nov 2008, Alexey Dobriyan wrote:
> > arch/powerpc/platforms/cell/ras.c:299: error: implicit declaration of function 'crash_shutdown_register'
> > 
> > Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> > ---
> > 
> >  arch/powerpc/platforms/cell/ras.c |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > --- a/arch/powerpc/platforms/cell/ras.c
> > +++ b/arch/powerpc/platforms/cell/ras.c
> > @@ -16,6 +16,7 @@
> >  #include <linux/kexec.h>
> >  #include <linux/crash_dump.h>
> >  
> > +#include <asm/kexec.h>
> 
> #include <linux/kexec.h>?

That struck me too. But the generic header only includes the arch one if
CONFIG_KEXEC=y, and this code needs things from asm/kexec.h even when it
=n. So I think this is the best fix.

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

Re: [PATCH] cell: fix ras.c compilation

[Geert Uytterhoeven]

[-- Attachment #1: Type: TEXT/PLAIN, Size: 1597 bytes --]

On Mon, 3 Nov 2008, Michael Ellerman wrote:
> On Mon, 2008-11-03 at 09:20 +0100, Geert Uytterhoeven wrote:
> > On Sun, 2 Nov 2008, Alexey Dobriyan wrote:
> > > arch/powerpc/platforms/cell/ras.c:299: error: implicit declaration of function 'crash_shutdown_register'
> > > 
> > > Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> > > ---
> > > 
> > >  arch/powerpc/platforms/cell/ras.c |    1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > --- a/arch/powerpc/platforms/cell/ras.c
> > > +++ b/arch/powerpc/platforms/cell/ras.c
> > > @@ -16,6 +16,7 @@
> > >  #include <linux/kexec.h>
> > >  #include <linux/crash_dump.h>
> > >  
> > > +#include <asm/kexec.h>
> > 
> > #include <linux/kexec.h>?
> 
> That struck me too. But the generic header only includes the arch one if
> CONFIG_KEXEC=y, and this code needs things from asm/kexec.h even when it
> =n. So I think this is the best fix.

IC.

The real crash_shutdown_register() in arch/powerpc/kernel/crash.c only exists
if CONFIG_KEXEC. So arch/powerpc/include/asm/kexec.h has a dummy one if
!CONFIG_KEXEC.

But as you point out you don't get to see asm/kexec.h if !CONFIG_KEXEC, so the
dummy one doesn't help much...

With kind regards,

Geert Uytterhoeven
Software Architect

Sony Techsoft Centre Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium

Phone:    +32 (0)2 700 8453
Fax:      +32 (0)2 700 8622
E-mail:   Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/

A division of Sony Europe (Belgium) N.V.
VAT BE 0413.825.160 · RPR Brussels
Fortis · BIC GEBABEBB · IBAN BE41293037680010