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.
> 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.
- Henry
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
