- 16 Aug, 2014 40 commits
-
-
Konrad Zapalowicz authored
This commit fixes the missing blank lines after declarations checkpath warnings found in dgnc_cls.c file. Signed-off-by:
Konrad Zapalowicz <bergo.torino@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Konrad Zapalowicz authored
This commit fixes the checkpath warning about misused 'volatile' modifier. In this case the 'volatile' was not needed as it was used for regular automatic variable. Thos commit removes the 'volatile'. Signed-off-by:
-
Konrad Zapalowicz authored
This commit fixes the following checkpath warning in dgnc_neo.c file: 'braces {} are not necessary for single statement blocks' Signed-off-by: -
Konrad Zapalowicz authored
This commit fixes the following checkpath error in dgnc_neo.c file: 'that open brace { should be on the previous line' Signed-off-by: -
Konrad Zapalowicz authored
This commit deals with the checkapth warnings 'missing line after declarations' in the dgnc_neo.c file. Signed-off-by:
-
Konrad Zapalowicz authored
This commit fixes the checkpatch warning: drivers/staging/dgnc/dgnc_neo.c:37: WARNING: Use #include <linux/io.h> instead of <asm/io.h> Signed-off-by: -
Konrad Zapalowicz authored
It turned out that the TODO file contained the invalid information as some of the work has already been done. This commit updates it with the current status of what is left to be done. Signed-off-by:
-
Konrad Zapalowicz authored
This commit slightly simplifies the sgnc_start() function by rearranging it. As a result the indentation level is reduced. This is not the functional change. Signed-off-by:
-
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 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:
-
