drivers: isdn: isdnloop: isdnloop.c: remove assignment of variables in if conditions...
authorBas Peters <baspeters93@gmail.com>
Wed, 4 Feb 2015 22:32:59 +0000 (23:32 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 5 Feb 2015 23:40:22 +0000 (15:40 -0800)
Signed-off-by: Bas Peters <baspeters93@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/isdnloop/isdnloop.c

index 5a4da94aefb056becbca88e5110282608b26df0c..af96317a52ee93b177b0ce047d749681ba4dbded 100644 (file)
@@ -59,7 +59,8 @@ isdnloop_bchan_send(isdnloop_card *card, int ch)
        isdn_ctrl cmd;
 
        while (card->sndcount[ch]) {
-               if ((skb = skb_dequeue(&card->bqueue[ch]))) {
+               skb = skb_dequeue(&card->bqueue[ch]);
+               if (skb) {
                        len = skb->len;
                        card->sndcount[ch] -= len;
                        ack = *(skb->head); /* used as scratch area */
@@ -317,7 +318,8 @@ isdnloop_polldchan(unsigned long data)
        u_char *p;
        isdn_ctrl cmd;
 
-       if ((skb = skb_dequeue(&card->dqueue)))
+       skb = skb_dequeue(&card->dqueue);
+       if (skb)
                avail = skb->len;
        else
                avail = 0;
@@ -471,8 +473,8 @@ isdnloop_fake(isdnloop_card *card, char *s, int ch)
 {
        struct sk_buff *skb;
        int len = strlen(s) + ((ch >= 0) ? 3 : 0);
-
-       if (!(skb = dev_alloc_skb(len))) {
+       skb = dev_alloc_skb(len);
+       if (!skb) {
                printk(KERN_WARNING "isdnloop: Out of memory in isdnloop_fake\n");
                return 1;
        }
@@ -1439,8 +1441,8 @@ isdnloop_initcard(char *id)
 {
        isdnloop_card *card;
        int i;
-
-       if (!(card = kzalloc(sizeof(isdnloop_card), GFP_KERNEL))) {
+       card = kzalloc(sizeof(isdnloop_card), GFP_KERNEL);
+       if (!card) {
                printk(KERN_WARNING
                       "isdnloop: (%s) Could not allocate card-struct.\n", id);
                return (isdnloop_card *) 0;
@@ -1489,8 +1491,8 @@ static int
 isdnloop_addcard(char *id1)
 {
        isdnloop_card *card;
-
-       if (!(card = isdnloop_initcard(id1))) {
+       card = isdnloop_initcard(id1);
+       if (!card) {
                return -EIO;
        }
        printk(KERN_INFO