mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] x86, mpparse: Move check_slot into CONFIG_X86_IO_APIC context.
@ 2011-03-22  9:24 Rakib Mullick
  2011-03-22 15:36 ` [tip:x86/urgent] " tip-bot for Rakib Mullick
  0 siblings, 1 reply; 2+ messages in thread
From: Rakib Mullick @ 2011-03-22  9:24 UTC (permalink / raw)
  To: Ingo Molnar; +Cc: Thomas Gleixner, H. Peter Anvin, x86, LKML

When CONFIG_X86_MPPARSE=y and CONFIG_X86_IO_APIC=n, then we get the
following warning:

arch/x86/kernel/mpparse.c:723: warning: ‘check_slot’ defined but not used

So, put check_slot into CONFIG_X86_IO_APIC context. Its only called
from CONFIG_X86_IO_APIC=y context.

Signed-off-by: Rakib Mullick <rakib.mullick@gmail.com>
---

diff --git a/arch/x86/kernel/mpparse.c b/arch/x86/kernel/mpparse.c
index 6f789a8..5a532ce 100644
--- a/arch/x86/kernel/mpparse.c
+++ b/arch/x86/kernel/mpparse.c
@@ -714,10 +714,6 @@ static void __init check_irq_src(struct
mpc_intsrc *m, int *nr_m_spare)
 		*nr_m_spare += 1;
 	}
 }
-#else /* CONFIG_X86_IO_APIC */
-static
-inline void __init check_irq_src(struct mpc_intsrc *m, int *nr_m_spare) {}
-#endif /* CONFIG_X86_IO_APIC */

 static int
 check_slot(unsigned long mpc_new_phys, unsigned long mpc_new_length, int count)
@@ -731,6 +727,10 @@ check_slot(unsigned long mpc_new_phys, unsigned
long mpc_new_length, int count)

 	return ret;
 }
+#else /* CONFIG_X86_IO_APIC */
+static
+inline void __init check_irq_src(struct mpc_intsrc *m, int *nr_m_spare) {}
+#endif /* CONFIG_X86_IO_APIC */

 static int  __init replace_intsrc_all(struct mpc_table *mpc,
 					unsigned long mpc_new_phys,

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-03-22 15:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-22  9:24 [PATCH] x86, mpparse: Move check_slot into CONFIG_X86_IO_APIC context Rakib Mullick
2011-03-22 15:36 ` [tip:x86/urgent] " tip-bot for Rakib Mullick

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome