linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Tobin C. Harding" <me@tobin.cc>
To: Joe Perches <joe@perches.com>
Cc: "Tobin C. Harding" <me@tobin.cc>,
	linux-kernel@vger.kernel.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Andy Whitcroft <apw@canonical.com>
Subject: [PATCH 0/4] checkpatch: update warnings
Date: Fri,  8 Dec 2017 10:27:47 +1100	[thread overview]
Message-ID: <1512689271-28051-1-git-send-email-me@tobin.cc> (raw)

This set is a combination of previous submissions

[PATCH v2 0/3] checkpatch: warn for use of %px
[PATCH] checkpatch: add check for tag Co-Developed-by

Although seemingly unrelated, they both patch checkpatch.pl and they
were both contributed to by Joe and myself. They are also tied together
by (previously incorrect) usage of the new Co-Developed-by tag.

Retains Joe's Ack'ed by tag on the %px patch from v2.

Retains Greg's Reviewed-by tag for the Co-Developed-by patch. The code
is totally different but the sentiment is the same, in this case, I
believe this is what the review was implying.

The Joe Perches patch is from LKML with minor adjustments after
testing. Needs review please.

As discussed, if this is all ok it will go through Tobin's tree.

thanks,
Tobin.

Joe Perches (1):
  checkpatch: add check for tag Co-Developed-by

Tobin C. Harding (3):
  checkpatch: add sub routine get_stat_real()
  checkpatch: add sub routine get_stat_here()
  checkpatch: warn for use of %px

 scripts/checkpatch.pl | 172 +++++++++++++++++++++++++++-----------------------
 1 file changed, 93 insertions(+), 79 deletions(-)

-- 
2.7.4

             reply	other threads:[~2017-12-07 23:28 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-07 23:27 Tobin C. Harding [this message]
2017-12-07 23:27 ` [PATCH 1/4] checkpatch: add sub routine get_stat_real() Tobin C. Harding
2017-12-07 23:27 ` [PATCH 2/4] checkpatch: add sub routine get_stat_here() Tobin C. Harding
2017-12-07 23:27 ` [PATCH 3/4] checkpatch: warn for use of %px Tobin C. Harding
2017-12-07 23:27 ` [PATCH 4/4] checkpatch: add check for tag Co-Developed-by Tobin C. Harding
2017-12-08  7:42   ` Greg Kroah-Hartman
2017-12-09  1:18     ` Tobin C. Harding

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=1512689271-28051-1-git-send-email-me@tobin.cc \
    --to=me@tobin.cc \
    --cc=apw@canonical.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=joe@perches.com \
    --cc=linux-kernel@vger.kernel.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).