Re: [PATCH] staging: android: binder: fix some sparse warnings and long lines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Dan,

Sorry will look at it again and send another patch, perhaps tomorrow.

Thanks for catching my mistakes!

Best regards,
Emil Goode

On Mon, 2012-04-23 at 20:57 +0300, Dan Carpenter wrote:
> On Mon, Apr 23, 2012 at 07:23:44PM +0200, Emil Goode wrote:
> > This patch silences some sparse warnings and adds some line breaks
> > to take the 80 char limit coding style rule in consideration.
> > 
> 
> Don't mix long line changes and sparse changes.
> 
> You missed some fp->binder casts so there are still Sparse warnings.
> Anyway, these sparse annotations should probably go into the
> struct binder_node definition in the .h file instead of always
> casting.  (I haven't verified.  So be careful).
> 
> Don't forget to get to run your patches through checkpatch.pl before
> sending them.  (There are whitespace problems in your patch).
> 
> regards,
> dan carpenter
> 


--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux