Re: [PATCH] eepc-wmi.c: Fix build error

From: Jan III Sobieski
Date: Wed Apr 07 2010 - 11:56:06 EST


2010/4/7 Ingo Molnar <mingo@xxxxxxx>:
>
> * Jan III Sobieski <jan3sobi3ski@xxxxxxxxx> wrote:
>
>> Hi,
>>
>>   GEN     .version
>>   CHK     include/generated/compile.h
>>   UPD     include/generated/compile.h
>>   CC      init/version.o
>>   LD      init/built-in.o
>>   LD      .tmp_vmlinux1
>> drivers/built-in.o: In function `eeepc_wmi_notify':
>> /home/test/linux-2.6/drivers/platform/x86/eeepc-wmi.c:86: undefined
>> reference to `sparse_keymap_report_event'
>
> -tip testing has triggered this bug too, see the fix below.

Works here. Thanks!

Tested-by: Jan III Sobieski <jan3sobi3ski@xxxxxxxxx>

>
>        Ingo
>
> --------------------------->
> From 763400ebb305711ebbd468c030db76d631400fe1 Mon Sep 17 00:00:00 2001
> From: Ingo Molnar <mingo@xxxxxxx>
> Date: Wed, 7 Apr 2010 16:15:51 +0200
> Subject: [PATCH] eepc-wmi.c: Fix build error
>
> -tip testing found:
>
> eeepc-wmi.c:(.text+0x36673c): undefined reference to `sparse_keymap_report_event'
> drivers/built-in.o: In function `eeepc_wmi_init':
> eeepc-wmi.c:(.init.text+0x19cd0): undefined reference to `sparse_keymap_setup'
> eeepc-wmi.c:(.init.text+0x19cf0): undefined reference to `sparse_keymap_free'
> eeepc-wmi.c:(.init.text+0x19d0b): undefined reference to `sparse_keymap_free'
> drivers/built-in.o: In function `eeepc_wmi_exit':
> eeepc-wmi.c:(.exit.text+0x2e87): undefined reference to `sparse_keymap_free'
>
> To fix this select INPUT_SPARSEKMAP, like the ASUS driver does.
>
> Signed-off-by: Ingo Molnar <mingo@xxxxxxx>
> ---
>  drivers/platform/x86/Kconfig |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
> index 3046776..301e379 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -392,6 +392,7 @@ config EEEPC_WMI
>        depends on ACPI_WMI
>        depends on INPUT
>        depends on EXPERIMENTAL
> +       select INPUT_SPARSEKMAP
>        ---help---
>          Say Y here if you want to support WMI-based hotkeys on Eee PC laptops.
>
>



--
Jan III Sobieski
--
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/