]> git.kernelconcepts.de Git - karo-tx-uboot.git/blobdiff - include/net.h
TX6 Release 2013-04-22
[karo-tx-uboot.git] / include / net.h
index ee11f82ccc87acf426b95f78337bb2de52d77abd..76c6d6605aaaa472829df04f4ef5a5bae3ce82b8 100644 (file)
 #include <asm/cache.h>
 #include <asm/byteorder.h>     /* for nton* / ntoh* stuff */
 
+#define DEBUG_LL_STATE 0       /* Link local state machine changes */
+#define DEBUG_DEV_PKT 0                /* Packets or info directed to the device */
+#define DEBUG_NET_PKT 0                /* Packets on info on the network at large */
+#define DEBUG_INT_STATE 0      /* Internal network state changes */
 
 /*
  *     The number of receive packet buffers, and the required packet buffer
@@ -80,14 +84,14 @@ struct eth_device {
        int iobase;
        int state;
 
-       int  (*init) (struct eth_device*, bd_t*);
-       int  (*send) (struct eth_device*, volatile void* packet, int length);
-       int  (*recv) (struct eth_device*);
-       void (*halt) (struct eth_device*);
+       int  (*init) (struct eth_device *, bd_t *);
+       int  (*send) (struct eth_device *, void *packet, int length);
+       int  (*recv) (struct eth_device *);
+       void (*halt) (struct eth_device *);
 #ifdef CONFIG_MCAST_TFTP
-       int (*mcast) (struct eth_device*, u32 ip, u8 set);
+       int (*mcast) (struct eth_device *, u32 ip, u8 set);
 #endif
-       int  (*write_hwaddr) (struct eth_device*);
+       int  (*write_hwaddr) (struct eth_device *);
        struct eth_device *next;
        int index;
        void *priv;
@@ -98,10 +102,18 @@ extern int eth_register(struct eth_device* dev);/* Register network device */
 extern int eth_unregister(struct eth_device *dev);/* Remove network device */
 extern void eth_try_another(int first_restart);        /* Change the device */
 extern void eth_set_current(void);             /* set nterface to ethcur var */
-extern struct eth_device *eth_get_dev(void);   /* get the current device MAC */
+
+/* get the current device MAC */
+extern struct eth_device *eth_current;
+
+static inline __attribute__((always_inline))
+struct eth_device *eth_get_dev(void)
+{
+       return eth_current;
+}
 extern struct eth_device *eth_get_dev_by_name(const char *devname);
 extern struct eth_device *eth_get_dev_by_index(int index); /* get dev @ index */
-extern int eth_get_dev_index (void);           /* get the device index */
+extern int eth_get_dev_index(void);            /* get the device index */
 extern void eth_parse_enetaddr(const char *addr, uchar *enetaddr);
 extern int eth_getenv_enetaddr(char *name, uchar *enetaddr);
 extern int eth_setenv_enetaddr(char *name, const uchar *enetaddr);
@@ -118,17 +130,48 @@ extern int eth_setenv_enetaddr(char *name, const uchar *enetaddr);
 extern int eth_getenv_enetaddr_by_index(const char *base_name, int index,
                                        uchar *enetaddr);
 
+#ifdef CONFIG_RANDOM_MACADDR
+/*
+ * The u-boot policy does not allow hardcoded ethernet addresses. Under the
+ * following circumstances a random generated address is allowed:
+ *  - in emergency cases, where you need a working network connection to set
+ *    the ethernet address.
+ *    Eg. you want a rescue boot and don't have a serial port to access the
+ *    CLI to set environment variables.
+ *
+ * In these cases, we generate a random locally administered ethernet address.
+ *
+ * Args:
+ *  enetaddr - returns 6 byte hardware address
+ */
+extern void eth_random_enetaddr(uchar *enetaddr);
+#endif
+
 extern int usb_eth_initialize(bd_t *bi);
 extern int eth_init(bd_t *bis);                        /* Initialize the device */
-extern int eth_send(volatile void *packet, int length);           /* Send a packet */
+extern int eth_send(void *packet, int length);    /* Send a packet */
 
 #ifdef CONFIG_API
-extern int eth_receive(volatile void *packet, int length); /* Receive a packet*/
+extern int eth_receive(void *packet, int length); /* Receive a packet*/
+extern void (*push_packet)(void *packet, int length);
 #endif
 extern int eth_rx(void);                       /* Check for received packets */
 extern void eth_halt(void);                    /* stop SCC */
 extern char *eth_get_name(void);               /* get name of current device */
 
+/* Set active state */
+static inline __attribute__((always_inline)) int eth_init_state_only(bd_t *bis)
+{
+       eth_get_dev()->state = ETH_STATE_ACTIVE;
+
+       return 0;
+}
+/* Set passive state */
+static inline __attribute__((always_inline)) void eth_halt_state_only(void)
+{
+       eth_get_dev()->state = ETH_STATE_PASSIVE;
+}
+
 /*
  * Set the hardware address for an ethernet interface based on 'eth%daddr'
  * environment variable (or just 'ethaddr' if eth_number is 0).
@@ -142,8 +185,8 @@ int eth_write_hwaddr(struct eth_device *dev, const char *base_name,
                     int eth_number);
 
 #ifdef CONFIG_MCAST_TFTP
-int eth_mcast_join( IPaddr_t mcast_addr, u8 join);
-u32 ether_crc (size_t len, unsigned char const *p);
+int eth_mcast_join(IPaddr_t mcast_addr, u8 join);
+u32 ether_crc(size_t len, unsigned char const *p);
 #endif
 
 
@@ -155,7 +198,17 @@ u32 ether_crc (size_t len, unsigned char const *p);
 /*
  *     Ethernet header
  */
-typedef struct {
+
+struct ethernet_hdr {
+       uchar           et_dest[6];     /* Destination node             */
+       uchar           et_src[6];      /* Source node                  */
+       ushort          et_protlen;     /* Protocol or length           */
+};
+
+/* Ethernet header size */
+#define ETHER_HDR_SIZE (sizeof(struct ethernet_hdr))
+
+struct e802_hdr {
        uchar           et_dest[6];     /* Destination node             */
        uchar           et_src[6];      /* Source node                  */
        ushort          et_protlen;     /* Protocol or length           */
@@ -166,23 +219,24 @@ typedef struct {
        uchar           et_snap2;
        uchar           et_snap3;
        ushort          et_prot;        /* 802 protocol                 */
-} Ethernet_t;
+};
 
-#define ETHER_HDR_SIZE 14              /* Ethernet header size         */
-#define E802_HDR_SIZE  22              /* 802 ethernet header size     */
+/* 802 + SNAP + ethernet header size */
+#define E802_HDR_SIZE  (sizeof(struct e802_hdr))
 
 /*
- *     Ethernet header
+ *     Virtual LAN Ethernet header
  */
-typedef struct {
+struct vlan_ethernet_hdr {
        uchar           vet_dest[6];    /* Destination node             */
        uchar           vet_src[6];     /* Source node                  */
        ushort          vet_vlan_type;  /* PROT_VLAN                    */
        ushort          vet_tag;        /* TAG of VLAN                  */
        ushort          vet_type;       /* protocol type                */
-} VLAN_Ethernet_t;
+};
 
-#define VLAN_ETHER_HDR_SIZE    18      /* VLAN Ethernet header size    */
+/* VLAN Ethernet header size */
+#define VLAN_ETHER_HDR_SIZE    (sizeof(struct vlan_ethernet_hdr))
 
 #define PROT_IP                0x0800          /* IP protocol                  */
 #define PROT_ARP       0x0806          /* IP ARP protocol              */
@@ -195,7 +249,7 @@ typedef struct {
 /*
  *     Internet Protocol (IP) header.
  */
-typedef struct {
+struct ip_hdr {
        uchar           ip_hl_v;        /* header length and version    */
        uchar           ip_tos;         /* type of service              */
        ushort          ip_len;         /* total length                 */
@@ -206,11 +260,7 @@ typedef struct {
        ushort          ip_sum;         /* checksum                     */
        IPaddr_t        ip_src;         /* Source IP address            */
        IPaddr_t        ip_dst;         /* Destination IP address       */
-       ushort          udp_src;        /* UDP source port              */
-       ushort          udp_dst;        /* UDP destination port         */
-       ushort          udp_len;        /* Length of UDP packet         */
-       ushort          udp_xsum;       /* Checksum                     */
-} IP_t;
+};
 
 #define IP_OFFS                0x1fff /* ip offset *= 8 */
 #define IP_FLAGS       0xe000 /* first 3 bits */
@@ -218,20 +268,42 @@ typedef struct {
 #define IP_FLAGS_DFRAG 0x4000 /* don't fragments */
 #define IP_FLAGS_MFRAG 0x2000 /* more fragments */
 
-#define IP_HDR_SIZE_NO_UDP     (sizeof (IP_t) - 8)
-#define IP_HDR_SIZE            (sizeof (IP_t))
+#define IP_HDR_SIZE            (sizeof(struct ip_hdr))
 
+/*
+ *     Internet Protocol (IP) + UDP header.
+ */
+struct ip_udp_hdr {
+       uchar           ip_hl_v;        /* header length and version    */
+       uchar           ip_tos;         /* type of service              */
+       ushort          ip_len;         /* total length                 */
+       ushort          ip_id;          /* identification               */
+       ushort          ip_off;         /* fragment offset field        */
+       uchar           ip_ttl;         /* time to live                 */
+       uchar           ip_p;           /* protocol                     */
+       ushort          ip_sum;         /* checksum                     */
+       IPaddr_t        ip_src;         /* Source IP address            */
+       IPaddr_t        ip_dst;         /* Destination IP address       */
+       ushort          udp_src;        /* UDP source port              */
+       ushort          udp_dst;        /* UDP destination port         */
+       ushort          udp_len;        /* Length of UDP packet         */
+       ushort          udp_xsum;       /* Checksum                     */
+};
+
+#define IP_UDP_HDR_SIZE                (sizeof(struct ip_udp_hdr))
+#define UDP_HDR_SIZE           (IP_UDP_HDR_SIZE - IP_HDR_SIZE)
 
 /*
  *     Address Resolution Protocol (ARP) header.
  */
-typedef struct
-{
+struct arp_hdr {
        ushort          ar_hrd;         /* Format of hardware address   */
 #   define ARP_ETHER       1           /* Ethernet  hardware address   */
        ushort          ar_pro;         /* Format of protocol address   */
        uchar           ar_hln;         /* Length of hardware address   */
+#   define ARP_HLEN    6
        uchar           ar_pln;         /* Length of protocol address   */
+#   define ARP_PLEN    4
        ushort          ar_op;          /* Operation                    */
 #   define ARPOP_REQUEST    1          /* Request  to resolve  address */
 #   define ARPOP_REPLY     2           /* Response to previous request */
@@ -245,13 +317,17 @@ typedef struct
         * specific hardware/protocol combinations.
         */
        uchar           ar_data[0];
+#define ar_sha         ar_data[0]
+#define ar_spa         ar_data[ARP_HLEN]
+#define ar_tha         ar_data[ARP_HLEN + ARP_PLEN]
+#define ar_tpa         ar_data[ARP_HLEN + ARP_PLEN + ARP_HLEN]
 #if 0
        uchar           ar_sha[];       /* Sender hardware address      */
        uchar           ar_spa[];       /* Sender protocol address      */
        uchar           ar_tha[];       /* Target hardware address      */
        uchar           ar_tpa[];       /* Target protocol address      */
 #endif /* 0 */
-} ARP_t;
+};
 
 #define ARP_HDR_SIZE   (8+20)          /* Size assuming ethernet       */
 
@@ -270,7 +346,7 @@ typedef struct
 /* Codes for NOT_REACH */
 #define ICMP_NOT_REACH_PORT    3       /* Port unreachable             */
 
-typedef struct icmphdr {
+struct icmp_hdr {
        uchar           type;
        uchar           code;
        ushort          checksum;
@@ -286,8 +362,10 @@ typedef struct icmphdr {
                } frag;
                uchar data[0];
        } un;
-} ICMP_t;
+};
 
+#define ICMP_HDR_SIZE          (sizeof(struct icmp_hdr))
+#define IP_ICMP_HDR_SIZE       (IP_HDR_SIZE + ICMP_HDR_SIZE)
 
 /*
  * Maximum packet size; used to allocate packet storage.
@@ -326,54 +404,44 @@ typedef struct icmphdr {
 
 /* net.c */
 /** BOOTP EXTENTIONS **/
-extern IPaddr_t                NetOurGatewayIP;        /* Our gateway IP addresse      */
-extern IPaddr_t                NetOurSubnetMask;       /* Our subnet mask (0 = unknown)*/
-extern IPaddr_t                NetOurDNSIP;     /* Our Domain Name Server (0 = unknown)*/
+extern IPaddr_t NetOurGatewayIP;       /* Our gateway IP address */
+extern IPaddr_t NetOurSubnetMask;      /* Our subnet mask (0 = unknown) */
+extern IPaddr_t NetOurDNSIP;   /* Our Domain Name Server (0 = unknown) */
 #if defined(CONFIG_BOOTP_DNS2)
-extern IPaddr_t                NetOurDNS2IP;    /* Our 2nd Domain Name Server (0 = unknown)*/
+extern IPaddr_t NetOurDNS2IP;  /* Our 2nd Domain Name Server (0 = unknown) */
 #endif
-extern char            NetOurNISDomain[32];    /* Our NIS domain               */
-extern char            NetOurHostName[32];     /* Our hostname                 */
-extern char            NetOurRootPath[64];     /* Our root path                */
-extern ushort          NetBootFileSize;        /* Our boot file size in blocks */
+extern char    NetOurNISDomain[32];    /* Our NIS domain */
+extern char    NetOurHostName[32];     /* Our hostname */
+extern char    NetOurRootPath[64];     /* Our root path */
+extern ushort  NetBootFileSize;        /* Our boot file size in blocks */
 /** END OF BOOTP EXTENTIONS **/
-extern ulong           NetBootFileXferSize;    /* size of bootfile in bytes    */
-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) */
-extern volatile uchar * NetTxPacket;           /* THE transmit packet          */
-extern volatile uchar * NetRxPackets[PKTBUFSRX];/* Receive packets             */
-extern volatile 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   */
+extern ulong           NetBootFileXferSize;    /* size of bootfile in bytes */
+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) */
+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 */
 extern uchar           NetEtherNullAddr[6];
 
-#define VLAN_NONE      4095                    /* untagged                     */
-#define VLAN_IDMASK    0x0fff                  /* mask of valid vlan id        */
-extern ushort          NetOurVLAN;             /* Our VLAN                     */
-extern ushort          NetOurNativeVLAN;       /* Our Native VLAN              */
-
-extern uchar           NetCDPAddr[6];          /* Ethernet CDP address         */
-extern ushort          CDPNativeVLAN;          /* CDP returned native VLAN     */
-extern ushort          CDPApplianceVLAN;       /* CDP returned appliance VLAN  */
+#define VLAN_NONE      4095                    /* untagged */
+#define VLAN_IDMASK    0x0fff                  /* mask of valid vlan id */
+extern ushort          NetOurVLAN;             /* Our VLAN */
+extern ushort          NetOurNativeVLAN;       /* Our Native VLAN */
 
-extern int             NetState;               /* Network loop state           */
-#define NETLOOP_CONTINUE       1
-#define NETLOOP_RESTART                2
-#define NETLOOP_SUCCESS                3
-#define NETLOOP_FAIL           4
-
-extern int             NetRestartWrap;         /* Tried all network devices    */
+extern int             NetRestartWrap;         /* Tried all network devices */
 
 enum proto_t {
        BOOTP, RARP, ARP, TFTPGET, DHCP, PING, DNS, NFS, CDP, NETCONS, SNTP,
-       TFTPSRV, TFTPPUT
+       TFTPSRV, TFTPPUT, LINKLOCAL, BOOTME
 };
 
 /* from net/net.c */
-extern char    BootFile[128];                  /* Boot File name               */
+extern char    BootFile[128];                  /* Boot File name */
 
 #if defined(CONFIG_CMD_DNS)
 extern char *NetDNSResolve;            /* The host to resolve  */
@@ -381,21 +449,36 @@ extern char *NetDNSenvvar;                /* the env var to put the ip into */
 #endif
 
 #if defined(CONFIG_CMD_PING)
-extern IPaddr_t        NetPingIP;                      /* the ip address to ping               */
+extern IPaddr_t        NetPingIP;                      /* the ip address to ping */
 #endif
 
 #if defined(CONFIG_CMD_CDP)
 /* when CDP completes these hold the return values */
-extern ushort CDPNativeVLAN;
-extern ushort CDPApplianceVLAN;
+extern ushort CDPNativeVLAN;           /* CDP returned native VLAN */
+extern ushort CDPApplianceVLAN;                /* CDP returned appliance VLAN */
+
+/*
+ * Check for a CDP packet by examining the received MAC address field
+ */
+static inline int is_cdp_packet(const uchar *et_addr)
+{
+       extern const uchar NetCDPAddr[6];
+
+       return memcmp(et_addr, NetCDPAddr, 6) == 0;
+}
 #endif
 
 #if defined(CONFIG_CMD_SNTP)
-extern IPaddr_t        NetNtpServerIP;                 /* the ip address to NTP        */
-extern int NetTimeOffset;                      /* offset time from UTC         */
+extern IPaddr_t        NetNtpServerIP;                 /* the ip address to NTP */
+extern int NetTimeOffset;                      /* offset time from UTC */
+#endif
+
+#if defined(CONFIG_MCAST_TFTP)
+extern IPaddr_t Mcast_addr;
 #endif
 
 /* Initialize the network adapter */
+extern void net_init(void);
 extern int NetLoop(enum proto_t);
 
 /* Shutdown adapters and cleanup */
@@ -408,28 +491,91 @@ extern void       NetStartAgain(void);
 extern int     NetEthHdrSize(void);
 
 /* Set ethernet header; returns the size of the header */
-extern int     NetSetEther(volatile uchar *, uchar *, uint);
+extern int NetSetEther(uchar *, uchar *, uint);
+extern int net_update_ether(struct ethernet_hdr *et, uchar *addr, uint prot);
 
 /* Set IP header */
-extern void    NetSetIP(volatile uchar *, IPaddr_t, int, int, int);
+extern void net_set_ip_header(uchar *pkt, IPaddr_t dest, IPaddr_t source);
+extern void net_set_udp_header(uchar *pkt, IPaddr_t dest, int dport,
+                               int sport, int len);
 
 /* Checksum */
-extern int     NetCksumOk(uchar *, int);       /* Return true if cksum OK      */
-extern uint    NetCksum(uchar *, int);         /* Calculate the checksum       */
-
-/* Set callbacks */
-extern void    NetSetHandler(rxhand_f *);      /* Set RX packet handler        */
+extern int     NetCksumOk(uchar *, int);       /* Return true if cksum OK */
+extern uint    NetCksum(uchar *, int);         /* Calculate the checksum */
+
+/* Callbacks */
+extern rxhand_f *net_get_udp_handler(void);    /* Get UDP RX packet handler */
+extern void net_set_udp_handler(rxhand_f *);   /* Set UDP RX packet handler */
+extern rxhand_f *net_get_arp_handler(void);    /* Get ARP RX packet handler */
+extern void net_set_arp_handler(rxhand_f *);   /* Set ARP 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          */
+extern void    NetSetTimeout(ulong, thand_f *);/* Set timeout handler */
+
+/* Network loop state */
+enum net_loop_state {
+       NETLOOP_CONTINUE,
+       NETLOOP_RESTART,
+       NETLOOP_SUCCESS,
+       NETLOOP_FAIL
+};
+extern enum net_loop_state net_state;
+
+static inline void net_set_state(enum net_loop_state state)
+{
+       debug_cond(DEBUG_INT_STATE, "--- NetState set to %d\n", state);
+       net_state = state;
+}
 
-/* Transmit "NetTxPacket" */
-extern void    NetSendPacket(volatile uchar *, int);
+/* Transmit a packet */
+static inline void NetSendPacket(uchar *pkt, int len)
+{
+       (void) eth_send(pkt, len);
+}
 
-/* Transmit UDP packet, performing ARP request if needed */
-extern int     NetSendUDPPacket(uchar *ether, IPaddr_t dest, int dport, int sport, int len);
+/*
+ * Transmit "NetTxPacket" as UDP packet, performing ARP request if needed
+ *  (ether will be populated)
+ *
+ * @param ether Raw packet buffer
+ * @param dest IP address to send the datagram to
+ * @param dport Destination UDP port
+ * @param sport Source UDP port
+ * @param payload_len Length of data after the UDP header
+ */
+extern int NetSendUDPPacket(uchar *ether, IPaddr_t dest, int dport,
+                       int sport, int payload_len);
 
 /* Processes a received packet */
-extern void    NetReceive(volatile uchar *, int);
+extern void NetReceive(uchar *, int);
+
+#ifdef CONFIG_NETCONSOLE
+void NcStart(void);
+int nc_input_packet(uchar *pkt, IPaddr_t src_ip, unsigned dest_port,
+       unsigned src_port, unsigned len);
+#endif
+
+static inline __attribute__((always_inline)) int eth_is_on_demand_init(void)
+{
+#ifdef CONFIG_NETCONSOLE
+       extern enum proto_t net_loop_last_protocol;
+
+       return net_loop_last_protocol != NETCONS;
+#else
+       return 1;
+#endif
+}
+
+static inline void eth_set_last_protocol(int protocol)
+{
+#ifdef CONFIG_NETCONSOLE
+       extern enum proto_t net_loop_last_protocol;
+
+       net_loop_last_protocol = protocol;
+#endif
+}
+#ifdef CONFIG_CMD_BOOTCE
+void BootmeStart(void);
+#endif
 
 /*
  * Check if autoload is enabled. If so, use either NFS or TFTP to download
@@ -445,10 +591,11 @@ void net_auto_load(void);
  * footprint in our tests.
  */
 /* return IP *in network byteorder* */
-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));
+
+       memcpy((void *)&ip, (void *)from, sizeof(ip));
        return ip;
 }
 
@@ -456,26 +603,27 @@ static inline IPaddr_t NetReadIP(volatile void *from)
 static inline ulong NetReadLong(ulong *from)
 {
        ulong l;
-       memcpy((void*)&l, (void*)from, sizeof(l));
+
+       memcpy((void *)&l, (void *)from, sizeof(l));
        return l;
 }
 
 /* write IP *in network byteorder* */
 static inline void NetWriteIP(void *to, IPaddr_t ip)
 {
-       memcpy(to, (void*)&ip, sizeof(ip));
+       memcpy(to, (void *)&ip, sizeof(ip));
 }
 
 /* copy IP */
-static inline void NetCopyIP(volatile void *to, void *from)
+static inline void NetCopyIP(void *to, void *from)
 {
-       memcpy((void*)to, from, sizeof(IPaddr_t));
+       memcpy((void *)to, from, sizeof(IPaddr_t));
 }
 
 /* copy ulong */
 static inline void NetCopyLong(ulong *to, ulong *from)
 {
-       memcpy((void*)to, (void*)from, sizeof(ulong));
+       memcpy((void *)to, (void *)from, sizeof(ulong));
 }
 
 /**
@@ -498,7 +646,7 @@ static inline int is_zero_ether_addr(const u8 *addr)
  */
 static inline int is_multicast_ether_addr(const u8 *addr)
 {
-       return (0x01 & addr[0]);
+       return 0x01 & addr[0];
 }
 
 /*
@@ -509,7 +657,8 @@ static inline int is_multicast_ether_addr(const u8 *addr)
  */
 static inline int is_broadcast_ether_addr(const u8 *addr)
 {
-       return (addr[0] & addr[1] & addr[2] & addr[3] & addr[4] & addr[5]) == 0xff;
+       return (addr[0] & addr[1] & addr[2] & addr[3] & addr[4] & addr[5]) ==
+               0xff;
 }
 
 /*
@@ -529,13 +678,13 @@ static inline int is_valid_ether_addr(const u8 *addr)
 }
 
 /* Convert an IP address to a string */
-extern void    ip_to_string (IPaddr_t x, char *s);
+extern void ip_to_string(IPaddr_t x, char *s);
 
 /* Convert a string to ip address */
 extern IPaddr_t string_to_ip(const char *s);
 
 /* Convert a VLAN id to a string */
-extern void    VLAN_to_string (ushort x, char *s);
+extern void VLAN_to_string(ushort x, char *s);
 
 /* Convert a string to a vlan id */
 extern ushort string_to_VLAN(const char *s);
@@ -544,7 +693,7 @@ extern ushort string_to_VLAN(const char *s);
 extern ushort getenv_VLAN(char *);
 
 /* copy a filename (allow for "..." notation, limit length) */
-extern void    copy_filename (char *dst, const char *src, int size);
+extern void copy_filename(char *dst, const char *src, int size);
 
 /* get a random source port */
 extern unsigned int random_port(void);