- 20 Sep, 2016 40 commits
-
-
Sandhya Bankar authored
No space is necessary after a cast. This issue was found by checkpatch. Signed-off-by:
Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Sandhya Bankar authored
No space is necessary after a cast. This issue was found by checkpatch. Signed-off-by:
-
Rehas Sachdeva authored
Adds spaces on either side of a '|'. Issue found by checkpatch. Signed-off-by:
Rehas Sachdeva <aquannie@gmail.com> Signed-off-by:
-
Rehas Sachdeva authored
Instead of storing the return value into a variable and then returning it, we can club the two into a single return statement. This change was made using the following semantic patch by Coccinelle: @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: -
Eva Rachel Retuya authored
Replace pr_warn() call with its respective dev_warn() counterpart. Semantic patch used to detect and apply the transformation: @A@ identifier dev; expression E; @@ struct device *dev = E; <+... when != dev == NULL - pr_warn( + dev_warn(dev, ...); ...+> Signed-off-by:
Eva Rachel Retuya <eraretuya@gmail.com> Signed-off-by:
-
Eva Rachel Retuya authored
Use dev_info() instead of printk(). Majority of the conversion was done previously on this commit: 011b2aad [media] staging/media: Use dev_ printks in cxd2099/cxd2099.[ch] Signed-off-by:
-
Rehas Sachdeva authored
Instead of storing the return value into a variable and then returning it, we can club the two into a single return statement. This change was made using the following semantic patch by Coccinelle: @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: -
Rehas Sachdeva authored
Instead of storing the return value into a variable and then returning it, we can club the two into a single return statement. This change was made using the following semantic patch by Coccinelle: @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: -
Sandhya Bankar authored
Add space around operator. This patch was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
Use x instead of x != NULL. This patch was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
Use x instead of x != NULL. This patch was found by checkpatch. Signed-off-by:
-
Cathal Mullaney authored
Fixed several minor brace coding style issues. Signed-off-by:
Cathal Mullaney <chuckleberryfinn@gmail.com> Signed-off-by:
-
Sandhya Bankar authored
This issue was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
This issue was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
This patch was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
This issue was found by checkpatch. Signed-off-by:
-
sayli karnik authored
This patch removes an unnecessary variable used to store return values in order to reduce memory usage. Done using coccinelle: @@ type T; constant C; identifier ret; @@ - T ret = C; ... when != ret return - ret + C ; Signed-off-by:
sayli karnik <karniksayli1995@gmail.com> Acked-by:
Julia Lawall <julia.lawall@lip6.fr> Signed-off-by:
-
Hariharan R authored
Fixes typo, FAILUER -> FAILURE Recieve -> Receive Signed-off-by:
Hariharan R <hariharanrangasamy@gmail.com> Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This issue was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
of_node_put needs to be called when the device node which is got from of_parse_phandle is no longer used. This patch is found by below coccinelle script: @@ expression e,e1,e2; @@ *e = of_parse_phandle(...) ... when != of_node_put(e) when != true e == NULL when != e2 = e e = e1 Signed-off-by: -
sayli karnik authored
This patch places assignments which are inside the if condition, above it. Done using coccinelle: @@ statement s; expression e; identifier id; @@ - if ((id = e)) + id = e; + if (id) s Signed-off-by: -
Sandhya Bankar authored
Remove unused variable. This change is made by below coccinelle script: @@ type T; identifier i; constant C; @@ ( extern T i; | - T i; <+... when != i - i = C; ...+> ) Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This issue was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
Do not initialise statics to 0. Static variable by default initialise to 0, so no need to explicit initialisation. This issue was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
This patch removes the explicit NULL comparison. This issue was found by checkpatch. Signed-off-by:
-
Brian Masney authored
Address warning from checkpatch: CHECK: Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. Linux already includes a copy of the GPL. Signed-off-by:
Brian Masney <masneyb@onstation.org> Signed-off-by:
-
Georgiana Rodica Chelu authored
Reorganize the parameters to improve the readability of the code in two manners: - parameters of distinctive variable types on different lines - parameters of similar variable types on the same line Also, take advantage of the horizontal space available and place all the parameters on a single line. Issues found by checkpatch.pl script. Signed-off-by:
Georgiana Rodica Chelu <georgiana.chelu93@gmail.com> Signed-off-by:
-
Georgiana Rodica Chelu authored
Organize the parameters on a single line to improve the readability of the code. Also, indent the line to match alignment with the open parenthesis. Ensure that the modifications do not break the rule of 80 characters per line. Issues found by checkpatch.pl script. Signed-off-by:
-
Rehas Sachdeva authored
Instead of storing the return value of a function call into a variable and then returning it, we can club the two into a single return statement. This change was made using the following semantic patch by Coccinelle: @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: -
Georgiana Rodica Chelu authored
Remove the braces {} in the following cases: - when there is a single branch with a single statement. - when both branches have a single statement The modifications improve the coding style. Issues found by checkpatch.pl script. Signed-off-by: -
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
-
Namrata A Shettar authored
Insert blank line after function declaration to resolve checkpatch issue. Signed-off-by:
Namrata A Shettar <namrataashettar@gmail.com> Signed-off-by:
-
Namrata A Shettar authored
Add space around binary operator '+' to resolve checkpatch issue. Signed-off-by:
-
Namrata A Shettar authored
Remove unneccessary blank lines to resolve checkpatch issue. Signed-off-by:
-
Namrata A Shettar authored
Add space around '+' to resolve checkpatch issue. Signed-off-by:
-
Bhumika Goyal authored
Some type conversions like casting a pointer to a pointer of same type, casting to the original type using addressof(&) operator etc. are not needed. Therefore, remove them. Done using coccinelle: @@ type t; t *p; t a; @@ ( - (t)(a) + a | - (t *)(p) + p | - (t *)(&a) + &a ) Signed-off-by:
Bhumika Goyal <bhumirks@gmail.com> Signed-off-by:
-
Xavier Roumegue authored
Fix the following warnings: Symbolic permissions are not preferred. Consider using octal permissions. Signed-off-by:
Xavier Roumegue <xroumegue@gmail.com> Signed-off-by:
-
