]> git.karo-electronics.de Git - karo-tx-linux.git/commit
Input: bcm5974 - only setup button urb for TYPE1 devices
authorHenrik Rydberg <rydberg@euromail.se>
Sun, 12 Aug 2012 22:18:44 +0000 (00:18 +0200)
committerHenrik Rydberg <rydberg@euromail.se>
Mon, 10 Sep 2012 17:59:59 +0000 (19:59 +0200)
commit5b8617d8a1f0d085856364fc276ab97c04b8f183
tree252eee252521be0680b901ce3cb54ecf136a96da
parent7be01807fc6b203b12110eb340ea9da7dfa55645
Input: bcm5974 - only setup button urb for TYPE1 devices

The early generations with this trackpad used the separate mouse
interface to produce button events. With the introduction of the
button pads, this information was moved to the trackpad interface,
leaving the mouse interface unused. The driver is still setting up
both interfaces, which has not caused any problems - until now.

It turns out that without the CONFIG_USB_EHCI_TT_NEWSCHED option, the
driver may return an ENOSPC upon bt_urb submission, resulting in a
failure to open the device. This happens everytime on the MacBookPro
Retina (and likely on other mid-2012 models), but earlier MacBook
models seem to work fine.

This patch skips the bt_urb setup for TYPE2 devices, which arguably
should have been done in the first place.

Tested-by: Benjamin Tissoires <benjamin.tissoires@enac.fr>
Signed-off-by: Henrik Rydberg <rydberg@euromail.se>
drivers/input/mouse/bcm5974.c