[U-Boot] [PATCH v2 5/6] netconsole: remove `serverip' check
Wolfgang Denk
wd at denx.de
Sun May 1 23:34:29 CEST 2011
Dear Simon Guinot,
In message <1304284220-15215-6-git-send-email-sguinot at lacie.com> you wrote:
> 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 | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/net/net.c b/net/net.c
> index e50bdf1..5bf3c8c 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");
> @@ -1729,6 +1728,7 @@ static int net_check_prereq (proto_t protocol)
> common:
> #endif
>
> + case NETCONS:
Please add a "/* fall through */" comment before the case label, so it
is clear to the reader that this is intended behaviour.
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
They're usually so busy thinking about what happens next that the
only time they ever find out what is happening now is when they come
to look back on it. - Terry Pratchett, _Wyrd Sisters_
More information about the U-Boot
mailing list