arcnet: Expand odd BUGLVL macro with if and uses
[cascardo/linux.git] / drivers / net / arcnet / capmode.c
index 42fce91..e7ec907 100644 (file)
@@ -66,17 +66,17 @@ static void rx(struct net_device *dev, int bufnum,
        pkt = (struct archdr *)skb_mac_header(skb);
        skb_pull(skb, ARC_HDR_SIZE);
 
-       /* up to sizeof(pkt->soft) has already been copied from the card */
-       /* squeeze in an int for the cap encapsulation */
-
-       /* use these variables to be sure we count in bytes, not in
-          sizeof(struct archdr) */
-       pktbuf=(char*)pkt;
-       pkthdrbuf=(char*)pkthdr;
-       memcpy(pktbuf, pkthdrbuf, ARC_HDR_SIZE+sizeof(pkt->soft.cap.proto));
-       memcpy(pktbuf+ARC_HDR_SIZE+sizeof(pkt->soft.cap.proto)+sizeof(int),
-              pkthdrbuf+ARC_HDR_SIZE+sizeof(pkt->soft.cap.proto),
-              sizeof(struct archdr)-ARC_HDR_SIZE-sizeof(pkt->soft.cap.proto));
+       /* up to sizeof(pkt->soft) has already been copied from the card
+        * squeeze in an int for the cap encapsulation
+        * use these variables to be sure we count in bytes, not in
+        * sizeof(struct archdr)
+        */
+       pktbuf = (char *)pkt;
+       pkthdrbuf = (char *)pkthdr;
+       memcpy(pktbuf, pkthdrbuf, ARC_HDR_SIZE + sizeof(pkt->soft.cap.proto));
+       memcpy(pktbuf + ARC_HDR_SIZE + sizeof(pkt->soft.cap.proto) + sizeof(int),
+              pkthdrbuf + ARC_HDR_SIZE + sizeof(pkt->soft.cap.proto),
+              sizeof(struct archdr) - ARC_HDR_SIZE - sizeof(pkt->soft.cap.proto));
 
        if (length > sizeof(pkt->soft))
                lp->hw.copy_from_card(dev, bufnum, ofs + sizeof(pkt->soft),
@@ -84,15 +84,14 @@ static void rx(struct net_device *dev, int bufnum,
                                      + sizeof(int),
                                      length - sizeof(pkt->soft));
 
-       BUGLVL(D_SKB) arcnet_dump_skb(dev, skb, "rx");
+       if (BUGLVL(D_SKB))
+               arcnet_dump_skb(dev, skb, "rx");
 
        skb->protocol = cpu_to_be16(ETH_P_ARCNET);
        netif_rx(skb);
 }
 
-
-/*
- * Create the ARCnet hard/soft headers for cap mode.
+/* Create the ARCnet hard/soft headers for cap mode.
  * There aren't any soft headers in cap mode - not even the protocol id.
  */
 static int build_header(struct sk_buff *skb,
@@ -101,12 +100,12 @@ static int build_header(struct sk_buff *skb,
                        uint8_t daddr)
 {
        int hdr_size = ARC_HDR_SIZE;
-       struct archdr *pkt = (struct archdr *) skb_push(skb, hdr_size);
+       struct archdr *pkt = (struct archdr *)skb_push(skb, hdr_size);
 
        BUGMSG(D_PROTO, "Preparing header for cap packet %x.\n",
-              *((int*)&pkt->soft.cap.cookie[0]));
-       /*
-        * Set the source hardware address.
+              *((int *)&pkt->soft.cap.cookie[0]));
+
+       /* Set the source hardware address.
         *
         * This is pretty pointless for most purposes, but it can help in
         * debugging.  ARCnet does not allow us to change the source address in
@@ -117,9 +116,8 @@ static int build_header(struct sk_buff *skb,
        /* see linux/net/ethernet/eth.c to see where I got the following */
 
        if (dev->flags & (IFF_LOOPBACK | IFF_NOARP)) {
-               /*
-                * FIXME: fill in the last byte of the dest ipaddr here to better
-                * comply with RFC1051 in "noarp" mode.
+               /* FIXME: fill in the last byte of the dest ipaddr here to
+                * better comply with RFC1051 in "noarp" mode.
                 */
                pkt->hard.dest = 0;
                return hdr_size;
@@ -130,7 +128,6 @@ static int build_header(struct sk_buff *skb,
        return hdr_size;        /* success */
 }
 
-
 static int prepare_tx(struct net_device *dev, struct archdr *pkt, int length,
                      int bufnum)
 {
@@ -138,7 +135,6 @@ static int prepare_tx(struct net_device *dev, struct archdr *pkt, int length,
        struct arc_hardware *hard = &pkt->hard;
        int ofs;
 
-
        /* hard header is not included in packet length */
        length -= ARC_HDR_SIZE;
        /* And neither is the cookie field */
@@ -148,7 +144,7 @@ static int prepare_tx(struct net_device *dev, struct archdr *pkt, int length,
               lp->next_tx, lp->cur_tx, bufnum);
 
        BUGMSG(D_PROTO, "Sending for cap packet %x.\n",
-              *((int*)&pkt->soft.cap.cookie[0]));
+              *((int *)&pkt->soft.cap.cookie[0]));
 
        if (length > XMTU) {
                /* should never happen! other people already check for this. */
@@ -162,11 +158,12 @@ static int prepare_tx(struct net_device *dev, struct archdr *pkt, int length,
        } else if (length > MTU) {
                hard->offset[0] = 0;
                hard->offset[1] = ofs = 512 - length - 3;
-       } else
+       } else {
                hard->offset[0] = ofs = 256 - length;
+       }
 
        BUGMSG(D_DURING, "prepare_tx: length=%d ofs=%d\n",
-              length,ofs);
+              length, ofs);
 
        /* Copy the arcnet-header + the protocol byte down: */
        lp->hw.copy_to_card(dev, bufnum, 0, hard, ARC_HDR_SIZE);
@@ -174,9 +171,10 @@ static int prepare_tx(struct net_device *dev, struct archdr *pkt, int length,
                            sizeof(pkt->soft.cap.proto));
 
        /* Skip the extra integer we have written into it as a cookie
-          but write the rest of the message: */
-       lp->hw.copy_to_card(dev, bufnum, ofs+1,
-                           ((unsigned char*)&pkt->soft.cap.mes),length-1);
+        * but write the rest of the message:
+        */
+       lp->hw.copy_to_card(dev, bufnum, ofs + 1,
+                           ((unsigned char *)&pkt->soft.cap.mes), length - 1);
 
        lp->lastload_dest = hard->dest;
 
@@ -188,21 +186,22 @@ static int ack_tx(struct net_device *dev, int acked)
        struct arcnet_local *lp = netdev_priv(dev);
        struct sk_buff *ackskb;
        struct archdr *ackpkt;
-       int length=sizeof(struct arc_cap);
+       int length = sizeof(struct arc_cap);
 
        BUGMSG(D_DURING, "capmode: ack_tx: protocol: %x: result: %d\n",
-               lp->outgoing.skb->protocol, acked);
+              lp->outgoing.skb->protocol, acked);
 
-       BUGLVL(D_SKB) arcnet_dump_skb(dev, lp->outgoing.skb, "ack_tx");
+       if (BUGLVL(D_SKB))
+               arcnet_dump_skb(dev, lp->outgoing.skb, "ack_tx");
 
        /* Now alloc a skb to send back up through the layers: */
-       ackskb = alloc_skb(length + ARC_HDR_SIZE , GFP_ATOMIC);
+       ackskb = alloc_skb(length + ARC_HDR_SIZE, GFP_ATOMIC);
        if (ackskb == NULL) {
                BUGMSG(D_NORMAL, "Memory squeeze, can't acknowledge.\n");
                goto free_outskb;
        }
 
-       skb_put(ackskb, length + ARC_HDR_SIZE );
+       skb_put(ackskb, length + ARC_HDR_SIZE);
        ackskb->dev = dev;
 
        skb_reset_mac_header(ackskb);
@@ -212,14 +211,15 @@ static int ack_tx(struct net_device *dev, int acked)
        skb_copy_from_linear_data(lp->outgoing.skb, ackpkt,
                                  ARC_HDR_SIZE + sizeof(struct arc_cap));
        ackpkt->soft.cap.proto = 0; /* using protocol 0 for acknowledge */
-       ackpkt->soft.cap.mes.ack=acked;
+       ackpkt->soft.cap.mes.ack = acked;
 
        BUGMSG(D_PROTO, "Ackknowledge for cap packet %x.\n",
-                       *((int*)&ackpkt->soft.cap.cookie[0]));
+              *((int *)&ackpkt->soft.cap.cookie[0]));
 
        ackskb->protocol = cpu_to_be16(ETH_P_ARCNET);
 
-       BUGLVL(D_SKB) arcnet_dump_skb(dev, ackskb, "ack_tx_recv");
+       if (BUGLVL(D_SKB))
+               arcnet_dump_skb(dev, ackskb, "ack_tx_recv");
        netif_rx(ackskb);
 
 free_outskb:
@@ -229,8 +229,7 @@ free_outskb:
        return 0;
 }
 
-static struct ArcProto capmode_proto =
-{
+static struct ArcProto capmode_proto = {
        'r',
        XMTU,
        0,