- 06 Mar, 2017 40 commits
-
-
James Simmons authored
Change lnet_me_t from typedef to proper structure. Signed-off-by:
James Simmons <uja.ornl@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831Reviewed-by:
Olaf Weber <olaf@sgi.com> Reviewed-by:
Doug Oucharek <doug.s.oucharek@intel.com> Reviewed-by:
Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
James Simmons authored
Change lnet_eq_t from typedef to proper structure. Signed-off-by:
-
James Simmons authored
Change lnet_libhandle_t from typedef to proper structure. Signed-off-by:
-
James Simmons authored
Change lnet_msg_t from typedef to proper structure. Signed-off-by:
-
James Simmons authored
With the creation of unique handlers we can remove the barely used generic handlers. Signed-off-by:
-
James Simmons authored
Change lnet_handle_me_t from a typedef of another typedef into a proper stand alone structure. Signed-off-by:
-
James Simmons authored
Change lnet_handle_md_t from a typedef of another typedef into a proper stand alone structure. Create the inline functions LNetInvalidateMDHandle and LNetMDHandleIsInvalid to handle this new piece of data. Signed-off-by:
-
James Simmons authored
Change lnet_handle_eq_t from a typedef of another typedef into a proper stand alone structure. Create the inline functions LNetInvalidateEQHandle and LNetEQHandleIsInvalid to handle this new piece of data. Signed-off-by:
-
James Simmons authored
The upstream kernel requires proper structures so convert all the UAPI typedefs in socklnd.h. Signed-off-by:
John L. Hammond <john.hammond@intel.com> Reviewed-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style fo block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments. Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Alexander Mikhalevich authored
WARNING: Avoid multiple line dereference - prefer 'param->u.crypt.key' Signed-off-by:
Alexander Mikhalevich <alex.mikhalevich@gmail.com> Acked-by:
Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by:
-
Varsha Rao authored
This patch fixes the following checkpatch issue: CHECK: No space is necessary after a cast Signed-off-by:
Varsha Rao <rvarsha016@gmail.com> Acked-by:
Julia Lawall <julia.lawall@lip6.fr> Signed-off-by:
-
Varsha Rao authored
This patch fixes the checkpatch issue by removing the Free Software Foundation's mailing address from the sample GPL notice. Because the FSF has changed address in the past, and may change again. Linux already includes a copy of the GPL. Signed-off-by:
-
Abhijit Naik authored
bcm2835-vchiq.c: fixing WARNING: braces {} are not necessary for single statement blocks Signed-off-by:Abhijit Naik <abhijitnaik27@gmail.com> Signed-off-by:
-
Abhijit Naik authored
bcm2835-vchiq.c: fixing WARNING: void function return statements are not generally useful Signed-off-by:
-
Nathan Howard authored
Fix checkpatch.pl warning of the form "CHECK: Macro argument 'vol' may be better as '(vol)' to avoid precedence issues." Signed-off-by:
Nathan Howard <adanhawthorn@gmail.com> Signed-off-by:
-
Nathan Howard authored
Fix checkpatch.pl warning of the form "WARNING: Prefer subsystem eg: netdev]_info([subsystem]dev, ... then dev_info(dev, ... then pr_info(... to printk(KERN_INFO ..." Signed-off-by:
-
Nathan Howard authored
Fix checkpatch.pl warning: "Block comments use * on subsequent lines." Signed-off-by:
-
Daniel Perez de Andres authored
A single line statement under an if clause doesn't require braces {} Signed-off-by:Daniel Perez de Andres <danielperezdeandres@gmail.com> Signed-off-by:
-
Dan Carpenter authored
The unwinding here isn't right. We don't free gdev[0] and instead free 1 step past what was allocated. Also we can't allocate "dev" then we should unwind instead of returning directly. Fixes: 7b3ad5ab ("staging: Import the BCM2835 MMAL-based V4L2 camera driver.") Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Acked-by:
walter harms <wharms@bfs.de> Signed-off-by:
-
Gargi Sharma authored
Relational and logical operators evaluate to either true or false. Explicit conversion is not needed so remove the ternary operator. Done using coccinelle: @r@ expression A,B; symbol true,false; binary operator b = {==,!=,&&,||,>=,<=,>,<}; @@ - (A b B) ? true : false + A b B Signed-off-by:Gargi Sharma <gs051095@gmail.com> Acked-by:
-
Gargi Sharma authored
Replace explicit NULL comparison with ! operator to simplify code. Found with Coccinelle script: @@ expression ptr; position p; statement s0, s1; @@ ptr@p = \(kmalloc\|devm_kzalloc\|kmalloc_array\|devm_ioremap\|usb_alloc_urb\| alloc_netdev\|dev_alloc_skb\)(...) ... when != ptr if ( ( + ! ptr - == NULL ) ) s0 else s1 Signed-off-by:
-
Yamanappagouda Patil authored
Fixed checkpath.pl erros "space prohibited after that open/close parenthesis" in vchiq_version.c file. Signed-off-by:
Yamanappagouda Patil <goudapatilk@gmail.com> Signed-off-by:
-
simran singhal authored
The macro DIV_ROUND_UP performs the computation (((n) + (d) - 1) /(d)). It clarifies the divisor calculations. This occurence was detected using the coccinelle script: @@ expression e1; expression e2; @@ ( - ((e1) + e2 - 1) / (e2) + DIV_ROUND_UP(e1,e2) | - ((e1) + (e2 - 1)) / (e2) + DIV_ROUND_UP(e1,e2) ) Signed-off-by:
simran singhal <singhalsimran0@gmail.com> Signed-off-by:
-
Yamanappagouda Patil authored
Fixed checkpatch.pl "missing a blank line after declarations" warning messages in vc04_services module. Signed-off-by:
-
simran singhal authored
Simplify function returns by merging assignment and return into one command line. Found with Coccinelle @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: -
Katie Dunne authored
Removes unnecessary blank lines after opening and before closing braces. These instances were found by checkpatch.pl. Signed-off-by:
Katie Dunne <kdunne@mail.ccsf.edu> Acked-by:
-
simran singhal authored
The macro DIV_ROUND_UP performs the computation (((n) + (d) - 1) /(d)). It clarifies the divisor calculations. This occurence was detected using the coccinelle script: @@ expression e1; expression e2; @@ ( - ((e1) + e2 - 1) / (e2) + DIV_ROUND_UP(e1,e2) | - ((e1) + (e2 - 1)) / (e2) + DIV_ROUND_UP(e1,e2) ) Signed-off-by:
-
Katie Dunne authored
Add spaces around operators -, *, ?:, >>, << to conform to kernel style. These instances were found with checkpatch.pl Signed-off-by:
-
Katie Dunne authored
Add curly braces to if-statements for style compliance. These cases are found by checkpatch.pl Signed-off-by:
-
Sreya Mittal authored
Put logical continuations on the next line; remove useless parentheses; and line up the tests. Signed-off-by:
Sreya Mittal <sreyamittal5@gmail.com> Acked-by:
-
sayli karnik authored
The patch adds spaces around binary operators as suggested by checkpatch.pl Signed-off-by:
sayli karnik <karniksayli1995@gmail.com> Signed-off-by:
-
sayli karnik authored
The patch removes the explicit NULL comparison by replacing 'x!=NULL' with 'x'. This issue was found by checkpatch.pl Signed-off-by:
-
