linux-stable.git/drivers/usb/class, branch linux-4.8.y

USB: cdc-acm: add device id for GW Instek AFG-125

2017-01-06T10:16:18+00:00

commit 301216044e4c27d5a7323c1fa766266fad00db5e upstream.

Add device-id entry for GW Instek AFG-125, which has a byte swapped
bInterfaceSubClass (0x20).

Signed-off-by: Nathaniel Quillin 
Acked-by: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman

cdc-acm: fix uninitialized variable

2016-11-18T09:51:47+00:00

commit 7309aa847ead3fa561663b16779a0dde8c64cc7c upstream.

variable struct usb_cdc_parsed_header h may be used
uninitialized in acm_probe.

In kernel 4.8.

    /* handle quirks deadly to normal probing*/
    if (quirks == NO_UNION_NORMAL)

        ...

        goto skip_normal_probe;
    }

we bypass call to

    cdc_parse_cdc_header(&h, intf, buffer, buflen);

but later use h in

    if (h.usb_cdc_country_functional_desc) { /* export the country data */

Signed-off-by: Oliver Neukum 
Reported-by: Victor Sologoubov 
Signed-off-by: Greg Kroah-Hartman

USB: cdc-acm: fix TIOCMIWAIT

2016-11-18T09:51:47+00:00

commit 18266403f3fe507f0246faa1d5432333a2f139ca upstream.

The TIOCMIWAIT implementation would return -EINVAL if any of the three
supported signals were included in the mask.

Instead of returning an error in case TIOCM_CTS is included, simply
drop the mask check completely, which is in accordance with how other
drivers implement this ioctl.

Fixes: 5a6a62bdb925 ("cdc-acm: add TIOCMIWAIT")
Signed-off-by: Johan Hovold 
Acked-by: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman

Revert "usbtmc: convert to devm_kzalloc"

2016-10-07T13:03:20+00:00

commit ab21b63e8aedfc73565dd9cdd51eb338341177cb upstream.

This reverts commit e6c7efdcb76f11b04e3d3f71c8d764ab75c9423b.

Turns out it was totally wrong.  The memory is supposed to be bound to
the kref, as the original code was doing correctly, not the
device/driver binding as the devm_kzalloc() would cause.

This fixes an oops when read would be called after the device was
unbound from the driver.

Reported-by: Ladislav Michl 
Cc: Andy Shevchenko 
Signed-off-by: Greg Kroah-Hartman

cdc-acm: fix wrong pipe type on rx interrupt xfers

2016-08-15T14:30:56+00:00

This fixes the "BOGUS urb xfer" warning logged by usb_submit_urb().

Signed-off-by: Gavin Li 
Acked-by: Oliver Neukum 
Cc: stable 
Signed-off-by: Greg Kroah-Hartman

cdc-acm: beautify probe()

2016-07-18T15:46:57+00:00

This removes some overly long lines by renaming variables and giving
them local scope.

Signed-off-by: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman

cdc-wdm: use the common CDC parser

2016-07-18T15:46:57+00:00

Now that the common parser resides in USB core, it can
be used for CDC-WDM.

Signed-off-by: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman

cdc-acm: cleanup error handling

2016-07-18T15:46:57+00:00

A small update to unify error handling during probe().

Signed-off-by: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman

cdc-acm: use the common parser

2016-07-18T15:46:57+00:00

This introduces the common parser for extra CDC headers now that it no longer
depends on usbnet.

Signed-off-by: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman

usb: cdc-acm: Space prohibited before close parenthesis ')'.

2016-06-08T05:18:39+00:00

Space prohibited before close parenthesis ')'.

Signed-off-by: Sandhya Bankar 
Signed-off-by: Greg Kroah-Hartman