Modified the if-else statements to remove unnecessary comparisons.
This change was detected with the help of coccinelle tool

Signed-off-by: Harisangam Sharvari <shari...@visteon.com>
---
 drivers/staging/vt6655/rxtx.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c
index 7468776..7fedde3 100644
--- a/drivers/staging/vt6655/rxtx.c
+++ b/drivers/staging/vt6655/rxtx.c
@@ -1093,7 +1093,7 @@ s_cbFillTxBufHead(struct vnt_private *pDevice, unsigned 
char byPktType,
        if (byPktType == PK_TYPE_11GB || byPktType == PK_TYPE_11GA) {/* 802.11g 
packet */
 
                if (byFBOption == AUTO_FB_NONE) {
-                       if (bRTS == true) {/* RTS_need */
+                       if (bRTS) {/* RTS_need */
                                pvRrvTime = (void *)(pbyTxBufferAddr + 
wTxBufSize);
                                pMICHDR = (struct vnt_mic_hdr 
*)(pbyTxBufferAddr + wTxBufSize + sizeof(struct vnt_rrv_time_rts));
                                pvRTS = (void *)(pbyTxBufferAddr + wTxBufSize + 
sizeof(struct vnt_rrv_time_rts) + cbMICHDR);
@@ -1115,7 +1115,7 @@ s_cbFillTxBufHead(struct vnt_private *pDevice, unsigned 
char byPktType,
                        }
                } else {
                        /* Auto Fall Back */
-                       if (bRTS == true) {/* RTS_need */
+                       if (bRTS) {/* RTS_need */
                                pvRrvTime = (void *)(pbyTxBufferAddr + 
wTxBufSize);
                                pMICHDR = (struct vnt_mic_hdr *) 
(pbyTxBufferAddr + wTxBufSize + sizeof(struct vnt_rrv_time_rts));
                                pvRTS = (void *) (pbyTxBufferAddr + wTxBufSize 
+ sizeof(struct vnt_rrv_time_rts) + cbMICHDR);
@@ -1138,7 +1138,7 @@ s_cbFillTxBufHead(struct vnt_private *pDevice, unsigned 
char byPktType,
        } else {/* 802.11a/b packet */
 
                if (byFBOption == AUTO_FB_NONE) {
-                       if (bRTS == true) {
+                       if (bRTS) {
                                pvRrvTime = (void *)(pbyTxBufferAddr + 
wTxBufSize);
                                pMICHDR = (struct vnt_mic_hdr *) 
(pbyTxBufferAddr + wTxBufSize + sizeof(struct vnt_rrv_time_ab));
                                pvRTS = (void *)(pbyTxBufferAddr + wTxBufSize + 
sizeof(struct vnt_rrv_time_ab) + cbMICHDR);
@@ -1158,7 +1158,7 @@ s_cbFillTxBufHead(struct vnt_private *pDevice, unsigned 
char byPktType,
                        }
                } else {
                        /* Auto Fall Back */
-                       if (bRTS == true) { /* RTS_need */
+                       if (bRTS) { /* RTS_need */
                                pvRrvTime = (void *)(pbyTxBufferAddr + 
wTxBufSize);
                                pMICHDR = (struct vnt_mic_hdr *) 
(pbyTxBufferAddr + wTxBufSize + sizeof(struct vnt_rrv_time_ab));
                                pvRTS = (void *)(pbyTxBufferAddr + wTxBufSize + 
sizeof(struct vnt_rrv_time_ab) + cbMICHDR);
-- 
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to