Re: [PATCH 0/3] checkpatch fixes

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

 



On 09/18/2012 12:23 AM, Dan Carpenter wrote:

> On Mon, Sep 17, 2012 at 02:56:01PM -0700, Greg KH wrote:
>> On Mon, Sep 17, 2012 at 07:59:16PM -0700, Daniel Cotey wrote:
>>> to keep files unique all changes of one type are in one patch.
>>>
>>> Daniel Cotey (3):
>>>   Staging: silicom: bp_proc.c: checkpatch cleanups: initial defines
>>>   Staging: silicom: bp_mod.c: checkpatch cleanup: fix returns
>>>   Staging: silicom: bp_mod.c: checkpatch cleanups: C99 comments
>>>
>>>  drivers/staging/silicom/bp_mod.c  | 389 +++++++++++++++-----------------------
>>>  drivers/staging/silicom/bp_proc.c |   9 +-
>>>  2 files changed, 153 insertions(+), 245 deletions(-)
>>
>> Please at least build the patches you send me, these broke the build in
>> massive ways, not nice.
> 
> Also, Daniel, you didn't CC the list again.  I don't know why you're
> not doing that.


Also, the subject should be about staging: or silicom:,
not checkpatch.  As it is, it looks like it would be patches
to scripts/checkpatch.pl

-- 
~Randy
_______________________________________________
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