Coding style correction of some wan drivers
authorRudy Matela <rudy.matela@gmail.com>
Wed, 9 Dec 2009 14:35:40 +0000 (11:35 -0300)
committerJiri Kosina <jkosina@suse.cz>
Thu, 4 Feb 2010 10:55:46 +0000 (11:55 +0100)
Added a space separating some if/switch/while keywords from the following
parenthesis to conform to the CodingStyle.

Signed-off-by: Rudy Matela <rudy.matela@gmail.com>
Acked-by: Krzysztof Halasa <khc@pm.waw.pl>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/net/wan/cosa.c
drivers/net/wan/hdlc_cisco.c
drivers/net/wan/hdlc_x25.c

index b36bf96eb502d18853b6ed1172c9ec4b4060e23f..f0bd70fb650cbd669a15cc07a6aaa02116f9f684 100644 (file)
@@ -811,7 +811,7 @@ static ssize_t cosa_read(struct file *file,
        cosa_enable_rx(chan);
        spin_lock_irqsave(&cosa->lock, flags);
        add_wait_queue(&chan->rxwaitq, &wait);
-       while(!chan->rx_status) {
+       while (!chan->rx_status) {
                current->state = TASK_INTERRUPTIBLE;
                spin_unlock_irqrestore(&cosa->lock, flags);
                schedule();
@@ -896,7 +896,7 @@ static ssize_t cosa_write(struct file *file,
 
        spin_lock_irqsave(&cosa->lock, flags);
        add_wait_queue(&chan->txwaitq, &wait);
-       while(!chan->tx_status) {
+       while (!chan->tx_status) {
                current->state = TASK_INTERRUPTIBLE;
                spin_unlock_irqrestore(&cosa->lock, flags);
                schedule();
@@ -1153,7 +1153,7 @@ static int cosa_ioctl_common(struct cosa_data *cosa,
        struct channel_data *channel, unsigned int cmd, unsigned long arg)
 {
        void __user *argp = (void __user *)arg;
-       switch(cmd) {
+       switch (cmd) {
        case COSAIORSET:        /* Reset the device */
                if (!capable(CAP_NET_ADMIN))
                        return -EACCES;
@@ -1704,7 +1704,7 @@ static inline void tx_interrupt(struct cosa_data *cosa, int status)
                        spin_unlock_irqrestore(&cosa->lock, flags);
                        return;
                }
-               while(1) {
+               while (1) {
                        cosa->txchan++;
                        i++;
                        if (cosa->txchan >= cosa->nchannels)
@@ -2010,7 +2010,7 @@ again:
 static void debug_status_in(struct cosa_data *cosa, int status)
 {
        char *s;
-       switch(status & SR_CMD_FROM_SRP_MASK) {
+       switch (status & SR_CMD_FROM_SRP_MASK) {
        case SR_UP_REQUEST:
                s = "RX_REQ";
                break;
index f1bff98acd1fc8e1765b88d7d7195a740933b3dc..1ceccf1ca6c7952e277dbf11e43421206657b8a6 100644 (file)
@@ -141,7 +141,7 @@ static __be16 cisco_type_trans(struct sk_buff *skb, struct net_device *dev)
            data->address != CISCO_UNICAST)
                return cpu_to_be16(ETH_P_HDLC);
 
-       switch(data->protocol) {
+       switch (data->protocol) {
        case cpu_to_be16(ETH_P_IP):
        case cpu_to_be16(ETH_P_IPX):
        case cpu_to_be16(ETH_P_IPV6):
@@ -190,7 +190,7 @@ static int cisco_rx(struct sk_buff *skb)
                cisco_data = (struct cisco_packet*)(skb->data + sizeof
                                                    (struct hdlc_header));
 
-               switch(ntohl (cisco_data->type)) {
+               switch (ntohl (cisco_data->type)) {
                case CISCO_ADDR_REQ: /* Stolen from syncppp.c :-) */
                        in_dev = dev->ip_ptr;
                        addr = 0;
@@ -245,8 +245,8 @@ static int cisco_rx(struct sk_buff *skb)
 
                        dev_kfree_skb_any(skb);
                        return NET_RX_SUCCESS;
-               } /* switch(keepalive type) */
-       } /* switch(protocol) */
+               } /* switch (keepalive type) */
+       } /* switch (protocol) */
 
        printk(KERN_INFO "%s: Unsupported protocol %x\n", dev->name,
               ntohs(data->protocol));
index aa9248f8eb1afd371c4834824d3da3c2f629a457..6e1ca256effd8f8752b3387c23ca75d7522cb1d8 100644 (file)
@@ -202,10 +202,10 @@ static int x25_ioctl(struct net_device *dev, struct ifreq *ifr)
                return 0; /* return protocol only, no settable parameters */
 
        case IF_PROTO_X25:
-               if(!capable(CAP_NET_ADMIN))
+               if (!capable(CAP_NET_ADMIN))
                        return -EPERM;
 
-               if(dev->flags & IFF_UP)
+               if (dev->flags & IFF_UP)
                        return -EBUSY;
 
                result=hdlc->attach(dev, ENCODING_NRZ,PARITY_CRC16_PR1_CCITT);