Hi! Sorry for long delay. Just tried to build module with your patch. It doesn't build. Build log: dmig@dmig-laptop:~/workspace/linux-source-2.6.29$ LANG=C make -j3 drivers/hid/usbhid/usbhid.ko CHK include/linux/version.h CHK include/linux/utsrelease.h SYMLINK include/asm -> include/asm-x86 CALL scripts/checksyscalls.sh CC [M] drivers/hid/usbhid/hid-core.o CC [M] drivers/hid/usbhid/hid-pidff.o drivers/hid/usbhid/hid-core.c: In function 'hid_submit_out': drivers/hid/usbhid/hid-core.c:254: error: too many arguments to function 'add_timer' drivers/hid/usbhid/hid-core.c: In function 'hid_submit_ctrl': drivers/hid/usbhid/hid-core.c:308: error: too many arguments to function 'add_timer' drivers/hid/usbhid/hid-core.c: In function 'usbhid_stop': drivers/hid/usbhid/hid-core.c:934: error: 'struct usbhid_device' has no member named 'hid_out_timeout' drivers/hid/usbhid/hid-core.c:935: error: 'struct usbhid_device' has no member named 'hid_ctrl_timeout' make[1]: *** [drivers/hid/usbhid/hid-core.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [drivers/hid/usbhid/usbhid.ko] Error 2 2009/5/8 Anssi Hannula <anssi.hannula@xxxxxxxxx>: > Anssi Hannula wrote: >> Dmitriy Geels wrote: >>> 2009/3/8 Anssi Hannula <anssi.hannula@xxxxxxxxx>: >>>> Dmitriy Geels wrote: >>>>> 2009/3/7 Anssi Hannula <anssi.hannula@xxxxxxxxx>: >>>>>> Provide the kernel log again, this time with debug=2 set for hid module >>>>>> and having run "fftest" with only one effect and then "ffmvforce". >>>>> here is it: http://paste.org.ru/index.pl?4bwah8 >>>> Apply attached patch that adds more debug output and try this again. >>> Here is log: http://paste.org.ru/index.pl?0mtjuz >>> >> >> Sorry for the long delay. >> >> Try the attached (untested) patch. It adds a simple timeout to the hid >> ctrl and out urbs, discarding the faulty urb instead of getting stuck. > > And of course there was an error, attached is a fixed patch :) -- 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
