See to it that sent data is 8-byte aligned

U-boot might use non-8-byte-aligned addresses for sending data, which
the kwgbe_send doesn't accept (bootp does this for me). This patch
copies the data to be sent to a malloced temporary buffer if it is
non-aligned.

v2: Malloc send buffer (comment from Stefan Roese)
v3: No need to use jumbo frames, use 1518 bytes buffer instead
    (comment from Ben Warren)

Signed-off-by: Simon Kagstrom <[email protected]>
---
 drivers/net/kirkwood_egiga.c |   26 ++++++++++++++++++++++----
 1 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/drivers/net/kirkwood_egiga.c b/drivers/net/kirkwood_egiga.c
index f31fefc..6627412 100644
--- a/drivers/net/kirkwood_egiga.c
+++ b/drivers/net/kirkwood_egiga.c
@@ -481,24 +481,42 @@ static int kwgbe_halt(struct eth_device *dev)
        return 0;
 }
 
+#define KWGBE_SEND_BUF_SIZE 1518
 static int kwgbe_send(struct eth_device *dev, volatile void *dataptr,
                      int datasize)
 {
        struct kwgbe_device *dkwgbe = to_dkwgbe(dev);
        struct kwgbe_registers *regs = dkwgbe->regs;
        struct kwgbe_txdesc *p_txdesc = dkwgbe->p_txdesc;
+       void *p = (void *)dataptr;
        u32 cmd_sts;
 
+       /* Copy buffer if it's misaligned */
        if ((u32) dataptr & 0x07) {
-               printf("Err..(%s) xmit dataptr not 64bit aligned\n",
-                       __FUNCTION__);
-               return -1;
+               static void *aligned_buf;
+
+               if (!aligned_buf)
+                       aligned_buf = memalign(sizeof(u32),
+                                       KWGBE_SEND_BUF_SIZE);
+               if (!aligned_buf) {
+                       printf("Err...(%s): Cannot allocate aligned buffer\n",
+                               __FUNCTION__);
+                       return -1;
+               }
+               if (datasize > KWGBE_SEND_BUF_SIZE) {
+                       printf("Err..(%s) Non-aligned data too large (%d)\n",
+                               __FUNCTION__, datasize);
+                       return -1;
+               }
+               memcpy(aligned_buf, p, datasize);
+               p = aligned_buf;
        }
+
        p_txdesc->cmd_sts = KWGBE_ZERO_PADDING | KWGBE_GEN_CRC;
        p_txdesc->cmd_sts |= KWGBE_TX_FIRST_DESC | KWGBE_TX_LAST_DESC;
        p_txdesc->cmd_sts |= KWGBE_BUFFER_OWNED_BY_DMA;
        p_txdesc->cmd_sts |= KWGBE_TX_EN_INTERRUPT;
-       p_txdesc->buf_ptr = (u8 *) dataptr;
+       p_txdesc->buf_ptr = (u8 *) p;
        p_txdesc->byte_cnt = datasize;
 
        /* Apply send command using zeroth RXUQ */
-- 
1.6.0.4

_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to