* [PATCH] staging: octeon: Add spaces around operators
@ 2016-02-10 2:26 Janani Ravichandran
2016-02-10 6:21 ` [Outreachy kernel] " Sudip Mukherjee
0 siblings, 1 reply; 3+ messages in thread
From: Janani Ravichandran @ 2016-02-10 2:26 UTC (permalink / raw)
To: outreachy-kernel
Add spaces around operators for better readability. Change suggested by
checkpatch.
Signed-off-by: Janani Ravichandran <janani.rvchndrn@gmail.com>
---
drivers/staging/octeon/ethernet-tx.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/octeon/ethernet-tx.c b/drivers/staging/octeon/ethernet-tx.c
index 82abaec..a08ea3f 100644
--- a/drivers/staging/octeon/ethernet-tx.c
+++ b/drivers/staging/octeon/ethernet-tx.c
@@ -419,7 +419,7 @@ dont_put_skbuff_in_hw:
cvmx_fau_fetch_and_add32(FAU_NUM_PACKET_BUFFERS_TO_FREE, 0);
}
- skb_to_free = cvm_oct_adjust_skb_to_free(skb_to_free, priv->fau+qos*4);
+ skb_to_free = cvm_oct_adjust_skb_to_free(skb_to_free, priv->fau + qos * 4);
/*
* If we're sending faster than the receive can free them then
@@ -430,7 +430,7 @@ dont_put_skbuff_in_hw:
if (pko_command.s.dontfree) {
queue_type = QUEUE_CORE;
- pko_command.s.reg0 = priv->fau+qos*4;
+ pko_command.s.reg0 = priv->fau + qos * 4;
} else {
queue_type = QUEUE_HW;
}
--
2.5.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [Outreachy kernel] [PATCH] staging: octeon: Add spaces around operators
2016-02-10 2:26 [PATCH] staging: octeon: Add spaces around operators Janani Ravichandran
@ 2016-02-10 6:21 ` Sudip Mukherjee
2016-02-10 8:44 ` Janani Ravichandran
0 siblings, 1 reply; 3+ messages in thread
From: Sudip Mukherjee @ 2016-02-10 6:21 UTC (permalink / raw)
To: Janani Ravichandran; +Cc: outreachy-kernel
On Tue, Feb 09, 2016 at 09:26:16PM -0500, Janani Ravichandran wrote:
> Add spaces around operators for better readability. Change suggested by
> checkpatch.
>
> Signed-off-by: Janani Ravichandran <janani.rvchndrn@gmail.com>
> ---
> drivers/staging/octeon/ethernet-tx.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/octeon/ethernet-tx.c b/drivers/staging/octeon/ethernet-tx.c
> index 82abaec..a08ea3f 100644
> --- a/drivers/staging/octeon/ethernet-tx.c
> +++ b/drivers/staging/octeon/ethernet-tx.c
> @@ -419,7 +419,7 @@ dont_put_skbuff_in_hw:
> cvmx_fau_fetch_and_add32(FAU_NUM_PACKET_BUFFERS_TO_FREE, 0);
> }
>
> - skb_to_free = cvm_oct_adjust_skb_to_free(skb_to_free, priv->fau+qos*4);
> + skb_to_free = cvm_oct_adjust_skb_to_free(skb_to_free, priv->fau + qos * 4);
oops.. some thing went wrong here. why so much space before 4?
regards
sudip
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-02-10 8:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-10 2:26 [PATCH] staging: octeon: Add spaces around operators Janani Ravichandran
2016-02-10 6:21 ` [Outreachy kernel] " Sudip Mukherjee
2016-02-10 8:44 ` Janani Ravichandran
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.