diff mbox

powerpc/fsl_rio: Fix compile error when CONFIG_FSL_RIO not set

Message ID 1307031956-11686-1-git-send-email-galak@kernel.crashing.org (mailing list archive)
State Accepted, archived
Delegated to: Kumar Gala
Headers show

Commit Message

Kumar Gala June 2, 2011, 4:25 p.m. UTC
arch/powerpc/kernel/built-in.o: In function `machine_check_e500mc':
arch/powerpc/kernel/traps.c:429: undefined reference to `fsl_rio_mcheck_exception'
arch/powerpc/kernel/built-in.o: In function `machine_check_e500': arch/powerpc/kernel/traps.c:519:
undefined reference to `fsl_rio_mcheck_exception'
make: *** [.tmp_vmlinux1] Error 1

Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
---
 arch/powerpc/include/asm/rio.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Josh Boyer June 2, 2011, 5:32 p.m. UTC | #1
On Thu, Jun 2, 2011 at 12:25 PM, Kumar Gala <galak@kernel.crashing.org> wrote:
> arch/powerpc/kernel/built-in.o: In function `machine_check_e500mc':
> arch/powerpc/kernel/traps.c:429: undefined reference to `fsl_rio_mcheck_exception'
> arch/powerpc/kernel/built-in.o: In function `machine_check_e500': arch/powerpc/kernel/traps.c:519:
> undefined reference to `fsl_rio_mcheck_exception'
> make: *** [.tmp_vmlinux1] Error 1
>
> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>

Should be Reported-by Timur, right?

josh
Tabi Timur-B04825 June 2, 2011, 7:11 p.m. UTC | #2
On Thu, Jun 2, 2011 at 12:32 PM, Josh Boyer <jwboyer@linux.vnet.ibm.com> wrote:
> Should be Reported-by Timur, right?

Technically, yes.  For the record, I don't care about these things.
It implies that only willing to report the problem, but too lazy to
fix it; which is true, but I'd hate to see that fact permanently
etched in the git history. :-)
Josh Boyer June 2, 2011, 7:22 p.m. UTC | #3
On Thu, Jun 02, 2011 at 07:11:18PM +0000, Tabi Timur-B04825 wrote:
>On Thu, Jun 2, 2011 at 12:32 PM, Josh Boyer <jwboyer@linux.vnet.ibm.com> wrote:
>> Should be Reported-by Timur, right?
>
>Technically, yes.  For the record, I don't care about these things.
>It implies that only willing to report the problem, but too lazy to
>fix it; which is true, but I'd hate to see that fact permanently
>etched in the git history. :-)

So for you, that might hold true.  (You said it, not me.)  But for some,
they can report compile errors but might not have the technical ability
to fix it.  They're still learning, or they are using randconfig and
don't understand the arch/interaction, etc.  The tag has value.

josh
diff mbox

Patch

diff --git a/arch/powerpc/include/asm/rio.h b/arch/powerpc/include/asm/rio.h
index d902abd..b1d2dec 100644
--- a/arch/powerpc/include/asm/rio.h
+++ b/arch/powerpc/include/asm/rio.h
@@ -14,7 +14,7 @@ 
 #define ASM_PPC_RIO_H
 
 extern void platform_rio_init(void);
-#ifdef CONFIG_RAPIDIO
+#ifdef CONFIG_FSL_RIO
 extern int fsl_rio_mcheck_exception(struct pt_regs *);
 #else
 static inline int fsl_rio_mcheck_exception(struct pt_regs *regs) {return 0; }