Return-path: Received: from mms2.broadcom.com ([216.31.210.18]:2165 "EHLO mms2.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751889Ab1GGVzt convert rfc822-to-8bit (ORCPT ); Thu, 7 Jul 2011 17:55:49 -0400 Date: Thu, 7 Jul 2011 14:55:37 -0700 From: "Henry Ptasinski" To: "Greg KH" cc: "Pavel Roskin" , =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= , "linville@tuxdriver.com" , "devel@linuxdriverproject.org" , "linux-wireless@vger.kernel.org" , "Brett Rudley" , "Arend Van Spriel" , "Roland Vossen" , "Franky (Zhenhui) Lin" , "Henry Ptasinski" Subject: Re: [RFC] Move brcm80211 to mainline Message-ID: <20110707215537.GS23944@broadcom.com> (sfid-20110707_235556_773466_88EAC035) References: <20110707002034.GA17885@broadcom.com> <4E1504A3.1070801@gnu.org> <20110707014512.GA21087@suse.de> <20110707144628.GF23944@broadcom.com> <20110707145822.GA29241@suse.de> MIME-Version: 1.0 In-Reply-To: <20110707145822.GA29241@suse.de> Content-Type: text/plain; charset=utf-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Thu, Jul 07, 2011 at 07:58:22AM -0700, Greg KH wrote: > On Thu, Jul 07, 2011 at 07:46:28AM -0700, Henry Ptasinski wrote: > > On Wed, Jul 06, 2011 at 06:45:12PM -0700, Greg KH wrote: > > > On Wed, Jul 06, 2011 at 08:58:11PM -0400, Pavel Roskin wrote: > > > > On 07/06/2011 08:40 PM, Rafał Miłecki wrote: > > > > >Short question, without commenting on brcm80211 code yet: > > > > > > > > > >Why should we want 2 mainline drivers for the same hardware? > > > > > > > > Also, checkpatch.pl finds a lot of bad stuff: > > > > > > > > total: 1 errors, 1505 warnings, 105454 lines checked > > > > > > > > $ scripts/checkpatch.pl 0001-wireless-testing-add-brcm80211.patch >Log > > > > $ egrep '^(WARNING|ERROR)' Log |sort |uniq -c > > > > 1 ERROR: do not use assignment in if condition > > > > 76 WARNING: braces {} are not necessary for any arm of this statement > > > > 205 WARNING: braces {} are not necessary for single statement blocks > > > > 4 WARNING: consider using a completion > > > > 11 WARNING: consider using kstrto* in preference to simple_strtoul > > > > 20 WARNING: do not add new typedefs > > > > 5 WARNING: externs should be avoided in .c files > > > > 1164 WARNING: line over 80 characters > > > > 20 WARNING: Use of volatile is usually wrong: see > > > > > > Almost all of these should be fixed, especially the volatile one, that's > > > just horrible. > > > > Fixes for these are in progress. We'll keep posting patches to address this > > cleanup in staging. > > Ok, at the least these should be resolved before ever asking for the > code to be moved out of staging. Sorry for me not checking this > previously, I wrongly assumed that you had finished this task already. No worries. We'll have almost all of these cleaned up shortly. The 80 char lines will be nice and tedious, though, unless somebody can suggest a decent tool that doesn't make the code unreadable. > > > > Also, there's the whole big endian mess that needs to be fixed up before > > > the driver can be moved out of staging. Henry, what's the status of > > > that? > > > > We're looking at BE MIPS, and I sent some cards to Benjamin Herrenschmidt so he > > can test on PPC platforms. > > Don't forget just building the thing on other arches (SPARC for example) > and what happens when you try to do that... Building some cross-compiler toolchains now. SPARC hardware to do anything native with seems to be getting very scarce, and trying to fit a PCIe minicard into one might be challenging ... - Henry