diff options
author | Miao Yan <yanmiaobest@gmail.com> | 2015-12-21 02:07:02 -0800 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2016-01-04 12:25:35 -0500 |
commit | a40b2dff7bec6c1656a59ce31c16207b8954eb68 (patch) | |
tree | 9f2ff24c83b669061f960ec0ab1957a210f4556b /drivers/net/e1000.c | |
parent | 18d3f46995319dfcb7f73c7c6a5e5da6b8e57e69 (diff) | |
download | u-boot-a40b2dff7bec6c1656a59ce31c16207b8954eb68.tar.gz |
net: e1000: use correct helper to do endianness conversion
In struct e1000_rx_desc, field 'length' is declared as
uint16_t, so use le16_to_cpu() to do endianness conversion.
Also drop conversion on 'status' which is declared as
uint8_t.
Signed-off-by: Miao Yan <yanmiaobest@gmail.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Diffstat (limited to 'drivers/net/e1000.c')
-rw-r--r-- | drivers/net/e1000.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/e1000.c b/drivers/net/e1000.c index 2ba03ed73e..6124bf0ab3 100644 --- a/drivers/net/e1000.c +++ b/drivers/net/e1000.c @@ -5165,11 +5165,11 @@ _e1000_poll(struct e1000_hw *hw) inval_end = inval_start + roundup(sizeof(*rd), ARCH_DMA_MINALIGN); invalidate_dcache_range(inval_start, inval_end); - if (!(le32_to_cpu(rd->status)) & E1000_RXD_STAT_DD) + if (!(rd->status & E1000_RXD_STAT_DD)) return 0; /* DEBUGOUT("recv: packet len=%d\n", rd->length); */ /* Packet received, make sure the data are re-loaded from RAM. */ - len = le32_to_cpu(rd->length); + len = le16_to_cpu(rd->length); invalidate_dcache_range((unsigned long)packet, (unsigned long)packet + roundup(len, ARCH_DMA_MINALIGN)); |