- 24 Sep, 2014 5 commits
-
-
Catalina Mocanu authored
This fixes the following checkpatch.pl warning: WARNING: Prefer [subsystem eg: netdev]_dbg([subsystem]dev, ... then dev_dbg(dev, ... then pr_debug(... to printk(KERN_DEBUG ... Removing the macro also fixes: WARNING: macros should not use a trailing semicolon Signed-off-by:
Catalina Mocanu <catalina.mocanu@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Roberta Dobrescu authored
This fixes the following coccinelle warning: WARNING: Assignment of bool to 0/1 Signed-off-by:
Roberta Dobrescu <roberta.dobrescu@gmail.com> Acked-by:
Daniel Baluta <daniel.baluta@intel.com> Signed-off-by:
-
Roberta Dobrescu authored
This fixes the following coccinelle warning: WARNING: Comparison of bool to 0/1 Signed-off-by:
-
Aybuke Ozdemir authored
Fix checkpatch.pl issues with line over 80 characters in parallel-display.c Signed-off-by:
Aybuke Ozdemir <aybuke.147@gmail.com> Signed-off-by:
-
Mahati Chamarthy authored
This patch merges an assignment with an immediately following return of the assigned variable. The following Coccinelle semantic patch was used to make this transformation: @@ expression ret; identifier f; @@ -ret = +return f(...); -return ret; Signed-off-by:Mahati Chamarthy <mahati.chamarthy@gmail.com> Signed-off-by:
-
- 23 Sep, 2014 35 commits
-
-
Mahati Chamarthy authored
This fixes the following checkpatch.pl warnings: WARNING: else is not generally useful after a break or return Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl error in file rtw_ap.c ERROR: need consistent spacing around '-' (ctx:WxV) Signed-off-by:
Tapasweni Pathak <tapaswenipathak@gmail.com> Signed-off-by:
-
Mahati Chamarthy authored
This patch merges an assignment with an immediately following return of the assigned variable. It also removes a variable that becomes unused due to this transformation. The following Coccinelle semantic patch was used to make this transformation: @@ expression ret; identifier f; @@ -ret = +return f(...); -return ret; @@ identifier ret; type t; identifier c; @@ -t ret = c; ... when != ret when strict Signed-off-by:Julia Lawall <julia.lawall@lip6.fr> Signed-off-by:
-
Heena Sirwani authored
The following patch fixes the checkpatch.pl warning: WARNING: line over 80 characters Signed-off-by:
Heena Sirwani <heenasirwani@gmail.com> Signed-off-by:
-
Heena Sirwani authored
The following patch fixes the checkpatch.pl warning: WARNING: else is generally not useful after a break or return. Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in headers.h file WARNING : please, no space before tabs Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl error in InterfaceMacros.h file ERROR: do not use C99 // comments Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in synaptics_i2c_rmi4.c file WARNING : Missing a blank line after declarations Signed-off-by:
-
Rajbinder Brar authored
This patch merges an assignment with an immediately following return of the assigned variable. The following Coccinelle semantic patch was used to make this transformation: @@ expression ret; identifier f; @@ -ret= +return f(...); -return ret; A variable that became unused due to this transformation was also removed. Signed-off-by:
Rajbinder Brar <brar.rajbinder@gmail.com> Acked-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in files of vt6656 WARNING: else is not generally useful after a break or return Signed-off-by:
-
Tapasweni Pathak authored
This patch fixes checkpatch.pl error in baseband.c file ERROR : that open brace { should be on the previous line Signed-off-by: -
Tapasweni Pathak authored
This patch fixes checkpatch.pl warning in aes_ccmp.c file WARNING : else is not generally useful after a break or return Signed-off-by:
-
Mahati Chamarthy authored
This patch merges an assignment with an immediately following return of the assigned variable. The following Coccinelle semantic patch was used to make this transformation: @@ expression ret; identifier f; @@ -ret = +return f(...); -return ret; A variable that became unused due to this transformation was also removed. Signed-off-by: -
Mahati Chamarthy authored
This patch merges an assignment with an immediately following return of the assigned variable.The following Coccinelle semantic patch was used to make this transformation: @@ expression ret; identifier f; @@ -ret = +return f(...); -return ret; A variable that became unused due to this transformation was also removed. Signed-off-by: -
Aybuke Ozdemir authored
This patch fixes these error messages found by checkpatch.pl: ERROR: do not use C99 // comments Signed-off-by:
-
Gulsah Kose authored
Fixes "braces {} are not necessary for single statement blocks" checkpatch.pl warnings in dgnc_sysfs.c Signed-off-by:Gulsah Kose <gulsah.1004@gmail.com> Signed-off-by:
-
Benjamin Romer authored
The function ULTRA_check_channel_client() was recently patched to use ioread8_rep(). Unfortunately the parameters were incorrectly ordered, and this causes the function to fail and the guest to not boot. This patch switches the parameters around and uses memcpy_fromio() instead, which is clearer since a structure is being copied, not an array, and it fixes the error message printout later in the function that was using the UUID value in the channel, rather than the value copied from the channel, which causes a misleading error to be reported. Signed-off-by:
Benjamin Romer <benjamin.romer@unisys.com> 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:
-
Catalina Mocanu authored
This fixes the following checkpatch.pl warning: WARNING: void function return statements are not generally useful Signed-off-by:
-
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:
-
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:
-
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:
-
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:
-
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:
-
