From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Walle Date: Tue, 22 May 2012 23:53:27 +0200 Subject: [U-Boot] [PATCH v5 4/5] net: add eth_setenv_enetaddr_by_index() In-Reply-To: <4FBC0823.6080800@gmail.com> References: <1336776649-9316-1-git-send-email-michael@walle.cc> <201205222318.23146.michael@walle.cc> <4FBC0823.6080800@gmail.com> Message-ID: <201205222353.27929.michael@walle.cc> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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