Commit f2e1fc9d authored by Andrew Kephart's avatar Andrew Kephart Committed by Florian Tobias Schandinat

udlfb: Search config descriptor if not at device level

For at least one DisplayLink device, the vendor-specific information can
be found in the config descriptor instead of as a separate, device-level
descriptor.  This patch searches the current interface (of the current
config descriptor) for the DL vendor-specific descriptor.
Signed-off-by: default avatarAndrew Kephart <akephart@akephart.org>
Signed-off-by: default avatarBernie Thompson <bernie@plugable.com>
Signed-off-by: default avatarFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>
parent 58e7c3b0
...@@ -1479,21 +1479,30 @@ static int dlfb_select_std_channel(struct dlfb_data *dev) ...@@ -1479,21 +1479,30 @@ static int dlfb_select_std_channel(struct dlfb_data *dev)
} }
static int dlfb_parse_vendor_descriptor(struct dlfb_data *dev, static int dlfb_parse_vendor_descriptor(struct dlfb_data *dev,
struct usb_device *usbdev) struct usb_interface *interface)
{ {
char *desc; char *desc;
char *buf; char *buf;
char *desc_end; char *desc_end;
u8 total_len = 0; int total_len = 0;
buf = kzalloc(MAX_VENDOR_DESCRIPTOR_SIZE, GFP_KERNEL); buf = kzalloc(MAX_VENDOR_DESCRIPTOR_SIZE, GFP_KERNEL);
if (!buf) if (!buf)
return false; return false;
desc = buf; desc = buf;
total_len = usb_get_descriptor(usbdev, 0x5f, /* vendor specific */ total_len = usb_get_descriptor(interface_to_usbdev(interface),
0, desc, MAX_VENDOR_DESCRIPTOR_SIZE); 0x5f, /* vendor specific */
0, desc, MAX_VENDOR_DESCRIPTOR_SIZE);
/* if not found, look in configuration descriptor */
if (total_len < 0) {
if (0 == usb_get_extra_descriptor(interface->cur_altsetting,
0x5f, &desc))
total_len = (int) desc[0];
}
if (total_len > 5) { if (total_len > 5) {
pr_info("vendor descriptor length:%x data:%02x %02x %02x %02x" \ pr_info("vendor descriptor length:%x data:%02x %02x %02x %02x" \
"%02x %02x %02x %02x %02x %02x %02x\n", "%02x %02x %02x %02x %02x %02x %02x\n",
...@@ -1534,6 +1543,8 @@ static int dlfb_parse_vendor_descriptor(struct dlfb_data *dev, ...@@ -1534,6 +1543,8 @@ static int dlfb_parse_vendor_descriptor(struct dlfb_data *dev,
} }
desc += length; desc += length;
} }
} else {
pr_info("vendor descriptor not available (%d)\n", total_len);
} }
goto success; goto success;
...@@ -1583,7 +1594,7 @@ static int dlfb_usb_probe(struct usb_interface *interface, ...@@ -1583,7 +1594,7 @@ static int dlfb_usb_probe(struct usb_interface *interface,
dev->sku_pixel_limit = 2048 * 1152; /* default to maximum */ dev->sku_pixel_limit = 2048 * 1152; /* default to maximum */
if (!dlfb_parse_vendor_descriptor(dev, usbdev)) { if (!dlfb_parse_vendor_descriptor(dev, interface)) {
pr_err("firmware not recognized. Assume incompatible device\n"); pr_err("firmware not recognized. Assume incompatible device\n");
goto error; goto error;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment