[U-Boot] [PATCH 5/6] netconsole: remove `serverip' check

Wolfgang Denk wd at denx.de
Sun May 1 17:19:15 CEST 2011


Dear Simon Guinot,

In message <1304262244-13788-6-git-send-email-simon.guinot at sequanux.org> you wrote:
> From: Simon Guinot <sguinot at lacie.com>
> 
> Netconsole use the environment variable `ncip' to configure the
> destination IP. `serverip' don't need to be defined.
> 
> Signed-off-by: Simon Guinot <sguinot at lacie.com>
> ---
>  net/net.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/net/net.c b/net/net.c
> index e50bdf1..63b4201 100644
> --- a/net/net.c
> +++ b/net/net.c
> @@ -1718,7 +1718,6 @@ static int net_check_prereq (proto_t protocol)
>  #if defined(CONFIG_CMD_NFS)
>  	case NFS:
>  #endif
> -	case NETCONS:
>  	case TFTP:
>  		if (NetServerIP == 0) {
>  			puts ("*** ERROR: `serverip' not set\n");

NAK, as this change would also disable the test if NetOurIP == 0,
i. e. if our own IP address is set.

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
Genitiv ins Wasser, weil's Dativ ist!


More information about the U-Boot mailing list