[U-Boot-Users] [PATCH] m68k fix for Ethernet Buffers at address 0
Wilson Callan
wcallan at savantav.com
Thu Jul 12 19:45:56 CEST 2007
Dear Wolfgang,
> First, a formal issue: you missed to include the Signed-off-by: line
> which is mandatory for each patch.
>
Apologies. I didnt know i should "sign off" my own patches, but now
i see that is what i'm supposed to do.
> This is a problem with the Linux driver then, and should be fixed in
> Linux. Please see the FAQ entry about this, and then let's get rid of
> this eth_init call in U-Boot. Philosophy here is that devices shall be
> only initialized when really used by U-Boot.
that makes sense. in this case its u-boot that knows the Ethernet
Address since its stored in its environment. Possibly the solution
is to create a routine in fec.c (the fast ethernet driver) which only
sets the MAC in the coldfire, instead of starting up the ethernet
port before the buffers are ready. This change would keep things
working as they are, but fix the bug where packets are received
before the buffer address is set.
should i go ahead with this fix, or instead remove the eth_init()
call and risk breaking all the m68k systems?
thanks, wilson
More information about the U-Boot
mailing list