Re: [PATCH] sony-laptop/thinkpad-acpi: fix build error

On Monday 17 September 2007 16:54, Andreas Herrmann wrote:

sony-laptop and thinkpad-acpi make use of input-core functions.
Hence you get link errors if those drivers are built but CONFIG_INPUT
is not set. Attached patch fixes this minor issue.


Regards,

Andreas

--
Build errors if CONFIG_SONY_LAPTOP && !INPUT or
if CONFIG_THINKPAD_ACPI && !INPUT:

 LD      vmlinux
    ...
drivers/built-in.o: In function `sony_laptop_remove_input':
sony-laptop.c:(.text+0x768fb): undefined reference to `input_unregister_device'
    ...
drivers/built-in.o: In function `thinkpad_acpi_module_exit':
thinkpad_acpi.c:(.text+0x78c1b): undefined reference to `input_free_device'
    ...

I suggest to auto-select CONFIG_INPUT for both drivers.

I'd prefer to use "depends on" here, as "select" is sort
of a last resort, given its inherent disfunctionalty
(is that a word?:-)

yes, I wish "select" worked the way programmers want it to,
but it doesn't.  Indeed, the only reason using it here is technically
correct is that both these invocations depends on X86 and that
avoids the conflict with !s390 INPUT.

Also, the reality is that all configs that want these drivers
will have INPUT set already -- something that probably could
not be said of BACKLIGHT_CLASS_DEVICE etc. -- so this patch
is really just to please randconfig.

Unless you object, I'll just change this to "depends on" when i apply it.

thanks,
-Len