arcnet: Remove assignments from ifs
authorJoe Perches <joe@perches.com>
Tue, 5 May 2015 17:05:59 +0000 (10:05 -0700)
committerMichael Grzeschik <m.grzeschik@pengutronix.de>
Wed, 23 Sep 2015 06:44:25 +0000 (08:44 +0200)
Move the assignment above the if like general kernel style.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
drivers/net/arcnet/com20020-isa.c
drivers/net/arcnet/com20020.c
drivers/net/arcnet/com90io.c

index f3b2222bcada4f4e447c28c7322aba89dbbc373a..352500b8d8752cafe7ed3d92d912d68a806a6c7e 100644 (file)
@@ -107,7 +107,9 @@ static int __init com20020isa_probe(struct net_device *dev)
        }
 
        lp->card_name = "ISA COM20020";
-       if ((err = com20020_found(dev, 0)) != 0)
+
+       err = com20020_found(dev, 0);
+       if (err != 0)
                goto out;
 
        return 0;
index e813fc619ac90bc121b204f62ba670a91f31b3e6..bea386222149453d55268cf05b47e2432d45de3d 100644 (file)
@@ -140,7 +140,8 @@ int com20020_check(struct net_device *dev)
        outb(0 | RDDATAflag | AUTOINCflag, _ADDR_HI);
        outb(0, _ADDR_LO);
 
-       if ((status = inb(_MEMDATA)) != TESTvalue) {
+       status = inb(_MEMDATA);
+       if (status != TESTvalue) {
                arc_printk(D_NORMAL, dev, "Signature byte not found (%02Xh != D1h).\n",
                           status);
                return -ENODEV;
index 32abaa8d0dc419bf41772fd153b659c93df0d517..11bc589aa8d120e2e2c534b7bd7b595772b19e80 100644 (file)
@@ -196,7 +196,8 @@ static int __init com90io_probe(struct net_device *dev)
        outb(AUTOINCflag, _ADDR_HI);
        outb(0, _ADDR_LO);
 
-       if ((status = inb(_MEMDATA)) != 0xd1) {
+       status = inb(_MEMDATA);
+       if (status != 0xd1) {
                arc_printk(D_INIT_REASONS, dev, "Signature byte not found (%Xh instead).\n",
                           status);
                goto err_out;