All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Walle <michael@walle.cc>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 3/3] net: add eth_setenv_enetaddr_by_index()
Date: Sat, 12 May 2012 00:12:45 +0200	[thread overview]
Message-ID: <201205120012.45444.michael@walle.cc> (raw)
In-Reply-To: <20120511211602.BED12206496@gemini.denx.de>

Am Freitag 11 Mai 2012, 23:16:02 schrieb Wolfgang Denk:
> Dear Michael Walle,
> 
> In message <201205112301.50311.michael@walle.cc> you wrote:
> > > Eventually this should be made configurable?
> > 
> > Given the above, i would keep it non-configurable if youre fine with
> > that.
> 
> Only if the code size for the existing boards that don't need this
> does not grow.
Well i can't add a new function without increasing the filesize, can i? 

Using inline functions would be one option, but that would be inconsistend 
with the eth_getenv_enetaddr_by_index().

phew, i wonder how the eth_getenv_enetaddr_by_index() made it into uboot ;)

-- 
michael

  reply	other threads:[~2012-05-11 22:12 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-10 17:32 [U-Boot] [PATCH v4 0/3] net: helper functions Michael Walle
2012-05-10 17:32 ` [U-Boot] [PATCH v4 1/3] lib: add rand() function Michael Walle
2012-05-11 19:20   ` Wolfgang Denk
2012-05-11 20:32     ` Michael Walle
2012-05-11 20:43       ` Wolfgang Denk
     [not found]         ` <201205112249.27871.michael@walle.cc>
2012-05-11 21:11           ` Wolfgang Denk
2012-05-11 21:20             ` Michael Walle
2012-05-11 21:29               ` Wolfgang Denk
2012-05-11 21:34                 ` Michael Walle
2012-05-10 17:32 ` [U-Boot] [PATCH v4 2/3] net: add helper to generate random mac address Michael Walle
2012-05-11 19:21   ` Wolfgang Denk
2012-05-14  5:43     ` Mike Frysinger
2012-05-10 17:32 ` [U-Boot] [PATCH v4 3/3] net: add eth_setenv_enetaddr_by_index() Michael Walle
2012-05-11 19:25   ` Wolfgang Denk
2012-05-11 21:01     ` Michael Walle
2012-05-11 21:16       ` Wolfgang Denk
2012-05-11 22:12         ` Michael Walle [this message]
2012-05-11 16:32 ` [U-Boot] [PATCH v4 0/3] net: helper functions Michael Walle
2012-05-11 16:46   ` Joe Hershberger
2012-05-11 16:51     ` Prafulla Wadaskar
2012-05-21 20:52       ` Michael Walle
2012-05-11 19:18 ` Wolfgang Denk
2012-05-11 20:29   ` Michael Walle
2012-05-11 20:43     ` Wolfgang Denk
2012-05-11 20:54       ` Michael Walle
2012-05-11 21:14         ` Wolfgang Denk
2012-05-11 21:22           ` Michael Walle
2012-05-11 21:30             ` Wolfgang Denk
2012-05-11 21:42               ` Michael Walle
2012-05-11 21:55                 ` Wolfgang Denk
2012-05-14  5:42             ` Mike Frysinger

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=201205120012.45444.michael@walle.cc \
    --to=michael@walle.cc \
    --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.