From: "Rick L. Vinyard, Jr." <rvinyard@cs.nmsu.edu>
To: felipe.balbi@nokia.com
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"krzysztof.h1@wp.pl" <krzysztof.h1@wp.pl>,
"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
"oliver@neukum.org" <oliver@neukum.org>,
"linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
"jkosina@suse.cz" <jkosina@suse.cz>
Subject: Re: [PATCH] Logitech G13 driver (fixed cc list --- ignore others)
Date: Mon, 14 Dec 2009 15:48:43 -0700 [thread overview]
Message-ID: <db910869762a655c288c762e82e83d3e.squirrel@intranet.cs.nmsu.edu> (raw)
In-Reply-To: <20091214220220.GB2052@nokia.com>
Hello,
Felipe Balbi wrote:
> On Mon, Dec 14, 2009 at 10:22:32PM +0100, ext Rick L. Vinyard Jr. wrote:
>>
>>Signed-off-by: Rick L. Vinyard, Jr <rvinyard@cs.nmsu.edu>
>
> there are plenty of style mistakes in this patch. the body message
> description which you put below should be above the signed-off-by line,
Fixed in the new patch labelled 0.0.2
> multi-lined comments are of the form:
>
> /*
> * multi lined comment
> * goes here
> */
>
Fixed
> remove the commented code like /* hid_debug = 0 */
>
Removed
> and some others.. please run your patch through scripts/checkpatch.pl
> --strict and fix the errors and warnings reported.
>
I ran it without the --strict option before. That's probably why I didn't
catch it. It's clean now with --strict.
>>+static int __init g13_init(void)
>>+{
>>+ pr_debug("g13 HID driver loaded");
>
> don't pr_debug() here.
>
Removed
>>+ return hid_register_driver(&g13_driver);
>>+}
>>+
>>+static void __exit g13_exit(void)
>>+{
>>+ pr_debug("g13 HID driver unloaded");
>
> nor here.
>
>>+ hid_unregister_driver(&g13_driver);
>>+}
Removed
Thanks for the input. Everything above is taken into account in v0.0.2 of
the patch.
---
Rick
next prev parent reply other threads:[~2009-12-14 22:48 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-14 21:22 [PATCH] Logitech G13 driver (fixed cc list --- ignore others) Rick L. Vinyard Jr.
2009-12-14 21:26 ` Rick L. Vinyard, Jr.
2009-12-14 22:02 ` Felipe Balbi
2009-12-14 22:48 ` Rick L. Vinyard, Jr. [this message]
2009-12-16 10:34 ` Pavel Machek
2009-12-16 14:08 ` Jiri Kosina
2010-01-04 22:23 ` Rick L. Vinyard, Jr.
[not found] ` <7f9100aac5f9b06ec78efff25c7a5a71.squirrel-2xSMGd46i5akveL4JqN78fZ8FUJU4vz8@public.gmane.org>
2010-01-04 22:48 ` Pavel Machek
2010-01-05 0:14 ` Jaya Kumar
[not found] ` <45a44e481001041614i35ceef84q5f12a068e2f0b97b-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-01-14 21:08 ` Miguel Ojeda
2010-01-14 21:48 ` Rick L. Vinyard, Jr.
[not found] ` <044387d146c2f91cb7f593736fcce28f.squirrel-2xSMGd46i5akveL4JqN78fZ8FUJU4vz8@public.gmane.org>
2010-01-14 22:34 ` Miguel Ojeda
2010-01-14 23:03 ` Rick L. Vinyard, Jr.
[not found] ` <8f404284c29a6e7736de49ede9a44a2c.squirrel-2xSMGd46i5akveL4JqN78fZ8FUJU4vz8@public.gmane.org>
2010-01-14 23:34 ` Miguel Ojeda
2010-01-05 9:52 ` Jiri Kosina
2010-01-04 23:57 ` Jaya Kumar
2010-01-07 15:59 ` Rick L. Vinyard, Jr.
2010-01-08 14:21 ` Giacomo A. Catenazzi
[not found] ` <4B473F64.2010203-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>
2010-01-08 16:45 ` Rick L. Vinyard, Jr.
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=db910869762a655c288c762e82e83d3e.squirrel@intranet.cs.nmsu.edu \
--to=rvinyard@cs.nmsu.edu \
--cc=akpm@linux-foundation.org \
--cc=felipe.balbi@nokia.com \
--cc=jkosina@suse.cz \
--cc=krzysztof.h1@wp.pl \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=oliver@neukum.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).