Re: [PATCH] fixed coding style issues

[Baodong Chen]

On Wed, Jul 25, 2012 at 4:01 PM, Joe Perches <joe@xxxxxxxxxxx> wrote:
> On Wed, 2012-07-25 at 15:44 +0800, Baodong Chen wrote:
>> i want to participate in and to start from
>> fixing simple issues!
>
> That's fine Baodong.  Welcome.
>
>> how could i choose issues form checkpatch's output to find which
>> should be fixed which are
>> not?
>
> checkpatch output is merely a guide for submission of patches
> to conform to some generic "kernel style".  I think there are
> times when really ugly code should have a pass to make it more
> "kernel style" like.  Most of those cleanup type changes should
> be done only to code in the "drivers/staging" directory.
>
> I think most of the time, unless you are also fixing some other
> underlying code defect or shortcoming, checkpatch only cleanups
> to existing code are best avoided.
>
> I suggest for now, until you are really comfortable with the
> preferred style, you confine yourself to drivers/staging.
>
hi,Joe;
thank for your explaining!
ok, i will start from drivers/staging

>> or checkpatch.pl should be updated??
>
> You could work on that too.
>
> One possibility is to write a utility to take checkpatch output
> and generate patches automatically, or perhaps extend checkpatch
> itself to do that.  (ie: add a --fix option)
>
> cheers, Joe
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/