USB: serial: metro-usb: add MS7820 device id

PATCH In review Johan Hovold 2 participants 3 messages Oct 12, 2017 1 message not in the archive

Subsystems (3)

Stable branch USB USB serial

lore

Share & tools

lore

Johan Hovold

author maintainer

2017-10-12 12:52 UTC #0 lore raw

PATCH 1/1 USB: serial: metro-usb: add MS7820 device id

Add device-id entry for (Honeywell) Metrologic MS7820 bar code scanner.

The device has two interfaces (in this mode?); a vendor-specific interface with two interrupt endpoints and a second HID interface, which we do not bind to.

Reported-by Ladislav Dobrovsky <[email protected]>

Cc stable <[email protected]>

Signed-off-by Johan Hovold <[email protected]>

---
 drivers/usb/serial/metro-usb.c | 1 +
 1 file changed, 1 insertion(+)

drivers/usb/serial/metro-usb.c +1 -0

index cc84da8dbb84..14511d6a7d44 100644
@@ -45,6 +45,7 @@ struct metrousb_private {
 static const struct usb_device_id id_table[] = {
 	{ USB_DEVICE(FOCUS_VENDOR_ID, FOCUS_PRODUCT_ID_BI) },
 	{ USB_DEVICE(FOCUS_VENDOR_ID, FOCUS_PRODUCT_ID_UNI) },
+	{ USB_DEVICE_INTERFACE_CLASS(0x0c2e, 0x0730, 0xff) },	/* MS7820 */
 	{ }, /* Terminating entry. */
 };
 MODULE_DEVICE_TABLE(usb, id_table);

Signature · 1 line

2.14.2

A message that is not in the archive (sent off-list or not yet received): CAGFbA8XdUFV1Lkp7kLLPaELjgCOPJ2UbrQUi-xtzc5F7HaZa-A@mail.gmail.com

Ladislav Dobrovsky

Tested-by

2017-10-12 14:36 UTC #2 lore raw

Tested-by Ladislav Dobrovsky <[email protected]>

Quoted 32 lines

2017-10-12 14:50 GMT+02:00 Johan Hovold <[email protected]>:
Quoted 29 lines
Add device-id entry for (Honeywell) Metrologic MS7820 bar code scanner.
The device has two interfaces (in this mode?); a vendor-specific interface with two interrupt endpoints and a second HID interface, which we do not bind to.
Reported-by Ladislav Dobrovsky <[email protected]>
Cc stable <[email protected]>
Signed-off-by Johan Hovold <[email protected]>
---
 drivers/usb/serial/metro-usb.c | 1 +
 1 file changed, 1 insertion(+)
drivers/usb/serial/metro-usb.c +1 -1
index cc84da8dbb84..14511d6a7d44 100644
@@ -45,6 +45,7 @@ struct metrousb_private {
 static const struct usb_device_id id_table[] = {
        { USB_DEVICE(FOCUS_VENDOR_ID, FOCUS_PRODUCT_ID_BI) },
        { USB_DEVICE(FOCUS_VENDOR_ID, FOCUS_PRODUCT_ID_UNI) },
+       { USB_DEVICE_INTERFACE_CLASS(0x0c2e, 0x0730, 0xff) },   /* MS7820 */
        { }, /* Terminating entry. */
 };
 MODULE_DEVICE_TABLE(usb, id_table);
--
2.14.2

Johan Hovold

author maintainer

2017-10-12 15:24 UTC #3 lore raw

In reply to a message not in the archive

Quoted 11 lines · Ladislav Dobrovsky

Tested-by Ladislav Dobrovsky <[email protected]>
2017-10-12 14:50 GMT+02:00 Johan Hovold <[email protected]>:
Quoted 7 lines
Add device-id entry for (Honeywell) Metrologic MS7820 bar code scanner.
The device has two interfaces (in this mode?); a vendor-specific interface with two interrupt endpoints and a second HID interface, which we do not bind to.
Reported-by Ladislav Dobrovsky <[email protected]>

Thanks for reporting and testing this.

Johan