Run the ethernet common code through indent, and fix other whitespace
issues.

Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]>

--- net-2.6.orig/net/ethernet/eth.c     2006-04-11 10:07:37.000000000 -0700
+++ net-2.6/net/ethernet/eth.c  2006-04-11 10:18:37.000000000 -0700
@@ -66,55 +66,46 @@
 __setup("ether=", netdev_boot_setup);
 
 /*
- *      Create the Ethernet MAC header for an arbitrary protocol layer 
+ *      Create the Ethernet MAC header for an arbitrary protocol layer
  *
  *     saddr=NULL      means use device source address
  *     daddr=NULL      means leave destination address (eg unresolved arp)
  */
 
 int eth_header(struct sk_buff *skb, struct net_device *dev, unsigned short 
type,
-          void *daddr, void *saddr, unsigned len)
+              void *daddr, void *saddr, unsigned len)
 {
-       struct ethhdr *eth = (struct ethhdr *)skb_push(skb,ETH_HLEN);
+       struct ethhdr *eth = (struct ethhdr *)skb_push(skb, ETH_HLEN);
 
-       /* 
-        *      Set the protocol type. For a packet of type ETH_P_802_3 we put 
the length
-        *      in here instead. It is up to the 802.2 layer to carry protocol 
information.
+       /*
+        * Set the protocol type. For a packet of type ETH_P_802_3 we put the 
length
+        * in here instead. It is up to the 802.2 layer to set protocol 
information.
         */
-       
-       if(type!=ETH_P_802_3) 
+
+       if (type != ETH_P_802_3)
                eth->h_proto = htons(type);
        else
                eth->h_proto = htons(len);
 
-       /*
-        *      Set the source hardware address. 
-        */
-        
-       if(!saddr)
+       /* Set the source hardware address. */
+       if (!saddr)
                saddr = dev->dev_addr;
-       memcpy(eth->h_source,saddr, ETH_ALEN);
+       memcpy(eth->h_source, saddr, ETH_ALEN);
 
-       if(daddr)
-       {
-               memcpy(eth->h_dest,daddr, ETH_ALEN);
+       if (daddr) {
+               memcpy(eth->h_dest, daddr, ETH_ALEN);
                return ETH_HLEN;
        }
-       
-       /*
-        *      Anyway, the loopback-device should never use this function... 
-        */
 
-       if (dev->flags & (IFF_LOOPBACK|IFF_NOARP)) 
-       {
+       /* Anyway, the loopback-device should never use this function... */
+       if (dev->flags & (IFF_LOOPBACK | IFF_NOARP)) {
                memset(eth->h_dest, 0, ETH_ALEN);
                return ETH_HLEN;
        }
-       
+
        return -ETH_HLEN;
 }
 
-
 /*
  *     Rebuild the Ethernet MAC header. This is called after an ARP
  *     (or in future other address resolution) has completed on this
@@ -129,17 +120,16 @@
        struct ethhdr *eth = (struct ethhdr *)skb->data;
        struct net_device *dev = skb->dev;
 
-       switch (eth->h_proto)
-       {
+       switch (eth->h_proto) {
 #ifdef CONFIG_INET
        case __constant_htons(ETH_P_IP):
-               return arp_find(eth->h_dest, skb);
-#endif 
+               return arp_find(eth->h_dest, skb);
+#endif
        default:
                printk(KERN_DEBUG
-                      "%s: unable to resolve type %X addresses.\n", 
+                      "%s: unable to resolve type %X addresses.\n",
                       dev->name, (int)eth->h_proto);
-               
+
                memcpy(eth->h_source, dev->dev_addr, ETH_ALEN);
                break;
        }
@@ -147,58 +137,57 @@
        return 0;
 }
 
-
 /*
- *     Determine the packet's protocol ID. The rule here is that we 
+ *     Determine the packet's protocol ID. The rule here is that we
  *     assume 802.3 if the type field is short enough to be a length.
  *     This is normal practice and works for any 'now in use' protocol.
  */
- 
-__be16 eth_type_trans(struct sk_buff *skb, struct net_device *dev)
+
+__be16 eth_type_trans(struct sk_buff * skb, struct net_device * dev)
 {
        struct ethhdr *eth;
        unsigned char *rawp;
-       
+
        skb->mac.raw = skb->data;
-       skb_pull(skb,ETH_HLEN);
+       skb_pull(skb, ETH_HLEN);
        eth = eth_hdr(skb);
-       
+
        if (is_multicast_ether_addr(eth->h_dest)) {
                if (!compare_ether_addr(eth->h_dest, dev->broadcast))
                        skb->pkt_type = PACKET_BROADCAST;
                else
                        skb->pkt_type = PACKET_MULTICAST;
        }
-       
+
        /*
-        *      This ALLMULTI check should be redundant by 1.4
-        *      so don't forget to remove it.
+        *      This ALLMULTI check should be redundant by 1.4
+        *      so don't forget to remove it.
         *
-        *      Seems, you forgot to remove it. All silly devices
-        *      seems to set IFF_PROMISC.
+        *      Seems, you forgot to remove it. All silly devices
+        *      seems to set IFF_PROMISC.
         */
-        
-       else if(1 /*dev->flags&IFF_PROMISC*/) {
+
+       else if (1 /*dev->flags&IFF_PROMISC */ ) {
                if (unlikely(compare_ether_addr(eth->h_dest, dev->dev_addr)))
                        skb->pkt_type = PACKET_OTHERHOST;
        }
-       
+
        if (ntohs(eth->h_proto) >= 1536)
                return eth->h_proto;
-               
+
        rawp = skb->data;
-       
+
        /*
-        *      This is a magic hack to spot IPX packets. Older Novell breaks
-        *      the protocol design and runs IPX over 802.3 without an 802.2 LLC
-        *      layer. We look for FFFF which isn't a used 802.2 SSAP/DSAP. This
-        *      won't work for fault tolerant netware but does for the rest.
+        *      This is a magic hack to spot IPX packets. Older Novell breaks
+        *      the protocol design and runs IPX over 802.3 without an 802.2 LLC
+        *      layer. We look for FFFF which isn't a used 802.2 SSAP/DSAP. This
+        *      won't work for fault tolerant netware but does for the rest.
         */
        if (*(unsigned short *)rawp == 0xFFFF)
                return htons(ETH_P_802_3);
-               
+
        /*
-        *      Real 802.2 LLC
+        *      Real 802.2 LLC
         */
        return htons(ETH_P_802_2);
 }
@@ -216,8 +205,8 @@
        struct ethhdr *eth;
        struct net_device *dev = neigh->dev;
 
-       eth = (struct ethhdr*)
-               (((u8*)hh->hh_data) + (HH_DATA_OFF(sizeof(*eth))));
+       eth = (struct ethhdr *)
+           (((u8 *) hh->hh_data) + (HH_DATA_OFF(sizeof(*eth))));
 
        if (type == __constant_htons(ETH_P_802_3))
                return -1;
@@ -233,17 +222,17 @@
  * Called by Address Resolution module to notify changes in address.
  */
 
-void eth_header_cache_update(struct hh_cache *hh, struct net_device *dev, 
unsigned char * haddr)
+void eth_header_cache_update(struct hh_cache *hh, struct net_device *dev,
+                            unsigned char *haddr)
 {
-       memcpy(((u8*)hh->hh_data) + HH_DATA_OFF(sizeof(struct ethhdr)),
+       memcpy(((u8 *) hh->hh_data) + HH_DATA_OFF(sizeof(struct ethhdr)),
               haddr, ETH_ALEN);
 }
-
 EXPORT_SYMBOL(eth_type_trans);
 
 static int eth_mac_addr(struct net_device *dev, void *p)
 {
-       struct sockaddr *addr=p;
+       struct sockaddr *addr = p;
        if (netif_running(dev))
                return -EBUSY;
        memcpy(dev->dev_addr, addr->sa_data, ETH_ALEN);
@@ -278,7 +267,7 @@
        dev->tx_queue_len       = 1000; /* Ethernet wants good queues */        
        dev->flags              = IFF_BROADCAST|IFF_MULTICAST;
        
-       memset(dev->broadcast,0xFF, ETH_ALEN);
+       memset(dev->broadcast, 0xFF, ETH_ALEN);
 
 }
 EXPORT_SYMBOL(ether_setup);

--

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to