diff options
| author | Lennert Buytenhek <buytenh@marvell.com> | 2009-01-19 17:17:18 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-01-19 17:58:07 -0800 |
| commit | 2b448334a255d34401562229f467ffd95d8ed6ef (patch) | |
| tree | f009fd5590b57ce83c69f8b1f1a87fb01869ef48 /drivers/net/mv643xx_eth.c | |
| parent | e0c6ef9388b58f297937fc9651331941d1579b25 (diff) | |
| download | linux-2b448334a255d34401562229f467ffd95d8ed6ef.tar.gz linux-2b448334a255d34401562229f467ffd95d8ed6ef.tar.bz2 linux-2b448334a255d34401562229f467ffd95d8ed6ef.zip | |
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>
Diffstat (limited to 'drivers/net/mv643xx_eth.c')
| -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: entry = addr_crc(a); } - table[entry >> 2] |= 1 << (entry & 3); + table[entry >> 2] |= 1 << (8 * (entry & 3)); } for (i = 0; i < 0x100; i += 4) { |