Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1631128

Re: [PATCH] staging: ks7010: fix block comment style in ks_wlan_net.c

Show all headers | View raw

On Tue, Apr 25, 2017 at 10:59:25PM +0200, Ilia Sergachev wrote:
> This is a patch to the ks_wlan_net.c file that fixes up the following warning
> found by the checkpatch.pl tool:
> WARNING: Block comments use a trailing */ on a separate line

Good work resubmitting this following Greg's comments. May I suggest
some ways to hasten your success.

Note firstly, that I appreciate that writing the gitlog message is
very difficult to do well. I am by no means an expert.

 - You don't need to mention the file, especially not in the subject. We
   can see the files changed from the patch itself.
 - You should refrain from using 'this patch', the subject (as in
   subject, object, verb) is implicit.

See Documentation/process/submitting-patches.rst (Section 2: Describe
your changes). 

Example

Checkpatch emits WARNING: Block comments use a trailing */ on a
separate line.

Move trailing */ to separate line.

Good luck,
Tobin.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] staging: ks7010: fix block comment style in ks_wlan_net.c Ilia Sergachev <ilia.sergachev@unibas.ch> - 2017-04-25 23:10 +0200
  Re: [PATCH] staging: ks7010: fix block comment style in ks_wlan_net.c "Tobin C. Harding" <me@tobin.cc> - 2017-04-26 05:30 +0200
  [PATCH v2] Staging: ks7010: style fix Ilia Sergachev <ilia.sergachev@unibas.ch> - 2017-04-26 10:30 +0200
    Re: [PATCH v2] Staging: ks7010: style fix Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-04-26 10:40 +0200
  [PATCH v2] Staging: ks7010: style fix Ilia Sergachev <ilia.sergachev@unibas.ch> - 2017-04-26 10:50 +0200
    Re: [PATCH v2] Staging: ks7010: style fix "Tobin C. Harding" <me@tobin.cc> - 2017-04-26 12:40 +0200
      Re: [PATCH v2] Staging: ks7010: style fix Dan Carpenter <dan.carpenter@oracle.com> - 2017-04-26 12:50 +0200
  [PATCH v3] staging: ks7010: fix block comment style Ilia Sergachev <ilia.sergachev@unibas.ch> - 2017-04-26 13:10 +0200
    Re: [PATCH v3] staging: ks7010: fix block comment style "Tobin C. Harding" <me@tobin.cc> - 2017-04-27 01:00 +0200
    Re: [PATCH v3] staging: ks7010: fix block comment style Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-04-28 11:50 +0200
  [PATCH v4] staging: ks7010: fix block comment style Ilia Sergachev <ilia.sergachev@unibas.ch> - 2017-04-28 12:40 +0200
    Re: [PATCH v4] staging: ks7010: fix block comment style Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-04-28 14:30 +0200
      Re: [PATCH v4] staging: ks7010: fix block comment style Ilia Sergachev <ilia.sergachev@unibas.ch> - 2017-04-28 15:10 +0200
        Re: [PATCH v4] staging: ks7010: fix block comment style "Tobin C. Harding" <me@tobin.cc> - 2017-04-30 11:20 +0200

csiph-web