- 09 Jul, 2014 2 commits
-
-
Andrey Utkin authored
The issue is discovered by static checker. The proposed change (0x000c0 -> 0x000c) is likely correct because: 1. 16-bit `map` holds value coming from struct ieee80211_vht_mcs_info.tx_mcs_map, which is described so: "TX MCS map 2 bits for each stream, total 8 streams". The changed code refers to case of 2 TX streams, and 0x000c mask filters two bits related to the second stream. Some codelines below 0x0003 mask is used to test first stream. 2. Mask 0x000c is used 3 more times in that place. 3. Specifying 5 digits of hex value is uncommon, especially while working with `u16` variable. So likely the trailing zero is a typo. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=78041Reported-by:
David Binderman <dcb314@hotmail.com> Signed-off-by:
Andrey Utkin <andrey.krieger.utkin@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Emil Goode authored
We need to make sure the struct rtllib_device pointer ieee is not NULL after the goto rx_dropped label since it is dereferenced there. Signed-off-by:
Emil Goode <emilgoode@gmail.com> Signed-off-by:
-
- 08 Jul, 2014 38 commits
-
-
Anil Belur authored
- this replaces jiffies comparision with safer function using time_after_eq() Signed-off-by:
Anil Belur <askb23@gmail.com> Signed-off-by:
-
Rasmus Villemoes authored
struct rtl_stats is rather huge (152 bytes), and since rtl92ee_rx_command_packet() does not modify it, it might as well be passed by const reference. Reported by Coverity: CID 1222131 Signed-off-by:
Rasmus Villemoes <linux@rasmusvillemoes.dk> Signed-off-by:
-
Rasmus Villemoes authored
struct rtl_stats is rather huge (152 bytes), and since rtl8812ae_rx_command_packet_handler() does not modify it, it might as well be passed by const reference. Reported by Coverity: CID 1167285 Signed-off-by:
-
Fabian Frederick authored
use macro definition Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: Josh Triplett <josh@joshtriplett.org> Cc: devel@driverdev.osuosl.org Signed-off-by:
Fabian Frederick <fabf@skynet.be> Signed-off-by:
-
Fabian Frederick authored
lprocfs_stats_seq_sops is only used in static int lprocfs_stats_seq_open with seq_open as const struct seq_operations * Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: Peng Tao <bergwolf@gmail.com> Cc: devel@driverdev.osuosl.org Signed-off-by:
-
Aaro Koskinen authored
wHubCharacteristics gets wrong value on big-endian CPUs. Signed-off-by:
Aaro Koskinen <aaro.koskinen@iki.fi> Signed-off-by:
-
Aaro Koskinen authored
Return transferred bytes only when transfer was successful. Signed-off-by:
-
Aaro Koskinen authored
On data toggle error, we don't know if and how many bytes were successfully transferred by DMA. We should just fail the transaction instead of trying a retry. Signed-off-by:
-
Aaro Koskinen authored
The driver did not use link_urb_to_ep() / unlink_urb_from_ep(). This caused odd behaviour in some error recovery situations, all requests would start to fail after the first failure. Signed-off-by:
-
Aaro Koskinen authored
Cancel requests synchronously instead of using the dequeue tasklet. Signed-off-by:
-
Geert Uytterhoeven authored
The handling of ip2= in drivers/char/ip2/ip2base.c was moved to drivers/char/ip2/ip2main.c in commit 47babd4c ("Char: merge ip2main and ip2base"). The ip2 driver was demoted to staging in commit 4a6514e6 ("tty: move obsolete and broken tty drivers to drivers/staging/tty/"), and finally deleted in commit 51c9d654 ("Staging: delete tty drivers"). Signed-off-by:
Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by:
-
Bernd Porr authored
The AD converter on the usbsigma turns vref (2.5V/2) into the binary value 0x780000. However the full scale range goes up to 0x800000 so that vref/2 needs to be upscaled by 0x800000/0x780000 which is 1.06bar. In the driver it's been 1.06 without the infinite 6666. I've put the fraction in the code so that the compiler can calculate the most precise scaling value by itself. Signed-off-by:
Bernd Porr <mail@berndporr.me.uk> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Ian Abbott authored
Comedi subdevices can be locked for mostly exclusive access by a particular file object by means of the COMEDI_LOCK ioctl. Some operations disregard the lock. For example, the read() and write() file operations are allowed for the file object that set up an asynchronous command on the subdevice (making it "busy") regardless of whether it is locked. The COMEDI_BUFINFO, COMEDI_CANCEL and COMEDI_POLL ioctls are also associated with the file object that made the subdevice "busy" with a command. They currently obey the exclusive access lock on the subdevice as well, but it makes more sense for them to ignore it like the read() and write() file operations. Remove the exclusive subdevice access checks for these ioctls. Signed-off-by:
-
Chase Southwood authored
Remove the checks for interrupts from unknown sources. This situation should never occur and the checks were doing nothing to help the situation. Also, the portion of the function for handling counter interrupts is reapeated four times (once for each counter), but is completely identical save for the register is is accessing, so we can handle all four counters with a for loop. Finally, the interrupt handler is incorrectly setting and then checking devpriv->timer_select_mode before processing some of the triggered interrupts, so just remove all occurrences of this. Signed-off-by:
Chase Southwood <chase.southwood@gmail.com> Cc: H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk.> Signed-off-by:
-
Chase Southwood authored
apci1564_cos_insn_config() is currently using apci1564_reset() to disable digital input interrupts when the configuration operation is COMEDI_DIGITAL_TRIG_DISABLE. However, this is incorrect as the device reset function also resets the registers for the digital outputs, timer, watchdog, and counters as well. Replace the reset function call with a direct disabling of just the digital input interrupts. Signed-off-by:
-
Chase Southwood authored
Commit aed3f9d4 (staging: comedi: addi_apci_1564: absorb apci1564_reset()) moved the only use of addi_watchdog.h from hwdrv_apci1564.c to addi_apci_1564.c, but left the include statement itself in the former file. Move this include to the file which actually uses it. Signed-off-by:
-
Matthias Beyer authored
Signed-off-by:
Matthias Beyer <mail@beyermatthias.de> Signed-off-by:
-
Matthias Beyer authored
This patch fixes line lengths in the drivers/staging/bcm/InterfaceMisc.c file. Signed-off-by: -
Kevin McKinney authored
Add myself and Matthias Beyer as maintainers for the bcm wimax driver. Signed-off-by:
Kevin McKinney <klmckinney1@gmail.com> Signed-off-by:
-
Kevin McKinney authored
Add myself and Matthias Beyer to the TODO file as the list of maintainers. Signed-off-by:
-
Matthias Beyer authored
Signed-off-by:
-
Matthias Beyer authored
Signed-off-by:
-
Matthias Beyer authored
Signed-off-by:
-
Matthias Beyer authored
Signed-off-by:
-
Matthias Beyer authored
Signed-off-by:
-
Matthias Beyer authored
Signed-off-by:
-
Matthias Beyer authored
This patch outsources a chunk of code into an own function and refactors the variable names inside the new function. Signed-off-by:
-
Matthias Beyer authored
Signed-off-by:
-
Matthias Beyer authored
This patch removes the `process_done` variable from both format_eth_hdr_to_stack() and read_bulk_callback() (where it is declared), as it is not used anywhere (it is set in the format_eth_hdr_to_stack() function, but afterwards not used). Acked-by:
-
Matthias Beyer authored
This patch outsources a chunk of code into a function. Acked-by:
-
Matthias Beyer authored
Acked-by:
-
Matthias Beyer authored
Acked-by:
-
Malcolm Priestley authored
The latest vendor code makes changes to uSIFS and uDIFS. The orginal codes have been rearranged into a fall through switch statement. These reduce the time that device allows to TX packets and improves throughput. Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
-
Malcolm Priestley authored
The device refuses to associate to older access points even though in the logs it appears to be associated in tkip or ccmp mode. Clear bit from key_entry_inuse in pairwise key so the entry is over written by group address. This doesn't seem to make any difference to newer access points and is a temporary fix until a better solution is found. Signed-off-by:
-
Malcolm Priestley authored
skb_push the headroom and trimmed back to 0 Signed-off-by:
-
Anton Protopopov authored
This commit fixes a few sparse warnings for missing the 'static' keyword in array definitions: drivers/staging//vt6655/rxtx.c:81:22: warning: symbol 'wTimeStampOff' was not declared. Should it be static? drivers/staging//vt6655/rxtx.c:86:22: warning: symbol 'wFB_Opt0' was not declared. Should it be static? drivers/staging//vt6655/rxtx.c:90:22: warning: symbol 'wFB_Opt1' was not declared. Should it be static? Signed-off-by:Anton Protopopov <a.s.protopopov@gmail.com> Signed-off-by:
-
Malcolm Priestley authored
The driver reports the rate tried in struct vnt_interrupt_data tsr* variables which is available in INTnsProcessData via interrupt urb context. Instead of closing apTD tx context in s_nsBulkOutIoCompleteWrite by setting in_use to false. Keep the context open and allow vnt_int_report_rate to close it. If the tx_retry value is correct it will report back the sucessful RATE tried. struct vnt_usb_send_context add pkt_no which is index of apTD Signed-off-by:
-
Malcolm Priestley authored
Remove variable re_alloc_skb and merge code within urb->actual_length. Signed-off-by:
-
