Skip to contentSkip to conversation

iio:hid-sensor-hub: Add in quirk for sensor hub in Lenovo Ideapad Yogas

PATCH In review Peter F. Patel-Schneider 3 participants 3 messages Mar 5, 2014 – Mar 6, 2014

Subsystems (3)
HID IIO HID sensor hub drivers
lore
Share & tools
lore

Peter F. Patel-Schneider

author
#0 lore raw

PATCH 1/1 iio:hid-sensor-hub: Add in quirk for sensor hub in Lenovo Ideapad Yogas

The sensor hub in Lenovo Yogas needs the ennumeration quirk. I've been running the patch for about a week with no problems, whereas theunpatched drivers reliably mis-initialized the sensors.

Signed-off-by Peter F. Patel-Schneider <[email protected]>
---
 drivers/hid/hid-ids.h        | 3 +++
 drivers/hid/hid-sensor-hub.c | 3 +++
 2 files changed, 6 insertions(+)
drivers/hid/hid-ids.h +3 -0
index 5a5248f..f944e37 100644
@@ -823,6 +823,9 @@
 #define USB_DEVICE_ID_SYNAPTICS_HD    0x0ac3
 #define USB_DEVICE_ID_SYNAPTICS_QUAD_HD    0x1ac3
 
+#define USB_VENDOR_ID_TEXAS_INSTRUMENTS    0x2047
+#define USB_DEVICE_ID_TEXAS_INSTRUMENTS_LENOVO_YOGA    0x0855
+
 #define USB_VENDOR_ID_THINGM        0x27b8
 #define USB_DEVICE_ID_BLINK1        0x01ed
drivers/hid/hid-sensor-hub.c +3 -0
index 46f4480..597732b 100644
@@ -665,6 +665,9 @@ static const struct hid_device_id sensor_hub_devices[] = {
     { HID_DEVICE(HID_BUS_ANY, HID_GROUP_SENSOR_HUB, USB_VENDOR_ID_INTEL_1,
             USB_DEVICE_ID_INTEL_HID_SENSOR),
             .driver_data = HID_SENSOR_HUB_ENUM_QUIRK},
+    { HID_DEVICE(HID_BUS_ANY, HID_GROUP_SENSOR_HUB, USB_VENDOR_ID_TEXAS_INSTRUMENTS,
+             USB_DEVICE_ID_TEXAS_INSTRUMENTS_LENOVO_YOGA),
+             .driver_data = HID_SENSOR_HUB_ENUM_QUIRK},
     { HID_DEVICE(HID_BUS_ANY, HID_GROUP_SENSOR_HUB, HID_ANY_ID,
              HID_ANY_ID) },
     { }
maintainer
#1 lore raw

Looks good, but need to post to linux_input mailing list for Jiri to pick up.

Thanks,
Srinivas

Quoted 45 lines · Peter F. Patel-Schneider

The sensor hub in Lenovo Yogas needs the ennumeration quirk. I've been running the patch for about a week with no problems, whereas theunpatched drivers reliably mis-initialized the sensors.

Signed-off-by Peter F. Patel-Schneider <[email protected]>
---
 drivers/hid/hid-ids.h        | 3 +++
 drivers/hid/hid-sensor-hub.c | 3 +++
 2 files changed, 6 insertions(+)
drivers/hid/hid-ids.h +3 -0
index 5a5248f..f944e37 100644
@@ -823,6 +823,9 @@
 #define USB_DEVICE_ID_SYNAPTICS_HD    0x0ac3
 #define USB_DEVICE_ID_SYNAPTICS_QUAD_HD    0x1ac3
 
+#define USB_VENDOR_ID_TEXAS_INSTRUMENTS    0x2047
+#define USB_DEVICE_ID_TEXAS_INSTRUMENTS_LENOVO_YOGA    0x0855
+
 #define USB_VENDOR_ID_THINGM        0x27b8
 #define USB_DEVICE_ID_BLINK1        0x01ed
drivers/hid/hid-sensor-hub.c +3 -0
index 46f4480..597732b 100644
@@ -665,6 +665,9 @@ static const struct hid_device_id 
 sensor_hub_devices[] = {
     { HID_DEVICE(HID_BUS_ANY, HID_GROUP_SENSOR_HUB, 
 USB_VENDOR_ID_INTEL_1,
             USB_DEVICE_ID_INTEL_HID_SENSOR),
             .driver_data = HID_SENSOR_HUB_ENUM_QUIRK},
+    { HID_DEVICE(HID_BUS_ANY, HID_GROUP_SENSOR_HUB, 
 USB_VENDOR_ID_TEXAS_INSTRUMENTS,
+             USB_DEVICE_ID_TEXAS_INSTRUMENTS_LENOVO_YOGA),
+             .driver_data = HID_SENSOR_HUB_ENUM_QUIRK},

     { HID_DEVICE(HID_BUS_ANY, HID_GROUP_SENSOR_HUB, HID_ANY_ID,
              HID_ANY_ID) },
     { }
Signature · 3 lines
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
#2 lore raw
Quoted 1 line · Srinivas Pandruvada

Looks good, but need to post to linux_input mailing list for Jiri to pick up.

Indeed, please resend so that I have non-quoted version to apply.

Also if Srinivas can provide his Ack/Signoff, that'd be helpful.

Signature · 2 lines
Jiri Kosina
SUSE Labs