diff options
author | Lennert Buytenhek <buytenh@marvell.com> | 2009-01-19 20:17:18 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-01-19 20:58:07 -0500 |
commit | 2b448334a255d34401562229f467ffd95d8ed6ef (patch) | |
tree | f009fd5590b57ce83c69f8b1f1a87fb01869ef48 | |
parent | e0c6ef9388b58f297937fc9651331941d1579b25 (diff) |
mv643xx_eth: fix multicast filter programming
Commit 66e63ffbc04706568d8789cbb00eaa8ddbcae648 ("mv643xx_eth:
implement ->set_rx_mode()") cleaned up mv643xx_eth's multicast filter
programming, but broke it as well.
The non-special multicast filter table (for multicast addresses that
are not of the form 01:00:5e:00:00:xx) consists of 256 hash table
buckets organised as 64 32-bit words, where the 'accept' bits are
in the LSB of each byte, so in bits 24 16 8 0 of each 32-bit word.
The old code got this right, but the referenced commit broke this by
using bits 3 2 1 0 instead. This commit fixes this up.
Signed-off-by: Lennert Buytenhek <buytenh@marvell.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/mv643xx_eth.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/mv643xx_eth.c b/drivers/net/mv643xx_eth.c index e2aa468d40f1..8c6979a26d61 100644 --- a/drivers/net/mv643xx_eth.c +++ b/drivers/net/mv643xx_eth.c | |||
@@ -1596,7 +1596,7 @@ oom: | |||
1596 | entry = addr_crc(a); | 1596 | entry = addr_crc(a); |
1597 | } | 1597 | } |
1598 | 1598 | ||
1599 | table[entry >> 2] |= 1 << (entry & 3); | 1599 | table[entry >> 2] |= 1 << (8 * (entry & 3)); |
1600 | } | 1600 | } |
1601 | 1601 | ||
1602 | for (i = 0; i < 0x100; i += 4) { | 1602 | for (i = 0; i < 0x100; i += 4) { |