]> git.baikalelectronics.ru Git - kernel.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>
Wed, 19 Sep 2012 17:50:21 +0000 (19:50 +0200)
commitf08fa4e6cb28942d2bd222bc6621f3cf8ec4b678
tree42e678f056b16544a4b181a734d1f784ec70cd59
parent6a62bcc4d1160b57f07f868b5dc2784d985b0a30
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>
Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Henrik Rydberg <rydberg@euromail.se>
drivers/input/mouse/bcm5974.c