diff mbox series

igc: Tidy up some white space

Message ID 20181019195747.e4h2oihuyroi5vwn@kili.mountain
State Accepted
Delegated to: Jeff Kirsher
Headers show
Series igc: Tidy up some white space | expand

Commit Message

Dan Carpenter Oct. 19, 2018, 7:57 p.m. UTC
I just cleaned up a couple small white space issues.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
---
 drivers/net/ethernet/intel/igc/igc_main.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Sasha Neftin Oct. 24, 2018, 11:03 a.m. UTC | #1
On 10/19/2018 22:57, Dan Carpenter wrote:
> I just cleaned up a couple small white space issues.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> ---
>   drivers/net/ethernet/intel/igc/igc_main.c | 10 +++++-----
>   1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/net/ethernet/intel/igc/igc_main.c b/drivers/net/ethernet/intel/igc/igc_main.c
> index 9d85707e8a81..3233141c594b 100644
> --- a/drivers/net/ethernet/intel/igc/igc_main.c
> +++ b/drivers/net/ethernet/intel/igc/igc_main.c
> @@ -1108,7 +1108,7 @@ static struct sk_buff *igc_build_skb(struct igc_ring *rx_ring,
>   
>   	/* update pointers within the skb to store the data */
>   	skb_reserve(skb, IGC_SKB_PAD);
> -	 __skb_put(skb, size);
> +	__skb_put(skb, size);
>   
>   	/* update buffer offset */
>   #if (PAGE_SIZE < 8192)
> @@ -1160,9 +1160,9 @@ static struct sk_buff *igc_construct_skb(struct igc_ring *rx_ring,
>   				(va + headlen) - page_address(rx_buffer->page),
>   				size, truesize);
>   #if (PAGE_SIZE < 8192)
> -	rx_buffer->page_offset ^= truesize;
> +		rx_buffer->page_offset ^= truesize;
>   #else
> -	rx_buffer->page_offset += truesize;
> +		rx_buffer->page_offset += truesize;
>   #endif
>   	} else {
>   		rx_buffer->pagecnt_bias++;
> @@ -1668,8 +1668,8 @@ static bool igc_clean_tx_irq(struct igc_q_vector *q_vector, int napi_budget)
>   				tx_buffer->next_to_watch,
>   				jiffies,
>   				tx_buffer->next_to_watch->wb.status);
> -				netif_stop_subqueue(tx_ring->netdev,
> -						    tx_ring->queue_index);
> +			netif_stop_subqueue(tx_ring->netdev,
> +					    tx_ring->queue_index);
>   
>   			/* we are about to reset, no point in enabling stuff */
>   			return true;
> 
Thanks for the patch. Good catch of typos.

Acked-by: Sasha Neftin <sasha.neftin@intel.com>
Brown, Aaron F Nov. 1, 2018, 7:29 p.m. UTC | #2
> From: Intel-wired-lan [mailto:intel-wired-lan-bounces@osuosl.org] On
> Behalf Of Dan Carpenter
> Sent: Friday, October 19, 2018 12:58 PM
> To: Kirsher, Jeffrey T <jeffrey.t.kirsher@intel.com>; Neftin, Sasha
> <sasha.neftin@intel.com>
> Cc: kernel-janitors@vger.kernel.org; intel-wired-lan@lists.osuosl.org
> Subject: [Intel-wired-lan] [PATCH] igc: Tidy up some white space
> 
> I just cleaned up a couple small white space issues.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> ---
>  drivers/net/ethernet/intel/igc/igc_main.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 

Tested-by: Aaron Brown <aaron.f.brown@intel.com>
diff mbox series

Patch

diff --git a/drivers/net/ethernet/intel/igc/igc_main.c b/drivers/net/ethernet/intel/igc/igc_main.c
index 9d85707e8a81..3233141c594b 100644
--- a/drivers/net/ethernet/intel/igc/igc_main.c
+++ b/drivers/net/ethernet/intel/igc/igc_main.c
@@ -1108,7 +1108,7 @@  static struct sk_buff *igc_build_skb(struct igc_ring *rx_ring,
 
 	/* update pointers within the skb to store the data */
 	skb_reserve(skb, IGC_SKB_PAD);
-	 __skb_put(skb, size);
+	__skb_put(skb, size);
 
 	/* update buffer offset */
 #if (PAGE_SIZE < 8192)
@@ -1160,9 +1160,9 @@  static struct sk_buff *igc_construct_skb(struct igc_ring *rx_ring,
 				(va + headlen) - page_address(rx_buffer->page),
 				size, truesize);
 #if (PAGE_SIZE < 8192)
-	rx_buffer->page_offset ^= truesize;
+		rx_buffer->page_offset ^= truesize;
 #else
-	rx_buffer->page_offset += truesize;
+		rx_buffer->page_offset += truesize;
 #endif
 	} else {
 		rx_buffer->pagecnt_bias++;
@@ -1668,8 +1668,8 @@  static bool igc_clean_tx_irq(struct igc_q_vector *q_vector, int napi_budget)
 				tx_buffer->next_to_watch,
 				jiffies,
 				tx_buffer->next_to_watch->wb.status);
-				netif_stop_subqueue(tx_ring->netdev,
-						    tx_ring->queue_index);
+			netif_stop_subqueue(tx_ring->netdev,
+					    tx_ring->queue_index);
 
 			/* we are about to reset, no point in enabling stuff */
 			return true;