diff options
| author |   Nickolai Zeldovich <nickolai@csail.mit.edu> | 2013-01-17 07:18:29 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2013-01-17 15:04:30 -0500 |
| commit | 13b6d2e6c25dd33f94873b771ad7de3550da5125 (patch) | |
| tree | af193686297ae611a69c5166a24ebc3f0c5a0d8d /drivers/net/ethernet/3com | |
| parent | Merge branch 'wireless' (diff) | |
| download | linux-dev-13b6d2e6c25dd33f94873b771ad7de3550da5125.tar.xz linux-dev-13b6d2e6c25dd33f94873b771ad7de3550da5125.zip | |
3c574_cs: fix operator precedence between << and &
The code to print the FIFO size in tc574_config computes it as:
8 << config & Ram_size
which evaluates the '<<' first, but the actual intent is to evaluate the
'&' first. Add parentheses to enforce desired evaluation order.
Signed-off-by: Nickolai Zeldovich <nickolai@csail.mit.edu>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/3com')
| -rw-r--r-- | drivers/net/ethernet/3com/3c574_cs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/3com/3c574_cs.c b/drivers/net/ethernet/3com/3c574_cs.c index 66df93638085..ffd8de28a76a 100644 --- a/drivers/net/ethernet/3com/3c574_cs.c +++ b/drivers/net/ethernet/3com/3c574_cs.c @@ -432,7 +432,7 @@ static int tc574_config(struct pcmcia_device *link) netdev_info(dev, "%s at io %#3lx, irq %d, hw_addr %pM\n", cardname, dev->base_addr, dev->irq, dev->dev_addr); netdev_info(dev, " %dK FIFO split %s Rx:Tx, %sMII interface.\n", - 8 << config & Ram_size, + 8 << (config & Ram_size), ram_split[(config & Ram_split) >> Ram_split_shift], config & Autoselect ? "autoselect " : ""); |