History

checkpatch: Fix false DT_SPLIT_BINDING_PATCH warnings

Patches which both remove and add/modify DT binding files are incorrectly flagged as needing to split the patch. The issue is the check sees "dev/null" as one of the files in the patch which is not a DT binding file. Add "dev/null" to the skipped files.

Rob Herring (Arm) committed 9 months ago

Commit a45ff9d1 parent 6de23f81 file · +1 −1Browse files

Showing the changes to scripts/checkpatch.pl (1 of 1 file). Show all

Unified · Split

scripts/checkpatch.pl+1−1
@@ -2928,7 +2928,7 @@ sub process { · 2927 unmodified lines
29282928 }
29292929 $checklicenseline = 1;
29302930
2931− if ($realfile !~ /^MAINTAINERS/) {
2931+ if ($realfile !~ /^(MAINTAINERS|dev\/null)/) {
29322932 my $last_binding_patch = $is_binding_patch;
29332933
29342934 $is_binding_patch = () = $realfile =~ m@^(?:Documentation/devicetree/|include/dt-bindings/)@;