summaryrefslogtreecommitdiffstats
path: root/include/net.h
diff options
context:
space:
mode:
authorAlexey Brodkin <Alexey.Brodkin@synopsys.com>2013-09-25 19:27:48 +0400
committerJoe Hershberger <joe.hershberger@ni.com>2013-11-22 16:50:55 -0600
commited102be70f762afc39bda165ba57ea84dc9be39e (patch)
tree1056dc7f0270deb8db3283264267a486b768ed0e /include/net.h
parent227ad7b2b6fab024fff6f60613b0e90c9e3a6724 (diff)
downloadtalos-obmc-uboot-ed102be70f762afc39bda165ba57ea84dc9be39e.tar.gz
talos-obmc-uboot-ed102be70f762afc39bda165ba57ea84dc9be39e.zip
net: designware: Fix alignment of buffer descriptors
It's important that buffer descriptors are aligned in accordance to GMAC data bus width (32/64/128-bit). It's safe to align to 128-bit (16-bytes) for every bus width type. If buffer descriptor is improperly aligned GMAC discards lower bits of provided address and as a result reads from improper location that doesn't match expected fields. Commit ef76025a99247cdb8f927a2c9f15400678dfb599 "net: Multiple updates/enhancements to designware.c" introduced another structure member "link_printed" right before buffer descriptors while "padding" member was left untouched. This together with alignment of structure itself to 16-byte boundary forces buffer descriptoprs always to be 4-byte aligned that causes driver complete disfunction if GMAC bus width is 64 or 128-bit. Proposed change makes sure all buffer descriptors are 16-byte (128-bit) aligned. Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> Patch: 277902
Diffstat (limited to 'include/net.h')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud