diff options
author | Eric Dumazet <eric.dumazet@gmail.com> | 2010-09-23 01:40:09 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-09-26 21:34:29 -0400 |
commit | 807540baae406c84dcb9c1c8ef07a56d2d2ae84a (patch) | |
tree | ccd5c2cb57710dd6b73cf8df11eedf67abc14ae4 /drivers/net/atarilance.c | |
parent | cb4dfe562cac6fcb544df752e40c1d78000d0712 (diff) |
drivers/net: return operator cleanup
Change "return (EXPR);" to "return EXPR;"
return is not a function, parentheses are not required.
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/atarilance.c')
-rw-r--r-- | drivers/net/atarilance.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/drivers/net/atarilance.c b/drivers/net/atarilance.c index b57d7dee389a..3134e5326231 100644 --- a/drivers/net/atarilance.c +++ b/drivers/net/atarilance.c | |||
@@ -362,7 +362,7 @@ static void *slow_memcpy( void *dst, const void *src, size_t len ) | |||
362 | *cto++ = *cfrom++; | 362 | *cto++ = *cfrom++; |
363 | MFPDELAY(); | 363 | MFPDELAY(); |
364 | } | 364 | } |
365 | return( dst ); | 365 | return dst; |
366 | } | 366 | } |
367 | 367 | ||
368 | 368 | ||
@@ -449,7 +449,7 @@ static noinline int __init addr_accessible(volatile void *regp, int wordflag, | |||
449 | vbr[2] = save_berr; | 449 | vbr[2] = save_berr; |
450 | local_irq_restore(flags); | 450 | local_irq_restore(flags); |
451 | 451 | ||
452 | return( ret ); | 452 | return ret; |
453 | } | 453 | } |
454 | 454 | ||
455 | static const struct net_device_ops lance_netdev_ops = { | 455 | static const struct net_device_ops lance_netdev_ops = { |
@@ -526,7 +526,7 @@ static unsigned long __init lance_probe1( struct net_device *dev, | |||
526 | goto probe_ok; | 526 | goto probe_ok; |
527 | 527 | ||
528 | probe_fail: | 528 | probe_fail: |
529 | return( 0 ); | 529 | return 0; |
530 | 530 | ||
531 | probe_ok: | 531 | probe_ok: |
532 | lp = netdev_priv(dev); | 532 | lp = netdev_priv(dev); |
@@ -556,7 +556,7 @@ static unsigned long __init lance_probe1( struct net_device *dev, | |||
556 | if (request_irq(IRQ_AUTO_5, lance_interrupt, IRQ_TYPE_PRIO, | 556 | if (request_irq(IRQ_AUTO_5, lance_interrupt, IRQ_TYPE_PRIO, |
557 | "PAM/Riebl-ST Ethernet", dev)) { | 557 | "PAM/Riebl-ST Ethernet", dev)) { |
558 | printk( "Lance: request for irq %d failed\n", IRQ_AUTO_5 ); | 558 | printk( "Lance: request for irq %d failed\n", IRQ_AUTO_5 ); |
559 | return( 0 ); | 559 | return 0; |
560 | } | 560 | } |
561 | dev->irq = (unsigned short)IRQ_AUTO_5; | 561 | dev->irq = (unsigned short)IRQ_AUTO_5; |
562 | } | 562 | } |
@@ -568,12 +568,12 @@ static unsigned long __init lance_probe1( struct net_device *dev, | |||
568 | unsigned long irq = atari_register_vme_int(); | 568 | unsigned long irq = atari_register_vme_int(); |
569 | if (!irq) { | 569 | if (!irq) { |
570 | printk( "Lance: request for VME interrupt failed\n" ); | 570 | printk( "Lance: request for VME interrupt failed\n" ); |
571 | return( 0 ); | 571 | return 0; |
572 | } | 572 | } |
573 | if (request_irq(irq, lance_interrupt, IRQ_TYPE_PRIO, | 573 | if (request_irq(irq, lance_interrupt, IRQ_TYPE_PRIO, |
574 | "Riebl-VME Ethernet", dev)) { | 574 | "Riebl-VME Ethernet", dev)) { |
575 | printk( "Lance: request for irq %ld failed\n", irq ); | 575 | printk( "Lance: request for irq %ld failed\n", irq ); |
576 | return( 0 ); | 576 | return 0; |
577 | } | 577 | } |
578 | dev->irq = irq; | 578 | dev->irq = irq; |
579 | } | 579 | } |
@@ -637,7 +637,7 @@ static unsigned long __init lance_probe1( struct net_device *dev, | |||
637 | /* XXX MSch */ | 637 | /* XXX MSch */ |
638 | dev->watchdog_timeo = TX_TIMEOUT; | 638 | dev->watchdog_timeo = TX_TIMEOUT; |
639 | 639 | ||
640 | return( 1 ); | 640 | return 1; |
641 | } | 641 | } |
642 | 642 | ||
643 | 643 | ||
@@ -666,7 +666,7 @@ static int lance_open( struct net_device *dev ) | |||
666 | DPRINTK( 2, ( "lance_open(): opening %s failed, i=%d, csr0=%04x\n", | 666 | DPRINTK( 2, ( "lance_open(): opening %s failed, i=%d, csr0=%04x\n", |
667 | dev->name, i, DREG )); | 667 | dev->name, i, DREG )); |
668 | DREG = CSR0_STOP; | 668 | DREG = CSR0_STOP; |
669 | return( -EIO ); | 669 | return -EIO; |
670 | } | 670 | } |
671 | DREG = CSR0_IDON; | 671 | DREG = CSR0_IDON; |
672 | DREG = CSR0_STRT; | 672 | DREG = CSR0_STRT; |
@@ -676,7 +676,7 @@ static int lance_open( struct net_device *dev ) | |||
676 | 676 | ||
677 | DPRINTK( 2, ( "%s: LANCE is open, csr0 %04x\n", dev->name, DREG )); | 677 | DPRINTK( 2, ( "%s: LANCE is open, csr0 %04x\n", dev->name, DREG )); |
678 | 678 | ||
679 | return( 0 ); | 679 | return 0; |
680 | } | 680 | } |
681 | 681 | ||
682 | 682 | ||
@@ -1126,13 +1126,13 @@ static int lance_set_mac_address( struct net_device *dev, void *addr ) | |||
1126 | int i; | 1126 | int i; |
1127 | 1127 | ||
1128 | if (lp->cardtype != OLD_RIEBL && lp->cardtype != NEW_RIEBL) | 1128 | if (lp->cardtype != OLD_RIEBL && lp->cardtype != NEW_RIEBL) |
1129 | return( -EOPNOTSUPP ); | 1129 | return -EOPNOTSUPP; |
1130 | 1130 | ||
1131 | if (netif_running(dev)) { | 1131 | if (netif_running(dev)) { |
1132 | /* Only possible while card isn't started */ | 1132 | /* Only possible while card isn't started */ |
1133 | DPRINTK( 1, ( "%s: hwaddr can be set only while card isn't open.\n", | 1133 | DPRINTK( 1, ( "%s: hwaddr can be set only while card isn't open.\n", |
1134 | dev->name )); | 1134 | dev->name )); |
1135 | return( -EIO ); | 1135 | return -EIO; |
1136 | } | 1136 | } |
1137 | 1137 | ||
1138 | memcpy( dev->dev_addr, saddr->sa_data, dev->addr_len ); | 1138 | memcpy( dev->dev_addr, saddr->sa_data, dev->addr_len ); |
@@ -1142,7 +1142,7 @@ static int lance_set_mac_address( struct net_device *dev, void *addr ) | |||
1142 | /* set also the magic for future sessions */ | 1142 | /* set also the magic for future sessions */ |
1143 | *RIEBL_MAGIC_ADDR = RIEBL_MAGIC; | 1143 | *RIEBL_MAGIC_ADDR = RIEBL_MAGIC; |
1144 | 1144 | ||
1145 | return( 0 ); | 1145 | return 0; |
1146 | } | 1146 | } |
1147 | 1147 | ||
1148 | 1148 | ||