Re: mmotm 2008-10-16-00-52 uploaded (HID)

[Randy.Dunlap]

On Thu, 16 Oct 2008, akpm@xxxxxxxxxxxxxxxxxxxx wrote:

> The mm-of-the-moment snapshot 2008-10-16-00-52 has been uploaded to
> 
>    http://userweb.kernel.org/~akpm/mmotm/
> 
> It contains the following patches against 2.6.27:

mmotm has several various build failures when
HID=y and USB=n
or
HID=y and USB_HID=m.

These are (mostly) caused by drivers/hid/Kconfig doing lots of "select"s
when EMBEDDED=n, such as

config HID
	tristate "Generic HID support"
	depends on INPUT
	default y
        select HID_A4TECH if !EMBEDDED
        select HID_APPLE if !EMBEDDED
        select HID_BELKIN if !EMBEDDED
        select HID_BRIGHT if !EMBEDDED

and HID_BRIGHT (just to pick one of them) says:

config HID_BRIGHT
	tristate "Bright"
	default m
	depends on USB_HID

but that "depends on USB_HID" isn't being "honored" because "select"
doesn't honor (or care) about dependencies.

I think that you already know about this, but drivers/hid/Kconfig
does need fixing.


Build failure examples:

9168:
HID=y, USB=n
hid-bright.c:(.text+0x1013f0): undefined reference to `usbhid_set_leds'
hid-dell.c:(.text+0x1019bc): undefined reference to `usbhid_set_leds'
hid-lg.c:(.text+0x102310): undefined reference to `usbhid_set_leds'
hid-lgff.c:(.text+0x102509): undefined reference to `usbhid_submit_report'
hid-lgff.c:(.text+0x102566): undefined reference to `usbhid_submit_report'
hid-pl.c:(.text+0x102bbb): undefined reference to `usbhid_submit_report'
hid-pl.c:(.text+0x102ddf): undefined reference to `usbhid_submit_report'
hid-sony.c:(.text+0x103339): undefined reference to `usb_control_msg'

9170:
HID=y, USB_HID=m
hid-bright.c:(.text+0xa9d2e): undefined reference to `usbhid_set_leds'
hid-dell.c:(.text+0xaa332): undefined reference to `usbhid_set_leds'
hid-lg.c:(.text+0xaa7e8): undefined reference to `usbhid_set_leds'
hid-pl.c:(.text+0xab669): undefined reference to `usbhid_submit_report'
hid-pl.c:(.text+0xab88d): undefined reference to `usbhid_submit_report'

9174:
HID=y, USB=n
hid-bright.c:(.text+0x1b4cf2): undefined reference to `usbhid_set_leds'
hid-dell.c:(.text+0x1b52e2): undefined reference to `usbhid_set_leds'
hid-lg.c:(.text+0x1b5790): undefined reference to `usbhid_set_leds'
hid-lg2ff.c:(.text+0x1b6077): undefined reference to `usbhid_submit_report'
(.text+0x1b61d2): undefined reference to `usbhid_submit_report'
hid-sony.c:(.text+0x1b6db7): undefined reference to `usb_control_msg'

-- 
~Randy
--
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/