Re: [PATCH 1/1] x86: hyperv: Fix a compilation warning

From: H. Peter Anvin
Date: Fri Oct 11 2013 - 01:52:22 EST


No real reason to do this.

"K. Y. Srinivasan" <kys@xxxxxxxxxxxxx> wrote:
>The variable hv_lapic_frequency is only used when CONFIG_X86_LOCAL_APIC
>is defined. Make the necessary adjustment.
>
>Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
>---
> arch/x86/kernel/cpu/mshyperv.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/arch/x86/kernel/cpu/mshyperv.c
>b/arch/x86/kernel/cpu/mshyperv.c
>index 628ff50..d7ff7b8 100644
>--- a/arch/x86/kernel/cpu/mshyperv.c
>+++ b/arch/x86/kernel/cpu/mshyperv.c
>@@ -69,7 +69,6 @@ static struct clocksource hyperv_cs = {
>
> static void __init ms_hyperv_init_platform(void)
> {
>- u64 hv_lapic_frequency;
>
> /*
> * Extract the features and hints
>@@ -82,6 +81,7 @@ static void __init ms_hyperv_init_platform(void)
>
> #ifdef CONFIG_X86_LOCAL_APIC
> if (ms_hyperv.features & HV_X64_MSR_APIC_FREQUENCY_AVAILABLE) {
>+ u64 hv_lapic_frequency;
> /*
> * Get the APIC frequency.
> */

--
Sent from my mobile phone. Please pardon brevity and lack of formatting.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/