From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752410AbbJQArO (ORCPT ); Fri, 16 Oct 2015 20:47:14 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:22372 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750957AbbJQArM (ORCPT ); Fri, 16 Oct 2015 20:47:12 -0400 Date: Sat, 17 Oct 2015 03:46:45 +0300 From: Dan Carpenter To: Luis de Bethencourt Cc: linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, hamohammed.sa@gmail.com, paul.gortmaker@windriver.com, gregkh@linuxfoundation.org, gdonald@gmail.com, cristina.opriceana@gmail.com Subject: Re: [PATCH v2] staging: rtl8192u: simplify conditional statement Message-ID: <20151017004645.GD7340@mwanda> References: <1445001991-24113-1-git-send-email-luisbg@osg.samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1445001991-24113-1-git-send-email-luisbg@osg.samsung.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Source-IP: aserv0021.oracle.com [141.146.126.233] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Oct 16, 2015 at 02:26:31PM +0100, Luis de Bethencourt wrote: > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > index c443e2e..f85e5ff 100644 > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > @@ -466,10 +466,7 @@ void ieee80211_softmac_scan_syncro(struct ieee80211_device *ieee) > /* this prevent excessive time wait when we > * need to wait for a syncro scan to end.. > */ > - if(ieee->state < IEEE80211_LINKED) > - ; > - else > - if (ieee->sync_scan_hurryup) > + if (ieee->state > IEEE80211_LINKED && ieee->sync_scan_hurryup) This should be >= instead of >. regards, dan carpenter