- 24 Sep, 2014 9 commits
-
-
Mahati Chamarthy authored
This patch merges an assignment with an immediately following return of the assigned variable. It also removes variables that became unused due to this transformation. The following Coccinelle semantic patch was used to make this transformation: @r@ identifier ret; identifier f; @@ -ret = +return f(...); -return ret; @@ identifier r.ret; expression e1; type t; @@ ( -t ret = e1; | -t ret; ) ... when != ret when strict Signed-off-by:Mahati Chamarthy <mahati.chamarthy@gmail.com> Acked-by:
Julia Lawall <julia.lawall@lip6.fr> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Tapasweni Pathak authored
This patch removes unncessary variable in file r8192U_core.c using Coccinelle. Semantic patch for this is as follows : @@ identifier ret; @@ -int ret = 0; ... when != ret when strict -return ret; +return 0; 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: @r@ identifier ret; identifier f; @@ -ret = +return f(...); -return ret; @@ identifier r.ret; expression e1; type t; @@ ( -t ret = e1; | -t ret; ) ... when != ret when strict Signed-off-by: -
Roberta Dobrescu authored
This patch replaces non-standard spinlock's macros. It is done using coccinelle and the following semantic patch: @@ expression x; @@ - DGNC_SPINLOCK_INIT(x) + spin_lock_init(&x) @@ expression x, y; @@ - DGNC_LOCK(x, y) + spin_lock_irqsave(&x, y) @@ expression x, y; @@ - DGNC_UNLOCK(x, y) + spin_unlock_irqrestore(&x, y) @used_by_lock exists@ typedef ulong; symbol lock_flags; position p1, p2; @@ ( ulong lock_flags@p1; | unsigned long lock_flags@p2; ) ... ( spin_lock_irqsave(..., lock_flags) | spin_unlock_irqrestore(..., lock_flags) ) @@ position used_by_lock.p1, used_by_lock.p2; @@ ( - ulong lock_flags@p1; + unsigned long flags; | - unsigned long lock_flags@p2; + unsigned long flags; ) <... - lock_flags + flags ...> Signed-off-by:
Roberta Dobrescu <roberta.dobrescu@gmail.com> Signed-off-by:
-
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:
-
Roberta Dobrescu authored
This fixes the following coccinelle warning: WARNING: Assignment of bool to 0/1 Signed-off-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:
-
- 23 Sep, 2014 31 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:
-
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: -
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:
-
