staging: i4l: act2000: do not assign in if
authorSudip Mukherjee <sudipm.mukherjee@gmail.com>
Sat, 30 Apr 2016 21:33:03 +0000 (22:33 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 1 May 2016 21:33:34 +0000 (14:33 -0700)
It is not the kernel coding style to assign values to some variable in
if statement. Split them up into different statements.

Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/i4l/act2000/act2000_isa.c

index 34c0a9ab0810a0db9b8b0f80dfb3a5aba48f8bdc..048507e9b9a81e3629d3a3edfe2303b2a7021283 100644 (file)
@@ -31,7 +31,8 @@ act2000_isa_reset(unsigned short portbase)
        int serial = 0;
 
        found = 0;
-       if ((reg = inb(portbase + ISA_COR)) != 0xff) {
+       reg = inb(portbase + ISA_COR);
+       if (reg != 0xff) {
                outb(reg | ISA_COR_RESET, portbase + ISA_COR);
                mdelay(10);
                outb(reg, portbase + ISA_COR);
@@ -303,7 +304,8 @@ act2000_isa_send(act2000_card *card)
        while (1) {
                spin_lock_irqsave(&card->lock, flags);
                if (!(card->sbuf)) {
-                       if ((card->sbuf = skb_dequeue(&card->sndq))) {
+                       card->sbuf = skb_dequeue(&card->sndq);
+                       if (card->sbuf) {
                                card->ack_msg = card->sbuf->data;
                                msg = (actcapi_msg *)card->sbuf->data;
                                if ((msg->hdr.cmd.cmd == 0x86) &&
@@ -378,7 +380,8 @@ act2000_isa_getid(act2000_card *card)
                printk(KERN_WARNING "act2000: Wrong Firmware-ID!\n");
                return -EPROTO;
        }
-       if ((p = strchr(fid.revision, '\n')))
+       p = strchr(fid.revision, '\n');
+       if (p)
                *p = '\0';
        printk(KERN_INFO "act2000: Firmware-ID: %s\n", fid.revision);
        if (card->flags & ACT2000_FLAGS_IVALID) {