fuse: remove unused arg in fuse_write_file_get()

[linux-2.6-microblaze.git] / scripts / checkpatch.pl

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 23697a6..461d422 100755 (executable)
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -1084,10 +1084,10 @@ sub is_maintained_obsolete {
 sub is_SPDX_License_valid {
        my ($license) = @_;
 
-       return 1 if (!$tree || which("python") eq "" || !(-e "$root/scripts/spdxcheck.py") || !(-e "$gitroot"));
+       return 1 if (!$tree || which("python3") eq "" || !(-x "$root/scripts/spdxcheck.py") || !(-e "$gitroot"));
 
        my $root_path = abs_path($root);
-       my $status = `cd "$root_path"; echo "$license" | python scripts/spdxcheck.py -`;
+       my $status = `cd "$root_path"; echo "$license" | scripts/spdxcheck.py -`;
        return 0 if ($status ne "");
        return 1;
 }
@@ -5361,9 +5361,13 @@ sub process {
                        }
                }
 
-#goto labels aren't indented, allow a single space however
-               if ($line=~/^.\s+[A-Za-z\d_]+:(?![0-9]+)/ and
-                  !($line=~/^. [A-Za-z\d_]+:/) and !($line=~/^.\s+default:/)) {
+# check that goto labels aren't indented (allow a single space indentation)
+# and ignore bitfield definitions like foo:1
+# Strictly, labels can have whitespace after the identifier and before the :
+# but this is not allowed here as many ?: uses would appear to be labels
+               if ($sline =~ /^.\s+[A-Za-z_][A-Za-z\d_]*:(?!\s*\d+)/ &&
+                   $sline !~ /^. [A-Za-z\d_][A-Za-z\d_]*:/ &&
+                   $sline !~ /^.\s+default:/) {
                        if (WARN("INDENTED_LABEL",
                                 "labels should not be indented\n" . $herecurr) &&
                            $fix) {
@@ -5458,7 +5462,7 @@ sub process {
 # Return of what appears to be an errno should normally be negative
                if ($sline =~ /\breturn(?:\s*\(+\s*|\s+)(E[A-Z]+)(?:\s*\)+\s*|\s*)[;:,]/) {
                        my $name = $1;
-                       if ($name ne 'EOF' && $name ne 'ERROR') {
+                       if ($name ne 'EOF' && $name ne 'ERROR' && $name !~ /^EPOLL/) {
                                WARN("USE_NEGATIVE_ERRNO",
                                     "return of an errno should typically be negative (ie: return -$1)\n" . $herecurr);
                        }