[U-Boot] [PATCH v5 4/5] net: add eth_setenv_enetaddr_by_index()

[Michael Walle <michael@walle.cc>]

Am Dienstag 22 Mai 2012, 23:41:55 schrieb Rob Herring:
> On 05/22/2012 04:18 PM, Michael Walle wrote:
> > Am Mittwoch 16 Mai 2012, 02:56:39 schrieb Joe Hershberger:
> >> You should synchronize with Rob Herring who is also trying to add this
> >> eth_setenv_enetaddr_by_index() function.
> >> http://patchwork.ozlabs.org/patch/152590/
> > 
> > Hi Rob,
> > 
> > Joe mentioned you have the same patch as me here:
> >   http://patchwork.ozlabs.org/patch/158624/
> > 
> > What is the current status of your patch?
> 
> I have no idea. It's been in the patch system for a month, gotten no
> comments, and delegated to jhersh.
> 
> > Wolfgang had some comments on my patch:
> >   http://patchwork.ozlabs.org/patch/158354/
> > 
> > Could you incorporate these changes into your patch?
> > 
> > Unfortunately, i don't see a way to add the helper function without
> > meeting Wolfgangs requirement not to increase the binary file for other
> > boards. Unless gc-sections and function-sections are used on ARM, or
> > adding an CONFIG_XX macro to mask the helper on compile time.
> 
> There was a patch series for gc-sections recently...
yeah saw that too, but someone mentioned there were problems on arm, iirc.

> I don't see a way to meet this impossible requirement either, and I
> don't really have the bandwidth right now.

ok, thanks for the clarification :)

-- 
Michael