Add link-local addressing support

cat sizeofMCP8313

git-test-sequence origin/master.. './sizeofMCP8313'

The mv_eth driver should not redefine the net function definition

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---

?board/Marvell/db64360/mv_eth.c | ? ?2 --
?board/Marvell/db64460/mv_eth.c | ? ?2 --
?board/esd/cpci750/mv_eth.c ? ? | ? ?2 --
?board/prodrive/p3mx/mv_eth.c ? | ? ?2 --
?include/net.h ? ? ? ? ? ? ? ? ?| ? 16 ++++++++--------
?net/bootp.c ? ? ? ? ? ? ? ? ? ?| ? ?4 ++--
?net/net.c ? ? ? ? ? ? ? ? ? ? ?| ? 40 ++++++++++++++++++++--------------------
?net/rarp.c ? ? ? ? ? ? ? ? ? ? | ? ?2 +-
?net/tftp.c ? ? ? ? ? ? ? ? ? ? | ? ?6 +++---

?9 files changed, 34 insertions(+), 42 deletions(-)

board/Marvell/db64360/mv_eth.c +0 -0

index 6340585..550610e 100644
@@ -95,8 +95,6 @@ int mv64360_eth_xmit (struct eth_device *, volatile void *packet, int length);

?#ifndef ?UPDATE_STATS_BY_SOFTWARE
?static void mv64360_eth_print_stat (struct eth_device *dev); ?#endif

board/Marvell/db64360/mv_eth.c +0 -2

-/* Processes a received packet */
-extern void NetReceive (volatile uchar *, int);

?extern unsigned int INTERNAL_REG_BASE_ADDR;

board/Marvell/db64460/mv_eth.c +0 -0

index 4aefbaf..c447229 100644
@@ -95,8 +95,6 @@ int mv64460_eth_xmit (struct eth_device *, volatile void *packet, int length);

?#ifndef ?UPDATE_STATS_BY_SOFTWARE
?static void mv64460_eth_print_stat (struct eth_device *dev); ?#endif

board/Marvell/db64460/mv_eth.c +0 -2

-/* Processes a received packet */
-extern void NetReceive (volatile uchar *, int);

?extern unsigned int INTERNAL_REG_BASE_ADDR;

board/esd/cpci750/mv_eth.c +0 -0

index 001c1ad..db2324b 100644
@@ -95,8 +95,6 @@ int mv64360_eth_xmit (struct eth_device *, volatile void *packet, int length);

?#ifndef ?UPDATE_STATS_BY_SOFTWARE
?static void mv64360_eth_print_stat (struct eth_device *dev); ?#endif

board/esd/cpci750/mv_eth.c +0 -2

-/* Processes a received packet */
-extern void NetReceive (volatile uchar *, int);

?extern unsigned int INTERNAL_REG_BASE_ADDR;

board/prodrive/p3mx/mv_eth.c +0 -0

index 15b3bfc..f26ded8 100644
@@ -109,8 +109,6 @@ int phy_setup_aneg (char *devname, unsigned char addr);

?#ifndef ? ? ? ? UPDATE_STATS_BY_SOFTWARE
?static void mv64460_eth_print_stat (struct eth_device *dev); ?#endif

board/prodrive/p3mx/mv_eth.c +0 -2

-/* Processes a received packet */
-extern void NetReceive (volatile uchar *, int);

?extern unsigned int INTERNAL_REG_BASE_ADDR;

include/net.h +0 -0

index e4d42c2..ccdb01d 100644
@@ -342,9 +342,9 @@ extern uchar ? ? ? ? ? ? ? ?NetOurEther[6]; ? ? ? ? /* Our ethernet address ? ? ? ? */

?extern uchar ? ? ? ? ? NetServerEther[6]; ? ? ?/* Boot server enet address ? ? */ ?extern IPaddr_t ? ? ? ? ? ? ? ?NetOurIP; ? ? ? ? ? ? ? /* Our ? ?IP addr (0 = unknown) */ ?extern IPaddr_t ? ? ? ? ? ? ? ?NetServerIP; ? ? ? ? ? ?/* Server IP addr (0 = unknown) */

include/net.h +3 -3

-extern volatile uchar * NetTxPacket; ? ? ? ? ? /* THE transmit packet ? ? ? ? ?*/
-extern volatile uchar * NetRxPackets[PKTBUFSRX];/* Receive packets ? ? ? ? ? ? */
-extern volatile uchar * NetRxPacket; ? ? ? ? ? /* Current receive packet ? ? ? */
+extern uchar ? ? ? ? ? *NetTxPacket; ? ? ? ? ? /* THE transmit packet */
+extern uchar ? ? ? ? ? *NetRxPackets[PKTBUFSRX];/* Receive packets */
+extern uchar ? ? ? ? ? *NetRxPacket; ? ? ? ? ? /* Current receive packet */

?extern int ? ? ? ? ? ? NetRxPacketLen; ? ? ? ? /* Current rx packet length ? ? */ ?extern unsigned ? ? ? ? ? ? ? ?NetIPID; ? ? ? ? ? ? ? ?/* IP ID (counting) ? ? ? ? ? ? */ ?extern uchar ? ? ? ? ? NetBcastAddr[6]; ? ? ? ?/* Ethernet boardcast address ? */

include/net.h +0 -0

@@ -408,10 +408,10 @@ extern void ? ? ? NetStartAgain(void);
 ?extern int ? ? NetEthHdrSize(void);
 

?/* Set ethernet header; returns the size of the header */

include/net.h +1 -1

-extern int ? ? NetSetEther(volatile uchar *, uchar *, uint);
+extern int NetSetEther(uchar *, uchar *, uint);

?/* Set IP header */

include/net.h +1 -1

-extern void ? ?NetSetIP(volatile uchar *, IPaddr_t, int, int, int);
+extern void NetSetIP(uchar *, IPaddr_t, int, int, int);

?/* Checksum */
?extern int ? ? NetCksumOk(uchar *, int); ? ? ? /* Return true if cksum OK ? ? ?*/

include/net.h +0 -0

@@ -423,7 +423,7 @@ extern void net_set_icmp_handler(rxhand_icmp_f *f); /* Set ICMP RX handler */

?extern void ? ?NetSetTimeout(ulong, thand_f *);/* Set timeout handler ? ? ? ? ?*/

?/* Transmit "NetTxPacket" */

include/net.h +1 -1

-extern void ? ?NetSendPacket(volatile uchar *, int);
+extern void NetSendPacket(uchar *, int);

?/* Transmit UDP packet, performing ARP request if needed */
?extern int ? ? NetSendUDPPacket(uchar *ether, IPaddr_t dest, int dport, int sport, int len);

include/net.h +0 -0

@@ -445,7 +445,7 @@ void net_auto_load(void);

?* footprint in our tests.
?*/
?/* return IP *in network byteorder* */

include/net.h +1 -1

-static inline IPaddr_t NetReadIP(volatile void *from)
+static inline IPaddr_t NetReadIP(void *from)

?{
? ? ? ?IPaddr_t ip;
? ? ? ?memcpy((void*)&ip, (void*)from, sizeof(ip));

include/net.h +0 -0

@@ -467,7 +467,7 @@ static inline void NetWriteIP(void *to, IPaddr_t ip)
 ?}
 

?/* copy IP */

include/net.h +1 -1

-static inline void NetCopyIP(volatile void *to, void *from)
+static inline void NetCopyIP(void *to, void *from)

?{
? ? ? ?memcpy((void*)to, from, sizeof(IPaddr_t));
?}

net/bootp.c +0 -0

index 34124b8..6d40608 100644
@@ -585,7 +585,7 @@ static int BootpExtended (u8 * e)

?void
?BootpRequest (void)
?{
- ? ? ? volatile uchar *pkt, *iphdr;
+ ? ? ? uchar *pkt, *iphdr;
? ? ? ?Bootp_t *bp;
? ? ? ?int ext_len, pktlen, iplen;

net/bootp.c +1 -1

@@ -837,7 +837,7 @@ static int DhcpMessageType(unsigned char *popt)
 
 ?static void DhcpSendRequestPkt(Bootp_t *bp_offer)
 ?{
- ? ? ? volatile uchar *pkt, *iphdr;
+ ? ? ? uchar *pkt, *iphdr;
 ? ? ? ?Bootp_t *bp;
 ? ? ? ?int pktlen, iplen, extlen;
 ? ? ? ?IPaddr_t OfferedIP;

net/net.c +0 -0

index 045405b..29e018c 100644
@@ -152,7 +152,7 @@ IPaddr_t ? ?NetOurIP;

?/* Server IP addr (0 = unknown) */
?IPaddr_t ? ? ? NetServerIP;
?/* Current receive packet */

net/net.c +1 -1

-volatile uchar *NetRxPacket;
+uchar *NetRxPacket;

?/* Current rx packet length */
?int ? ? ? ? ? ?NetRxPacketLen;
?/* IP packet ID */

net/net.c +0 -0

@@ -208,10 +208,10 @@ void NcStart(void);
 ?int nc_input_packet(uchar *pkt, unsigned dest, unsigned src, unsigned len);

?#endif

net/net.c +1 -1

-volatile uchar PktBuf[(PKTBUFSRX+1) * PKTSIZE_ALIGN + PKTALIGN];
+uchar PktBuf[(PKTBUFSRX+1) * PKTSIZE_ALIGN + PKTALIGN];

?/* Receive packet */

net/net.c +1 -1

-volatile uchar *NetRxPackets[PKTBUFSRX];
+uchar *NetRxPackets[PKTBUFSRX];

?/* Current RX packet handler */
?static rxhand_f *packetHandler;

net/net.c +0 -0

@@ -225,7 +225,7 @@ static ulong ? ? ? ?timeStart;

?/* Current timeout value */
?static ulong ? timeDelta;
?/* THE transmit packet */

net/net.c +1 -1

-volatile uchar *NetTxPacket;
+uchar *NetTxPacket;

?static int net_check_prereq(enum proto_t protocol);

net/net.c +1 -1

@@ -246,7 +246,7 @@ int ? ? ? ? NetArpWaitTry;
 
 ?void ArpRequest(void)
 ?{
- ? ? ? volatile uchar *pkt;
+ ? ? ? uchar *pkt;
 ? ? ? ?ARP_t *arp;
 
 ? ? ? ?debug("ARP broadcast %d\n", NetArpWaitTry);

@@ -704,7 +704,7 @@ NetSetTimeout(ulong iv, thand_f *f)
 
 

?void

net/net.c +1 -1

-NetSendPacket(volatile uchar *pkt, int len)
+NetSendPacket(uchar *pkt, int len)

?{
? ? ? ?(void) eth_send(pkt, len);
?}

net/net.c +2 -2

@@ -767,8 +767,8 @@ static ushort PingSeqNo;
 ?int PingSend(void)
 ?{
 ? ? ? ?static uchar mac[6];
- ? ? ? volatile IP_t *ip;
- ? ? ? volatile ushort *s;
+ ? ? ? IP_t *ip;
+ ? ? ? ushort *s;
 ? ? ? ?uchar *pkt;
 

? ? ? ?/* XXX always send arp request */

net/net.c +1 -1

@@ -783,7 +783,7 @@ int PingSend(void)
 ? ? ? ?pkt = NetArpWaitTxPacket;
 ? ? ? ?pkt += NetSetEther(pkt, mac, PROT_IP);
 
- ? ? ? ip = (volatile IP_t *)pkt;
+ ? ? ? ip = (IP_t *)pkt;
 

? ? ? ?/*
? ? ? ? * Construct an IP and ICMP header.

net/net.c +3 -3

@@ -935,9 +935,9 @@ static ushort CDP_compute_csum(const uchar *buff, ushort len)
 
 ?int CDPSendTrigger(void)
 ?{
- ? ? ? volatile uchar *pkt;
- ? ? ? volatile ushort *s;
- ? ? ? volatile ushort *cp;
+ ? ? ? uchar *pkt;
+ ? ? ? ushort *s;
+ ? ? ? ushort *cp;
 ? ? ? ?Ethernet_t *et;
 ? ? ? ?int len;
 ? ? ? ?ushort chksum;

@@ -964,7 +964,7 @@ int CDPSendTrigger(void)

? ? ? ?/* CDP header */
? ? ? ?*pkt++ = 0x02; ? ? ? ? ? ? ? ? ? ? ? ? ?/* CDP version 2 */ ? ? ? ?*pkt++ = 180; ? ? ? ? ? ? ? ? ? ? ? ? ? /* TTL */
- ? ? ? s = (volatile ushort *)pkt;
+ ? ? ? s = (ushort *)pkt;
? ? ? ?cp = s;
? ? ? ?/* checksum (0 for later calculation) */
? ? ? ?*s++ = htons(0);

? ? ? ?/* too small packet? */
? ? ? ?if (len < ETHER_HDR_SIZE)

net/net.c +0 -0

@@ -1491,11 +1491,11 @@ NetReceive(volatile uchar *inpkt, int len)

? ? ? ? ? ? ? ? */
? ? ? ? ? ? ? ?x = ntohs(et->et_prot);

- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + E802_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + E802_HDR_SIZE);

? ? ? ? ? ? ? ?len -= E802_HDR_SIZE;

? ? ? ?} else if (x != PROT_VLAN) { ? ?/* normal packet */
- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + ETHER_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + ETHER_HDR_SIZE);

? ? ? ? ? ? ? ?len -= ETHER_HDR_SIZE;

? ? ? ?} else { ? ? ? ? ? ? ? ? ? ? ? ?/* VLAN packet */

net/net.c +1 -1

@@ -1519,7 +1519,7 @@ NetReceive(volatile uchar *inpkt, int len)
 ? ? ? ? ? ? ? ?vlanid = cti & VLAN_IDMASK;
 ? ? ? ? ? ? ? ?x = ntohs(vet->vet_type);
 
- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + VLAN_ETHER_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + VLAN_ETHER_HDR_SIZE);
 
 
 ? ? ? ? ? ? ? ?len -= VLAN_ETHER_HDR_SIZE;

? ? ? ?}

net/net.c +0 -0

@@ -1917,7 +1917,7 @@ NetEthHdrSize(void)
 ?}
 

?int

net/net.c +1 -1

-NetSetEther(volatile uchar *xet, uchar * addr, uint prot)
+NetSetEther(uchar *xet, uchar * addr, uint prot)

?{
? ? ? ?Ethernet_t *et = (Ethernet_t *)xet;
? ? ? ?ushort myvlanid;

net/net.c +0 -0

@@ -1942,7 +1942,7 @@ NetSetEther(volatile uchar *xet, uchar * addr, uint prot)
 ?}
 

?void

net/net.c +1 -1

-NetSetIP(volatile uchar *xip, IPaddr_t dest, int dport, int sport, int len)
+NetSetIP(uchar *xip, IPaddr_t dest, int dport, int sport, int len)

?{
? ? ? ?IP_t *ip = (IP_t *)xip;

net/rarp.c +1 -1

index 097f970..77d63e8 100644
@@ -71,7 +71,7 @@ void
 ?RarpRequest (void)
 ?{
 ? ? ? ?int i;
- ? ? ? volatile uchar *pkt;
+ ? ? ? uchar *pkt;
 ? ? ? ?ARP_t * rarp;
 
 ? ? ? ?printf("RARP broadcast %d\n", ++RarpTry);

net/tftp.c +3 -3

index 7aa3e23..e62f229 100644
@@ -310,9 +310,9 @@ static void
 ?TftpSend(void)
 ?{
 ? ? ? ?uchar *pkt;
- ? ? ? volatile uchar *xp;
- ? ? ? int ? ? ? ? ? ? len = 0;
- ? ? ? volatile ushort *s;
+ ? ? ? uchar *xp;
+ ? ? ? int len = 0;
+ ? ? ? ushort *s;
 

?#ifdef CONFIG_MCAST_TFTP
? ? ? ?/* Multicast TFTP.. non-MasterClients do not ACK data. */ --
1.6.0.2

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Hi Joe,

Quoted 8 lines · Joe Hershberger

The mv_eth driver should not redefine the net function definition

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---
?<snip>

Review NetReceive(volatile uchar *inpkt, int len) · line 1459

Quoted 8 lines

(quoted diff) +2 -2

@@ -1454,9 +1454,9 @@ NetReceive(volatile uchar *inpkt, int len)
 
 ? ? ? ?debug("packet received\n");
 
- ? ? ? NetRxPacket = inpkt;
+ ? ? ? NetRxPacket = (uchar *)inpkt;
 ? ? ? ?NetRxPacketLen = len;
- ? ? ? et = (Ethernet_t *)inpkt;
+ ? ? ? et = (Ethernet_t *)NetRxPacket;

Why change this?

Quoted 7 lines

? ? ? ?/* too small packet? */
? ? ? ?if (len < ETHER_HDR_SIZE)

(quoted diff) +0 -0

@@ -1491,11 +1491,11 @@ NetReceive(volatile uchar *inpkt, int len)

? ? ? ? ? ? ? ? */
? ? ? ? ? ? ? ?x = ntohs(et->et_prot);

Quoted 2 lines

- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + E802_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + E802_HDR_SIZE);

and these? You are using a global instead of the passed-in local.

Quoted 21 lines

? ? ? ? ? ? ? ?len -= E802_HDR_SIZE;

? ? ? ?} else if (x != PROT_VLAN) { ? ?/* normal packet */
- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + ETHER_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + ETHER_HDR_SIZE);

? ? ? ? ? ? ? ?len -= ETHER_HDR_SIZE;

? ? ? ?} else { ? ? ? ? ? ? ? ? ? ? ? ?/* VLAN packet */

(quoted diff) +1 -1

@@ -1519,7 +1519,7 @@ NetReceive(volatile uchar *inpkt, int len)
 ? ? ? ? ? ? ? ?vlanid = cti & VLAN_IDMASK;
 ? ? ? ? ? ? ? ?x = ntohs(vet->vet_type);
 
- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + VLAN_ETHER_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + VLAN_ETHER_HDR_SIZE);
 
 
 ? ? ? ? ? ? ? ?len -= VLAN_ETHER_HDR_SIZE;

? ? ? ?}

net/net.c NetReceive(volatile uchar *inpkt, int len) · line 1457

Quoted 4 lines

net/net.c +1 -1

@@ -1454,9 +1454,9 @@ NetReceive(volatile uchar *inpkt, int len)
<snip>
-       NetRxPacket = inpkt;
+       NetRxPacket = (uchar *)inpkt;

Hi Simon,

Quoted 52 lines · Simon Glass

Hi Joe,

Quoted 8 lines · Joe Hershberger

The mv_eth driver should not redefine the net function definition

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---
?<snip>

Review NetReceive(volatile uchar *inpkt, int len) · line 1459

Quoted 8 lines

(quoted diff) +2 -2

@@ -1454,9 +1454,9 @@ NetReceive(volatile uchar *inpkt, int len)
 
 ? ? ? ?debug("packet received\n");
 
- ? ? ? NetRxPacket = inpkt;
+ ? ? ? NetRxPacket = (uchar *)inpkt;
 ? ? ? ?NetRxPacketLen = len;
- ? ? ? et = (Ethernet_t *)inpkt;
+ ? ? ? et = (Ethernet_t *)NetRxPacket;

Why change this?

Quoted 9 lines

? ? ? ?/* too small packet? */
? ? ? ?if (len < ETHER_HDR_SIZE)

(quoted diff) +0 -0

@@ -1491,11 +1491,11 @@ NetReceive(volatile uchar *inpkt, int len)

? ? ? ? ? ? ? ? */
? ? ? ? ? ? ? ?x = ntohs(et->et_prot);

- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + E802_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + E802_HDR_SIZE);

and these? You are using a global instead of the passed-in local.

Quoted 16 lines

? ? ? ? ? ? ? ?len -= E802_HDR_SIZE;

? ? ? ?} else if (x != PROT_VLAN) { ? ?/* normal packet */
- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + ETHER_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + ETHER_HDR_SIZE);

? ? ? ? ? ? ? ?len -= ETHER_HDR_SIZE;

? ? ? ?} else { ? ? ? ? ? ? ? ? ? ? ? ?/* VLAN packet */

(quoted diff) +1 -1

@@ -1519,7 +1519,7 @@ NetReceive(volatile uchar *inpkt, int len)
 ? ? ? ? ? ? ? ?vlanid = cti & VLAN_IDMASK;
 ? ? ? ? ? ? ? ?x = ntohs(vet->vet_type);
 
- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + VLAN_ETHER_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + VLAN_ETHER_HDR_SIZE);

Quoted 3 lines

Review NetReceive(volatile uchar *inpkt, int len) · line 1525

Quoted 3 lines

(quoted diff) +0 -0

 ? ? ? ? ? ? ? ?len -= VLAN_ETHER_HDR_SIZE;
 ? ? ? ?}
 

This patch removes volatile from the NetRxPacket and all but 1 of the other places that inpkt was assigned. ?You will notice that the first assignment of inpkt to NetRxPacket casts away the volatile:

Quoted 5 lines

Review NetReceive(volatile uchar *inpkt, int len)

Quoted 5 lines

(quoted diff) +1 -1

@@ -1454,9 +1454,9 @@ NetReceive(volatile uchar *inpkt, int len)
<snip>
- ? ? ? NetRxPacket = inpkt;
+ ? ? ? NetRxPacket = (uchar *)inpkt;

All the assignments that need a non-volatile pointer now use NetRxPacket instead of inpkt, since it is already assigned and and the same type minus volatile.

Best regards,
-Joe

Hi Joe,

Quoted 59 lines · Joe Hershberger

Hi Simon,

Quoted 55 lines · Simon Glass

Hi Joe,

Quoted 8 lines · Joe Hershberger

The mv_eth driver should not redefine the net function definition

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---
?<snip>

Review NetReceive(volatile uchar *inpkt, int len) · line 1459

Quoted 8 lines

(quoted diff) +2 -2

@@ -1454,9 +1454,9 @@ NetReceive(volatile uchar *inpkt, int len)
 
 ? ? ? ?debug("packet received\n");
 
- ? ? ? NetRxPacket = inpkt;
+ ? ? ? NetRxPacket = (uchar *)inpkt;
 ? ? ? ?NetRxPacketLen = len;
- ? ? ? et = (Ethernet_t *)inpkt;
+ ? ? ? et = (Ethernet_t *)NetRxPacket;

Why change this?

Quoted 9 lines

? ? ? ?/* too small packet? */
? ? ? ?if (len < ETHER_HDR_SIZE)

(quoted diff) +0 -0

@@ -1491,11 +1491,11 @@ NetReceive(volatile uchar *inpkt, int len)

? ? ? ? ? ? ? ? */
? ? ? ? ? ? ? ?x = ntohs(et->et_prot);

- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + E802_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + E802_HDR_SIZE);

and these? You are using a global instead of the passed-in local.

Quoted 21 lines

? ? ? ? ? ? ? ?len -= E802_HDR_SIZE;

? ? ? ?} else if (x != PROT_VLAN) { ? ?/* normal packet */
- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + ETHER_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + ETHER_HDR_SIZE);

? ? ? ? ? ? ? ?len -= ETHER_HDR_SIZE;

? ? ? ?} else { ? ? ? ? ? ? ? ? ? ? ? ?/* VLAN packet */

(quoted diff) +1 -1

@@ -1519,7 +1519,7 @@ NetReceive(volatile uchar *inpkt, int len)
 ? ? ? ? ? ? ? ?vlanid = cti & VLAN_IDMASK;
 ? ? ? ? ? ? ? ?x = ntohs(vet->vet_type);
 
- ? ? ? ? ? ? ? ip = (IP_t *)(inpkt + VLAN_ETHER_HDR_SIZE);
+ ? ? ? ? ? ? ? ip = (IP_t *)(NetRxPacket + VLAN_ETHER_HDR_SIZE);
 
 
 ? ? ? ? ? ? ? ?len -= VLAN_ETHER_HDR_SIZE;

? ? ? ?}

Quoted 3 lines

This patch removes volatile from the NetRxPacket and all but 1 of the other places that inpkt was assigned. ?You will notice that the first assignment of inpkt to NetRxPacket casts away the volatile:

Yes - I wonder why NetReceive needs to remain volatile?

Quoted 8 lines

Quoted 4 lines

Review NetReceive(volatile uchar *inpkt, int len)

Quoted 4 lines

(quoted diff) +1 -1

@@ -1454,9 +1454,9 @@ NetReceive(volatile uchar *inpkt, int len)
<snip>
- ? ? ? NetRxPacket = inpkt;
+ ? ? ? NetRxPacket = (uchar *)inpkt;

All the assignments that need a non-volatile pointer now use NetRxPacket instead of inpkt, since it is already assigned and and the same type minus volatile.

Yes, I am only sensitive to this because it is a global and there is enough use of globals in the net code already.

Quoted 12 lines · Simon Glass

Quoted 10 lines · Joe Hershberger

Quoted 5 lines · Simon Glass

net/net.c NetReceive(volatile uchar *inpkt, int len) · line 1457

Quoted 4 lines · Joe Hershberger

net/net.c +1 -1

@@ -1454,9 +1454,9 @@ NetReceive(volatile uchar *inpkt, int len)
<snip>
-       NetRxPacket = inpkt;
+       NetRxPacket = (uchar *)inpkt;

All the assignments that need a non-volatile pointer now use NetRxPacket instead of inpkt, since it is already assigned and and the same type minus volatile.

Quoted 2 lines

Yes, I am only sensitive to this because it is a global and there is enough use of globals in the net code already.

I chose to not add a local variable to hold the non-volatile pointer since there was already a global. Since u-boot is single threaded, this should not be a problem and could be easily changed later when the Ethernet driver interface is cleaned up.

in general, i like this. ?my only concern would be the drivers that might break due to incorrect cache management (which the volatile markers happen to work around for them).

having the API accept a volatile but then casting it away puts us in a worse place i think. ?on one hand, our API is saying "we treat it as volatile" when we really don't, so any drivers that call it with a volatile don't get build warnings. ?i'd just drop it from all of the net API so that the drivers which do call with a volatile pointer get a build warning -- now the driver maintainer knows they have to at least look at something.
-mike

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---

?common/main.c ? ? ? ? ? ?| ? ?1 -
?drivers/net/netconsole.c | ?122 +++++++++--------
?include/common.h ? ? ? ? | ? ?4 +
?include/net.h ? ? ? ? ? ?| ?123 +++++++++--------
?net/bootp.c ? ? ? ? ? ? ?| ?338 +++++++++++++++++++++++-----------------------
?net/bootp.h ? ? ? ? ? ? ?| ? 56 ++++----
?net/eth.c ? ? ? ? ? ? ? ?| ? 66 +++++-----
?net/net.c ? ? ? ? ? ? ? ?| ? 19 +--
?net/nfs.c ? ? ? ? ? ? ? ?| ?300 ++++++++++++++++++++---------------------
?net/nfs.h ? ? ? ? ? ? ? ?| ? ?2 +-
?net/rarp.c ? ? ? ? ? ? ? | ? 28 ++--
?net/rarp.h ? ? ? ? ? ? ? | ? ?4 +-
?net/sntp.c ? ? ? ? ? ? ? | ? 31 +++--
?net/sntp.h ? ? ? ? ? ? ? | ? ?2 +-
?net/tftp.c ? ? ? ? ? ? ? | ? 30 ++---
?net/tftp.h ? ? ? ? ? ? ? | ? ?2 +-

?16 files changed, 572 insertions(+), 556 deletions(-)

Hi Joe,

Quoted 20 lines · Joe Hershberger

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---

?common/main.c ? ? ? ? ? ?| ? ?1 -
?drivers/net/netconsole.c | ?122 +++++++++--------
?include/common.h ? ? ? ? | ? ?4 +
?include/net.h ? ? ? ? ? ?| ?123 +++++++++--------
?net/bootp.c ? ? ? ? ? ? ?| ?338 +++++++++++++++++++++++-----------------------
?net/bootp.h ? ? ? ? ? ? ?| ? 56 ++++----
?net/eth.c ? ? ? ? ? ? ? ?| ? 66 +++++-----
?net/net.c ? ? ? ? ? ? ? ?| ? 19 +--
?net/nfs.c ? ? ? ? ? ? ? ?| ?300 ++++++++++++++++++++---------------------
?net/nfs.h ? ? ? ? ? ? ? ?| ? ?2 +-
?net/rarp.c ? ? ? ? ? ? ? | ? 28 ++--
?net/rarp.h ? ? ? ? ? ? ? | ? ?4 +-
?net/sntp.c ? ? ? ? ? ? ? | ? 31 +++--
?net/sntp.h ? ? ? ? ? ? ? | ? ?2 +-
?net/tftp.c ? ? ? ? ? ? ? | ? 30 ++---
?net/tftp.h ? ? ? ? ? ? ? | ? ?2 +-

Quoted 2 lines

?16 files changed, 572 insertions(+), 556 deletions(-)

Others may fine with it, but it might be worth splitting this very long patch, perhaps split out some of the larger files or the show_activity() change.

Hi Simon,

Quoted 26 lines · Simon Glass

Hi Joe,

Quoted 20 lines · Joe Hershberger

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---

?common/main.c ? ? ? ? ? ?| ? ?1 -
?drivers/net/netconsole.c | ?122 +++++++++--------
?include/common.h ? ? ? ? | ? ?4 +
?include/net.h ? ? ? ? ? ?| ?123 +++++++++--------
?net/bootp.c ? ? ? ? ? ? ?| ?338 +++++++++++++++++++++++-----------------------
?net/bootp.h ? ? ? ? ? ? ?| ? 56 ++++----
?net/eth.c ? ? ? ? ? ? ? ?| ? 66 +++++-----
?net/net.c ? ? ? ? ? ? ? ?| ? 19 +--
?net/nfs.c ? ? ? ? ? ? ? ?| ?300 ++++++++++++++++++++---------------------
?net/nfs.h ? ? ? ? ? ? ? ?| ? ?2 +-
?net/rarp.c ? ? ? ? ? ? ? | ? 28 ++--
?net/rarp.h ? ? ? ? ? ? ? | ? ?4 +-
?net/sntp.c ? ? ? ? ? ? ? | ? 31 +++--
?net/sntp.h ? ? ? ? ? ? ? | ? ?2 +-
?net/tftp.c ? ? ? ? ? ? ? | ? 30 ++---
?net/tftp.h ? ? ? ? ? ? ? | ? ?2 +-

Quoted 2 lines

?16 files changed, 572 insertions(+), 556 deletions(-)

Quoted 3 lines

Others may fine with it, but it might be worth splitting this very long patch, perhaps split out some of the larger files or the show_activity() change.

I had this separated by file, but figured since the series was pretty long, it is all cosmetic (making separate patches not especially helpful), and it was under the 100kB mail limit that it would be better as a single patch. ?If it will be a barrier to review, I can split it up again.

Best regards,
-Joe

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---

Clears up checkpatch.pl complaint about extern in c file Completes the effort started by Ben Warren in 2008 to generalize eth init

Quoted 3 lines · On Thursday 19 January 2012 19:53:06 Joe Hershberger wrote:

Clears up checkpatch.pl complaint about extern in c file Completes the effort started by Ben Warren in 2008 to generalize eth init

awesome. these boards have been around "forever", so if Stefan doesn't have any feedback, i'm good to merge it.

MAC-seeded rand() will be needed by link-local as well, so give it an interface

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---

?net/Makefile ? | ? ?1 +
?net/bootp.c ? ?| ? 69 +++++++++++--------------------------------------------
?net/bootp.h ? ?| ? ?3 --
?net/net_rand.c | ? 68 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
?net/net_rand.h | ? 18 ++++++++++++++

?5 files changed, 101 insertions(+), 58 deletions(-) ?create mode 100644 net/net_rand.c
?create mode 100644 net/net_rand.h

net/Makefile +0 -0

index 0544f6b..5901046 100644
@@ -31,6 +31,7 @@ COBJS-$(CONFIG_CMD_NET) ?+= bootp.o

?COBJS-$(CONFIG_CMD_DNS) ?+= dns.o ?COBJS-$(CONFIG_CMD_NET) ?+= eth.o ?COBJS-$(CONFIG_CMD_NET) ?+= net.o +COBJS-$(CONFIG_BOOTP_RANDOM_DELAY) += net_rand.o ?COBJS-$(CONFIG_CMD_NFS) ?+= nfs.o ?COBJS-$(CONFIG_CMD_RARP) += rarp.o ?COBJS-$(CONFIG_CMD_SNTP) += sntp.o

net/bootp.c +0 -0

index 07870d0..9824cd1 100644
@@ -12,6 +12,9 @@

?#include <command.h>
?#include <net.h>
?#include "bootp.h"

net/bootp.c +3 -0

+#ifdef CONFIG_BOOTP_RANDOM_DELAY
+#include "net_rand.h"
+#endif

?#include "tftp.h"
?#include "nfs.h"
?#ifdef CONFIG_STATUS_LED

net/bootp.c +3 -3

@@ -37,9 +40,6 @@
 
 ?ulong ? ? ? ? ?BootpID;
 ?int ? ? ? ? ? ?BootpTry;
-#ifdef CONFIG_BOOTP_RANDOM_DELAY
-ulong ? ? ? ? ?seed1, seed2;
-#endif
 
 ?#if defined(CONFIG_CMD_DHCP)
 ?dhcp_state_t dhcp_state = INIT;

@@ -583,66 +583,25 @@ BootpRequest(void)
 ? ? ? ?uchar *pkt, *iphdr;
 ? ? ? ?struct Bootp_t *bp;
 ? ? ? ?int ext_len, pktlen, iplen;
+#ifdef CONFIG_BOOTP_RANDOM_DELAY
+ ? ? ? ulong i, rand_ms;
+#endif
 
 ?#if defined(CONFIG_CMD_DHCP)
 ? ? ? ?dhcp_state = INIT;

?#endif

?#ifdef CONFIG_BOOTP_RANDOM_DELAY ? ? ? ? ? ? ? /* Random BOOTP delay */
- ? ? ? unsigned char bi_enetaddr[6];
- ? ? ? int ? reg;
- ? ? ? ulong tst1, tst2, sum, m_mask, m_value = 0;

net/bootp.c +8 -49

-
- ? ? ? if (BootpTry == 0) {
- ? ? ? ? ? ? ? /* get our mac */
- ? ? ? ? ? ? ? eth_getenv_enetaddr("ethaddr", bi_enetaddr);
-
- ? ? ? ? ? ? ? debug("BootpRequest => Our Mac: ");
- ? ? ? ? ? ? ? for (reg = 0; reg < 6; reg++)
- ? ? ? ? ? ? ? ? ? ? ? debug("%x%c", bi_enetaddr[reg], reg == 5 ? '\n' : ':');
-
- ? ? ? ? ? ? ? /* Mac-Manipulation 2 get seed1 */
- ? ? ? ? ? ? ? tst1 = 0;
- ? ? ? ? ? ? ? tst2 = 0;
- ? ? ? ? ? ? ? for (reg = 2; reg < 6; reg++) {
- ? ? ? ? ? ? ? ? ? ? ? tst1 = tst1 << 8;
- ? ? ? ? ? ? ? ? ? ? ? tst1 = tst1 | bi_enetaddr[reg];
- ? ? ? ? ? ? ? }
- ? ? ? ? ? ? ? for (reg = 0; reg < 2; reg++) {
- ? ? ? ? ? ? ? ? ? ? ? tst2 = tst2 | bi_enetaddr[reg];
- ? ? ? ? ? ? ? ? ? ? ? tst2 = tst2 << 8;
- ? ? ? ? ? ? ? }
-
- ? ? ? ? ? ? ? seed1 = tst1^tst2;
+ ? ? ? if (BootpTry == 0)
+ ? ? ? ? ? ? ? srand_mac();
 
- ? ? ? ? ? ? ? /* Mirror seed1*/
- ? ? ? ? ? ? ? m_mask = 0x1;
- ? ? ? ? ? ? ? for (reg = 1; reg <= 32; reg++) {
- ? ? ? ? ? ? ? ? ? ? ? m_value |= (m_mask & seed1);
- ? ? ? ? ? ? ? ? ? ? ? seed1 = seed1 >> 1;
- ? ? ? ? ? ? ? ? ? ? ? m_value = m_value << 1;
- ? ? ? ? ? ? ? }
- ? ? ? ? ? ? ? seed1 = m_value;
- ? ? ? ? ? ? ? seed2 = 0xB78D0945;
- ? ? ? }
-
- ? ? ? /* Random Number Generator */
- ? ? ? for (reg = 0; reg <= 0; reg++) {
- ? ? ? ? ? ? ? sum = seed1 + seed2;
- ? ? ? ? ? ? ? if (sum < seed1 || sum < seed2)
- ? ? ? ? ? ? ? ? ? ? ? sum++;
- ? ? ? ? ? ? ? seed2 = seed1;
- ? ? ? ? ? ? ? seed1 = sum;
-
- ? ? ? ? ? ? ? if (BootpTry <= 2) { ? ?/* Start with max 1024 * 1ms */
- ? ? ? ? ? ? ? ? ? ? ? sum = sum >> (22-BootpTry);
- ? ? ? ? ? ? ? } else { ? ? ? ?/*After 3rd BOOTP request max 8192 * 1ms */
- ? ? ? ? ? ? ? ? ? ? ? sum = sum >> 19;
- ? ? ? ? ? ? ? }
- ? ? ? }
+ ? ? ? if (BootpTry <= 2) ? ? ?/* Start with max 1024 * 1ms */
+ ? ? ? ? ? ? ? rand_ms = rand() >> (22-BootpTry);
+ ? ? ? else ? ? ? ? ? ?/* After 3rd BOOTP request max 8192 * 1ms */
+ ? ? ? ? ? ? ? rand_ms = rand() >> 19;
 
- ? ? ? printf("Random delay: %ld ms...\n", sum);
- ? ? ? for (reg = 0; reg < sum; reg++)
+ ? ? ? printf("Random delay: %ld ms...\n", rand_ms);
+ ? ? ? for (i = 0; i < rand_ms; i++)

? ? ? ? ? ? ? ?udelay(1000); /*Wait 1ms*/

?#endif /* CONFIG_BOOTP_RANDOM_DELAY */

net/bootp.h +0 -0

index ce73734..bf4e875 100644
@@ -63,9 +63,6 @@ struct Bootp_t {

?extern ulong ? BootpID; ? ? ? ? ? ? ? ?/* ID of cur BOOTP request ? ? ?*/ ?extern char ? ?BootFile[128]; ? ? ? ? ?/* Boot file name ? ? ? ? ? ? ? */ ?extern int ? ? BootpTry;

net/bootp.h +0 -3

-#ifdef CONFIG_BOOTP_RANDOM_DELAY
-extern ulong ? seed1, seed2; ? ? ? ? ? /* seed for random BOOTP delay ?*/
-#endif

?/* Send a BOOTP request */

net/net_rand.c new file +68 -0

new file mode 100644
index 0000000..744e2f9
@@ -0,0 +1,68 @@
+/*
+ * ? ? Based on LiMon - BOOTP.
+ *
+ * ? ? Copyright 1994, 1995, 2000 Neil Russell.
+ * ? ? (See License)
+ * ? ? Copyright 2000 Roland Borde
+ * ? ? Copyright 2000 Paolo Scaffardi
+ * ? ? Copyright 2000-2004 Wolfgang Denk, wd at denx.de
+ */
+
+#include <common.h>
+#include <net.h>
+#include "net_rand.h"
+
+static ulong seed1, seed2;
+
+void srand_mac(void)
+{
+ ? ? ? ulong tst1, tst2, m_mask;
+ ? ? ? ulong m_value = 0;
+ ? ? ? int reg;
+ ? ? ? unsigned char bi_enetaddr[6];
+
+ ? ? ? /* get our mac */
+ ? ? ? eth_getenv_enetaddr("ethaddr", bi_enetaddr);
+
+ ? ? ? debug("BootpRequest => Our Mac: ");
+ ? ? ? for (reg = 0; reg < 6; reg++)
+ ? ? ? ? ? ? ? debug("%x%c", bi_enetaddr[reg], reg == 5 ? '\n' : ':');
+
+ ? ? ? /* Mac-Manipulation 2 get seed1 */
+ ? ? ? tst1 = 0;
+ ? ? ? tst2 = 0;
+ ? ? ? for (reg = 2; reg < 6; reg++) {
+ ? ? ? ? ? ? ? tst1 = tst1 << 8;
+ ? ? ? ? ? ? ? tst1 = tst1 | bi_enetaddr[reg];
+ ? ? ? }
+ ? ? ? for (reg = 0; reg < 2; reg++) {
+ ? ? ? ? ? ? ? tst2 = tst2 | bi_enetaddr[reg];
+ ? ? ? ? ? ? ? tst2 = tst2 << 8;
+ ? ? ? }
+
+ ? ? ? seed1 = tst1^tst2;
+
+ ? ? ? /* Mirror seed1*/
+ ? ? ? m_mask = 0x1;
+ ? ? ? for (reg = 1; reg <= 32; reg++) {
+ ? ? ? ? ? ? ? m_value |= (m_mask & seed1);
+ ? ? ? ? ? ? ? seed1 = seed1 >> 1;
+ ? ? ? ? ? ? ? m_value = m_value << 1;
+ ? ? ? }
+ ? ? ? seed1 = m_value;
+ ? ? ? seed2 = 0xB78D0945;
+}
+
+unsigned long rand(void)
+{
+ ? ? ? ulong sum;
+
+ ? ? ? /* Random Number Generator */
+ ? ? ? sum = seed1 + seed2;
+ ? ? ? if (sum < seed1 || sum < seed2)
+ ? ? ? ? ? ? ? sum++;
+ ? ? ? seed2 = seed1;
+ ? ? ? seed1 = sum;
+
+ ? ? ? return sum;
+}

net/net_rand.h new file +18 -1

new file mode 100644
index 0000000..2c6a6ac
@@ -0,0 +1,18 @@
+/*
+ * ? ? Copied from LiMon - BOOTP.
+ *
+ * ? ? Copyright 1994, 1995, 2000 Neil Russell.
+ * ? ? (See License)
+ * ? ? Copyright 2000 Paolo Scaffardi
+ */
+
+#ifndef __NET_RAND_H__
+#define __NET_RAND_H__
+
+#define RAND_MAX 0xFFFFFFFF
+
+void srand_mac(void);
+unsigned long rand(void);
+
+#endif /* __NET_RAND_H__ */
+
--

1.6.0.2

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

net/bootp.c line 17

Quoted 6 lines · On Thursday 19 January 2012 19:53:07 Joe Hershberger wrote:

net/bootp.c +3 -0

 
+#ifdef CONFIG_BOOTP_RANDOM_DELAY
+#include "net_rand.h"
+#endif

let's push this CONFIG_xxx check into net_rand.h so we don't have to check it every time we include the file

Quoted 10 lines · Mike Frysinger

net/bootp.c line 17

Quoted 6 lines · On Thursday 19 January 2012 19:53:07 Joe Hershberger wrote:

net/bootp.c +3 -0

 
+#ifdef CONFIG_BOOTP_RANDOM_DELAY
+#include "net_rand.h"
+#endif

let's push this CONFIG_xxx check into net_rand.h so we don't have to check it every time we include the file

In this case, it doesn't make sense. The rand is enabled based on other features that need it... it does not have a config of its own. As a result, putting this in the header would make it worse. This guard is still needed in bootp.c, since rand is only needed when that subfeature is enabled. For link-local, it will always be included.

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---

?arch/powerpc/cpu/mpc8xx/fec.c | ? ?2 +-
?include/net.h ? ? ? ? ? ? ? ? | ? ?9 +-
?net/Makefile ? ? ? ? ? ? ? ? ?| ? ?1 +
?net/cdp.c ? ? ? ? ? ? ? ? ? ? | ?380 +++++++++++++++++++++++++++++++++++++++++
?net/cdp.h ? ? ? ? ? ? ? ? ? ? | ? 18 ++
?net/net.c ? ? ? ? ? ? ? ? ? ? | ?368 +---------------------------------------

?6 files changed, 406 insertions(+), 372 deletions(-) ?create mode 100644 net/cdp.c
?create mode 100644 net/cdp.h

arch/powerpc/cpu/mpc8xx/fec.c +0 -0

index f2a2c3a..0bc1e5d 100644
@@ -274,7 +274,7 @@ static int fec_recv (struct eth_device *dev)
 ?#if defined(CONFIG_CMD_CDP)

? ? ? ? ? ? ? ? ? ? ? ?if ((rx[0] & 1) != 0
? ? ? ? ? ? ? ? ? ? ? ? ? ?&& memcmp ((uchar *) rx, NetBcastAddr, 6) != 0
- ? ? ? ? ? ? ? ? ? ? ? ? ? && memcmp ((uchar *) rx, NetCDPAddr, 6) != 0)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? && !is_cdp_packet((uchar *)rx))

? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?rx = NULL;
?#endif
? ? ? ? ? ? ? ? ? ? ? ?/*

include/net.h +0 -0

index 09d1b51..0396b69 100644
@@ -355,10 +355,6 @@ extern uchar ? ? ? ? ? ? ? NetEtherNullAddr[6];

?extern ushort ? ? ? ? ?NetOurVLAN; ? ? ? ? ? ? /* Our VLAN */ ?extern ushort ? ? ? ? ?NetOurNativeVLAN; ? ? ? /* Our Native VLAN */

include/net.h +0 -4

-extern uchar ? NetCDPAddr[6]; ? ? ? ? ?/* Ethernet CDP address */
-extern ushort ?CDPNativeVLAN; ? ? ? ? ?/* CDP returned native VLAN */
-extern ushort ?CDPApplianceVLAN; ? ? ? /* CDP returned appliance VLAN */
-

?extern int ? ? ? ? ? ? NetState; ? ? ? ? ? ? ? /* Network loop state */ ?#define NETLOOP_CONTINUE ? ? ? 1
?#define NETLOOP_RESTART ? ? ? ? ? ? ? ?2

include/net.h +0 -0

@@ -386,8 +382,9 @@ extern IPaddr_t ? ? NetPingIP; ? ? ? ? ? ? ? ? ? ? ?/* the ip address to ping */
 
 ?#if defined(CONFIG_CMD_CDP)

?/* when CDP completes these hold the return values */

include/net.h +3 -2

-extern ushort CDPNativeVLAN;
-extern ushort CDPApplianceVLAN;
+extern ushort CDPNativeVLAN; ? ? ? ? ? /* CDP returned native VLAN */
+extern ushort CDPApplianceVLAN; ? ? ? ? ? ? ? ?/* CDP returned appliance VLAN */
+int is_cdp_packet(const uchar *et_addr);

?#endif

?#if defined(CONFIG_CMD_SNTP)

net/Makefile +0 -0

index 5901046..b350bfc 100644
@@ -28,6 +28,7 @@ include $(TOPDIR)/config.mk

?LIB ? ?= $(obj)libnet.o

?COBJS-$(CONFIG_CMD_NET) ?+= bootp.o
+COBJS-$(CONFIG_CMD_CDP) ?+= cdp.o
?COBJS-$(CONFIG_CMD_DNS) ?+= dns.o
?COBJS-$(CONFIG_CMD_NET) ?+= eth.o
?COBJS-$(CONFIG_CMD_NET) ?+= net.o

net/cdp.c new file +380 -0

new file mode 100644
index 0000000..c5e54b2
@@ -0,0 +1,380 @@
+/*
+ * ? ? Copied from Linux Monitor (LiMon) - Networking.
+ *
+ * ? ? Copyright 1994 - 2000 Neil Russell.
+ * ? ? (See License)
+ * ? ? Copyright 2000 Roland Borde
+ * ? ? Copyright 2000 Paolo Scaffardi
+ * ? ? Copyright 2000-2002 Wolfgang Denk, wd at denx.de
+ */
+
+#include <common.h>
+#include <net.h>
+#if defined(CONFIG_CDP_VERSION)
+#include <timestamp.h>
+#endif
+
+#include "cdp.h"
+
+/* Ethernet bcast address */
+static const uchar NetCDPAddr[6] = { 0x01, 0x00, 0x0c, 0xcc, 0xcc, 0xcc };
+
+#define CDP_DEVICE_ID_TLV ? ? ? ? ? ? ?0x0001
+#define CDP_ADDRESS_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x0002
+#define CDP_PORT_ID_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x0003
+#define CDP_CAPABILITIES_TLV ? ? ? ? ? 0x0004
+#define CDP_VERSION_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x0005
+#define CDP_PLATFORM_TLV ? ? ? ? ? ? ? 0x0006
+#define CDP_NATIVE_VLAN_TLV ? ? ? ? ? ?0x000a
+#define CDP_APPLIANCE_VLAN_TLV ? ? ? ? 0x000e
+#define CDP_TRIGGER_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x000f
+#define CDP_POWER_CONSUMPTION_TLV ? ? ?0x0010
+#define CDP_SYSNAME_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x0014
+#define CDP_SYSOBJECT_TLV ? ? ? ? ? ? ?0x0015
+#define CDP_MANAGEMENT_ADDRESS_TLV ? ? 0x0016
+
+#define CDP_TIMEOUT ? ? ? ? ? ? ? ? ? ?250UL ? /* one packet every 250ms */
+
+static int CDPSeq;
+static int CDPOK;
+
+ushort CDPNativeVLAN;
+ushort CDPApplianceVLAN;
+
+static const uchar CDP_SNAP_hdr[8] = { 0xAA, 0xAA, 0x03, 0x00, 0x00, 0x0C, 0x20,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?0x00 };
+
+static ushort
+CDP_compute_csum(const uchar *buff, ushort len)
+{
+ ? ? ? ushort csum;
+ ? ? ? int ? ? odd;
+ ? ? ? ulong ? result = 0;
+ ? ? ? ushort ?leftover;
+ ? ? ? ushort *p;
+
+ ? ? ? if (len > 0) {
+ ? ? ? ? ? ? ? odd = 1 & (ulong)buff;
+ ? ? ? ? ? ? ? if (odd) {
+ ? ? ? ? ? ? ? ? ? ? ? result = *buff << 8;
+ ? ? ? ? ? ? ? ? ? ? ? len--;
+ ? ? ? ? ? ? ? ? ? ? ? buff++;
+ ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? while (len > 1) {
+ ? ? ? ? ? ? ? ? ? ? ? p = (ushort *)buff;
+ ? ? ? ? ? ? ? ? ? ? ? result += *p++;
+ ? ? ? ? ? ? ? ? ? ? ? buff = (uchar *)p;
+ ? ? ? ? ? ? ? ? ? ? ? if (result & 0x80000000)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? result = (result & 0xFFFF) + (result >> 16);
+ ? ? ? ? ? ? ? ? ? ? ? len -= 2;
+ ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? if (len) {
+ ? ? ? ? ? ? ? ? ? ? ? leftover = (signed short)(*(const signed char *)buff);
+ ? ? ? ? ? ? ? ? ? ? ? /* CISCO SUCKS big time! (and blows too):
+ ? ? ? ? ? ? ? ? ? ? ? ?* CDP uses the IP checksum algorithm with a twist;
+ ? ? ? ? ? ? ? ? ? ? ? ?* for the last byte it *sign* extends and sums.
+ ? ? ? ? ? ? ? ? ? ? ? ?*/
+ ? ? ? ? ? ? ? ? ? ? ? result = (result & 0xffff0000) |
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?((result + leftover) & 0x0000ffff);
+ ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? while (result >> 16)
+ ? ? ? ? ? ? ? ? ? ? ? result = (result & 0xFFFF) + (result >> 16);
+
+ ? ? ? ? ? ? ? if (odd)
+ ? ? ? ? ? ? ? ? ? ? ? result = ((result >> 8) & 0xff) |
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?((result & 0xff) << 8);
+ ? ? ? }
+
+ ? ? ? /* add up 16-bit and 17-bit words for 17+c bits */
+ ? ? ? result = (result & 0xffff) + (result >> 16);
+ ? ? ? /* add up 16-bit and 2-bit for 16+c bit */
+ ? ? ? result = (result & 0xffff) + (result >> 16);
+ ? ? ? /* add up carry.. */
+ ? ? ? result = (result & 0xffff) + (result >> 16);
+
+ ? ? ? /* negate */
+ ? ? ? csum = ~(ushort)result;
+
+ ? ? ? /* run time endian detection */
+ ? ? ? if (csum != htons(csum)) ? ? ? ?/* little endian */
+ ? ? ? ? ? ? ? csum = htons(csum);
+
+ ? ? ? return csum;
+}
+
+static int
+CDPSendTrigger(void)
+{
+ ? ? ? uchar *pkt;
+ ? ? ? ushort *s;
+ ? ? ? ushort *cp;
+ ? ? ? Ethernet_t *et;
+ ? ? ? int len;
+ ? ? ? ushort chksum;
+#if ? ?defined(CONFIG_CDP_DEVICE_ID) || defined(CONFIG_CDP_PORT_ID) ? || \
+ ? ? ? defined(CONFIG_CDP_VERSION) ? || defined(CONFIG_CDP_PLATFORM)
+ ? ? ? char buf[32];
+#endif
+
+ ? ? ? pkt = NetTxPacket;
+ ? ? ? et = (Ethernet_t *)pkt;
+
+ ? ? ? /* NOTE: trigger sent not on any VLAN */
+
+ ? ? ? /* form ethernet header */
+ ? ? ? memcpy(et->et_dest, NetCDPAddr, 6);
+ ? ? ? memcpy(et->et_src, NetOurEther, 6);
+
+ ? ? ? pkt += ETHER_HDR_SIZE;
+
+ ? ? ? /* SNAP header */
+ ? ? ? memcpy((uchar *)pkt, CDP_SNAP_hdr, sizeof(CDP_SNAP_hdr));
+ ? ? ? pkt += sizeof(CDP_SNAP_hdr);
+
+ ? ? ? /* CDP header */
+ ? ? ? *pkt++ = 0x02; ? ? ? ? ? ? ? ? ? ? ? ? ?/* CDP version 2 */
+ ? ? ? *pkt++ = 180; ? ? ? ? ? ? ? ? ? ? ? ? ? /* TTL */
+ ? ? ? s = (ushort *)pkt;
+ ? ? ? cp = s;
+ ? ? ? /* checksum (0 for later calculation) */
+ ? ? ? *s++ = htons(0);
+
+ ? ? ? /* CDP fields */
+#ifdef CONFIG_CDP_DEVICE_ID
+ ? ? ? *s++ = htons(CDP_DEVICE_ID_TLV);
+ ? ? ? *s++ = htons(CONFIG_CDP_DEVICE_ID);
+ ? ? ? sprintf(buf, CONFIG_CDP_DEVICE_ID_PREFIX "%pm", NetOurEther);
+ ? ? ? memcpy((uchar *)s, buf, 16);
+ ? ? ? s += 16 / 2;
+#endif
+
+#ifdef CONFIG_CDP_PORT_ID
+ ? ? ? *s++ = htons(CDP_PORT_ID_TLV);
+ ? ? ? memset(buf, 0, sizeof(buf));
+ ? ? ? sprintf(buf, CONFIG_CDP_PORT_ID, eth_get_dev_index());
+ ? ? ? len = strlen(buf);
+ ? ? ? if (len & 1) ? ?/* make it even */
+ ? ? ? ? ? ? ? len++;
+ ? ? ? *s++ = htons(len + 4);
+ ? ? ? memcpy((uchar *)s, buf, len);
+ ? ? ? s += len / 2;
+#endif
+
+#ifdef CONFIG_CDP_CAPABILITIES
+ ? ? ? *s++ = htons(CDP_CAPABILITIES_TLV);
+ ? ? ? *s++ = htons(8);
+ ? ? ? *(ulong *)s = htonl(CONFIG_CDP_CAPABILITIES);
+ ? ? ? s += 2;
+#endif
+
+#ifdef CONFIG_CDP_VERSION
+ ? ? ? *s++ = htons(CDP_VERSION_TLV);
+ ? ? ? memset(buf, 0, sizeof(buf));
+ ? ? ? strcpy(buf, CONFIG_CDP_VERSION);
+ ? ? ? len = strlen(buf);
+ ? ? ? if (len & 1) ? ?/* make it even */
+ ? ? ? ? ? ? ? len++;
+ ? ? ? *s++ = htons(len + 4);
+ ? ? ? memcpy((uchar *)s, buf, len);
+ ? ? ? s += len / 2;
+#endif
+
+#ifdef CONFIG_CDP_PLATFORM
+ ? ? ? *s++ = htons(CDP_PLATFORM_TLV);
+ ? ? ? memset(buf, 0, sizeof(buf));
+ ? ? ? strcpy(buf, CONFIG_CDP_PLATFORM);
+ ? ? ? len = strlen(buf);
+ ? ? ? if (len & 1) ? ?/* make it even */
+ ? ? ? ? ? ? ? len++;
+ ? ? ? *s++ = htons(len + 4);
+ ? ? ? memcpy((uchar *)s, buf, len);
+ ? ? ? s += len / 2;
+#endif
+
+#ifdef CONFIG_CDP_TRIGGER
+ ? ? ? *s++ = htons(CDP_TRIGGER_TLV);
+ ? ? ? *s++ = htons(8);
+ ? ? ? *(ulong *)s = htonl(CONFIG_CDP_TRIGGER);
+ ? ? ? s += 2;
+#endif
+
+#ifdef CONFIG_CDP_POWER_CONSUMPTION
+ ? ? ? *s++ = htons(CDP_POWER_CONSUMPTION_TLV);
+ ? ? ? *s++ = htons(6);
+ ? ? ? *s++ = htons(CONFIG_CDP_POWER_CONSUMPTION);
+#endif
+
+ ? ? ? /* length of ethernet packet */
+ ? ? ? len = (uchar *)s - ((uchar *)NetTxPacket + ETHER_HDR_SIZE);
+ ? ? ? et->et_protlen = htons(len);
+
+ ? ? ? len = ETHER_HDR_SIZE + sizeof(CDP_SNAP_hdr);
+ ? ? ? chksum = CDP_compute_csum((uchar *)NetTxPacket + len,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? (uchar *)s - (NetTxPacket + len));
+ ? ? ? if (chksum == 0)
+ ? ? ? ? ? ? ? chksum = 0xFFFF;
+ ? ? ? *cp = htons(chksum);
+
+ ? ? ? (void) eth_send(NetTxPacket, (uchar *)s - NetTxPacket);
+ ? ? ? return 0;
+}
+
+static void
+CDPTimeout(void)
+{
+ ? ? ? CDPSeq++;
+
+ ? ? ? if (CDPSeq < 3) {
+ ? ? ? ? ? ? ? NetSetTimeout(CDP_TIMEOUT, CDPTimeout);
+ ? ? ? ? ? ? ? CDPSendTrigger();
+ ? ? ? ? ? ? ? return;
+ ? ? ? }
+
+ ? ? ? /* if not OK try again */
+ ? ? ? if (!CDPOK)
+ ? ? ? ? ? ? ? NetStartAgain();
+ ? ? ? else
+ ? ? ? ? ? ? ? NetState = NETLOOP_SUCCESS;
+}
+
+static void
+CDPDummyHandler(uchar *pkt, unsigned dest, IPaddr_t sip, unsigned src,
+ ? ? ? ? ? ? ? unsigned len)
+{
+ ? ? ? /* nothing */
+}
+
+void
+CDPHandler(const uchar *pkt, unsigned len)
+{
+ ? ? ? const uchar *t;
+ ? ? ? const ushort *ss;
+ ? ? ? ushort type, tlen;
+ ? ? ? ushort vlan, nvlan;
+
+ ? ? ? /* minimum size? */
+ ? ? ? if (len < sizeof(CDP_SNAP_hdr) + 4)
+ ? ? ? ? ? ? ? goto pkt_short;
+
+ ? ? ? /* check for valid CDP SNAP header */
+ ? ? ? if (memcmp(pkt, CDP_SNAP_hdr, sizeof(CDP_SNAP_hdr)) != 0)
+ ? ? ? ? ? ? ? return;
+
+ ? ? ? pkt += sizeof(CDP_SNAP_hdr);
+ ? ? ? len -= sizeof(CDP_SNAP_hdr);
+
+ ? ? ? /* Version of CDP protocol must be >= 2 and TTL != 0 */
+ ? ? ? if (pkt[0] < 0x02 || pkt[1] == 0)
+ ? ? ? ? ? ? ? return;
+
+ ? ? ? /*
+ ? ? ? ?* if version is greater than 0x02 maybe we'll have a problem;
+ ? ? ? ?* output a warning
+ ? ? ? ?*/
+ ? ? ? if (pkt[0] != 0x02)
+ ? ? ? ? ? ? ? printf("**WARNING: CDP packet received with a protocol version "
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? "%d > 2\n", pkt[0] & 0xff);
+
+ ? ? ? if (CDP_compute_csum(pkt, len) != 0)
+ ? ? ? ? ? ? ? return;
+
+ ? ? ? pkt += 4;
+ ? ? ? len -= 4;
+
+ ? ? ? vlan = htons(-1);
+ ? ? ? nvlan = htons(-1);
+ ? ? ? while (len > 0) {
+ ? ? ? ? ? ? ? if (len < 4)
+ ? ? ? ? ? ? ? ? ? ? ? goto pkt_short;
+
+ ? ? ? ? ? ? ? ss = (const ushort *)pkt;
+ ? ? ? ? ? ? ? type = ntohs(ss[0]);
+ ? ? ? ? ? ? ? tlen = ntohs(ss[1]);
+ ? ? ? ? ? ? ? if (tlen > len)
+ ? ? ? ? ? ? ? ? ? ? ? goto pkt_short;
+
+ ? ? ? ? ? ? ? pkt += tlen;
+ ? ? ? ? ? ? ? len -= tlen;
+
+ ? ? ? ? ? ? ? ss += 2; ? ? ? ?/* point ss to the data of the TLV */
+ ? ? ? ? ? ? ? tlen -= 4;
+
+ ? ? ? ? ? ? ? switch (type) {
+ ? ? ? ? ? ? ? case CDP_DEVICE_ID_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_ADDRESS_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_PORT_ID_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_CAPABILITIES_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_VERSION_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_PLATFORM_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_NATIVE_VLAN_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? nvlan = *ss;
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_APPLIANCE_VLAN_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? t = (const uchar *)ss;
+ ? ? ? ? ? ? ? ? ? ? ? while (tlen > 0) {
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? if (tlen < 3)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? goto pkt_short;
+
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ss = (const ushort *)(t + 1);
+
+#ifdef CONFIG_CDP_APPLIANCE_VLAN_TYPE
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? if (t[0] == CONFIG_CDP_APPLIANCE_VLAN_TYPE)
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? vlan = *ss;
+#else
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? /* XXX will this work; dunno */
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? vlan = ntohs(*ss);
+#endif
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? t += 3; tlen -= 3;
+ ? ? ? ? ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_TRIGGER_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_POWER_CONSUMPTION_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_SYSNAME_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_SYSOBJECT_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? case CDP_MANAGEMENT_ADDRESS_TLV:
+ ? ? ? ? ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? }
+ ? ? ? }
+
+ ? ? ? CDPApplianceVLAN = vlan;
+ ? ? ? CDPNativeVLAN = nvlan;
+
+ ? ? ? CDPOK = 1;
+ ? ? ? return;
+
+ pkt_short:
+ ? ? ? printf("** CDP packet is too short\n");
+ ? ? ? return;
+}
+
+void
+CDPStart(void)
+{
+ ? ? ? printf("Using %s device\n", eth_get_name());
+ ? ? ? CDPSeq = 0;
+ ? ? ? CDPOK = 0;
+
+ ? ? ? CDPNativeVLAN = htons(-1);
+ ? ? ? CDPApplianceVLAN = htons(-1);
+
+ ? ? ? NetSetTimeout(CDP_TIMEOUT, CDPTimeout);
+ ? ? ? NetSetHandler(CDPDummyHandler);
+
+ ? ? ? CDPSendTrigger();
+}
+
+int
+is_cdp_packet(const uchar *et_addr)
+{
+ ? ? ? return memcmp(et_addr, NetCDPAddr, 6) == 0;
+}

net/cdp.h new file +18 -0

new file mode 100644
index 0000000..fef744e
@@ -0,0 +1,18 @@
+/*
+ * ? ? Copied from Linux Monitor (LiMon) - Networking.
+ *
+ * ? ? Copyright 1994 - 2000 Neil Russell.
+ * ? ? (See License)
+ * ? ? Copyright 2000 Roland Borde
+ * ? ? Copyright 2000 Paolo Scaffardi
+ * ? ? Copyright 2000-2002 Wolfgang Denk, wd at denx.de
+ */
+
+#ifndef __CDP_H__
+#define __CDP_H__
+
+void CDPStart(void);
+void CDPHandler(const uchar *pkt, unsigned len);
+
+#endif /* __CDP_H__ */
+

net/net.c +0 -0

index 1d6252f..48d3ca8 100644
@@ -91,8 +91,8 @@
 ?#if defined(CONFIG_CMD_SNTP)

?#include "sntp.h"
?#endif

net/net.c +2 -2

-#if defined(CONFIG_CDP_VERSION)
-#include <timestamp.h>
+#if defined(CONFIG_CMD_CDP)
+#include "cdp.h"

?#endif
?#if defined(CONFIG_CMD_DNS)
?#include "dns.h"

net/net.c +0 -0

@@ -163,10 +163,6 @@ uchar ? ? ? ? ? ? ?NetEtherNullAddr[6];

?#ifdef CONFIG_API
?void ? ? ? ? ? (*push_packet)(volatile void *, int len) = 0; ?#endif

net/net.c +0 -4

-#if defined(CONFIG_CMD_CDP)
-/* Ethernet bcast address */
-uchar ? ? ? ? ?NetCDPAddr[6] = { 0x01, 0x00, 0x0c, 0xcc, 0xcc, 0xcc };
-#endif

?/* Network loop state */
?int ? ? ? ? ? ?NetState;
?/* Tried all network devices */

net/net.c +0 -0

@@ -192,10 +188,6 @@ IPaddr_t ? NetPingIP;
 ?static void PingStart(void);

?#endif

net/net.c +0 -4

-#if defined(CONFIG_CMD_CDP)
-static void CDPStart(void);
-#endif
-

?#if defined(CONFIG_CMD_SNTP)
?/* NTP server IP address */
?IPaddr_t ? ? ? NetNtpServerIP;

net/net.c +0 -0

@@ -841,360 +833,6 @@ static void PingStart(void)
 ?}

?#endif

net/net.c +0 -354

-#if defined(CONFIG_CMD_CDP)
-
-#define CDP_DEVICE_ID_TLV ? ? ? ? ? ? ?0x0001
-#define CDP_ADDRESS_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x0002
-#define CDP_PORT_ID_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x0003
-#define CDP_CAPABILITIES_TLV ? ? ? ? ? 0x0004
-#define CDP_VERSION_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x0005
-#define CDP_PLATFORM_TLV ? ? ? ? ? ? ? 0x0006
-#define CDP_NATIVE_VLAN_TLV ? ? ? ? ? ?0x000a
-#define CDP_APPLIANCE_VLAN_TLV ? ? ? ? 0x000e
-#define CDP_TRIGGER_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x000f
-#define CDP_POWER_CONSUMPTION_TLV ? ? ?0x0010
-#define CDP_SYSNAME_TLV ? ? ? ? ? ? ? ? ? ? ? ?0x0014
-#define CDP_SYSOBJECT_TLV ? ? ? ? ? ? ?0x0015
-#define CDP_MANAGEMENT_ADDRESS_TLV ? ? 0x0016
-
-#define CDP_TIMEOUT ? ? ? ? ? ? ? ? ? ?250UL ? /* one packet every 250ms */
-
-static int CDPSeq;
-static int CDPOK;
-
-ushort CDPNativeVLAN;
-ushort CDPApplianceVLAN;
-
-static const uchar CDP_SNAP_hdr[8] = { 0xAA, 0xAA, 0x03, 0x00, 0x00, 0x0C, 0x20,
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?0x00 };
-
-static ushort CDP_compute_csum(const uchar *buff, ushort len)
-{
- ? ? ? ushort csum;
- ? ? ? int ? ? odd;
- ? ? ? ulong ? result = 0;
- ? ? ? ushort ?leftover;
- ? ? ? ushort *p;
-
- ? ? ? if (len > 0) {
- ? ? ? ? ? ? ? odd = 1 & (ulong)buff;
- ? ? ? ? ? ? ? if (odd) {
- ? ? ? ? ? ? ? ? ? ? ? result = *buff << 8;
- ? ? ? ? ? ? ? ? ? ? ? len--;
- ? ? ? ? ? ? ? ? ? ? ? buff++;
- ? ? ? ? ? ? ? }
- ? ? ? ? ? ? ? while (len > 1) {
- ? ? ? ? ? ? ? ? ? ? ? p = (ushort *)buff;
- ? ? ? ? ? ? ? ? ? ? ? result += *p++;
- ? ? ? ? ? ? ? ? ? ? ? buff = (uchar *)p;
- ? ? ? ? ? ? ? ? ? ? ? if (result & 0x80000000)
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? result = (result & 0xFFFF) + (result >> 16);
- ? ? ? ? ? ? ? ? ? ? ? len -= 2;
- ? ? ? ? ? ? ? }
- ? ? ? ? ? ? ? if (len) {
- ? ? ? ? ? ? ? ? ? ? ? leftover = (signed short)(*(const signed char *)buff);
- ? ? ? ? ? ? ? ? ? ? ? /* CISCO SUCKS big time! (and blows too):
- ? ? ? ? ? ? ? ? ? ? ? ?* CDP uses the IP checksum algorithm with a twist;
- ? ? ? ? ? ? ? ? ? ? ? ?* for the last byte it *sign* extends and sums.
- ? ? ? ? ? ? ? ? ? ? ? ?*/
- ? ? ? ? ? ? ? ? ? ? ? result = (result & 0xffff0000) |
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?((result + leftover) & 0x0000ffff);
- ? ? ? ? ? ? ? }
- ? ? ? ? ? ? ? while (result >> 16)
- ? ? ? ? ? ? ? ? ? ? ? result = (result & 0xFFFF) + (result >> 16);
-
- ? ? ? ? ? ? ? if (odd)
- ? ? ? ? ? ? ? ? ? ? ? result = ((result >> 8) & 0xff) |
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?((result & 0xff) << 8);
- ? ? ? }
-
- ? ? ? /* add up 16-bit and 17-bit words for 17+c bits */
- ? ? ? result = (result & 0xffff) + (result >> 16);
- ? ? ? /* add up 16-bit and 2-bit for 16+c bit */
- ? ? ? result = (result & 0xffff) + (result >> 16);
- ? ? ? /* add up carry.. */
- ? ? ? result = (result & 0xffff) + (result >> 16);
-
- ? ? ? /* negate */
- ? ? ? csum = ~(ushort)result;
-
- ? ? ? /* run time endian detection */
- ? ? ? if (csum != htons(csum)) ? ? ? ?/* little endian */
- ? ? ? ? ? ? ? csum = htons(csum);
-
- ? ? ? return csum;
-}
-
-int CDPSendTrigger(void)
-{
- ? ? ? uchar *pkt;
- ? ? ? ushort *s;
- ? ? ? ushort *cp;
- ? ? ? Ethernet_t *et;
- ? ? ? int len;
- ? ? ? ushort chksum;
-#if ? ?defined(CONFIG_CDP_DEVICE_ID) || defined(CONFIG_CDP_PORT_ID) ? || \
- ? ? ? defined(CONFIG_CDP_VERSION) ? || defined(CONFIG_CDP_PLATFORM)
- ? ? ? char buf[32];
-#endif
-
- ? ? ? pkt = NetTxPacket;
- ? ? ? et = (Ethernet_t *)pkt;
-
- ? ? ? /* NOTE: trigger sent not on any VLAN */
-
- ? ? ? /* form ethernet header */
- ? ? ? memcpy(et->et_dest, NetCDPAddr, 6);
- ? ? ? memcpy(et->et_src, NetOurEther, 6);
-
- ? ? ? pkt += ETHER_HDR_SIZE;
-
- ? ? ? /* SNAP header */
- ? ? ? memcpy((uchar *)pkt, CDP_SNAP_hdr, sizeof(CDP_SNAP_hdr));
- ? ? ? pkt += sizeof(CDP_SNAP_hdr);
-
- ? ? ? /* CDP header */
- ? ? ? *pkt++ = 0x02; ? ? ? ? ? ? ? ? ? ? ? ? ?/* CDP version 2 */
- ? ? ? *pkt++ = 180; ? ? ? ? ? ? ? ? ? ? ? ? ? /* TTL */
- ? ? ? s = (ushort *)pkt;
- ? ? ? cp = s;
- ? ? ? /* checksum (0 for later calculation) */
- ? ? ? *s++ = htons(0);
-
- ? ? ? /* CDP fields */
-#ifdef CONFIG_CDP_DEVICE_ID
- ? ? ? *s++ = htons(CDP_DEVICE_ID_TLV);
- ? ? ? *s++ = htons(CONFIG_CDP_DEVICE_ID);
- ? ? ? sprintf(buf, CONFIG_CDP_DEVICE_ID_PREFIX "%pm", NetOurEther);
- ? ? ? memcpy((uchar *)s, buf, 16);
- ? ? ? s += 16 / 2;
-#endif
-
-#ifdef CONFIG_CDP_PORT_ID
- ? ? ? *s++ = htons(CDP_PORT_ID_TLV);
- ? ? ? memset(buf, 0, sizeof(buf));
- ? ? ? sprintf(buf, CONFIG_CDP_PORT_ID, eth_get_dev_index());
- ? ? ? len = strlen(buf);
- ? ? ? if (len & 1) ? ?/* make it even */
- ? ? ? ? ? ? ? len++;
- ? ? ? *s++ = htons(len + 4);
- ? ? ? memcpy((uchar *)s, buf, len);
- ? ? ? s += len / 2;
-#endif
-
-#ifdef CONFIG_CDP_CAPABILITIES
- ? ? ? *s++ = htons(CDP_CAPABILITIES_TLV);
- ? ? ? *s++ = htons(8);
- ? ? ? *(ulong *)s = htonl(CONFIG_CDP_CAPABILITIES);
- ? ? ? s += 2;
-#endif
-
-#ifdef CONFIG_CDP_VERSION
- ? ? ? *s++ = htons(CDP_VERSION_TLV);
- ? ? ? memset(buf, 0, sizeof(buf));
- ? ? ? strcpy(buf, CONFIG_CDP_VERSION);
- ? ? ? len = strlen(buf);
- ? ? ? if (len & 1) ? ?/* make it even */
- ? ? ? ? ? ? ? len++;
- ? ? ? *s++ = htons(len + 4);
- ? ? ? memcpy((uchar *)s, buf, len);
- ? ? ? s += len / 2;
-#endif
-
-#ifdef CONFIG_CDP_PLATFORM
- ? ? ? *s++ = htons(CDP_PLATFORM_TLV);
- ? ? ? memset(buf, 0, sizeof(buf));
- ? ? ? strcpy(buf, CONFIG_CDP_PLATFORM);
- ? ? ? len = strlen(buf);
- ? ? ? if (len & 1) ? ?/* make it even */
- ? ? ? ? ? ? ? len++;
- ? ? ? *s++ = htons(len + 4);
- ? ? ? memcpy((uchar *)s, buf, len);
- ? ? ? s += len / 2;
-#endif
-
-#ifdef CONFIG_CDP_TRIGGER
- ? ? ? *s++ = htons(CDP_TRIGGER_TLV);
- ? ? ? *s++ = htons(8);
- ? ? ? *(ulong *)s = htonl(CONFIG_CDP_TRIGGER);
- ? ? ? s += 2;
-#endif
-
-#ifdef CONFIG_CDP_POWER_CONSUMPTION
- ? ? ? *s++ = htons(CDP_POWER_CONSUMPTION_TLV);
- ? ? ? *s++ = htons(6);
- ? ? ? *s++ = htons(CONFIG_CDP_POWER_CONSUMPTION);
-#endif
-
- ? ? ? /* length of ethernet packet */
- ? ? ? len = (uchar *)s - ((uchar *)NetTxPacket + ETHER_HDR_SIZE);
- ? ? ? et->et_protlen = htons(len);
-
- ? ? ? len = ETHER_HDR_SIZE + sizeof(CDP_SNAP_hdr);
- ? ? ? chksum = CDP_compute_csum((uchar *)NetTxPacket + len,
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? (uchar *)s - (NetTxPacket + len));
- ? ? ? if (chksum == 0)
- ? ? ? ? ? ? ? chksum = 0xFFFF;
- ? ? ? *cp = htons(chksum);
-
- ? ? ? (void) eth_send(NetTxPacket, (uchar *)s - NetTxPacket);
- ? ? ? return 0;
-}
-
-static void
-CDPTimeout(void)
-{
- ? ? ? CDPSeq++;
-
- ? ? ? if (CDPSeq < 3) {
- ? ? ? ? ? ? ? NetSetTimeout(CDP_TIMEOUT, CDPTimeout);
- ? ? ? ? ? ? ? CDPSendTrigger();
- ? ? ? ? ? ? ? return;
- ? ? ? }
-
- ? ? ? /* if not OK try again */
- ? ? ? if (!CDPOK)
- ? ? ? ? ? ? ? NetStartAgain();
- ? ? ? else
- ? ? ? ? ? ? ? NetState = NETLOOP_SUCCESS;
-}
-
-static void
-CDPDummyHandler(uchar *pkt, unsigned dest, IPaddr_t sip, unsigned src,
- ? ? ? ? ? ? ? unsigned len)
-{
- ? ? ? /* nothing */
-}
-
-static void
-CDPHandler(const uchar *pkt, unsigned len)
-{
- ? ? ? const uchar *t;
- ? ? ? const ushort *ss;
- ? ? ? ushort type, tlen;
- ? ? ? ushort vlan, nvlan;
-
- ? ? ? /* minimum size? */
- ? ? ? if (len < sizeof(CDP_SNAP_hdr) + 4)
- ? ? ? ? ? ? ? goto pkt_short;
-
- ? ? ? /* check for valid CDP SNAP header */
- ? ? ? if (memcmp(pkt, CDP_SNAP_hdr, sizeof(CDP_SNAP_hdr)) != 0)
- ? ? ? ? ? ? ? return;
-
- ? ? ? pkt += sizeof(CDP_SNAP_hdr);
- ? ? ? len -= sizeof(CDP_SNAP_hdr);
-
- ? ? ? /* Version of CDP protocol must be >= 2 and TTL != 0 */
- ? ? ? if (pkt[0] < 0x02 || pkt[1] == 0)
- ? ? ? ? ? ? ? return;
-
- ? ? ? /*
- ? ? ? ?* if version is greater than 0x02 maybe we'll have a problem;
- ? ? ? ?* output a warning
- ? ? ? ?*/
- ? ? ? if (pkt[0] != 0x02)
- ? ? ? ? ? ? ? printf("**WARNING: CDP packet received with a protocol version "
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? "%d > 2\n", pkt[0] & 0xff);
-
- ? ? ? if (CDP_compute_csum(pkt, len) != 0)
- ? ? ? ? ? ? ? return;
-
- ? ? ? pkt += 4;
- ? ? ? len -= 4;
-
- ? ? ? vlan = htons(-1);
- ? ? ? nvlan = htons(-1);
- ? ? ? while (len > 0) {
- ? ? ? ? ? ? ? if (len < 4)
- ? ? ? ? ? ? ? ? ? ? ? goto pkt_short;
-
- ? ? ? ? ? ? ? ss = (const ushort *)pkt;
- ? ? ? ? ? ? ? type = ntohs(ss[0]);
- ? ? ? ? ? ? ? tlen = ntohs(ss[1]);
- ? ? ? ? ? ? ? if (tlen > len)
- ? ? ? ? ? ? ? ? ? ? ? goto pkt_short;
-
- ? ? ? ? ? ? ? pkt += tlen;
- ? ? ? ? ? ? ? len -= tlen;
-
- ? ? ? ? ? ? ? ss += 2; ? ? ? ?/* point ss to the data of the TLV */
- ? ? ? ? ? ? ? tlen -= 4;
-
- ? ? ? ? ? ? ? switch (type) {
- ? ? ? ? ? ? ? case CDP_DEVICE_ID_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_ADDRESS_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_PORT_ID_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_CAPABILITIES_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_VERSION_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_PLATFORM_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_NATIVE_VLAN_TLV:
- ? ? ? ? ? ? ? ? ? ? ? nvlan = *ss;
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_APPLIANCE_VLAN_TLV:
- ? ? ? ? ? ? ? ? ? ? ? t = (const uchar *)ss;
- ? ? ? ? ? ? ? ? ? ? ? while (tlen > 0) {
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? if (tlen < 3)
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? goto pkt_short;
-
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ss = (const ushort *)(t + 1);
-
-#ifdef CONFIG_CDP_APPLIANCE_VLAN_TYPE
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? if (t[0] == CONFIG_CDP_APPLIANCE_VLAN_TYPE)
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? vlan = *ss;
-#else
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? /* XXX will this work; dunno */
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? vlan = ntohs(*ss);
-#endif
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? t += 3; tlen -= 3;
- ? ? ? ? ? ? ? ? ? ? ? }
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_TRIGGER_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_POWER_CONSUMPTION_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_SYSNAME_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_SYSOBJECT_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? case CDP_MANAGEMENT_ADDRESS_TLV:
- ? ? ? ? ? ? ? ? ? ? ? break;
- ? ? ? ? ? ? ? }
- ? ? ? }
-
- ? ? ? CDPApplianceVLAN = vlan;
- ? ? ? CDPNativeVLAN = nvlan;
-
- ? ? ? CDPOK = 1;
- ? ? ? return;
-
- pkt_short:
- ? ? ? printf("** CDP packet is too short\n");
- ? ? ? return;
-}
-
-static void CDPStart(void)
-{
- ? ? ? printf("Using %s device\n", eth_get_name());
- ? ? ? CDPSeq = 0;
- ? ? ? CDPOK = 0;
-
- ? ? ? CDPNativeVLAN = htons(-1);
- ? ? ? CDPApplianceVLAN = htons(-1);
-
- ? ? ? NetSetTimeout(CDP_TIMEOUT, CDPTimeout);
- ? ? ? NetSetHandler(CDPDummyHandler);
-
- ? ? ? CDPSendTrigger();
-}
-#endif
-

?#ifdef CONFIG_IP_DEFRAG
?/*
?* This function collects fragments in a single packet, according

net/net.c +0 -0

@@ -1463,7 +1101,7 @@ NetReceive(volatile uchar *inpkt, int len)
 
 ?#if defined(CONFIG_CMD_CDP)

? ? ? ?/* keep track if packet is CDP */
- ? ? ? iscdp = memcmp(et->et_dest, NetCDPAddr, 6) == 0;
+ ? ? ? iscdp = is_cdp_packet(et->et_dest); ?#endif

? ? ? ?myvlanid = ntohs(NetOurVLAN);
--
1.6.0.2

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

net/cdp.h new file +8 -0

@@ -0,0 +1,18 @@
+/*
+ *	Copied from Linux Monitor (LiMon) - Networking.
+ *
+ *	Copyright 1994 - 2000 Neil Russell.
+ *	(See License)
+ *	Copyright 2000 Roland Borde
+ *	Copyright 2000 Paolo Scaffardi
+ *	Copyright 2000-2002 Wolfgang Denk, wd at denx.de

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---

?include/net.h | ? ?3 +-
?net/Makefile ?| ? ?1 +
?net/arp.c ? ? | ?213 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
?net/arp.h ? ? | ? 30 ++++++++
?net/net.c ? ? | ?209 ++++----------------------------------------------------

?5 files changed, 259 insertions(+), 197 deletions(-) ?create mode 100644 net/arp.c
?create mode 100644 net/arp.h

include/net.h +0 -0

index 0396b69..2d00233 100644
@@ -418,7 +418,8 @@ extern void NetSetIP(uchar *, IPaddr_t, int, int, int);

?extern int ? ? NetCksumOk(uchar *, int); ? ? ? /* Return true if cksum OK */ ?extern uint ? ?NetCksum(uchar *, int); ? ? ? ? /* Calculate the checksum */

include/net.h +2 -1

-/* Set callbacks */
+/* Callbacks */
+extern rxhand_f *NetGetHandler(void); ? ? ? ? ?/* Get RX packet handler */

?extern void ? ?NetSetHandler(rxhand_f *); ? ? ?/* Set RX packet handler */ ?extern void net_set_icmp_handler(rxhand_icmp_f *f); /* Set ICMP RX handler */ ?extern void ? ?NetSetTimeout(ulong, thand_f *);/* Set timeout handler */

net/Makefile +0 -0

index b350bfc..0916a56 100644
@@ -27,6 +27,7 @@ include $(TOPDIR)/config.mk
 

?LIB ? ?= $(obj)libnet.o

+COBJS-$(CONFIG_CMD_NET) ?+= arp.o
?COBJS-$(CONFIG_CMD_NET) ?+= bootp.o
?COBJS-$(CONFIG_CMD_CDP) ?+= cdp.o
?COBJS-$(CONFIG_CMD_DNS) ?+= dns.o

net/arp.c new file +213 -0

new file mode 100644
index 0000000..f75217c
@@ -0,0 +1,213 @@
+/*
+ * ? ? Copied from Linux Monitor (LiMon) - Networking.
+ *
+ * ? ? Copyright 1994 - 2000 Neil Russell.
+ * ? ? (See License)
+ * ? ? Copyright 2000 Roland Borde
+ * ? ? Copyright 2000 Paolo Scaffardi
+ * ? ? Copyright 2000-2002 Wolfgang Denk, wd at denx.de
+ */
+
+#include <common.h>
+
+#include "arp.h"
+
+#ifndef ? ? ? ?CONFIG_ARP_TIMEOUT
+/* Milliseconds before trying ARP again */
+# define ARP_TIMEOUT ? ? ? ? ? 5000UL
+#else
+# define ARP_TIMEOUT ? ? ? ? ? CONFIG_ARP_TIMEOUT
+#endif
+
+
+#ifndef ? ? ? ?CONFIG_NET_RETRY_COUNT
+# define ARP_TIMEOUT_COUNT ? ? 5 ? ? ? /* # of timeouts before giving up ?*/
+#else
+# define ARP_TIMEOUT_COUNT ? ? CONFIG_NET_RETRY_COUNT
+#endif
+
+IPaddr_t ? ? ? NetArpWaitPacketIP;
+IPaddr_t ? ? ? NetArpWaitReplyIP;
+/* MAC address of waiting packet's destination */
+uchar ? ? ? ? *NetArpWaitPacketMAC;
+/* THE transmit packet */
+uchar ? ? ? ? *NetArpWaitTxPacket;
+int ? ? ? ? ? ?NetArpWaitTxPacketSize;
+uchar ? ? ? ? ?NetArpWaitPacketBuf[PKTSIZE_ALIGN + PKTALIGN];
+ulong ? ? ? ? ?NetArpWaitTimerStart;
+int ? ? ? ? ? ?NetArpWaitTry;
+
+void ArpInit(void)
+{
+ ? ? ? /* XXX problem with bss workaround */
+ ? ? ? NetArpWaitPacketMAC = NULL;
+ ? ? ? NetArpWaitPacketIP = 0;
+ ? ? ? NetArpWaitReplyIP = 0;
+ ? ? ? NetArpWaitTxPacket = &NetArpWaitPacketBuf[0] + (PKTALIGN - 1);
+ ? ? ? NetArpWaitTxPacket -= (ulong)NetArpWaitTxPacket % PKTALIGN;
+ ? ? ? NetArpWaitTxPacketSize = 0;
+}
+
+void ArpRequest(void)
+{
+ ? ? ? uchar *pkt;
+ ? ? ? ARP_t *arp;
+
+ ? ? ? debug("ARP broadcast %d\n", NetArpWaitTry);
+
+ ? ? ? pkt = NetTxPacket;
+
+ ? ? ? pkt += NetSetEther(pkt, NetBcastAddr, PROT_ARP);
+
+ ? ? ? arp = (ARP_t *) pkt;
+
+ ? ? ? arp->ar_hrd = htons(ARP_ETHER);
+ ? ? ? arp->ar_pro = htons(PROT_IP);
+ ? ? ? arp->ar_hln = 6;
+ ? ? ? arp->ar_pln = 4;
+ ? ? ? arp->ar_op = htons(ARPOP_REQUEST);
+
+ ? ? ? /* source ET addr */
+ ? ? ? memcpy(&arp->ar_data[0], NetOurEther, 6);
+ ? ? ? /* source IP addr */
+ ? ? ? NetWriteIP((uchar *) &arp->ar_data[6], NetOurIP);
+ ? ? ? /* dest ET addr = 0 */
+ ? ? ? memset(&arp->ar_data[10], '\0', 6);
+ ? ? ? if ((NetArpWaitPacketIP & NetOurSubnetMask) !=
+ ? ? ? ? ? (NetOurIP & NetOurSubnetMask)) {
+ ? ? ? ? ? ? ? if (NetOurGatewayIP == 0) {
+ ? ? ? ? ? ? ? ? ? ? ? puts("## Warning: gatewayip needed but not set\n");
+ ? ? ? ? ? ? ? ? ? ? ? NetArpWaitReplyIP = NetArpWaitPacketIP;
+ ? ? ? ? ? ? ? } else {
+ ? ? ? ? ? ? ? ? ? ? ? NetArpWaitReplyIP = NetOurGatewayIP;
+ ? ? ? ? ? ? ? }
+ ? ? ? } else {
+ ? ? ? ? ? ? ? NetArpWaitReplyIP = NetArpWaitPacketIP;
+ ? ? ? }
+
+ ? ? ? NetWriteIP((uchar *) &arp->ar_data[16], NetArpWaitReplyIP);
+ ? ? ? (void) eth_send(NetTxPacket, (pkt - NetTxPacket) + ARP_HDR_SIZE);
+}
+
+void ArpTimeoutCheck(void)
+{
+ ? ? ? ulong t;
+
+ ? ? ? if (!NetArpWaitPacketIP)
+ ? ? ? ? ? ? ? return;
+
+ ? ? ? t = get_timer(0);
+
+ ? ? ? /* check for arp timeout */
+ ? ? ? if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT) {
+ ? ? ? ? ? ? ? NetArpWaitTry++;
+
+ ? ? ? ? ? ? ? if (NetArpWaitTry >= ARP_TIMEOUT_COUNT) {
+ ? ? ? ? ? ? ? ? ? ? ? puts("\nARP Retry count exceeded; starting again\n");
+ ? ? ? ? ? ? ? ? ? ? ? NetArpWaitTry = 0;
+ ? ? ? ? ? ? ? ? ? ? ? NetStartAgain();
+ ? ? ? ? ? ? ? } else {
+ ? ? ? ? ? ? ? ? ? ? ? NetArpWaitTimerStart = t;
+ ? ? ? ? ? ? ? ? ? ? ? ArpRequest();
+ ? ? ? ? ? ? ? }
+ ? ? ? }
+}
+
+void ArpReceive(Ethernet_t *et, IP_t *ip, int len)
+{
+ ? ? ? ARP_t *arp;
+ ? ? ? IPaddr_t tmp;
+ ? ? ? uchar *pkt;
+
+ ? ? ? /*
+ ? ? ? ?* We have to deal with two types of ARP packets:
+ ? ? ? ?* - REQUEST packets will be answered by sending ?our
+ ? ? ? ?* ? IP address - if we know it.
+ ? ? ? ?* - REPLY packates are expected only after we asked
+ ? ? ? ?* ? for the TFTP server's or the gateway's ethernet
+ ? ? ? ?* ? address; so if we receive such a packet, we set
+ ? ? ? ?* ? the server ethernet address
+ ? ? ? ?*/
+ ? ? ? debug("Got ARP\n");
+
+ ? ? ? arp = (ARP_t *)ip;
+ ? ? ? if (len < ARP_HDR_SIZE) {
+ ? ? ? ? ? ? ? printf("bad length %d < %d\n", len, ARP_HDR_SIZE);
+ ? ? ? ? ? ? ? return;
+ ? ? ? }
+ ? ? ? if (ntohs(arp->ar_hrd) != ARP_ETHER)
+ ? ? ? ? ? ? ? return;
+ ? ? ? if (ntohs(arp->ar_pro) != PROT_IP)
+ ? ? ? ? ? ? ? return;
+ ? ? ? if (arp->ar_hln != 6)
+ ? ? ? ? ? ? ? return;
+ ? ? ? if (arp->ar_pln != 4)
+ ? ? ? ? ? ? ? return;
+
+ ? ? ? if (NetOurIP == 0)
+ ? ? ? ? ? ? ? return;
+
+ ? ? ? if (NetReadIP(&arp->ar_data[16]) != NetOurIP)
+ ? ? ? ? ? ? ? return;
+
+ ? ? ? switch (ntohs(arp->ar_op)) {
+ ? ? ? case ARPOP_REQUEST:
+ ? ? ? ? ? ? ? /* reply with our IP address */
+ ? ? ? ? ? ? ? debug("Got ARP REQUEST, return our IP\n");
+ ? ? ? ? ? ? ? pkt = (uchar *)et;
+ ? ? ? ? ? ? ? pkt += NetSetEther(pkt, et->et_src, PROT_ARP);
+ ? ? ? ? ? ? ? arp->ar_op = htons(ARPOP_REPLY);
+ ? ? ? ? ? ? ? memcpy(&arp->ar_data[10], &arp->ar_data[0], 6);
+ ? ? ? ? ? ? ? NetCopyIP(&arp->ar_data[16], &arp->ar_data[6]);
+ ? ? ? ? ? ? ? memcpy(&arp->ar_data[0], NetOurEther, 6);
+ ? ? ? ? ? ? ? NetCopyIP(&arp->ar_data[6], &NetOurIP);
+ ? ? ? ? ? ? ? (void) eth_send((uchar *)et,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? (pkt - (uchar *)et) + ARP_HDR_SIZE);
+ ? ? ? ? ? ? ? return;
+
+ ? ? ? case ARPOP_REPLY: ? ? ? ? ? ? ? /* arp reply */
+ ? ? ? ? ? ? ? /* are we waiting for a reply */
+ ? ? ? ? ? ? ? if (!NetArpWaitPacketIP || !NetArpWaitPacketMAC)
+ ? ? ? ? ? ? ? ? ? ? ? break;
+
+#ifdef CONFIG_KEEP_SERVERADDR
+ ? ? ? ? ? ? ? if (NetServerIP == NetArpWaitPacketIP) {
+ ? ? ? ? ? ? ? ? ? ? ? char buf[20];
+ ? ? ? ? ? ? ? ? ? ? ? sprintf(buf, "%pM", arp->ar_data);
+ ? ? ? ? ? ? ? ? ? ? ? setenv("serveraddr", buf);
+ ? ? ? ? ? ? ? }
+#endif
+
+ ? ? ? ? ? ? ? tmp = NetReadIP(&arp->ar_data[6]);
+
+ ? ? ? ? ? ? ? /* matched waiting packet's address */
+ ? ? ? ? ? ? ? if (tmp == NetArpWaitReplyIP) {
+ ? ? ? ? ? ? ? ? ? ? ? debug("Got ARP REPLY, set eth addr (%pM)\n",
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? arp->ar_data);
+
+ ? ? ? ? ? ? ? ? ? ? ? /* save address for later use */
+ ? ? ? ? ? ? ? ? ? ? ? memcpy(NetArpWaitPacketMAC,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&arp->ar_data[0], 6);
+
+#ifdef CONFIG_NETCONSOLE
+ ? ? ? ? ? ? ? ? ? ? ? NetGetHandler()(0, 0, 0, 0, 0);
+#endif
+ ? ? ? ? ? ? ? ? ? ? ? /* modify header, and transmit it */
+ ? ? ? ? ? ? ? ? ? ? ? memcpy(((Ethernet_t *)NetArpWaitTxPacket)->
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? et_dest, NetArpWaitPacketMAC, 6);
+ ? ? ? ? ? ? ? ? ? ? ? (void) eth_send(NetArpWaitTxPacket,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? NetArpWaitTxPacketSize);
+
+ ? ? ? ? ? ? ? ? ? ? ? /* no arp request pending now */
+ ? ? ? ? ? ? ? ? ? ? ? NetArpWaitPacketIP = 0;
+ ? ? ? ? ? ? ? ? ? ? ? NetArpWaitTxPacketSize = 0;
+ ? ? ? ? ? ? ? ? ? ? ? NetArpWaitPacketMAC = NULL;
+
+ ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? return;
+ ? ? ? default:
+ ? ? ? ? ? ? ? debug("Unexpected ARP opcode 0x%x\n",
+ ? ? ? ? ? ? ? ? ? ? ntohs(arp->ar_op));
+ ? ? ? ? ? ? ? return;
+ ? ? ? }
+}

net/arp.h new file +30 -0

new file mode 100644
index 0000000..09c763d
@@ -0,0 +1,30 @@
+/*
+ * ? ? Copied from Linux Monitor (LiMon) - Networking.
+ *
+ * ? ? Copyright 1994 - 2000 Neil Russell.
+ * ? ? (See License)
+ * ? ? Copyright 2000 Roland Borde
+ * ? ? Copyright 2000 Paolo Scaffardi
+ * ? ? Copyright 2000-2002 Wolfgang Denk, wd at denx.de
+ */
+
+#ifndef __ARP_H__
+#define __ARP_H__
+
+#include <net.h>
+
+extern IPaddr_t ? ? ? ?NetArpWaitPacketIP;
+/* MAC address of waiting packet's destination */
+extern uchar *NetArpWaitPacketMAC;
+/* THE transmit packet */
+extern uchar *NetArpWaitTxPacket;
+extern int NetArpWaitTxPacketSize;
+extern ulong NetArpWaitTimerStart;
+extern int NetArpWaitTry;
+
+void ArpInit(void);
+void ArpRequest(void);
+void ArpTimeoutCheck(void);
+void ArpReceive(Ethernet_t *et, IP_t *ip, int len);
+
+#endif /* __ARP_H__ */

net/net.c +0 -0

index 48d3ca8..2dae5a0 100644
@@ -78,6 +78,7 @@

?#include <watchdog.h>
?#include <command.h>
?#include <net.h>
+#include "arp.h"
?#include "bootp.h"
?#include "tftp.h"
?#ifdef CONFIG_CMD_RARP

net/net.c +0 -14

@@ -100,20 +101,6 @@
 
 ?DECLARE_GLOBAL_DATA_PTR;
 
-#ifndef ? ? ? ?CONFIG_ARP_TIMEOUT
-/* Milliseconds before trying ARP again */
-# define ARP_TIMEOUT ? ? ? ? ? 5000UL
-#else
-# define ARP_TIMEOUT ? ? ? ? ? CONFIG_ARP_TIMEOUT
-#endif
-
-
-#ifndef ? ? ? ?CONFIG_NET_RETRY_COUNT
-# define ARP_TIMEOUT_COUNT ? ? 5 ? ? ? /* # of timeouts before giving up ?*/
-#else
-# define ARP_TIMEOUT_COUNT ? ? CONFIG_NET_RETRY_COUNT
-#endif
-

?/** BOOTP EXTENTIONS **/

?/* Our subnet mask (0=unknown) */

net/net.c +0 -0

@@ -220,82 +207,6 @@ static int NetTryCount;
 

?/**********************************************************************/

net/net.c +0 -76

-IPaddr_t ? ? ? NetArpWaitPacketIP;
-IPaddr_t ? ? ? NetArpWaitReplyIP;
-/* MAC address of waiting packet's destination */
-uchar ? ? ? ? *NetArpWaitPacketMAC;
-/* THE transmit packet */
-uchar ? ? ? ? *NetArpWaitTxPacket;
-int ? ? ? ? ? ?NetArpWaitTxPacketSize;
-uchar ? ? ? ? ?NetArpWaitPacketBuf[PKTSIZE_ALIGN + PKTALIGN];
-ulong ? ? ? ? ?NetArpWaitTimerStart;
-int ? ? ? ? ? ?NetArpWaitTry;
-
-void ArpRequest(void)
-{
- ? ? ? uchar *pkt;
- ? ? ? ARP_t *arp;
-
- ? ? ? debug("ARP broadcast %d\n", NetArpWaitTry);
-
- ? ? ? pkt = NetTxPacket;
-
- ? ? ? pkt += NetSetEther(pkt, NetBcastAddr, PROT_ARP);
-
- ? ? ? arp = (ARP_t *) pkt;
-
- ? ? ? arp->ar_hrd = htons(ARP_ETHER);
- ? ? ? arp->ar_pro = htons(PROT_IP);
- ? ? ? arp->ar_hln = 6;
- ? ? ? arp->ar_pln = 4;
- ? ? ? arp->ar_op = htons(ARPOP_REQUEST);
-
- ? ? ? /* source ET addr */
- ? ? ? memcpy(&arp->ar_data[0], NetOurEther, 6);
- ? ? ? /* source IP addr */
- ? ? ? NetWriteIP((uchar *) &arp->ar_data[6], NetOurIP);
- ? ? ? /* dest ET addr = 0 */
- ? ? ? memset(&arp->ar_data[10], '\0', 6);
- ? ? ? if ((NetArpWaitPacketIP & NetOurSubnetMask) !=
- ? ? ? ? ? (NetOurIP & NetOurSubnetMask)) {
- ? ? ? ? ? ? ? if (NetOurGatewayIP == 0) {
- ? ? ? ? ? ? ? ? ? ? ? puts("## Warning: gatewayip needed but not set\n");
- ? ? ? ? ? ? ? ? ? ? ? NetArpWaitReplyIP = NetArpWaitPacketIP;
- ? ? ? ? ? ? ? } else {
- ? ? ? ? ? ? ? ? ? ? ? NetArpWaitReplyIP = NetOurGatewayIP;
- ? ? ? ? ? ? ? }
- ? ? ? } else {
- ? ? ? ? ? ? ? NetArpWaitReplyIP = NetArpWaitPacketIP;
- ? ? ? }
-
- ? ? ? NetWriteIP((uchar *) &arp->ar_data[16], NetArpWaitReplyIP);
- ? ? ? (void) eth_send(NetTxPacket, (pkt - NetTxPacket) + ARP_HDR_SIZE);
-}
-
-void ArpTimeoutCheck(void)
-{
- ? ? ? ulong t;
-
- ? ? ? if (!NetArpWaitPacketIP)
- ? ? ? ? ? ? ? return;
-
- ? ? ? t = get_timer(0);
-
- ? ? ? /* check for arp timeout */
- ? ? ? if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT) {
- ? ? ? ? ? ? ? NetArpWaitTry++;
-
- ? ? ? ? ? ? ? if (NetArpWaitTry >= ARP_TIMEOUT_COUNT) {
- ? ? ? ? ? ? ? ? ? ? ? puts("\nARP Retry count exceeded; starting again\n");
- ? ? ? ? ? ? ? ? ? ? ? NetArpWaitTry = 0;
- ? ? ? ? ? ? ? ? ? ? ? NetStartAgain();
- ? ? ? ? ? ? ? } else {
- ? ? ? ? ? ? ? ? ? ? ? NetArpWaitTimerStart = t;
- ? ? ? ? ? ? ? ? ? ? ? ArpRequest();
- ? ? ? ? ? ? ? }
- ? ? ? }
-}
-

?/*
?* Check if autoload is enabled. If so, use either NFS or TFTP to download ?* the boot file.

net/net.c +2 -6

@@ -363,15 +274,11 @@ int NetLoop(enum proto_t protocol)
 ? ? ? ?NetRestarted = 0;
 ? ? ? ?NetDevExists = 0;
 
- ? ? ? /* XXX problem with bss workaround */
- ? ? ? NetArpWaitPacketMAC = NULL;
- ? ? ? NetArpWaitTxPacket = NULL;
- ? ? ? NetArpWaitPacketIP = 0;
- ? ? ? NetArpWaitReplyIP = 0;
- ? ? ? NetArpWaitTxPacket = NULL;
 ? ? ? ?NetTxPacket = NULL;
 ? ? ? ?NetTryCount = 1;
 
+ ? ? ? ArpInit();
+
 ? ? ? ?if (!NetTxPacket) {
 ? ? ? ? ? ? ? ?int ? ? i;

? ? ? ? ? ? ? ?/*

net/net.c +0 -6

@@ -383,12 +290,6 @@ int NetLoop(enum proto_t protocol)
 ? ? ? ? ? ? ? ? ? ? ? ?NetRxPackets[i] = NetTxPacket + (i+1)*PKTSIZE_ALIGN;
 ? ? ? ?}
 
- ? ? ? if (!NetArpWaitTxPacket) {
- ? ? ? ? ? ? ? NetArpWaitTxPacket = &NetArpWaitPacketBuf[0] + (PKTALIGN - 1);
- ? ? ? ? ? ? ? NetArpWaitTxPacket -= (ulong)NetArpWaitTxPacket % PKTALIGN;
- ? ? ? ? ? ? ? NetArpWaitTxPacketSize = 0;
- ? ? ? }
-
 ? ? ? ?eth_halt();
 ? ? ? ?eth_set_current();
 ? ? ? ?if (eth_init(bd) < 0) {

@@ -661,6 +562,13 @@ void NetStartAgain(void)

?* ? ? Miscelaneous bits.
?*/

net/net.c +7 -0

+rxhand_f *
+NetGetHandler(void)
+{
+ ? ? ? return packetHandler;
+}
+
+

?void
?NetSetHandler(rxhand_f *f)
?{

net/net.c +2 -1

@@ -1072,11 +980,12 @@ NetReceive(volatile uchar *inpkt, int len)
 ?{
 ? ? ? ?Ethernet_t *et;
 ? ? ? ?IP_t ? ?*ip;
+#ifdef CONFIG_CMD_RARP
 ? ? ? ?ARP_t ? *arp;
+#endif
 ? ? ? ?IPaddr_t tmp;
 ? ? ? ?IPaddr_t src_ip;
 ? ? ? ?int ? ? x;
- ? ? ? uchar *pkt;
 ?#if defined(CONFIG_CMD_CDP)
 ? ? ? ?int iscdp;

?#endif

net/net.c +0 -0

@@ -1173,99 +1082,7 @@ NetReceive(volatile uchar *inpkt, int len)
 ? ? ? ?switch (x) {
 

? ? ? ?case PROT_ARP:
- ? ? ? ? ? ? ? /*
- ? ? ? ? ? ? ? ?* We have to deal with two types of ARP packets:
- ? ? ? ? ? ? ? ?* - REQUEST packets will be answered by sending ?our
- ? ? ? ? ? ? ? ?* ? IP address - if we know it.
- ? ? ? ? ? ? ? ?* - REPLY packates are expected only after we asked
- ? ? ? ? ? ? ? ?* ? for the TFTP server's or the gateway's ethernet
- ? ? ? ? ? ? ? ?* ? address; so if we receive such a packet, we set
- ? ? ? ? ? ? ? ?* ? the server ethernet address
- ? ? ? ? ? ? ? ?*/
- ? ? ? ? ? ? ? debug("Got ARP\n");

net/net.c +1 -83

-
- ? ? ? ? ? ? ? arp = (ARP_t *)ip;
- ? ? ? ? ? ? ? if (len < ARP_HDR_SIZE) {
- ? ? ? ? ? ? ? ? ? ? ? printf("bad length %d < %d\n", len, ARP_HDR_SIZE);
- ? ? ? ? ? ? ? ? ? ? ? return;
- ? ? ? ? ? ? ? }
- ? ? ? ? ? ? ? if (ntohs(arp->ar_hrd) != ARP_ETHER)
- ? ? ? ? ? ? ? ? ? ? ? return;
- ? ? ? ? ? ? ? if (ntohs(arp->ar_pro) != PROT_IP)
- ? ? ? ? ? ? ? ? ? ? ? return;
- ? ? ? ? ? ? ? if (arp->ar_hln != 6)
- ? ? ? ? ? ? ? ? ? ? ? return;
- ? ? ? ? ? ? ? if (arp->ar_pln != 4)
- ? ? ? ? ? ? ? ? ? ? ? return;
-
- ? ? ? ? ? ? ? if (NetOurIP == 0)
- ? ? ? ? ? ? ? ? ? ? ? return;
-
- ? ? ? ? ? ? ? if (NetReadIP(&arp->ar_data[16]) != NetOurIP)
- ? ? ? ? ? ? ? ? ? ? ? return;
-
- ? ? ? ? ? ? ? switch (ntohs(arp->ar_op)) {
- ? ? ? ? ? ? ? case ARPOP_REQUEST:
- ? ? ? ? ? ? ? ? ? ? ? /* reply with our IP address */
- ? ? ? ? ? ? ? ? ? ? ? debug("Got ARP REQUEST, return our IP\n");
- ? ? ? ? ? ? ? ? ? ? ? pkt = (uchar *)et;
- ? ? ? ? ? ? ? ? ? ? ? pkt += NetSetEther(pkt, et->et_src, PROT_ARP);
- ? ? ? ? ? ? ? ? ? ? ? arp->ar_op = htons(ARPOP_REPLY);
- ? ? ? ? ? ? ? ? ? ? ? memcpy(&arp->ar_data[10], &arp->ar_data[0], 6);
- ? ? ? ? ? ? ? ? ? ? ? NetCopyIP(&arp->ar_data[16], &arp->ar_data[6]);
- ? ? ? ? ? ? ? ? ? ? ? memcpy(&arp->ar_data[0], NetOurEther, 6);
- ? ? ? ? ? ? ? ? ? ? ? NetCopyIP(&arp->ar_data[6], &NetOurIP);
- ? ? ? ? ? ? ? ? ? ? ? (void) eth_send((uchar *)et,
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? (pkt - (uchar *)et) + ARP_HDR_SIZE);
- ? ? ? ? ? ? ? ? ? ? ? return;
-
- ? ? ? ? ? ? ? case ARPOP_REPLY: ? ? ? ? ? ? ? /* arp reply */
- ? ? ? ? ? ? ? ? ? ? ? /* are we waiting for a reply */
- ? ? ? ? ? ? ? ? ? ? ? if (!NetArpWaitPacketIP || !NetArpWaitPacketMAC)
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? break;
-
-#ifdef CONFIG_KEEP_SERVERADDR
- ? ? ? ? ? ? ? ? ? ? ? if (NetServerIP == NetArpWaitPacketIP) {
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? char buf[20];
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? sprintf(buf, "%pM", arp->ar_data);
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? setenv("serveraddr", buf);
- ? ? ? ? ? ? ? ? ? ? ? }
-#endif
-
- ? ? ? ? ? ? ? ? ? ? ? debug("Got ARP REPLY, set server/gtwy eth addr (%pM)\n",
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? arp->ar_data);
-
- ? ? ? ? ? ? ? ? ? ? ? tmp = NetReadIP(&arp->ar_data[6]);
-
- ? ? ? ? ? ? ? ? ? ? ? /* matched waiting packet's address */
- ? ? ? ? ? ? ? ? ? ? ? if (tmp == NetArpWaitReplyIP) {
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? debug("Got it\n");
-
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? /* save address for later use */
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? memcpy(NetArpWaitPacketMAC,
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&arp->ar_data[0], 6);
-
-#ifdef CONFIG_NETCONSOLE
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? (*packetHandler)(0, 0, 0, 0, 0);
-#endif
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? /* modify header, and transmit it */
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? memcpy(((Ethernet_t *)NetArpWaitTxPacket)->
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? et_dest, NetArpWaitPacketMAC, 6);
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? (void) eth_send(NetArpWaitTxPacket,
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? NetArpWaitTxPacketSize);
-
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? /* no arp request pending now */
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? NetArpWaitPacketIP = 0;
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? NetArpWaitTxPacketSize = 0;
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? NetArpWaitPacketMAC = NULL;
-
- ? ? ? ? ? ? ? ? ? ? ? }
- ? ? ? ? ? ? ? ? ? ? ? return;
- ? ? ? ? ? ? ? default:
- ? ? ? ? ? ? ? ? ? ? ? debug("Unexpected ARP opcode 0x%x\n",
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ntohs(arp->ar_op));
- ? ? ? ? ? ? ? ? ? ? ? return;
- ? ? ? ? ? ? ? }
+ ? ? ? ? ? ? ? ArpReceive(et, ip, len);

? ? ? ? ? ? ? ?break;

?#ifdef CONFIG_CMD_RARP
--
1.6.0.2

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Signed-off-by Joe Hershberger <[email protected]>

Cc Joe Hershberger <[email protected]>

Cc Wolfgang Denk <[email protected]>

---

?include/configs/am3517_crane.h | ? ?2 +-
?include/configs/am3517_evm.h ? | ? ?2 +-
?include/configs/origen.h ? ? ? | ? ?2 +-

?net/Makefile ? ? ? ? ? ? ? ? ? | ? ?1 +
?net/net.c ? ? ? ? ? ? ? ? ? ? ?| ?133 ++-----------------------------------
?net/ping.c ? ? ? ? ? ? ? ? ? ? | ?143 ++++++++++++++++++++++++++++++++++++++++
?net/ping.h ? ? ? ? ? ? ? ? ? ? | ? 20 ++++++

?7 files changed, 172 insertions(+), 131 deletions(-) ?create mode 100644 net/ping.c
?create mode 100644 net/ping.h

include/configs/am3517_crane.h +0 -0

index 0a0c261..acd4e3f 100644
@@ -146,7 +146,7 @@

?#define CONFIG_CMD_MMC ? ? ? ? /* MMC support ? ? ? ? ? ? ? ? ?*/ ?#define CONFIG_CMD_NAND ? ? ? ? ? ? ? ?/* NAND support ? ? ? ? ? ? ? ? */ ?#define CONFIG_CMD_DHCP

include/configs/am3517_crane.h +1 -1

-#define CONFIG_CMD_PING
+#undef CONFIG_CMD_PING

?#undef CONFIG_CMD_FLASH ? ? ? ? ? ? ? ?/* flinfo, erase, protect ? ? ? */ ?#undef CONFIG_CMD_FPGA ? ? ? ? /* FPGA configuration Support ? */

include/configs/am3517_evm.h +0 -0

index d44eeec..9775107 100644
@@ -145,7 +145,7 @@

?#define CONFIG_CMD_MMC ? ? ? ? /* MMC support ? ? ? ? ? ? ? ? ?*/ ?#define CONFIG_CMD_NAND ? ? ? ? ? ? ? ?/* NAND support ? ? ? ? ? ? ? ? */ ?#define CONFIG_CMD_DHCP

include/configs/am3517_evm.h +1 -1

-#define CONFIG_CMD_PING
+#undef CONFIG_CMD_PING

?#undef CONFIG_CMD_FLASH ? ? ? ? ? ? ? ?/* flinfo, erase, protect ? ? ? */ ?#undef CONFIG_CMD_FPGA ? ? ? ? /* FPGA configuration Support ? */

include/configs/origen.h +0 -0

index cd502d1..c9bac30 100644
@@ -84,7 +84,7 @@

?/* Command definition*/
?#include <config_cmd_default.h>

include/configs/origen.h +1 -1

-#define CONFIG_CMD_PING
+#undef CONFIG_CMD_PING

?#define CONFIG_CMD_ELF
?#define CONFIG_CMD_DHCP
?#define CONFIG_CMD_MMC

net/Makefile +0 -0

index 0916a56..f1c4859 100644
@@ -35,6 +35,7 @@ COBJS-$(CONFIG_CMD_NET) ?+= eth.o

?COBJS-$(CONFIG_CMD_NET) ?+= net.o
?COBJS-$(CONFIG_BOOTP_RANDOM_DELAY) += net_rand.o
?COBJS-$(CONFIG_CMD_NFS) ?+= nfs.o
+COBJS-$(CONFIG_CMD_PING) += ping.o
?COBJS-$(CONFIG_CMD_RARP) += rarp.o
?COBJS-$(CONFIG_CMD_SNTP) += sntp.o
?COBJS-$(CONFIG_CMD_NET) ?+= tftp.o

net/net.c +0 -0

index 2dae5a0..c2fbcb5 100644
@@ -98,6 +98,9 @@
 ?#if defined(CONFIG_CMD_DNS)

?#include "dns.h"
?#endif

net/net.c +3 -0

+#if defined(CONFIG_CMD_PING)
+#include "ping.h"
+#endif

?DECLARE_GLOBAL_DATA_PTR;

net/net.c +0 -0

@@ -168,13 +171,6 @@ ushort ? ? ? ? ? ? NetOurNativeVLAN = 0xFFFF;

?/* Boot File name */
?char ? ? ? ? ? BootFile[128];

net/net.c +0 -7

-#if defined(CONFIG_CMD_PING)
-/* the ip address to ping */
-IPaddr_t ? ? ? NetPingIP;
-
-static void PingStart(void);
-#endif
-

?#if defined(CONFIG_CMD_SNTP)
?/* NTP server IP address */
?IPaddr_t ? ? ? NetNtpServerIP;

net/net.c +0 -0

@@ -653,94 +649,6 @@ NetSendUDPPacket(uchar *ether, IPaddr_t dest, int dport, int sport, int len)

? ? ? ?return 0; ? ? ? /* transmitted */
?}

net/net.c +0 -88

-#if defined(CONFIG_CMD_PING)
-static ushort PingSeqNo;
-
-int PingSend(void)
-{
- ? ? ? static uchar mac[6];
- ? ? ? IP_t *ip;
- ? ? ? ushort *s;
- ? ? ? uchar *pkt;
-
- ? ? ? /* XXX always send arp request */
-
- ? ? ? memcpy(mac, NetEtherNullAddr, 6);
-
- ? ? ? debug("sending ARP for %08x\n", NetPingIP);
-
- ? ? ? NetArpWaitPacketIP = NetPingIP;
- ? ? ? NetArpWaitPacketMAC = mac;
-
- ? ? ? pkt = NetArpWaitTxPacket;
- ? ? ? pkt += NetSetEther(pkt, mac, PROT_IP);
-
- ? ? ? ip = (IP_t *)pkt;
-
- ? ? ? /*
- ? ? ? ?* Construct an IP and ICMP header.
- ? ? ? ?* (need to set no fragment bit - XXX)
- ? ? ? ?*/
- ? ? ? /* IP_HDR_SIZE / 4 (not including UDP) */
- ? ? ? ip->ip_hl_v ?= 0x45;
- ? ? ? ip->ip_tos ? = 0;
- ? ? ? ip->ip_len ? = htons(IP_HDR_SIZE_NO_UDP + 8);
- ? ? ? ip->ip_id ? ?= htons(NetIPID++);
- ? ? ? ip->ip_off ? = htons(IP_FLAGS_DFRAG); ? /* Don't fragment */
- ? ? ? ip->ip_ttl ? = 255;
- ? ? ? ip->ip_p ? ? = 0x01; ? ? ? ? ? ?/* ICMP */
- ? ? ? ip->ip_sum ? = 0;
- ? ? ? /* already in network byte order */
- ? ? ? NetCopyIP((void *)&ip->ip_src, &NetOurIP);
- ? ? ? /* - "" - */
- ? ? ? NetCopyIP((void *)&ip->ip_dst, &NetPingIP);
- ? ? ? ip->ip_sum ? = ~NetCksum((uchar *)ip, IP_HDR_SIZE_NO_UDP / 2);
-
- ? ? ? s = &ip->udp_src; ? ? ? ? ? ? ? /* XXX ICMP starts here */
- ? ? ? s[0] = htons(0x0800); ? ? ? ? ? /* echo-request, code */
- ? ? ? s[1] = 0; ? ? ? ? ? ? ? ? ? ? ? /* checksum */
- ? ? ? s[2] = 0; ? ? ? ? ? ? ? ? ? ? ? /* identifier */
- ? ? ? s[3] = htons(PingSeqNo++); ? ? ?/* sequence number */
- ? ? ? s[1] = ~NetCksum((uchar *)s, 8/2);
-
- ? ? ? /* size of the waiting packet */
- ? ? ? NetArpWaitTxPacketSize =
- ? ? ? ? ? ? ? (pkt - NetArpWaitTxPacket) + IP_HDR_SIZE_NO_UDP + 8;
-
- ? ? ? /* and do the ARP request */
- ? ? ? NetArpWaitTry = 1;
- ? ? ? NetArpWaitTimerStart = get_timer(0);
- ? ? ? ArpRequest();
- ? ? ? return 1; ? ? ? /* waiting */
-}
-
-static void
-PingTimeout(void)
-{
- ? ? ? eth_halt();
- ? ? ? NetState = NETLOOP_FAIL; ? ? ? ?/* we did not get the reply */
-}
-
-static void
-PingHandler(uchar *pkt, unsigned dest, IPaddr_t sip, unsigned src,
- ? ? ? ? ? unsigned len)
-{
- ? ? ? if (sip != NetPingIP)
- ? ? ? ? ? ? ? return;
-
- ? ? ? NetState = NETLOOP_SUCCESS;
-}
-
-static void PingStart(void)
-{
- ? ? ? printf("Using %s device\n", eth_get_name());
- ? ? ? NetSetTimeout(10000UL, PingTimeout);
- ? ? ? NetSetHandler(PingHandler);
-
- ? ? ? PingSend();
-}
-#endif
-

?#ifdef CONFIG_IP_DEFRAG
?/*
?* This function collects fragments in a single packet, according

net/net.c +2 -32

@@ -930,41 +838,10 @@ static void receive_icmp(IP_t *ip, int len, IPaddr_t src_ip, Ethernet_t *et)
 ? ? ? ? ? ? ? ?printf(" ICMP Host Redirect to %pI4 ",
 ? ? ? ? ? ? ? ? ? ? ? ?&icmph->un.gateway);
 ? ? ? ? ? ? ? ?break;
+ ? ? ? default:
 ?#if defined(CONFIG_CMD_PING)
- ? ? ? case ICMP_ECHO_REPLY:
- ? ? ? ? ? ? ? /*
- ? ? ? ? ? ? ? ? ? ? ? * IP header OK. ?Pass the packet to the
- ? ? ? ? ? ? ? ? ? ? ? * current handler.
- ? ? ? ? ? ? ? ? ? ? ? */
- ? ? ? ? ? ? ? /*
- ? ? ? ? ? ? ? ?* XXX point to ip packet - should this use
- ? ? ? ? ? ? ? ?* packet_icmp_handler?
- ? ? ? ? ? ? ? ?*/
- ? ? ? ? ? ? ? (*packetHandler)((uchar *)ip, 0, src_ip, 0, 0);
- ? ? ? ? ? ? ? break;
- ? ? ? case ICMP_ECHO_REQUEST:
- ? ? ? ? ? ? ? debug("Got ICMP ECHO REQUEST, return %d bytes\n",
- ? ? ? ? ? ? ? ? ? ? ? ETHER_HDR_SIZE + len);
-
- ? ? ? ? ? ? ? memcpy(&et->et_dest[0], &et->et_src[0], 6);
- ? ? ? ? ? ? ? memcpy(&et->et_src[0], NetOurEther, 6);
-
- ? ? ? ? ? ? ? ip->ip_sum = 0;
- ? ? ? ? ? ? ? ip->ip_off = 0;
- ? ? ? ? ? ? ? NetCopyIP((void *)&ip->ip_dst, &ip->ip_src);
- ? ? ? ? ? ? ? NetCopyIP((void *)&ip->ip_src, &NetOurIP);
- ? ? ? ? ? ? ? ip->ip_sum = ~NetCksum((uchar *)ip,
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? IP_HDR_SIZE_NO_UDP >> 1);
-
- ? ? ? ? ? ? ? icmph->type = ICMP_ECHO_REPLY;
- ? ? ? ? ? ? ? icmph->checksum = 0;
- ? ? ? ? ? ? ? icmph->checksum = ~NetCksum((uchar *)icmph,
- ? ? ? ? ? ? ? ? ? ? ? (len - IP_HDR_SIZE_NO_UDP) >> 1);
- ? ? ? ? ? ? ? (void) eth_send((uchar *)et,
- ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ETHER_HDR_SIZE + len);
- ? ? ? ? ? ? ? break;
+ ? ? ? ? ? ? ? PingReceive(et, ip, len);

?#endif
- ? ? ? default:
?#ifdef CONFIG_CMD_TFTPPUT
? ? ? ? ? ? ? ?if (packet_icmp_handler)
? ? ? ? ? ? ? ? ? ? ? ?packet_icmp_handler(icmph->type, icmph->code,

net/ping.c new file +143 -0

new file mode 100644
index 0000000..4a66b8d
@@ -0,0 +1,143 @@
+/*
+ * ? ? Copied from Linux Monitor (LiMon) - Networking.
+ *
+ * ? ? Copyright 1994 - 2000 Neil Russell.
+ * ? ? (See License)
+ * ? ? Copyright 2000 Roland Borde
+ * ? ? Copyright 2000 Paolo Scaffardi
+ * ? ? Copyright 2000-2002 Wolfgang Denk, wd at denx.de
+ */
+
+#include "ping.h"
+#include "arp.h"
+
+static ushort PingSeqNo;
+
+/* The ip address to ping */
+IPaddr_t NetPingIP;
+
+static int PingSend(void)
+{
+ ? ? ? static uchar mac[6];
+ ? ? ? IP_t *ip;
+ ? ? ? ushort *s;
+ ? ? ? uchar *pkt;
+
+ ? ? ? /* XXX always send arp request */
+
+ ? ? ? memcpy(mac, NetEtherNullAddr, 6);
+
+ ? ? ? debug("sending ARP for %pI4\n", &NetPingIP);
+
+ ? ? ? NetArpWaitPacketIP = NetPingIP;
+ ? ? ? NetArpWaitPacketMAC = mac;
+
+ ? ? ? pkt = NetArpWaitTxPacket;
+ ? ? ? pkt += NetSetEther(pkt, mac, PROT_IP);
+
+ ? ? ? ip = (IP_t *)pkt;
+
+ ? ? ? /*
+ ? ? ? ?* Construct an IP and ICMP header.
+ ? ? ? ?* (need to set no fragment bit - XXX)
+ ? ? ? ?*/
+ ? ? ? /* IP_HDR_SIZE / 4 (not including UDP) */
+ ? ? ? ip->ip_hl_v ?= 0x45;
+ ? ? ? ip->ip_tos ? = 0;
+ ? ? ? ip->ip_len ? = htons(IP_HDR_SIZE_NO_UDP + 8);
+ ? ? ? ip->ip_id ? ?= htons(NetIPID++);
+ ? ? ? ip->ip_off ? = htons(IP_FLAGS_DFRAG); ? /* Don't fragment */
+ ? ? ? ip->ip_ttl ? = 255;
+ ? ? ? ip->ip_p ? ? = 0x01; ? ? ? ? ? ?/* ICMP */
+ ? ? ? ip->ip_sum ? = 0;
+ ? ? ? /* already in network byte order */
+ ? ? ? NetCopyIP((void *)&ip->ip_src, &NetOurIP);
+ ? ? ? /* - "" - */
+ ? ? ? NetCopyIP((void *)&ip->ip_dst, &NetPingIP);
+ ? ? ? ip->ip_sum ? = ~NetCksum((uchar *)ip, IP_HDR_SIZE_NO_UDP / 2);
+
+ ? ? ? s = &ip->udp_src; ? ? ? ? ? ? ? /* XXX ICMP starts here */
+ ? ? ? s[0] = htons(0x0800); ? ? ? ? ? /* echo-request, code */
+ ? ? ? s[1] = 0; ? ? ? ? ? ? ? ? ? ? ? /* checksum */
+ ? ? ? s[2] = 0; ? ? ? ? ? ? ? ? ? ? ? /* identifier */
+ ? ? ? s[3] = htons(PingSeqNo++); ? ? ?/* sequence number */
+ ? ? ? s[1] = ~NetCksum((uchar *)s, 8/2);
+
+ ? ? ? /* size of the waiting packet */
+ ? ? ? NetArpWaitTxPacketSize =
+ ? ? ? ? ? ? ? (pkt - NetArpWaitTxPacket) + IP_HDR_SIZE_NO_UDP + 8;
+
+ ? ? ? /* and do the ARP request */
+ ? ? ? NetArpWaitTry = 1;
+ ? ? ? NetArpWaitTimerStart = get_timer(0);
+ ? ? ? ArpRequest();
+ ? ? ? return 1; ? ? ? /* waiting */
+}
+
+static void
+PingTimeout(void)
+{
+ ? ? ? eth_halt();
+ ? ? ? NetState = NETLOOP_FAIL; ? ? ? ?/* we did not get the reply */
+}
+
+static void
+PingHandler(uchar *pkt, unsigned dest, IPaddr_t sip, unsigned src,
+ ? ? ? ? ? unsigned len)
+{
+ ? ? ? if (sip != NetPingIP)
+ ? ? ? ? ? ? ? return;
+
+ ? ? ? NetState = NETLOOP_SUCCESS;
+}
+
+void
+PingStart(void)
+{
+ ? ? ? printf("Using %s device\n", eth_get_name());
+ ? ? ? NetSetTimeout(10000UL, PingTimeout);
+ ? ? ? NetSetHandler(PingHandler);
+
+ ? ? ? PingSend();
+}
+
+void PingReceive(Ethernet_t *et, IP_t *ip, int len)
+{
+ ? ? ? ICMP_t *icmph = (ICMP_t *)&(ip->udp_src);
+ ? ? ? IPaddr_t src_ip;
+
+ ? ? ? switch (icmph->type) {
+ ? ? ? case ICMP_ECHO_REPLY:
+ ? ? ? ? ? ? ? /*
+ ? ? ? ? ? ? ? ?* IP header OK. ?Pass the packet to the
+ ? ? ? ? ? ? ? ?* current handler.
+ ? ? ? ? ? ? ? ?*/
+ ? ? ? ? ? ? ? /* XXX point to ip packet */
+ ? ? ? ? ? ? ? src_ip = NetReadIP((void *)&ip->ip_src);
+ ? ? ? ? ? ? ? NetGetHandler()((uchar *)ip, 0, src_ip, 0, 0);
+ ? ? ? ? ? ? ? return;
+ ? ? ? case ICMP_ECHO_REQUEST:
+ ? ? ? ? ? ? ? debug("Got ICMP ECHO REQUEST, return "
+ ? ? ? ? ? ? ? ? ? ? ? "%d bytes\n", ETHER_HDR_SIZE + len);
+
+ ? ? ? ? ? ? ? memcpy(&et->et_dest[0], &et->et_src[0], 6);
+ ? ? ? ? ? ? ? memcpy(&et->et_src[0], NetOurEther, 6);
+
+ ? ? ? ? ? ? ? ip->ip_sum = 0;
+ ? ? ? ? ? ? ? ip->ip_off = 0;
+ ? ? ? ? ? ? ? NetCopyIP((void *)&ip->ip_dst, &ip->ip_src);
+ ? ? ? ? ? ? ? NetCopyIP((void *)&ip->ip_src, &NetOurIP);
+ ? ? ? ? ? ? ? ip->ip_sum = ~NetCksum((uchar *)ip,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?IP_HDR_SIZE_NO_UDP >> 1);
+
+ ? ? ? ? ? ? ? icmph->type = ICMP_ECHO_REPLY;
+ ? ? ? ? ? ? ? icmph->checksum = 0;
+ ? ? ? ? ? ? ? icmph->checksum = ~NetCksum((uchar *)icmph,
+ ? ? ? ? ? ? ? ? ? ? ? (len - IP_HDR_SIZE_NO_UDP) >> 1);
+ ? ? ? ? ? ? ? (void) eth_send((uchar *)et,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ETHER_HDR_SIZE + len);
+ ? ? ? ? ? ? ? return;
+/* ? ? default:
+ ? ? ? ? ? ? ? return;*/
+ ? ? ? }
+}

net/ping.h new file +20 -1

new file mode 100644
index 0000000..56aad43
@@ -0,0 +1,20 @@
+/*
+ * ? ? Copied from Linux Monitor (LiMon) - Networking.
+ *
+ * ? ? Copyright 1994 - 2000 Neil Russell.
+ * ? ? (See License)
+ * ? ? Copyright 2000 Roland Borde
+ * ? ? Copyright 2000 Paolo Scaffardi
+ * ? ? Copyright 2000-2002 Wolfgang Denk, wd at denx.de
+ */
+
+#ifndef __PING_H__
+#define __PING_H__
+
+#include <common.h>
+#include <net.h>
+
+void PingStart(void);
+void PingReceive(Ethernet_t *et, IP_t *ip, int len);
+
+#endif /* __PING_H__ */
--

1.6.0.2

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot