[U-Boot] [PATCH v5 3/5] net: fix potential compiler warning

[Joe Hershberger]

Hi Michael,

On Fri, May 11, 2012 at 5:50 PM, Michael Walle <michael at walle.cc> wrote:
> Future compiler versions may generate a "too many arguments for functions"
> warning.
>
> Signed-off-by: Michael Walle <michael at walle.cc>
> Cc: Joe Hershberger <joe.hershberger at gmail.com>
> ---
>  net/eth.c |   14 ++++++++++++--
>  1 files changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/net/eth.c b/net/eth.c
> index c9f05d8..afce863 100644
> --- a/net/eth.c
> +++ b/net/eth.c
> @@ -58,7 +58,12 @@ int eth_getenv_enetaddr_by_index(const char *base_name, int index,
>                                 uchar *enetaddr)
>  {
>        char enetvar[32];
> -       sprintf(enetvar, index ? "%s%daddr" : "%saddr", base_name, index);
> +
> +       if (index)
> +               sprintf(enetvar, "%s%daddr", base_name, index);
> +       else
> +               sprintf(enetvar, "%saddr", base_name);
> +
>        return eth_getenv_enetaddr(enetvar, enetaddr);
>  }
>
> @@ -66,7 +71,12 @@ static int eth_mac_skip(int index)
>  {
>        char enetvar[15];
>        char *skip_state;
> -       sprintf(enetvar, index ? "eth%dmacskip" : "ethmacskip", index);
> +
> +       if (index)
> +               sprintf(enetvar, "eth%dmacskip", index);
> +       else
> +               sprintf(enetvar, "ethmacskip");
> +
>        return ((skip_state = getenv(enetvar)) != NULL);
>  }

This seems like this could improve by consolidating the logic to come
up with the ethaddr variable name in an inline function.  Especially
since you are about to add another use in the setenv case.  Do you
agree, Mike?  Or is 3 uses of a relatively simple algorithm better
inlined than separated?

-Joe