From: Rob Herring <robherring2@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v5 4/5] net: add eth_setenv_enetaddr_by_index()
Date: Tue, 22 May 2012 16:41:55 -0500 [thread overview]
Message-ID: <4FBC0823.6080800@gmail.com> (raw)
In-Reply-To: <201205222318.23146.michael@walle.cc>
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...
I don't see a way to meet this impossible requirement either, and I
don't really have the bandwidth right now.
Rob
next prev parent reply other threads:[~2012-05-22 21:41 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-11 22:50 [U-Boot] [PATCH v5 0/5] Kirkwood: add lschlv2 and lsxhl board support Michael Walle
2012-05-11 22:50 ` [U-Boot] [PATCH v5 1/5] lib: add rand() function Michael Walle
2012-05-23 16:34 ` Michael Walle
2012-05-24 4:32 ` Prafulla Wadaskar
2012-05-24 10:34 ` Michael Walle
2012-05-11 22:50 ` [U-Boot] [PATCH v5 2/5] net: add helper to generate random mac address Michael Walle
2012-05-25 17:52 ` Joe Hershberger
2012-05-11 22:50 ` [U-Boot] [PATCH v5 3/5] net: fix potential compiler warning Michael Walle
2012-05-16 1:11 ` Joe Hershberger
2012-05-11 22:50 ` [U-Boot] [PATCH v5 4/5] net: add eth_setenv_enetaddr_by_index() Michael Walle
2012-05-11 22:53 ` Michael Walle
2012-05-16 0:56 ` Joe Hershberger
2012-05-17 20:43 ` Michael Walle
2012-05-25 18:50 ` Joe Hershberger
2012-05-25 19:54 ` Michael Walle
2012-05-25 20:50 ` Joe Hershberger
2012-05-22 21:18 ` Michael Walle
2012-05-22 21:41 ` Rob Herring [this message]
2012-05-22 21:53 ` Michael Walle
2012-05-11 22:50 ` [U-Boot] [PATCH v5 5/5] Kirkwood: add lschlv2 and lsxhl board support Michael Walle
2012-05-22 20:58 ` Michael Walle
2012-05-24 4:50 ` Prafulla Wadaskar
2012-05-24 7:24 ` Prafulla Wadaskar
2012-05-24 10:30 ` Michael Walle
2012-05-25 23:34 ` Michael Walle
2012-05-25 14:13 ` [U-Boot] [PATCH v5 0/5] " Prafulla Wadaskar
2012-05-25 16:05 ` Joe Hershberger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FBC0823.6080800@gmail.com \
--to=robherring2@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.