linux/drivers/usb/class, branch v3.12

USB: usbtmc: fix up attribute permissions

2013-08-25T22:12:03Z

In auditing the usbtmc sysfs files, a bunch of them were being created as "read only", yet they have logic to handle writing to. So fix them up by setting the permissions properly. Signed-off-by: Greg Kroah-Hartman

Merge 3.11-rc6 into usb-next

2013-08-19T03:33:01Z

We want these USB fixes in this branch as well. Signed-off-by: Greg Kroah-Hartman

USB: cdc-wdm: fix race between interrupt handler and tasklet

2013-08-12T22:41:07Z

Both could want to submit the same URB. Some checks of the flag intended to prevent that were missing. Signed-off-by: Oliver Neukum CC: stable@vger.kernel.org Signed-off-by: Greg Kroah-Hartman

USB: usbtmc: fix big-endian probe of Rigol devices

2013-08-12T20:52:35Z

Fix probe of Rigol devices on big-endian machines. A quirk for these devices was introduced by commit c2e314835 ("USB: usbtmc: Set rigol_quirk if device is listed") but was only enabled on little-endian machines. Cc: stable@vger.kernel.org Signed-off-by: Johan Hovold Signed-off-by: Greg Kroah-Hartman

usbtmc: convert to devm_kzalloc

2013-07-25T19:01:12Z

kfree(data) will be called implicitly. Signed-off-by: Andy Shevchenko Signed-off-by: Greg Kroah-Hartman

usbtmc: remove redundant braces

2013-07-25T19:01:12Z

There is a few cases where braces are not needed. This patch removes unnecessary '& 255' pieces as well when lvalue type is u8. Signed-off-by: Andy Shevchenko Signed-off-by: Greg Kroah-Hartman

usbtmc: call pr_err instead of plain printk

2013-07-25T19:01:12Z

Additionally remove useless label. Signed-off-by: Andy Shevchenko Signed-off-by: Greg Kroah-Hartman

usbtmc: remove trailing spaces

2013-07-25T19:01:12Z

Recent patch series introduces few trailing spaces. This patch removes them. No functional change. Signed-off-by: Andy Shevchenko Signed-off-by: Greg Kroah-Hartman

usb: class: cdc-acm: be careful with bInterval

2013-07-25T18:49:29Z

bInterval must be on the range 1 - 16, if we want to pass the maximum allowed, we should be passing 16 Signed-off-by: Felipe Balbi Signed-off-by: Greg Kroah-Hartman

USB: cdc-acm: remove unneeded spin_lock_irqsave/restore on write path

2013-06-17T20:37:07Z

When writing data we were: lock do some work unlock call function lock do some work unlock return return It turns out, that "function" was only ever called in the one place, so instead of locking/unlocking for no good reason, just inline the function and only grab the lock once. This has sped up the pathological case of sending 1 byte packets to a loop-back cdc-acm device from 49600 bytes per second to 50100 bytes a second on my workstation. A tiny increase yes, but noticable, and now the spinlock isn't the hottest thing on the perf graph anymore. Yes, we are still waiting for the hardware for the most part, but getting rid of a spinlock_irq_save() call for every packet is still a good thing. And we end up deleting lines of code, always a win overall. This was found by using a Teensy 3.0 device and the test program and firmware located at: http://www.pjrc.com/teensy/benchmark_usb_serial_receive.html Reported-by: Paul Stoffregen Signed-off-by: Greg Kroah-Hartman