Re: Fix files with checkpatch errors/warnings

Javier Martinez Canillas <[email protected]>
Newsgroups org.kernel.vger.linux-newbie
Message-ID <CAAwP0s3cF3ydYx4FgUKOppk52yzQq-mPMXsKJxxP9OT=ZsE6EQ@mail.gmail.com>
On Thu, Dec 22, 2011 at 1:49 AM, Alejandro del Rio <[email protected]> wrote:
> Hi all,
>
> I'm completely new to linux kernel development and I would like to
> make a very first contribution by "cleaning" some files. I noticed
> that the script checkpatch.pl (inside scripts/ folder) report lots of
> style errors on some drivers.
> For example try running ./scripts/checkpatch.pl --file
> drivers/video/*, you will notice that most of the files contain style
> issues.
>
> My questions are: Is people actively fixing this? Maybe they are not
> very interested on fixing this typos... Otherwise, could I contribute
> by submitting patches on this?

Hi Alejandro,

The best to figure out is sending the patches ;-)

> And finally, if I do so, to who should I show this fixes?
>

Once you have the patch. You can use ./scripts/get_maintainer.pl to
know who is expecting to receive your patch.

It is only a suggestion not something to follow strictly. Common sense
will tell you if you need to cc someone else or not.

Hope it helps,

-- 
Javier Martínez Canillas
(+34) 682 39 81 69
Barcelona, Spain
--
To unsubscribe from this list: send the line "unsubscribe linux-newbie" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.linux-learn.org/faqs
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.