From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org, dim-tools@lists.freedesktop.org
Cc: jani.nikula@intel.com, Rodrigo Vivi <rodrigo.vivi@intel.com>
Subject: [DIM PATCH 2/3] dim: add checkpatch profiles to allow different checkpatch options
Date: Tue, 13 Mar 2018 13:30:09 +0200 [thread overview]
Message-ID: <20180313113010.13078-2-jani.nikula@intel.com> (raw)
In-Reply-To: <20180313113010.13078-1-jani.nikula@intel.com>
To reduce noise on CI checkpatch reports, we want to silence some
checkpatch warnings. Different branches may end up having different
rules, and users may want to get unfiltered results, so introduce
checkpatch profiles. Add some placeholder profiles to be filled later
on.
The idea is that CI would run 'dim checkpatch HEAD^ drm-intel' (or some
other commit range-ish as the case may be).
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
dim | 46 +++++++++++++++++++++++++++++++++++++++++-----
dim.rst | 9 +++++++--
2 files changed, 48 insertions(+), 7 deletions(-)
diff --git a/dim b/dim
index 81e2bc1511ac..4ba1c7ff490a 100755
--- a/dim
+++ b/dim
@@ -836,7 +836,7 @@ function apply_patch #patch_file
rv=1
fi
- if ! checkpatch_commit HEAD; then
+ if ! checkpatch_commit HEAD branch; then
rv=1
fi
if ! check_maintainer $branch HEAD; then
@@ -1358,12 +1358,47 @@ function check_maintainer
}
# $1 is the git sha1 to check
+# $2 is the checkpatch profile
function checkpatch_commit
{
- local commit rv checkpatch_options
+ local commit rv checkpatch_options profile profile_options
commit=$1
- checkpatch_options="-q --emacs --strict --show-types -"
+ profile=${2:-default}
+
+ # special branch profile maps branches to profiles
+ if [[ "$profile" = "branch" ]]; then
+ case "$(git_current_branch)" in
+ drm-intel-next-queued|drm-intel-next-fixes|drm-intel-fixes)
+ profile=drm-intel
+ ;;
+ drm-misc-next|drm-misc-next-fixes|drm-misc-fixes)
+ profile=drm-misc
+ ;;
+ *)
+ profile=default
+ ;;
+ esac
+ fi
+
+ # map profiles to checkpatch options
+ case "$profile" in
+ default)
+ profile_options=""
+ ;;
+ drm-misc)
+ profile_options=""
+ ;;
+ drm-intel)
+ profile_options=""
+ ;;
+ *)
+ echoerr "Unknown checkpatch profile $profile"
+ profile_options=""
+ ;;
+ esac
+
+ checkpatch_options="-q --emacs --strict --show-types $profile_options -"
git --no-pager log --oneline -1 $commit
if ! git show --pretty=email $commit |\
@@ -1430,12 +1465,13 @@ function dim_extract_next_fixes
dim_alias_cp=checkpatch
function dim_checkpatch
{
- local range rv
+ local range profile rv
range=$(rangeish "${1:-}")
+ profile=${2:-}
for commit in $(git rev-list --reverse $range); do
- if ! checkpatch_commit $commit; then
+ if ! checkpatch_commit $commit $profile; then
rv=1
fi
done
diff --git a/dim.rst b/dim.rst
index e2c5fd6e6d0a..cc930959e497 100644
--- a/dim.rst
+++ b/dim.rst
@@ -130,13 +130,18 @@ fixes *commit-ish*
Print the Fixes: and Cc: lines for the supplied *commit-ish* in the linux kernel
CodingStyle approved format.
-checkpatch [*commit-ish* [.. *commit-ish*]]
--------------------------------------------
+checkpatch [*commit-ish* [.. *commit-ish*]] [*profile*]
+-------------------------------------------------------
Runs the given commit range commit-ish..commit-ish through the check tools.
If no commit-ish is passed, defaults to HEAD^..HEAD. If one commit-ish is passed
instead of a range, the range commit-ish..HEAD is used.
+If profile is given, uses specific options for checkpatch error
+filtering. Current profiles are "default", "branch", "drm-intel", and
+"drm-misc". The "branch" profile maps the current git branch to the appropriate
+profile, or if the branch is not known, to "default".
+
sparse [*commit-ish* [.. *commit-ish*]]
---------------------------------------
Run sparse on the files changed by the given commit range.
--
2.11.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2018-03-13 11:30 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-13 11:30 [DIM PATCH 1/3] dim: cleanup checkpatch_commit Jani Nikula
2018-03-13 11:30 ` Jani Nikula [this message]
2018-03-13 11:48 ` [DIM PATCH 2/3] dim: add checkpatch profiles to allow different checkpatch options Daniel Vetter
2018-03-13 15:07 ` Jani Nikula
2018-03-14 6:29 ` Daniel Vetter
2018-03-14 8:57 ` Jani Nikula
2018-03-13 11:30 ` [DIM PATCH 3/3] dim: loosen some drm-intel checkpatch rules Jani Nikula
2018-03-13 11:55 ` Daniel Vetter
2018-03-14 9:03 ` Jani Nikula
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=20180313113010.13078-2-jani.nikula@intel.com \
--to=jani.nikula@intel.com \
--cc=dim-tools@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=rodrigo.vivi@intel.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.