[U-Boot] [PATCH v4 13/42] common: Move env_get_ip() to net.h
Joe Hershberger
joe.hershberger at gmail.com
Mon Nov 18 21:27:36 UTC 2019
On Thu, Nov 14, 2019 at 2:05 PM Simon Glass <sjg at chromium.org> wrote:
>
> This function relates to networking, so move it out of the common.h
> header file.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> Reviewed-by: Tom Rini <trini at konsulko.com>
Nit below, but...
Acked-by: Joe Hershberger <joe.hershberger at ni.com>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2: None
>
> include/common.h | 4 ----
> include/net.h | 21 ++++++++++++++++++++-
> 2 files changed, 20 insertions(+), 5 deletions(-)
>
> diff --git a/include/common.h b/include/common.h
> index f993e4bef3..4fda40cc1d 100644
> --- a/include/common.h
> +++ b/include/common.h
> @@ -319,10 +319,6 @@ int serial_printf (const char *fmt, ...)
>
> /* lib/net_utils.c */
> #include <net.h>
> -static inline struct in_addr env_get_ip(char *var)
> -{
> - return string_to_ip(env_get(var));
> -}
>
> #include <bootstage.h>
>
> diff --git a/include/net.h b/include/net.h
> index d8d187d8af..11eca1bc6c 100644
> --- a/include/net.h
> +++ b/include/net.h
> @@ -829,7 +829,13 @@ static inline void net_random_ethaddr(uchar *addr)
> /* Convert an IP address to a string */
> void ip_to_string(struct in_addr x, char *s);
>
> -/* Convert a string to ip address */
> +/**
> + * string_to_ip() - Convert a string to ip address
> + *
> + * @s: String to conver, in the format format a.b.c.d, where each value is a
> + * decimal number from 0 to 255
> + * @return IP address, or 0 if invalid
> + */
Please drop this unrelated documentation update... it conflicts with [1]
[1] - https://patchwork.ozlabs.org/patch/1162279/
> struct in_addr string_to_ip(const char *s);
>
> /* Convert a VLAN id to a string */
> @@ -887,4 +893,17 @@ int update_tftp(ulong addr, char *interface, char *devstring);
> */
> void eth_parse_enetaddr(const char *addr, uint8_t *enetaddr);
>
> +/**
> + * env_get_ip() - Convert an environment value to to an ip address
> + *
> + * @var: Environment variable to convert. The value of this variable must be
> + * in the format format a.b.c.d, where each value is a decimal number from
> + * 0 to 255
> + * @return IP address, or 0 if invalid
> + */
> +static inline struct in_addr env_get_ip(char *var)
> +{
> + return string_to_ip(env_get(var));
> +}
> +
> #endif /* __NET_H__ */
> --
> 2.24.0.rc1.363.gb1bccd3e3d-goog
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
More information about the U-Boot
mailing list