Hi On Fri, May 24, 2013 at 12:46 AM, Daniel Nicoletti <[email protected]> wrote: > Ok, just tested with my two Apple mouses (Trackpad & MagicMouse), > and with the ENODATA patch it now works perfectly, the other mouse > doesn't freeze when connecting and the udev signals are also properly > emitted, is there some formal way of adding a tested-by? > > tested-by: Daniel Nicoletti <[email protected]>
Just this line is enough. Thanks! It's weird, though, that the other patch is required. With this patch here, hid_get_raw_report() should work just fine on bluetooth _and_ usb devices. And the fact that you no longer get a delay means that I/O is working properly. However, the GET_REPORT request seems to fail after device-initialization for some reason. My guess is that we send REPORT_INIT requests in hid_start() and Bluetooth devices seem to not work well with that. Hence, they send an HANDSHAKE error for each report once. This seems to fall into the same time-span where we send our first battery-request during registration and hence HIDP-core thinks the battery-request failed. @Jiri, @Marcel are you sure report-initialization is needed for BT-HIDP devices? BT-HID-profile doesn't mention it, but it relies heavily on USB-HID (which I haven't read..).. Anyway, would be nice to see both patches applied. Cheers David -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
