diff options
author | Alexey <darkover@corbina.com.ua> | 2012-05-24 22:41:40 +0300 |
---|---|---|
committer | Michael Brown <mcb30@ipxe.org> | 2012-07-23 13:22:43 +0100 |
commit | 8f17955c03c139ca8a1b728d86d7b36a992941a8 (patch) | |
tree | bea252dac0893ed285390d806a93811d0af72b94 | |
parent | c3b4860ce3fb25b907a2ca3e46955df34c0ae9fd (diff) | |
download | ipxe-8f17955c03c139ca8a1b728d86d7b36a992941a8.zip ipxe-8f17955c03c139ca8a1b728d86d7b36a992941a8.tar.gz ipxe-8f17955c03c139ca8a1b728d86d7b36a992941a8.tar.bz2 |
[epic100] Fix wrong field used as rx packet length
Datasheet pp. 41-42 defines 'rx packet length' as upper word of
'status' dword field of the receive descriptor table.
http://www.smsc.com/media/Downloads_Archive/discontinued/83c171.pdf
Tested on SMC EtherPower II.
Signed-off-by: Alexey Smazhenko <darkover@corbina.com.ua>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
-rw-r--r-- | src/drivers/net/epic100.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/drivers/net/epic100.c b/src/drivers/net/epic100.c index b478fab..884eb72 100644 --- a/src/drivers/net/epic100.c +++ b/src/drivers/net/epic100.c @@ -376,7 +376,7 @@ epic100_poll(struct nic *nic, int retrieve) { int entry; int retcode; - int status; + unsigned long status; entry = cur_rx % RX_RING_SIZE; if ((rx_ring[entry].status & cpu_to_le32(RRING_OWN)) == RRING_OWN) @@ -401,7 +401,7 @@ epic100_poll(struct nic *nic, int retrieve) retcode = 0; } else { /* Omit the four octet CRC from the length. */ - nic->packetlen = le32_to_cpu((rx_ring[entry].buflength))- 4; + nic->packetlen = (status >> 16) - 4; memcpy(nic->packet, &rx_packet[entry * PKT_BUF_SZ], nic->packetlen); retcode = 1; } |