[U-Boot] [PATCH] net: fec: Avoid MX28 bus sync issue

Marek Vasut marex at denx.de
Thu Sep 12 16:05:04 CEST 2013


Dear Robert Hodaszi,

> Hi,
> 
> Sorry, hopefully that will be a plain-text.
> 
> There are a lot of bug announcement, just make a search:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33721

This was apparently fixed three years ago.

> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16660

And this one six years ago ...

> Also, I printed out the buffer addresses, and that temporary RX buffer
> was not aligned. So the transmit function rounded it down to the
> alignment boundary, and so caused invalid data transmission. (By the
> way. Shouldn't the transmit function check whether the alignment is
> proper, and throw an error message, instead of round it down? That would
> make more sense.)

Looking at the code one more time, it'd make most sense to simply allocate the 
buffer NOT on stack, but with some memalign-kind-of call to avoid this abuse of 
stack. You see, the max packet size is around 2k, which is quite a lot. How does 
this proposal sound to you ?

Best regards,
Marek Vasut


More information about the U-Boot mailing list