On Wed, 15 Oct 2008, Adrian Bunk wrote: > Commit 9be7bbd54df3c9c393ccd19acc49f90c517d1291 > (HID: build drivers for all quirky devices by default) > was wrong in that it select'ed the drivers, causing > build errors like the following with CONFIG_USB_HID=n: > <-- snip --> > > ... > LD .tmp_vmlinux1 > drivers/built-in.o: In function `bright_probe': > drivers/hid/hid-bright.c:38: undefined reference to `usbhid_set_leds' > drivers/hid/hid-bright.c:38: undefined reference to `usbhid_set_leds' > drivers/built-in.o: In function `dell_probe': > drivers/hid/hid-dell.c:41: undefined reference to `usbhid_set_leds' > drivers/hid/hid-dell.c:41: undefined reference to `usbhid_set_leds' > drivers/built-in.o: In function `lg_probe': > drivers/hid/hid-lg.c:252: undefined reference to `usbhid_set_leds' > drivers/hid/hid-lg.c:252: more undefined references to `usbhid_set_leds' follow > drivers/built-in.o: In function `sony_set_operational': > drivers/hid/hid-sony.c:42: undefined reference to `usb_control_msg' > drivers/hid/hid-sony.c:42: undefined reference to `usb_control_msg' > make[1]: *** [.tmp_vmlinux1] Error 1 Hmm ... I am not really sure, isn't this a bug in Kconfig? What we have right now: - CONFIG_HID selects all the individual quirk-drivers automatically - the individual quirk drivers have dependency on CONFIG_USB_HID Therefore I'd expect when CONFIG_HID is selected and CONFIG_USB_HID deselected the individual drivers not to get built (due to their dependency on unset option). But currently what happens is when I de-select CONFIG_USB_HID in menuconfig, the configuration options for individual drivers vanish from the menuconfig menu (which is expected, they depend on unset option), but after saving .config they are still there. Is this how it is expected to behave? (i.e. select having preference over dependency). Thanks, -- Jiri Kosina SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
