]> Pileus Git - ~andy/linux/commitdiff
staging: vt6655: no braces needed for single statement if , for blocks
authorDevendra Naga <devendra.aaru@gmail.com>
Tue, 8 Jan 2013 17:52:35 +0000 (12:52 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 17 Jan 2013 21:17:03 +0000 (13:17 -0800)
there is only one line inside the if and for blocks, so the braces are
not needed

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vt6655/channel.c

index 47c156bb70a9ff934c2a624dab5225c7ae5f1c52..aa76e39a46f49e8aed182e8994b5fc8bf059b3d5 100644 (file)
@@ -749,9 +749,9 @@ bool get_channel_map_info(void *pDeviceHandler, unsigned int uChannelIndex,
                unsigned char *pbyChannelNumber, unsigned char *pbyMap)
 {
 
-       if (uChannelIndex > CB_MAX_CHANNEL) {
+       if (uChannelIndex > CB_MAX_CHANNEL)
                return false;
-       }
+
        *pbyChannelNumber = sChannelTbl[uChannelIndex].byChannelNumber;
        *pbyMap = sChannelTbl[uChannelIndex].byMAP;
        return sChannelTbl[uChannelIndex].bValid;
@@ -761,9 +761,9 @@ void set_channel_map_info(void *pDeviceHandler, unsigned int uChannelIndex,
                unsigned char byMap)
 {
 
-       if (uChannelIndex > CB_MAX_CHANNEL) {
+       if (uChannelIndex > CB_MAX_CHANNEL)
                return;
-       }
+
        sChannelTbl[uChannelIndex].byMAP |= byMap;
 }
 
@@ -771,9 +771,8 @@ void clear_channel_map_info(void *pDeviceHandler)
 {
        unsigned int ii = 0;
 
-       for (ii = 1; ii <=  CB_MAX_CHANNEL; ii++) {
+       for (ii = 1; ii <=  CB_MAX_CHANNEL; ii++)
                sChannelTbl[ii].byMAP = 0;
-       }
 }
 
 unsigned char auto_channel_select(void *pDeviceHandler, CARD_PHY_TYPE ePHYType)