[OPW,kernel,v2,4/7] Staging: lustre: Fix space required after ','
diff mbox

Message ID 8d64e4de1c46d2176360131ba39e8ac4a9780221.1382784520.git.rashika.kheria@gmail.com
State Accepted
Headers show

Commit Message

Rashika Oct. 26, 2013, 10:53 a.m. UTC
This patch fixes the following checkpatch.pl warning in lustre/ldlm/interval_tree.c-
ERROR: space required after that ',' (ctx:VxV)

Signed-off-by: Rashika Kheria <rashika.kheria@gmail.com>
---

This revision fixes the following issues of the previous revision-
Incorrect file name

 drivers/staging/lustre/lustre/ldlm/interval_tree.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Waskiewicz Jr, Peter P Oct. 26, 2013, 8:24 p.m. UTC | #1
On Sat, 2013-10-26 at 16:23 +0530, Rashika Kheria wrote:
> This patch fixes the following checkpatch.pl warning in lustre/ldlm/interval_tree.c-
> ERROR: space required after that ',' (ctx:VxV)
> 
> Signed-off-by: Rashika Kheria <rashika.kheria@gmail.com>

Reviewed-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>

> ---
> 
> This revision fixes the following issues of the previous revision-
> Incorrect file name
> 
>  drivers/staging/lustre/lustre/ldlm/interval_tree.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/lustre/lustre/ldlm/interval_tree.c b/drivers/staging/lustre/lustre/ldlm/interval_tree.c
> index 361f692..8669969 100644
> --- a/drivers/staging/lustre/lustre/ldlm/interval_tree.c
> +++ b/drivers/staging/lustre/lustre/ldlm/interval_tree.c
> @@ -239,7 +239,7 @@ static void __rotate_change_maxhigh(struct interval_node *node,
>  	left_max = node->in_left ? node->in_left->in_max_high : 0;
>  	right_max = node->in_right ? node->in_right->in_max_high : 0;
>  	node->in_max_high  = max_u64(interval_high(node),
> -				     max_u64(left_max,right_max));
> +				     max_u64(left_max, right_max));
>  }
>  
>  /* The left rotation "pivots" around the link from node to node->right, and
> -- 
> 1.7.9.5
>

Patch
diff mbox

diff --git a/drivers/staging/lustre/lustre/ldlm/interval_tree.c b/drivers/staging/lustre/lustre/ldlm/interval_tree.c
index 361f692..8669969 100644
--- a/drivers/staging/lustre/lustre/ldlm/interval_tree.c
+++ b/drivers/staging/lustre/lustre/ldlm/interval_tree.c
@@ -239,7 +239,7 @@  static void __rotate_change_maxhigh(struct interval_node *node,
 	left_max = node->in_left ? node->in_left->in_max_high : 0;
 	right_max = node->in_right ? node->in_right->in_max_high : 0;
 	node->in_max_high  = max_u64(interval_high(node),
-				     max_u64(left_max,right_max));
+				     max_u64(left_max, right_max));
 }
 
 /* The left rotation "pivots" around the link from node to node->right, and