- 16 Aug, 2014 40 commits
-
-
Konrad Zapalowicz authored
This commit moves the utility functions out of dgnc_driver.c file and puts them in the new dgnc_utils.{c,h} files. The accompanying changes adjust the existing code to work with this design. Signed-off-by:Konrad Zapalowicz <bergo.torino@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Konrad Zapalowicz authored
This commit slightly cleans up the dgnc_driver.c file. The changes include removing one-line proxy functions as they were not needed. Additionaly the pci 'remove' function is deleted because it was no need for it. Signed-off-by:
-
Konrad Zapalowicz authored
This comit fixes the following sparse warnign: drivers/staging/dgnc/dgnc_tty.c:572:1: warning: the frame size of 1060 bytes is larger than 1024 bytes [-Wframe-larger-than=] This was caused by having buffer as an automatic variable. This commit moves it from the stack to the heap. Signed-off-by: -
Konrad Zapalowicz authored
This commit corrects the 'no space before tabs' checkpath warning. Signed-off-by:
-
Konrad Zapalowicz authored
This commit fixes the following checkpatch warnings: WARNING: externs should be avoided in .c files #80: FILE: drivers/staging/dgnc/dgnc_driver.c:80: +int dgnc_init_module(void); #81: FILE: drivers/staging/dgnc/dgnc_driver.c:81: +void dgnc_cleanup_module(void); This was caused by putting the declarations for module init and module exit fucntions on the top of the file. The fix removes these declarations plus it also corrects the type of the init/exit functions. Due to the dependency between init and exit functions the dgnc_cleanup_module had to be put first. Signed-off-by: -
Lars Hamre authored
This is a patch to the nokia_fw.c file that removes an unnecessary return statement found by the checkpatch.pl tool Signed-off-by:
Lars Hamre <hamham91@gmail.com> Signed-off-by:
-
A Raghavendra Rao authored
Fix the following sparse warning : In file included from drivers/staging/wlan-ng/prism2usb.c:5:0: drivers/staging/wlan-ng/prism2fw.c: In function ‘read_cardpda.constprop.43’: drivers/staging/wlan-ng/prism2fw.c:792:1: warning: the frame size of 1068 bytes is larger than 1024 bytes [-Wframe-larger-than=] The variable to 'struct p80211msg_p2req_readpda' was previously being created on the stack, which inturn exeeded the frame size limit, resulting in a sparse warning. This patch alloctes the memory to the structure dynamically and the operations are left unchanged. Signed-off-by:
A Raghavendra Rao <arrao@cdac.in> Signed-off-by:
-
Quentin Lambert authored
Add 2 missing blank lines after declaration. Signed-off-by:
Quentin Lambert <lambert.quentin@gmail.com> Signed-off-by:
-
Fabio Falzoi authored
CONFIG_RTS5208_DEBUG is no more needed, we rely on dynamic debug config options instead. Signed-off-by:
Fabio Falzoi <fabio.falzoi84@gmail.com> Signed-off-by:
-
Fabio Falzoi authored
Use dev_dbg with %*ph format specifier and print_hex_dump_bytes to dump memory instead of relying on custom macro. Signed-off-by:
-
Quentin Lambert authored
Add missing blank lines after declaration. Signed-off-by:
-
Julia Lawall authored
Of_node_put supports NULL as its argument, so the initial test is not necessary. Suggested by Uwe Kleine-König. The semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression e; @@ -if (e) of_node_put(e); // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Christoph Jaeger authored
Make use of the various list helper functions to improve readability. Signed-off-by:
Christoph Jaeger <email@christophjaeger.info> Signed-off-by:
-
Christoph Jaeger authored
Use a slab cache rather than rolling our own free list. Signed-off-by:
-
Christoph Jaeger authored
Use a slab cache rather than rolling our own free list. Signed-off-by:
-
Christoph Jaeger authored
Use a slab cache rather than rolling our own free list. Signed-off-by:
-
Christoph Jaeger authored
Simplify the somewhat overcomplicated application interface; improves readability and saves a bunch of lines. Use designated struct initializers for clarity. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Christoph Jaeger authored
No need to return a value from elt_buf_init(). Signed-off-by:
-
Christoph Jaeger authored
Member 'ops' has already been initialized by calling cdev_init(). Signed-off-by:
-
Christoph Jaeger authored
OZ_MAX_TIMER_POOL_SIZE is not used anywhere; remove it. Signed-off-by:
-
Christoph Jaeger authored
Signed-off-by:
-
Christoph Jaeger authored
Signed-off-by:
-
Hans Wennborg authored
Signed-off-by:
Hans Wennborg <hans@hanshq.net> Signed-off-by:
-
Hans Wennborg authored
Signed-off-by:
-
Benedict Boerger authored
drivers: staging: lustre: fix sparse warning: symbol XYZ was not declared. Should it be static? This was done by declaring them static. This could be done because the functions were used only in this file. Deleted the function lnet_print_text_bufs because it were unused. Compiled without an error. Done to complete a eudyptula task. Signed-off-by:
Benedict Boerger <benedict.boerger@cs.tu-dortmund.de> Signed-off-by:
-
Tom Wales authored
Remove spaces between the function names and open paranthesis and use the kernel coding style. Signed-off-by:
Tom Wales <twales518@gmail.com> Signed-off-by:
-
Fernando Apesteguia authored
Remove ComputeTxTime since it is not used. Signed-off-by:
Fernando Apesteguia <fernando.apesteguia@gmail.com> Signed-off-by:
-
Adrian Remonda authored
This is a patch to the rtl8188e_xmit.c file that fixes up a missing blank line warning found by the checkpatch.pl tool. Signed-off-by:
Adrian Remonda <adrianremonda@gmail.com> Signed-off-by:
-
Joe Perches authored
RT_TRACE uses require unnecessary parentheses. Fix the #define and remove the uses. Neaten the RT_TRACE lines and multi-line argument wrapping. dynamic debug can add KBUILD_MODNAME and __func__ so remove those from the #define. Use a single statement pr_debug instead of pr_debug and pr_cont which doesn't support dynamic debug anyway. Miscellaneous grammar and spelling fixes on the RT_TRACE uses. Compile tested only. Signed-off-by:
Joe Perches <joe@perches.com> Signed-off-by:
-
Antoine Schweitzer-Chaput authored
Signed-off-by:
Antoine Schweitzer-Chaput <antoine@schweitzer-chaput.fr> Signed-off-by:
-
Antoine Schweitzer-Chaput authored
Signed-off-by:
-
Antoine Schweitzer-Chaput authored
Also remove related unreachable code. Signed-off-by:
-
Antoine Schweitzer-Chaput authored
Also remove the unreachable code. Signed-off-by:
-
Antoine Schweitzer-Chaput authored
Also remove the code previously under #ifndef USE_ONE_PIPE. Signed-off-by:
-
Antoine Schweitzer-Chaput authored
Also remove the code now unreachable. Signed-off-by:
-
Luca Ellero authored
Enable support for the National Instruments USB-6501 module. The NI USB-6501 is a Full-Speed USB 2.0 (12 Mbit/s) device that provides 24 digital I/O lines channels and one 32-bit counter. This is a preliminary version: GPIO: works counter: doesn't work Signed-off-by:
Luca Ellero <luca.ellero@brickedbrain.com> Signed-off-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Niklas Svensson authored
This patch fixes warnings of checkpatch.pl script: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around devpriv->timer + init_timer(&(devpriv->timer)); CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis + dev_info(dev->class_dev, + "%s: %i microvolt, %li microsecond waveform attached\n", Task of Eudyptula challenge. Signed-off-by:
Niklas Svensson <nks@flawful.org> Reviewed-by:
-
H Hartley Sweeten authored
The local variable 'irq' is not necessary in dio200_attach(). Just pass the it->options[1] value directly. Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
-
H Hartley Sweeten authored
This member of the boardinfo is not used by the drivers. Remove it. Signed-off-by:
-
H Hartley Sweeten authored
The 'mainsize' member in the boardinfo for the DIO200 PCI boards is only used for a sanity check of the pci_resource_len(). This sanity check is not needed. Remove the sanity check along with the 'mainsize' values in the boardinfo. Signed-off-by:
-
