The patch titled
Subject: checkpatch-dt-bindings-should-be-a-separate-patch-v2
has been removed from the -mm tree. Its filename was
checkpatch-dt-bindings-should-be-a-separate-patch-v2.patch
This patch was dropped because it was folded into checkpatch-dt-bindings-should-be-a-separate-patch.patch
------------------------------------------------------
From: Rob Herring <robh@xxxxxxxxxx>
Subject: checkpatch-dt-bindings-should-be-a-separate-patch-v2
add doc pointer to warning, simplify logic
Link: http://lkml.kernel.org/r/20180810170513.26284-1-robh@xxxxxxxxxx
Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
Cc: Andy Whitcroft <apw@xxxxxxxxxxxxx>
Cc: Joe Perches <joe@xxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
scripts/checkpatch.pl | 22 +++++++---------------
1 file changed, 7 insertions(+), 15 deletions(-)
--- a/scripts/checkpatch.pl~checkpatch-dt-bindings-should-be-a-separate-patch-v2
+++ a/scripts/checkpatch.pl
@@ -2503,23 +2503,15 @@ sub process {
}
$checklicenseline = 1;
- if ($realfile !~ /MAINTAINERS/) {
- my $mixed = 0;
- if ($realfile =~ /(Documentation\/devicetree|include\/dt-bindings).*/) {
- if ($is_binding_patch == 0) {
- $mixed = 1;
- }
- $is_binding_patch = 1;
- } else {
- if ($is_binding_patch == 1) {
- $mixed = 1;
- }
- $is_binding_patch = 0;
- }
+ if ($realfile !~ /^MAINTAINERS/) {
+ my $last_binding_patch = $is_binding_patch;
+
+ $is_binding_patch = ($realfile =~ m@^(?:Documentation/devicetree/|include/dt-bindings/)@);
- if ($mixed == 1) {
+ if (($last_binding_patch != -1) &&
+ ($last_binding_patch ^ $is_binding_patch)) {
WARN("DT_SPLIT_BINDING_PATCH",
- "DT binding docs and includes should be a separate patch\n");
+ "DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.txt");
}
}
_
Patches currently in -mm which might be from robh@xxxxxxxxxx are
checkpatch-dt-bindings-should-be-a-separate-patch.patch
checkpatch-dt-bindings-should-be-a-separate-patch-v3.patch
