sfc: Remove unnecessary condition for processing the TX timestamp queue
authorBen Hutchings <bhutchings@solarflare.com>
Thu, 28 Nov 2013 18:58:12 +0000 (18:58 +0000)
committerBen Hutchings <bhutchings@solarflare.com>
Thu, 12 Dec 2013 22:07:18 +0000 (22:07 +0000)
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
drivers/net/ethernet/sfc/ptp.c

index 06c6420f40d7b23e2a6b4d4e6fcb7eb1b364f33b..084e2d44790f7bdac2a9b170fbef54edfdd575df 100644 (file)
@@ -890,13 +890,10 @@ static enum ptp_packet_state efx_ptp_match_rx(struct efx_nic *efx,
 /* Process any queued receive events and corresponding packets
  *
  * q is returned with all the packets that are ready for delivery.
- * true is returned if at least one of those packets requires
- * synchronisation.
  */
-static bool efx_ptp_process_events(struct efx_nic *efx, struct sk_buff_head *q)
+static void efx_ptp_process_events(struct efx_nic *efx, struct sk_buff_head *q)
 {
        struct efx_ptp_data *ptp = efx->ptp_data;
-       bool rc = false;
        struct sk_buff *skb;
 
        while ((skb = skb_dequeue(&ptp->rxq))) {
@@ -907,7 +904,6 @@ static bool efx_ptp_process_events(struct efx_nic *efx, struct sk_buff_head *q)
                        __skb_queue_tail(q, skb);
                } else if (efx_ptp_match_rx(efx, skb) ==
                           PTP_PACKET_STATE_MATCHED) {
-                       rc = true;
                        __skb_queue_tail(q, skb);
                } else if (time_after(jiffies, match->expiry)) {
                        match->state = PTP_PACKET_STATE_TIMED_OUT;
@@ -921,8 +917,6 @@ static bool efx_ptp_process_events(struct efx_nic *efx, struct sk_buff_head *q)
                        break;
                }
        }
-
-       return rc;
 }
 
 /* Complete processing of a received packet */
@@ -1088,12 +1082,10 @@ static void efx_ptp_worker(struct work_struct *work)
        efx_ptp_drop_time_expired_events(efx);
 
        __skb_queue_head_init(&tempq);
-       if (efx_ptp_process_events(efx, &tempq) ||
-           !skb_queue_empty(&ptp_data->txq)) {
+       efx_ptp_process_events(efx, &tempq);
 
-               while ((skb = skb_dequeue(&ptp_data->txq)))
-                       efx_ptp_xmit_skb(efx, skb);
-       }
+       while ((skb = skb_dequeue(&ptp_data->txq)))
+               efx_ptp_xmit_skb(efx, skb);
 
        while ((skb = __skb_dequeue(&tempq)))
                efx_ptp_process_rx(efx, skb);