staging:rtl8187se: Change argument type in function to bool
authorHimangi Saraogi <himangi774@gmail.com>
Fri, 7 Mar 2014 17:35:26 +0000 (23:05 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 8 Mar 2014 00:23:18 +0000 (16:23 -0800)
Since the argument morefrag is being used as a booleon in the function
rtl8180_tx, the type for the argument is changed to booleon in the
function definition and declaration. This ensures that the variable
morefrag is always in the range 0-1 and overflow dooes not cause errors
if it is assigned a value. Also, this improves code readability.

Signed-off-by: Himangi Saraogi <himangi774@gmail.com>
Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8187se/r8180.h
drivers/staging/rtl8187se/r8180_core.c

index 59237a5..00e38b5 100644 (file)
@@ -648,7 +648,7 @@ struct r8180_priv {
                BE_PRIORITY)
 
 short rtl8180_tx(struct net_device *dev, u8 *skbuf, int len, int priority,
-                short morefrag, short fragdesc, int rate);
+                bool morefrag, short fragdesc, int rate);
 
 u8 read_nic_byte(struct net_device *dev, int x);
 u32 read_nic_dword(struct net_device *dev, int x);
index dc434b8..d191750 100644 (file)
@@ -1679,7 +1679,7 @@ static void rtl8180_prepare_beacon(struct net_device *dev)
  * it does the DMA transfer.
  */
 short rtl8180_tx(struct net_device *dev, u8 *txbuf, int len, int priority,
-                short morefrag, short descfrag, int rate)
+                bool morefrag, short descfrag, int rate)
 {
        struct r8180_priv *priv = ieee80211_priv(dev);
        u32 *tail, *temp_tail;