diff options
author | Al Viro <viro@ftp.linux.org.uk> | 2007-02-09 11:38:30 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-02-09 12:14:07 -0500 |
commit | 4ec031166f6a466a443f462e567f7551096b1741 (patch) | |
tree | 2f59cd620ddd83436a16c8e5a494d6147a398716 /drivers/net/e2100.c | |
parent | b81831c69afb82c0545d3de729290fab4e50d429 (diff) |
[PATCH] kill eth_io_copy_and_sum()
On all targets that sucker boils down to memcpy_fromio(sbk->data, from, len).
The function name is highly misguiding (it _never_ does any checksums), the
last argument is just a noise and simply expanding the call to memcpy_fromio()
gives shorter and more readable source. For a lot of reasons it has almost
no remaining users, so it's better to just outright kill it.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/net/e2100.c')
-rw-r--r-- | drivers/net/e2100.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/e2100.c b/drivers/net/e2100.c index c62d9c6363c6..b2b0a96218ca 100644 --- a/drivers/net/e2100.c +++ b/drivers/net/e2100.c | |||
@@ -355,8 +355,7 @@ e21_block_input(struct net_device *dev, int count, struct sk_buff *skb, int ring | |||
355 | 355 | ||
356 | mem_on(ioaddr, shared_mem, (ring_offset>>8)); | 356 | mem_on(ioaddr, shared_mem, (ring_offset>>8)); |
357 | 357 | ||
358 | /* Packet is always in one chunk -- we can copy + cksum. */ | 358 | memcpy_fromio(skb->data, ei_status.mem + (ring_offset & 0xff), count); |
359 | eth_io_copy_and_sum(skb, ei_status.mem + (ring_offset & 0xff), count, 0); | ||
360 | 359 | ||
361 | mem_off(ioaddr); | 360 | mem_off(ioaddr); |
362 | } | 361 | } |