On Sat, Nov 19, 2011 at 02:19:54AM +0000, Ken O'Brien wrote: > Removed all checkpatch errors. > > Checkpatch status before: total: 131 errors, 200 warnings, 648 lines checked > Checkpatch status after: total: 0 errors, 194 warnings, 620 lines checked >
I don't have strong opinions personally, but I think the prefered way
is to do this for one class of errors at a time. It makes the patch
smaller and easier to review. Then send the series of patches.
> +typedef union _U_IP_ADDRESS {
> + struct {
> + ULONG
> ulIpv4Addr[MAX_IP_RANGE_LENGTH];/*Source Ip Address Range */
^ ^ ^
^ ^
tab tab one space only here
spaces after the ';' and '*'.
The whitespace on the line above isn't right.
> + /*
> + * bit 1: 1 Idlemode enable;
> + * bit 2: 1 Sleepmode Enable
> + */
The asterisks should all line up here.
regards,
dan carpenter
signature.asc
Description: Digital signature
_______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
