[U-Boot] [PATCH] net: fec: Avoid MX28 bus sync issue
Wolfgang Denk
wd at denx.de
Fri Sep 13 18:06:44 CEST 2013
Dear Robert Hodaszi,
In message <5232F2E7.4050407 at digi.com> you wrote:
>
> Ok. Then what about if I would use the stack, but align the buffer manually.
Has this been tested? Does it work?
> - uchar buff[FEC_MAX_PKT_SIZE] __aligned(ARCH_DMA_MINALIGN);
> + /* Align the receive buffer */
> + uchar buff_unaligned[FEC_MAX_PKT_SIZE + (ARCH_DMA_MINALIGN - 1)];
> + uchar *buff = ((uint32_t)buff_unaligned + (ARCH_DMA_MINALIGN - 1)) & ~(ARCH_DMA_MINALIGN - 1);
You should use the ALIGN() macro here.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Niklaus Wirth has lamented that, whereas Europeans pronounce his name
correctly (Ni-klows Virt), Americans invariably mangle it into (Nick-
les Worth). Which is to say that Europeans call him by name, but
Americans call him by value.
More information about the U-Boot
mailing list