]> 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)
commit94f2e71df396387e9b7f5e618ac8fe0cce653a8f
tree42e678f056b16544a4b181a734d1f784ec70cd59
parent5d26722030b4e0935f90c4a54e069413211be56f
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