- 23 Sep, 2014 35 commits
-
-
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:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Roxana Blaj authored
This fixes the warning: WARNING: void function return statements are not generally useful Signed-off-by:
Roxana Blaj <roxanagabriela10@gmail.com> 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:
Daniel Baluta <daniel.baluta@intel.com> Signed-off-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:
Aybuke Ozdemir <aybuke.147@gmail.com> 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 <tapaswenipathak@gmail.com> 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:Vaishali Thakkar <vthakkar1994@gmail.com> Acked-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:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in rtl871x_mp.h file WARNING : Macro should not use a trailing semicolon 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 authored
This patch fixes checkpatch.pl warning Unnecessary parentheses in rtl871x_xmit.c Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in rtl871x_ioctl_linux.c file WARNING : break is not useful after goto or return Signed-off-by:
-
- 22 Sep, 2014 5 commits
-
-
Aybuke Ozdemir authored
Fixed "Unnecessary space before function pointer argument" checkpatch.pl warning in ade7854.h Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in files of iio: accel WARNING : else is not generally useful after a break or return Signed-off-by:
-
Darshana Padmadas authored
This patch fixes checkpatch.pl warning: WARNING: else is not generally useful after a break or return Signed-off-by:
Darshana Padmadas <darshanapadmadas@gmail.com> Acked-by:
-
Darshana Padmadas authored
This patch fixes the checkpatch warning: WARNING: quoted string split across lines Signed-off-by:
-
Darshana Padmadas authored
This patch fixes checkpatch.pl warning: WARNING: msleep < 20ms can sleep for up to 20ms; see Documentation/timers/timers-howto.txt As per Documentation/timers/timers-howto.txt usleep_range should be used for sleeping for 10us-20ms Signed-off-by:
-
