net/can/mscan: drop assignment in while-construct

As suggested by Wolfgang Grandegger.

Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 4 insertions, 2 deletions

diff --git a/drivers/net/can/mscan/mscan.c b/drivers/net/can/mscan/mscan.c
index 76e413e7fdb1..20d1991b9094 100644
--- a/drivers/net/can/mscan/mscan.c
+++ b/drivers/net/can/mscan/mscan.c
@@ -379,8 +379,10 @@ static int mscan_rx_poll(struct napi_struct *napi, int quota)
         struct can_frame *frame;
         u8 canrflg;
 
-        while (npackets < quota && ((canrflg = in_8(&regs->canrflg)) &
-                                    (MSCAN_RXF | MSCAN_ERR_IF))) {
+        while (npackets < quota) {
+                canrflg = in_8(&regs->canrflg);
+                if (!(canrflg & (MSCAN_RXF | MSCAN_ERR_IF)))
+                        break;
 
                 skb = alloc_can_skb(dev, &frame);
                 if (!skb) {