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.
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?
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
