- 16 Nov, 2015 40 commits
-
-
Chaehyun Lim authored
This patch renames DIM_Startup to dim_startup to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: <DIM_Startup> FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:653: Signed-off-by:
Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Chaehyun Lim authored
This patch adds space around '<<' found by checkpatch. CHECK: spaces preferred around that '<<' (ctx:VxV) FILE: drivers/staging/most/hdm-dim2/dim2_reg.h:69: Signed-off-by:
-
Chaehyun Lim authored
This patch removes blank line after an open brace found by checkpatch. CHECK: Blank lines aren't necessary after an open brace '{' Signed-off-by: -
Chaehyun Lim authored
This patch removes multiple blank lines found by checkpatch. CHECK: Please don't use multiple blank lines Signed-off-by:
-
Ivan Safonov authored
malloc error handling moved into one place. Signed-off-by:
Ivan Safonov <insafonov@gmail.com> Signed-off-by:
-
Ivan Safonov authored
goto is not needed here. Signed-off-by:
-
Ivan Safonov authored
IS_* and GET_CVID_* macros have not been used. Signed-off-by:
-
Ivan Safonov authored
IS_* macros (except one) occur only once. Signed-off-by:
-
Ivan Safonov authored
The range of elements to fill with zeros is determined by using a roundup macro Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Ivan Safonov authored
This name is better suited for this variable. Signed-off-by:
-
Ivan Safonov authored
This name is better suited for this variable. Signed-off-by:
-
Ivan Safonov authored
This name is better suited for this variable. Signed-off-by:
-
Ivan Safonov authored
checkpatch fix: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: -
Ivan Safonov authored
It is superfluous. Signed-off-by:
-
Ivan Safonov authored
The array should not change in any case. Signed-off-by:
-
Ivan Safonov authored
It is simpler. Signed-off-by:
-
Ivan Safonov authored
It makes the code little easier. Signed-off-by:
-
Ivan Safonov authored
If the 'remain' is zero, the loop is not executed at all. Signed-off-by:
-
Ivan Safonov authored
Variable value calculated in the previous loop. Signed-off-by:
-
Ivan Safonov authored
It is now possible to get rid of re-initializing the offset variable. Signed-off-by:
-
Ivan Safonov authored
Line become shorter. After the loop offset variable points to the location following insertion. Signed-off-by:
-
Ivan Safonov authored
It is made to the initial value could be placed in the offset variable. Signed-off-by:
-
Ivan Safonov authored
It is better to read. Signed-off-by:
-
Abdul Hussain authored
This patch fixes the following checkpatch.pl error: fix space prohibited before that ',' Signed-off-by:
Abdul Hussain <hussain.abdul@outlook.com> Signed-off-by:
-
Ivan Safonov authored
The body of this loop is executed only once, so it can be removed. In this loop no keyword 'continue', only 'break' at the end. Signed-off-by:
-
Ivan Safonov authored
Here is more suitable for loop. Signed-off-by:
-
Ivan Safonov authored
It is a simple and clear representation of this loop. Signed-off-by:
-
Rémy Oudompheng authored
Previously the code could exit with failure too early. Signed-off-by:
Rémy Oudompheng <remyoudompheng@gmail.com> Signed-off-by:
-
Rémy Oudompheng authored
Remove rtw_get_passing_time_ms function and adjust type of relevant variables. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function ldlm_pool_set_limit() and replace its calls with the function it wrapped. Signed-off-by:
Shivani Bhardwaj <shivanib134@gmail.com> Acked-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function ldlm_pool_get_limit() and replace its calls with the function it wrapped. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function ldlm_pl2ns() and replace its calls with the function it wrapped. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the prototype of function cfs_trace_free_string_buffer() as it is no longer needed. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function cfs_trace_free_string_buffer() as it can be replaced with the standard function kfree(). Signed-off-by:
-
Shivani Bhardwaj authored
Replace the calls of function cfs_trace_free_string_buffer() with kfree() as the former function is not required. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function ll_check_page() and replace all its calls with the function it wrapped. Signed-off-by:
-
Amitoj Kaur Chawla authored
This patch moves extern declarations in socklnd_lib.c to the respective header file, 'socklnd.h'. This patch also removes extern keyword from function declarations since functions have the extern specifier by default. Signed-off-by:
Amitoj Kaur Chawla <amitoj1606@gmail.com> Acked-by:
-
Markus Elfring authored
The variable "rc" will be eventually set to an error return code in the class_register_type() function. Thus let us omit the explicit initialisation at the beginning. Signed-off-by:
Markus Elfring <elfring@users.sourceforge.net> Signed-off-by:
-
Sebastien Buisson authored
Fix 'error handling' issues found by Coverity version 6.5.1: Unchecked return value (CHECKED_RETURN) Calling function without checking return value. Signed-off-by:
Sebastien Buisson <sebastien.buisson@bull.net> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3427 Reviewed-on: http://review.whamcloud.com/7103Reviewed-by:
Bobbie Lind <bobbie.j.lind@intel.com> Reviewed-by:
Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
-
Liang Zhen authored
page_collection::pc_lock is supposed to protect race between functions called by smp_call_function(), however we don't have this use-case for ages and page_collection only lives in stack of thread, so it is safe to remove it. Signed-off-by:
Liang Zhen <liang.zhen@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3055 Reviewed-on: http://review.whamcloud.com/7660Reviewed-by:
Bobi Jam <bobijam@gmail.com> Reviewed-by:
-
