]> 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>
Sat, 15 Sep 2012 15:02:14 +0000 (17:02 +0200)
commit4164bdc63e5f52d80e2d7147802f100067a7e39c
tree42e678f056b16544a4b181a734d1f784ec70cd59
parent8b996b2e0e14aab227b9481d9c829f0098a8393e
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