From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751150AbbEYM4k (ORCPT ); Mon, 25 May 2015 08:56:40 -0400 Received: from mail-am1on0102.outbound.protection.outlook.com ([157.56.112.102]:64309 "EHLO emea01-am1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750757AbbEYM4j convert rfc822-to-8bit (ORCPT ); Mon, 25 May 2015 08:56:39 -0400 X-Greylist: delayed 1109 seconds by postgrey-1.27 at vger.kernel.org; Mon, 25 May 2015 08:56:39 EDT From: "Harisangam, Sharvari (S.)" To: "forest@alittletooquiet.net" , "gregkh@linuxfoundation.org" , "tvboxspy@gmail.com" , "gclement@baobob.org" , "haticeerturk27@gmail.com" , "mahfouz.saif.elyazal@gmail.com" CC: "devel@driverdev.osuosl.org" , "linux-kernel@vger.kernel.org" Subject: [PATCH] Staging: vt6655: Remove unnecessary equality checks for a bool variable Thread-Topic: [PATCH] Staging: vt6655: Remove unnecessary equality checks for a bool variable Thread-Index: AQHQluV147JUjy9uAkmcldnntxOMeg== Date: Mon, 25 May 2015 12:22:25 +0000 Message-ID: <20150525122207.GA13676@ubuntu> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=sharisan@visteon.com; x-ms-exchange-messagesentrepresentingtype: 1 x-originating-ip: [202.0.77.198] x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DB5PR06MB1350; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(601004)(5005006)(520002)(3002001);SRVR:DB5PR06MB1350;BCL:0;PCL:0;RULEID:;SRVR:DB5PR06MB1350; x-forefront-prvs: 058707456E x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(6009001)(199003)(189002)(19580405001)(50986999)(101416001)(19580395003)(33656002)(54356999)(2201001)(86362001)(87936001)(2656002)(66066001)(64706001)(105586002)(102836002)(68736005)(97736004)(4001540100001)(5001770100001)(81156007)(2860100001)(2900100001)(229853001)(33716001)(106356001)(106116001)(46102003)(40100003)(122556002)(77156002)(62966003)(189998001)(5001830100001)(5001960100002)(5001860100001)(2501003)(92566002);DIR:OUT;SFP:1102;SCL:1;SRVR:DB5PR06MB1350;H:DB5PR06MB1352.eurprd06.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; Content-Type: text/plain; charset="us-ascii" Content-ID: <7D9E5F255EA62E448A81F2B85FA1B36B@eurprd06.prod.outlook.com> Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-OriginatorOrg: visteon.com X-MS-Exchange-CrossTenant-originalarrivaltime: 25 May 2015 12:22:25.9520 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 7a147aaf-01ec-498c-80a1-e34a8c63c548 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB5PR06MB1350 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Modified the if-else statements to remove unnecessary comparisons. This change was detected with the help of coccinelle tool Signed-off-by: Harisangam Sharvari S --- 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