Re: [PATCH 2/4] staging: vt6655: checkpatch fixes

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

 



On Mon, Jan 14, 2013 at 12:41:32PM -0500, devendra.aaru wrote:
> On Tue, Jan 8, 2013 at 1:41 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> > On Tue, Jan 08, 2013 at 12:52:33PM -0500, Devendra Naga wrote:
> >> put opening brace in a new line
> >> 4 ---> 8 spaces
> >>
> >
> > This is fine, but if you wanted it would also have been fine to do
> > this in the first patch.  It would still fit under the one thing
> > per patch rule.
> 
> ok Dan, i will resend again if this is a problem, and follow your
> guidelines from next onwards.
> 

Don't resend.  The other patch is fine.

regards,
dan

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux