diff options
author | Malcolm Priestley <tvboxspy@gmail.com> | 2012-08-29 23:08:21 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-09-04 13:54:23 -0700 |
commit | aa209eef3ce8419ff2926c2fa944dfbfb5afbacb (patch) | |
tree | 477da19a81b43da810a0cc70d0ac52fa136755d0 /drivers/staging/vt6656/dpc.c | |
parent | 07732be2efa7d4ade66d224ea84d2c30740d0b77 (diff) | |
download | linux-stable-aa209eef3ce8419ff2926c2fa944dfbfb5afbacb.tar.gz linux-stable-aa209eef3ce8419ff2926c2fa944dfbfb5afbacb.tar.bz2 linux-stable-aa209eef3ce8419ff2926c2fa944dfbfb5afbacb.zip |
staging: vt6656: [BUG] - Failed connection, incorrect endian.
Hi,
This patch fixes a bug with driver failing to negotiate a connection.
The bug was traced to commit
203e4615ee9d9fa8d3506b9d0ef30095e4d5bc90
staging: vt6656: removed custom definitions of Ethernet packet types
In that patch, definitions in include/linux/if_ether.h replaced ones
in tether.h which had both big and little endian definitions.
include/linux/if_ether.h only refers to big endian values, cpu_to_be16
should be used for the correct endian architectures.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Cc: stable <stable@vger.kernel.org> # 2.6.37+
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/vt6656/dpc.c')
-rw-r--r-- | drivers/staging/vt6656/dpc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/vt6656/dpc.c b/drivers/staging/vt6656/dpc.c index e4bdf2a2b582..3aa895ec6507 100644 --- a/drivers/staging/vt6656/dpc.c +++ b/drivers/staging/vt6656/dpc.c @@ -200,7 +200,7 @@ s_vProcessRxMACHeader ( } else if (!compare_ether_addr(pbyRxBuffer, &pDevice->abySNAP_RFC1042[0])) { cbHeaderSize += 6; pwType = (PWORD) (pbyRxBufferAddr + cbHeaderSize); - if ((*pwType == cpu_to_le16(ETH_P_IPX)) || + if ((*pwType == cpu_to_be16(ETH_P_IPX)) || (*pwType == cpu_to_le16(0xF380))) { cbHeaderSize -= 8; pwType = (PWORD) (pbyRxBufferAddr + cbHeaderSize); |