- 23 Sep, 2014 40 commits
-
-
Catalina Mocanu authored
This fixes the following checkpatch.pl warning: WARNING: void function return statements are not generally useful Signed-off-by:
Catalina Mocanu <catalina.mocanu@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Catalina Mocanu authored
This fixes the following checkpatch.pl warning: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: -
Vaishali Thakkar authored
This patch fixes following checkpatch.pl warning: WARNING: msleep < 20ms can sleep for up to 20ms; see Documentation/timers/timers-howto.txt Signed-off-by:
Vaishali Thakkar <vthakkar1994@gmail.com> Acked-by:
Daniel Baluta <daniel.baluta@intel.com> Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in files of speakup WARNING : Comparing jiffies is almost always wrong; prefer time_after, time_before and friends Signed-off-by:
Tapasweni Pathak <tapaswenipathak@gmail.com> Signed-off-by:
-
Erin Grasmick authored
Removes void function return statements. Signed-off-by:
Erin Grasmick <ecg@unseen.is> Signed-off-by:
-
Erin Grasmick authored
Signed-off-by:
-
Aybuke Ozdemir authored
This patch fixes these warning messages found by checkpatch.pl: WARNING : Missing a blank line after declarations. Signed-off-by:
Aybuke Ozdemir <aybuke.147@gmail.com> Signed-off-by:
-
Erin Grasmick authored
Fix for unnecessary return at end of function as diagnosed by checkpatch.pl. Signed-off-by:
-
Roxana Blaj authored
This fixes the warning: WARNING: Missing a blank line after declarations Signed-off-by:
Roxana Blaj <roxanagabriela10@gmail.com> Acked-by:
-
Tina Johnson authored
The following checkpatch warning was fixed : WARNING: Missing a blank line after declarations Signed-off-by:
Tina Johnson <tinajohnson.1234@gmail.com> Signed-off-by:
-
Roxana Blaj authored
This fixes the warning: WARNING: void function return statements are not generally useful Signed-off-by:
-
Roxana Blaj authored
This fixes the warning: WARNING: else is not generally useful after a break or return Signed-off-by:
-
Gulsah Kose authored
This patch fixes "void function return statements are not generally useful" checkpatch.pl warning in base.c Signed-off-by:
Gulsah Kose <gulsah.1004@gmail.com> Acked-by:
-
Mark Einon authored
Signed-off-by:
Mark Einon <mark.einon@gmail.com> Signed-off-by:
-
Mark Einon authored
Rename adapter->flowcontrol to adapter->flow, reducing split lines. Signed-off-by:
-
Mark Einon authored
The 'if' blocks have braces, so the 'else' blocks should too. Signed-off-by:
-
Mark Einon authored
Modify some packet status ring associated variables to save a few split lines: pktstat_ringsize -> psr_size psr_num_entries -> psr_entries index -> ii Signed-off-by:
-
Mark Einon authored
Modify temporary variables so that split lines can be reduced: index -> k fbr_tmp_physaddr -> fbr_physaddr Signed-off-by:
-
Mark Einon authored
This ce_stats struct member is no longer used. Remove it. Signed-off-by:
-
Mark Einon authored
These two defines are not used, so remove them. Signed-off-by:
-
Mark Einon authored
These two adapter struct members are not used, remove them. Signed-off-by:
-
Aybuke Ozdemir authored
Fixes "Missing a blank line after declarations" checkpatch.pl warning in ethernet-xaui.c Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in lirc_serial.c file WARNING : break is not useful after goto or return Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in file lirc_serial.c WARNING : else is not generally useful after a break or return Signed-off-by:
-
Gulsah Kose authored
This patch fixes "quoted string split across lines" checkpatch.pl warning in lirc_imon.c. Signed-off-by:
-
Gulsah Kose authored
Fixes "Missing a blank line after declarations" checkpatch.pl warning in iss_video.c Signed-off-by:
-
Gulsah Kose authored
Fixes "Missing a blank line after declarations" checkpatch.pl warning in iss_csi2.c Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl "open brace '{' following struct go on the same line" errors Signed-off-by:Greg Donald <gdonald@gmail.com> Signed-off-by:
-
Aybuke Ozdemir authored
The following patch fixes the checkpatch.pl warning: WARNING: line over 80 characters. Signed-off-by:
-
Vaishali Thakkar authored
This patch fixes following checkpatch.pl warning using coccinelle: WARNING: else is not generally useful after a break or return Semantic patch used for this is as follows: @rule1@ expression e1; @@ if (e1) { ... return ...; } - else{ ... - } @rule2@ expression e2; statement s1; @@ if(e2) { ... return ...; } - else s1 Signed-off-by:Julia Lawall <julia.lawall@lip6.fr> Signed-off-by:
-
Vaishali Thakkar authored
This patch fixes following checkpatch.pl warning using coccinelle: WARNING: else is not generally useful after a break or return Semantic patch used for this is as follows: @rule1@ expression e1; @@ if (e1) { ... return ...; } - else{ ... - } @rule2@ expression e2; statement s1; @@ if(e2) { ... return ...; } - else s1 Signed-off-by: -
Tina Johnson authored
The GOTO macro is neither standard in Linux nor does its definiton contain much useful code. Hence GOTO can be replaced with useful parts of its definition. In a statement like GOTO(label, rc), the replacing code will be goto label if rc is a constant or a variable. But in cases like GOTO(label, e) where e is an assignment statement, both assignment and goto statements are kept. This patch was done using Coccinelle and the following semantic patch was used: @@ identifier rc,label; expression e; constant c; @@ ( -GOTO(label,rc = e); +rc = e; +goto label; | -GOTO(label,rc); +goto label; | -GOTO(label,c); +goto label; ) Signed-off-by:
-
Tapasweni Pathak authored
This patch merge two lines in a single line if immediate return is found. Unused variables in each case were removed manually as they are no longer needed. This is done using Coccinelle. Semantic patch used for this is as follows : @@ expression ret; identifier f; @@ -ret = +return f(...); -return ret; Signed-off-by: -
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in file rtl871x_ioctl.h WARNING : Missing space after return type Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in files of rel8712 WARNING : Unnecessary parentheses - maybe == should be = ? Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in files of rtl8712 WARNING : line over 80 characters Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in file rtl871x_xmit.c WARNING : prefer kmalloc_array over kmalloc with multiply Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in file usb_ops_linux.c WARNING : possible unecessary 'out of memory' message Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in file rtl871x_io.h WARNING : space prohibited before semicolon Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in files of rtl8712 WARNING : void function return statement are not generally useful Signed-off-by:
-
