Re: [PATCH 1/2] HID: sensor-hub add quirk for Microsoft Surface 3

From: Benjamin Tissoires
Date: Fri Nov 25 2016 - 08:00:29 EST


On Nov 25 2016 or thereabouts, Bastien Nocera wrote:
> On Fri, 2016-11-25 at 11:03 +0100, Benjamin Tissoires wrote:
> > One more device requiring a quirk :/
> >
> > Signed-off-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx>
>
> Tested-by: Bastien Nocera <hadess@xxxxxxxxxx>
>
> > ---
> > Âdrivers/hid/hid-sensor-hub.c | 3 +++
> > Â1 file changed, 3 insertions(+)
> >
> > diff --git a/drivers/hid/hid-sensor-hub.c b/drivers/hid/hid-sensor-
> > hub.c
> > index 50aad26..f9440b6 100644
> > --- a/drivers/hid/hid-sensor-hub.c
> > +++ b/drivers/hid/hid-sensor-hub.c
> > @@ -798,6 +798,9 @@ static const struct hid_device_id
> > sensor_hub_devices[] = {
> > Â { HID_DEVICE(HID_BUS_ANY, HID_GROUP_SENSOR_HUB,
> > USB_VENDOR_ID_MICROSOFT,
> > Â USB_DEVICE_ID_MS_TYPE_COVER_2),
> > Â .driver_data = HID_SENSOR_HUB_ENUM_QUIRK},
> > + { HID_DEVICE(HID_BUS_ANY, HID_GROUP_SENSOR_HUB,
> > USB_VENDOR_ID_MICROSOFT,
> > + 0x07bd), /* Microsoft Surface 3 TypeCover */
>
> The sensor is not in the TypeCover though, it's in the main device.

Oops, yes, sorry. I have been traumatised by the type cover apparently
:)

Benjamin

>
> > + .driver_data = HID_SENSOR_HUB_ENUM_QUIRK},
> > Â { HID_DEVICE(HID_BUS_ANY, HID_GROUP_SENSOR_HUB,
> > USB_VENDOR_ID_STM_0,
> > Â USB_DEVICE_ID_STM_HID_SENSOR),
> > Â .driver_data = HID_SENSOR_HUB_ENUM_QUIRK},