staging: vt6656: [BUG] - Failed connection, incorrect endian.
authorMalcolm Priestley <tvboxspy@gmail.com>
Wed, 29 Aug 2012 22:08:21 +0000 (23:08 +0100)
committerBen Hutchings <ben@decadent.org.uk>
Wed, 19 Sep 2012 14:05:08 +0000 (15:05 +0100)
commit7081f1bf324ad65e421bb83fd3c1cb7c974248ad
tree6779c9f9dedc0a4a8444e0f5c1cb667ee6ff9594
parentf707d86130d5d292c0f2b19d1bd8f2cad128c929
staging: vt6656: [BUG] - Failed connection, incorrect endian.

commit aa209eef3ce8419ff2926c2fa944dfbfb5afbacb upstream.

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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/staging/vt6656/dpc.c
drivers/staging/vt6656/rxtx.c