- 08 Sep, 2014 40 commits
-
-
navin patidar authored
Rename CamelCase function name. Signed-off-by:
navin patidar <navin.patidar@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
navin patidar authored
Signed-off-by:
-
navin patidar authored
Rename CamelCase function name. Signed-off-by:
-
Ilja Sidoroff authored
This patch removes redundant goto or break statements in hfa384x_usb.c as found by checkpatch.pl Signed-off-by:
Ilja Sidoroff <ilja.sidoroff@iki.fi> Signed-off-by:
-
Martin Kepplinger authored
remove a typedef that is not even really used. Signed-off-by:
Martin Kepplinger <martink@posteo.de> Signed-off-by:
-
SeeChen Ng authored
Follow the checkpatch.pl "LINE_SPACING" indication: 1. Insert a blank line after function declaration. 2. Remove multiple blank lines. Signed-off-by:
SeeChen Ng <seechen81@gmail.com> Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl '"(foo*)" should be "(foo *)"' errors Signed-off-by:
Greg Donald <gdonald@gmail.com> Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl '"(foo*)" should be "(foo *)"' errors Signed-off-by:
-
Omar Sandoval authored
Fix checkpatch.pl "Alignment should match open parenthesis". Signed-off-by:
Omar Sandoval <osandov@osandov.com> Reviewed-by:
Matthias Beyer <mail@beyermatthias.de> Signed-off-by:
-
Malcolm Priestley authored
rate control algorithm pid was removed from wireless tree The device defaults to algorithm 'minstrel_ht' There has been no problems using this algorithm. Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
-
Malcolm Priestley authored
pDevice is never NULL when this function is called remove the check. This fixes smatch warning drivers/staging/vt6655/device_main.c:1023 device_free_info() warn: variable dereferenced before check 'pDevice' (see line 1021) Reported-by:
Dan Carpenter <dan.carpenter@oracle.com> Cc: kbuild@01.org Signed-off-by:
-
Paul Bolle authored
There are two checks for CONFIG_B4860G100 in this driver. But there's no Kconfig symbol B4860G100. These checks will always evaluate to false. Remove them and the code they hide. Signed-off-by:
Paul Bolle <pebolle@tiscali.nl> Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl "space required before that '*'" errors Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl "space required before that '*'" errors Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl "space required before that '*'" errors Signed-off-by:
-
Eli Billauer authored
Suggested-by:
Eli Billauer <eli.billauer@gmail.com> Signed-off-by:
-
Eli Billauer authored
* Blank lines between a function call and its return status check were removed. * Double blank lines were removed. * Blank lines were added and removed as to silence checkpatch.pl --strict Suggested-by:
-
Eli Billauer authored
Signed-off-by:
-
Eli Billauer authored
Suggested-by:
-
Eli Billauer authored
Suggested-by:
-
Eli Billauer authored
Trivial comments removed, and one comment clarified Suggested-by:
-
Eli Billauer authored
Suggested-by:
-
Eli Billauer authored
Suggested-by:
-
Eli Billauer authored
Rather than checking the wait condition, the return value of wait_event_interruptible() and wait_event_interruptible_timeout() is used. Suggested-by:
-
Eli Billauer authored
"rc" is often used to hold the return value of a function call. This patch removes unnecessary assignments to this variable, and makes a few related execution flow improvements. Suggested-by:
-
Eli Billauer authored
The -EAGAIN status is passed through an "rc" variable instead of a less common flow. Suggested-by:
-
Eli Billauer authored
Suggested-by:
-
Eli Billauer authored
xilly_scan_idt() now returns an error status code, rather than being a void function and hint the status through an entry in a data structure. Suggested-by:
-
Eli Billauer authored
Suggested-by:
-
Eli Billauer authored
Suggested-by:
-
Eli Billauer authored
Responding to an interrupt while handling the "release" method can't end well anyhow. In practical cases, this can force the user to wait up to one second while flushing remaining data is attempted (a timeout on the flush mechanism limits the time for flushing). Suggested-by:
-
Eli Billauer authored
Suggested-by:
-
Eli Billauer authored
Replace devm_kzalloc where it applies. Suggested-by:
-
H Hartley Sweeten authored
For aesthetics, move the analog input (*insn_read) function so it is not located in the middle of the async command support functions. Also, rename it for consistency. Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
H Hartley Sweeten authored
Signed-off-by:
-
H Hartley Sweeten authored
The analog input FIFO is reset at the start of every (*insn_read) and (*do_cmd) operation. It's also reset for the analog input (*cancel). There's no reason to reset the FIFO if an (*insn_read) times out or after all the samples have been acquired. Signed-off-by:
-
H Hartley Sweeten authored
Introduce a helper function to program the AMCC master write address and transfer count registers to setup a DMA transaction. Signed-off-by:
-
H Hartley Sweeten authored
Introduce a helper function to program the AMCC master control/status register to enable/disable dma. Signed-off-by:
-
H Hartley Sweeten authored
Introduce a helper function to program the AMCC interrupt control/ status register to enable/disable interrupts. Fix the analog input (*cancel). The analog input async command support is the only interrupt generator. The (*cancel) should always disable interrupts. Signed-off-by:
-
H Hartley Sweeten authored
Rename this CamelCase member of the private data. Signed-off-by:
-
