[U-Boot] [PATCH] Make getenv_IPaddr() global
Wolfgang Denk
wd at denx.de
Sun Jan 3 00:50:18 CET 2010
Dear Ben,
In message <f8328f7c0912301331m7ae655c0ke6891526776cf0c3 at mail.gmail.com> you wrote:
>
> > We should not make the passing of the IP configuration to Linux
> > dependent on U-Boot features / settings.
> >
> > For this, make getenv_IPaddr() global. This fixes build error
> >
> > u-boot/lib_xxx/board.c:360: undefined reference to `getenv_IPaddr'
...
> I agree that this function should be generic, but it's more of an
> environment function than a networking one. Why not put this somewhere like
> common/env_common.c or create a new file called common/env_net.c?
After re-thinking what I wrote before, I still think that lib_generic/
is the right place. The code defines two functions:
IPaddr_t string_to_ip(char *s)
and
IPaddr_t getenv_IPaddr (char *var)
The former does all the actual work and has nothing to do with
environment things, and the latter should eventually be moved as a
"static inline" to some header file.
I'm not sure how to interpret your comment - was it just a question,
or was it intended to be a NAK ?
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: wd at denx.de
An expert is a person who avoids the small errors while sweeping on
to the grand fallacy.
More information about the U-Boot
mailing list