[U-Boot] [PATCH 2/2] net: bootp: Fix compile error processing ntpserver option

Joe Hershberger joe.hershberger at ni.com
Thu May 3 00:23:07 UTC 2018


On Fri, Apr 27, 2018 at 4:55 AM, Chris Packham <judge.packham at gmail.com> wrote:
> When the following configuration is set
>
>   # CONFIG_CMD_DHCP is not set
>   CONFIG_CMD_BOOTP=y
>   CONFIG_BOOTP_NTPSERVER=y
>
> The following compile error is observed
>
>   error: used struct type value where scalar is required
>     if (net_ntp_server)
>         ^~~~~~~~~~~~~~

We should probably enable CMD_SNTP in devkit8000 so that this code
path is compiled on travis. Can you add a patch for that to the
series?

>
> Resolve this by checking net_ntp_server.s_addr instead.
>
> Signed-off-by: Chris Packham <judge.packham at gmail.com>

Acked-by: Joe Hershberger <joe.hershberger at ni.com>

> ---
>
>  net/bootp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/net/bootp.c b/net/bootp.c
> index efa959971c27..9d7cb5d30c14 100644
> --- a/net/bootp.c
> +++ b/net/bootp.c
> @@ -333,7 +333,7 @@ static void bootp_process_vendor(u8 *ext, int size)
>                 debug("net_nis_domain : %s\n", net_nis_domain);
>
>  #if defined(CONFIG_CMD_SNTP) && defined(CONFIG_BOOTP_NTPSERVER)
> -       if (net_ntp_server)
> +       if (net_ntp_server.s_addr)
>                 debug("net_ntp_server : %pI4\n", &net_ntp_server);
>  #endif
>  }
> --
> 2.17.0
>
> _______________________________________________
> 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