* [PATCH 00/28] Staging: comedi: Fixed numerous file so that it had little or no complaints from checkpatch.pl
@ 2010-10-06 1:15 T Dent
2010-10-06 1:27 ` Greg KH
2010-10-06 1:52 ` Greg KH
0 siblings, 2 replies; 4+ messages in thread
From: T Dent @ 2010-10-06 1:15 UTC (permalink / raw)
To: greg, abbotti, fmhess; +Cc: kernel-janitors, linux-kernel, Tracey Dent
I fixed a lot of coding style errors, that checkpatch.pl had found. I
didn't do some files in the drivers/ subdirectory because the warnings
that were being produce was spacing errors. So spacing wasn't that
serious ,so I skim and look for if it was anymore wranings or errors.
There was none so I skip the rest.
This is my last full scale checkpatch.pl patch series, as I'm going to
try to contribute more interesting things to the Linux kernel. :)
Thanks,
Tracey D
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 00/28] Staging: comedi: Fixed numerous file so that it had little or no complaints from checkpatch.pl
2010-10-06 1:15 [PATCH 00/28] Staging: comedi: Fixed numerous file so that it had little or no complaints from checkpatch.pl T Dent
@ 2010-10-06 1:27 ` Greg KH
2010-10-06 1:52 ` Greg KH
1 sibling, 0 replies; 4+ messages in thread
From: Greg KH @ 2010-10-06 1:27 UTC (permalink / raw)
To: T Dent; +Cc: abbotti, fmhess, kernel-janitors, linux-kernel
On Tue, Oct 05, 2010 at 09:15:16PM -0400, T Dent wrote:
> I fixed a lot of coding style errors, that checkpatch.pl had found. I
> didn't do some files in the drivers/ subdirectory because the warnings
> that were being produce was spacing errors. So spacing wasn't that
> serious ,so I skim and look for if it was anymore wranings or errors.
> There was none so I skip the rest.
>
> This is my last full scale checkpatch.pl patch series, as I'm going to
> try to contribute more interesting things to the Linux kernel. :)
Did you make this against Linus's tree, or the linux-next tree?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 00/28] Staging: comedi: Fixed numerous file so that it had little or no complaints from checkpatch.pl
2010-10-06 1:15 [PATCH 00/28] Staging: comedi: Fixed numerous file so that it had little or no complaints from checkpatch.pl T Dent
2010-10-06 1:27 ` Greg KH
@ 2010-10-06 1:52 ` Greg KH
2010-10-06 2:06 ` T Dent
1 sibling, 1 reply; 4+ messages in thread
From: Greg KH @ 2010-10-06 1:52 UTC (permalink / raw)
To: T Dent; +Cc: abbotti, fmhess, kernel-janitors, linux-kernel
On Tue, Oct 05, 2010 at 09:15:16PM -0400, T Dent wrote:
> I fixed a lot of coding style errors, that checkpatch.pl had found. I
> didn't do some files in the drivers/ subdirectory because the warnings
> that were being produce was spacing errors. So spacing wasn't that
> serious ,so I skim and look for if it was anymore wranings or errors.
> There was none so I skip the rest.
>
> This is my last full scale checkpatch.pl patch series, as I'm going to
> try to contribute more interesting things to the Linux kernel. :)
As mentioned in the response to 01/28, these are all just fine as-is and
don't need to be changed, sorry. Go bug the checkpatch author to remove
this from the tool.
sorry,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 00/28] Staging: comedi: Fixed numerous file so that it had little or no complaints from checkpatch.pl
2010-10-06 1:52 ` Greg KH
@ 2010-10-06 2:06 ` T Dent
0 siblings, 0 replies; 4+ messages in thread
From: T Dent @ 2010-10-06 2:06 UTC (permalink / raw)
To: Greg KH; +Cc: abbotti, fmhess, kernel-janitors, linux-kernel
On 10/5/10, Greg KH <greg@kroah.com> wrote:
> On Tue, Oct 05, 2010 at 09:15:16PM -0400, T Dent wrote:
>> I fixed a lot of coding style errors, that checkpatch.pl had found. I
>> didn't do some files in the drivers/ subdirectory because the warnings
>> that were being produce was spacing errors. So spacing wasn't that
>> serious ,so I skim and look for if it was anymore wranings or errors.
>> There was none so I skip the rest.
>>
>> This is my last full scale checkpatch.pl patch series, as I'm going to
>> try to contribute more interesting things to the Linux kernel. :)
>
> As mentioned in the response to 01/28, these are all just fine as-is and
> don't need to be changed, sorry.
Okay. Doesn't matter that much it was the last patche series I was
going to do like these.
> Go bug the checkpatch author to remove
> this from the tool.
I intend to.
>
> sorry,
>
> greg k-h
>
Thanks,
Tracey D
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-10-06 2:06 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-10-06 1:15 [PATCH 00/28] Staging: comedi: Fixed numerous file so that it had little or no complaints from checkpatch.pl T Dent
2010-10-06 1:27 ` Greg KH
2010-10-06 1:52 ` Greg KH
2010-10-06 2:06 ` T Dent
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox