Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754274Ab1FPBg3 (ORCPT ); Wed, 15 Jun 2011 21:36:29 -0400 Received: from mms2.broadcom.com ([216.31.210.18]:4607 "EHLO mms2.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752035Ab1FPBg1 (ORCPT ); Wed, 15 Jun 2011 21:36:27 -0400 X-Server-Uuid: D3C04415-6FA8-4F2C-93C1-920E106A2031 Date: Wed, 15 Jun 2011 18:36:16 -0700 From: "Henry Ptasinski" To: "Greg Dietsche" cc: "gregkh@suse.de" , "Brett Rudley" , "Dowan Kim" , "Roland Vossen" , "Arend Van Spriel" , "linux-wireless@vger.kernel.org" , "devel@driverdev.osuosl.org" , "linux-kernel@vger.kernel.org" , "Henry Ptasinski" Subject: Re: [PATCH] staging: brcm80211: return false if not a broadcom board Message-ID: <20110616013615.GA14700@broadcom.com> References: <1308176709-14765-1-git-send-email-Gregory.Dietsche@cuw.edu> MIME-Version: 1.0 In-Reply-To: <1308176709-14765-1-git-send-email-Gregory.Dietsche@cuw.edu> User-Agent: Mutt/1.5.20 (2009-06-14) X-WSS-ID: 61E7809962O9546291-01-01 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1536 Lines: 43 On Wed, Jun 15, 2011 at 03:25:09PM -0700, Greg Dietsche wrote: > This code looks wrong to me. I think it meant to return false > if the board's vendor id isn't Broadcom's. > > Compile tested only. > > Signed-off-by: Greg Dietsche > --- > drivers/staging/brcm80211/brcmsmac/wlc_bmac.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/staging/brcm80211/brcmsmac/wlc_bmac.c b/drivers/staging/brcm80211/brcmsmac/wlc_bmac.c > index 4534926..ee13238 100644 > --- a/drivers/staging/brcm80211/brcmsmac/wlc_bmac.c > +++ b/drivers/staging/brcm80211/brcmsmac/wlc_bmac.c > @@ -1943,7 +1943,7 @@ static bool wlc_validboardtype(struct wlc_hw_info *wlc_hw) > } > > if (wlc_hw->sih->boardvendor != PCI_VENDOR_ID_BROADCOM) > - return goodboard; > + goodboard = false; > > return goodboard; > } Actually, the original code is correct. For boards with the Broadcom ID, this function checks that the boardrev is sane. Dev boards that aren't properly configured may have invalid info, so this check is mainly to catch that problem. For boards with any other vendor, we don't have any sanity checks that we know should be done, so the board info is always considered good. The device ID is of course always checked. - Henry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/