Dear Ben Warren,

In message <f8328f7c0910251228p4d5bd05j811b32553f37d...@mail.gmail.com> you 
wrote:
> 
> > IMHO we should aim to make this driver less complicated, not more.  Do we
> really need to malloc and free each time this interface is turned on or off?
>  Why not just initialize the pointers to NULL, and malloc() only the first
> time the init() function is called?  You could then get rid of the free()
> calls in the halt function.
> 
> U-boot is very transient in nature.  Once we launch a kernel all the memory
> essentially gets freed anyway.

Full ACK from me.  And we don't use that many different Ethernet
drivers in parallel either that the RAM footprint would hurt.

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: w...@denx.de
Genitiv ins Wasser, weil's Dativ ist!
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to