can: rx-offload: can_rx_offload_irq_offload_timestamp(): don't use assignment in...
authorMarc Kleine-Budde <mkl@pengutronix.de>
Mon, 7 Oct 2019 08:00:52 +0000 (10:00 +0200)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Mon, 11 Nov 2019 20:58:10 +0000 (21:58 +0100)
This patch moves the assignment of queue_len out of the if condition.

Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/rx-offload.c

index c277c51..412a8ce 100644 (file)
@@ -238,8 +238,8 @@ int can_rx_offload_irq_offload_timestamp(struct can_rx_offload *offload,
                skb_queue_splice_tail(&skb_queue, &offload->skb_queue);
                spin_unlock_irqrestore(&offload->skb_queue.lock, flags);
 
-               if ((queue_len = skb_queue_len(&offload->skb_queue)) >
-                   (offload->skb_queue_len_max / 8))
+               queue_len = skb_queue_len(&offload->skb_queue);
+               if (queue_len > offload->skb_queue_len_max / 8)
                        netdev_dbg(offload->dev, "%s: queue_len=%d\n",
                                   __func__, queue_len);