Clearer constant name.
Also remove related BOOTP_SIZE which was unused and doesn't take
into account VLAN packets.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

Separate the Ethernet header from the 802 header.
Base the size constants on the structs.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

Rename IP header related things to IP_UDP. The existing definition
of IP_t includes UDP header, so name it to accurately describe the
structure.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

Make the MAC-seeded random number generator available to /net in
general.  MAC-seeded rand will be needed by link-local as well, so
give it an interface.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>
Acked-by: NSimon Glass <sjg@chromium.org>

Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>
Acked-by: NSimon Glass <sjg@chromium.org>

Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>
Acked-by: NSimon Glass <sjg@chromium.org>

This inserts bootstage calls into tftp, usb start and bootm. We
could go further, but this is a reasonable start to illustrate
the concept.
Signed-off-by: NSimon Glass <sjg@chromium.org>

Fix:
bootp.c: In function 'BootpCopyNetParams':
bootp.c:108:11: warning: unused variable 'tmp_ip' [-Wunused-variable]
Signed-off-by: NAnatolij Gustschin <agust@denx.de>

The rarp code includes another instance of the auto_load logic, so call
what is now net_auto_load() instead.

This also fixes an incorrect call to TftpStart() which was never seen
since apparently no boards enable rarp.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: NMike Frysinger <vapier@gentoo.org>

TftpStart should support starting either a get or a put.
Signed-off-by: NSimon Glass <sjg@chromium.org>

These options are required to be present in RFC 4578 compliant DHCP
requests. They give more information to DHCP servers to allow serving

different DHCP responses to different systems based on client
architecture, client capabilities, UUID, or vendor.
Signed-off-by: NJason Hobbs <jason.hobbs@calxeda.com>

Commit 09349866 (Put common autoload code into auto_load() function)
broke handling of autoload environment variable not being set.
The bootp/dhcp code will just keep on requesting IP address forever
and never start TFTP download.

Fix it by moving TftpStart() outside the conditional like it was before.
Signed-off-by: NPeter Korsgaard <jacmet@sunsite.dk>
Tested-by: NFabio Estevam <fabio.estevam@freescale.com>
Acked-by: NSimon Glass <sjg@chromium.org>

This is a small clean-up patch.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Tested-by: NEric Bénard <eric@eukrea.com>

Signed-off-by: NLuuk Paulussen <luuk.paulussen@alliedtelesis.co.nz>
Acked-by: NChris Packham <chris.packham@alliedtelesis.co.nz>
Cc: Ben Warren <biggerbadderben@gmail.com>

This is needed for the upcoming TFTP server implementation.

This also simplifies PingHandler() and fixes rxhand_f documentation.
Signed-off-by: NLuca Ceresoli <luca.ceresoli@comelit.it>
Cc: Wolfgang Denk <wd@denx.de>
Acked-by: NDetlev Zundel <dzu@denx.de>

This seems to pad to one byte longer than required
Signed-off-by: NSimon Glass <sjg@chromium.org>

Can't get IP address with dhcp due to the dhcp server not
allow the empty param list request under some network env

This patch is based on Gray Remlin's initial patch.
Signed-off-by: NJason Liu <r64343@freescale.com>
Signed-off-by: NGray Remlin <g_remlin@rocketmail.com>

Signed-off-by: NMike Frysinger <vapier@gentoo.org>
Signed-off-by: NBen Warren <biggerbadderben@gmail.com>

Fix warning: bootp.c:695: warning: dereferencing type-punned pointer
will break strict-aliasing rules
Signed-off-by: NWolfgang Denk <wd@denx.de>
Cc: Ben Warren <biggerbadderben@gmail.com>

Fix warning: bootp.c:695: warning: dereferencing type-punned pointer
will break strict-aliasing rules
Signed-off-by: NWolfgang Denk <wd@denx.de>
Cc: Ben Warren <biggerbadderben@gmail.com>

 Minor ./net cleanups - no functional changes
  - change #ifdef DEBUG printf(); #endif to just debug()
  - changed __FUNCTION__ to __func__
  - got rid of extra whitespace between function and opening brace
  - removed unnecessary braces on if statements

 gcc dead code elimination should make this functionally/size equivalent
 when DEBUG is not defined. (confirmed on Blackfin, with gcc 4.3.3).
Signed-off-by: NRobin Getz <rgetz@blackfin.uclinux.org>
Signed-off-by: NBen Warren <biggerbadderben@gmail.com>

The net code is mostly consistent in using 'Packet' rather than 'Pkt', so
rename the minor detractor to follow suite.
Signed-off-by: NMike Frysinger <vapier@gentoo.org>
Signed-off-by: NBen Warren <biggerbadderben@gmail.com>

The problem is that BOOTP_SIZE uses ETHER_HDR_SIZE which is 14 bytes.
If sending a VLAN tagged frame (when env variable vlan is set) this
should be VLAN_ETHER_HDR_SIZE=18 which is what NetSetEther returns.
Signed-off-by: NNorbert van Bolhuis <nvbolhuis@aimvalley.nl>
Signed-off-by: NBen Warren <biggerbadderben@gmail.com>

Signed-off-by: NMike Frysinger <vapier@gentoo.org>
CC: Ben Warren <biggerbadderben@gmail.com>

Now that our printf functions support the %pI4 modifier like the kernel,
let's drop the inflexible print_IPaddr() function and covert over to the
%pI4 modifier.
Signed-off-by: NMike Frysinger <vapier@gentoo.org>
CC: Ben Warren <biggerbadderben@gmail.com>

Signed-off-by: NJean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

Enforce millisecond semantics of the first argument to NetSetTimeout() --
the change is transparent for well-behaving boards (CFG_HZ == 1000 and
get_timer() countiing in milliseconds).

Rationale for this patch is to enable millisecond granularity for
network-related timeouts, which is needed for the upcoming automatic
software update feature.

Summary of changes:
- do not scale the first argument to NetSetTimeout() by CFG_HZ
- change timeout values used in the networking code to milliseconds
Signed-off-by: NRafal Czubak <rcz@semihalf.com>
Signed-off-by: NBartlomiej Sieka <tur@semihalf.com>
Signed-off-by: NBen Warren <biggerbadderben@gmail.com>

The DHCP handler has 1 state that is not listed in this case, causing a
failure message when there is actually no failure.
Signed-off-by: NRemy Bohmer <linux@bohmer.net>
Signed-off-by: NBen Warren <biggerbadderben@gmail.com>

on the network with it's offered IP number; it should not reply until
after it has received a DHCP ACK message. Also ensures that U-Boot
does it's DHCPREQUEST as broadcast (per RFC 2131).
Signed-off-by: NRobin Getz <rgetz@blackfin.uclinux.org>
Acked-by: NBen Warren <biggerbadderben@gmail.com>
Signed-off-by: NWolfgang Denk <wd@denx.de>

Signed-off-by: NJean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Signed-off-by: NWolfgang Denk <wd@denx.de>

Signed-off-by: NWolfgang Denk <wd@denx.de>

Added option CONFIG_BOOTP_DHCP_REQUEST_DELAY. This provides an optional
delay before sending "DHCP Request" in net/bootp.c. Required to overcome
interoperability problems with Windows Server 200x DHCP server when U-Boot
client responds too fast for server to handle.
Signed-off-by: NAras Vaichas <arasv@magtech.com.au>
Signed-off-by: NBen Warren <biggerbadderben@gmail.com>

Signed-off-by: NWolfgang Denk <wd@denx.de>

This is a multi-part message in MIME format.

commit e6e505eae94ed721e123e177489291fc4544b7b8
Author: Justin Flammia <jflammia@savantav.com>
Date:   Mon Oct 29 17:19:03 2007 -0400

    Found a bug in the way the DHCP Request packet is built, where the IP address
    that is offered by the server is bound to prematurely. This patch is a fix of
    that bug where the IP address offered by the DHCP server is not used until
    after the DHCP ACK from the server is received.
Signed-off-by: NJustin Flammia <jflammia@savantav.com>
Signed-off-by: NBen Warren <bwarren@qstreams.com>

This is a multi-part message in MIME format.

commit e6e505eae94ed721e123e177489291fc4544b7b8
Author: Justin Flammia <jflammia@savantav.com>
Date:   Mon Oct 29 17:19:03 2007 -0400

    Found a bug in the way the DHCP Request packet is built, where the IP address
    that is offered by the server is bound to prematurely. This patch is a fix of
    that bug where the IP address offered by the DHCP server is not used until
    after the DHCP ACK from the server is received.
Signed-off-by: NJustin Flammia <jflammia@savantav.com>
Signed-off-by: NBen Warren <bwarren@qstreams.com>

Signed-off-by: NWolfgang Denk <wd@denx.de>

Added CONFIG_BOOTP_SERVERIP to allow the tftp server to be different
from the bootp server
Signed-off-by: NWilson Callan <wcallan@savantav.com>
Signed-off-by: NBen Warren <bwarren@qstreams.com>

Fixed some broken instances of "#ifdef CMD_CFG_IDE" too.
Those always evaluated TRUE, and thus were always compiled
even when IDE really wasn't defined/wanted.
Signed-off-by: NJon Loeliger <jdl@freescale.com>

All of the choices for CONFIG_BOOTP_ are now documented in
the README file.  You must now individually select exactly
the set that you want using a series of
    #define CONFIG_BOOTP_<x>
statements in the board port config files now.
Signed-off-by: NJon Loeliger <jdl@freescale.com>