Tom Rini | 10e4779 | 2018-05-06 17:58:06 -0400 | [diff] [blame] | 1 | # SPDX-License-Identifier: GPL-2.0+ |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 2 | # Copyright (c) 2011 The Chromium OS Authors. |
| 3 | # |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 4 | |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 5 | import collections |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 6 | import os |
| 7 | import re |
Vadim Bendebury | c549f08 | 2013-01-09 16:00:10 +0000 | [diff] [blame] | 8 | import sys |
Simon Glass | a997ea5 | 2020-04-17 18:09:04 -0600 | [diff] [blame] | 9 | |
| 10 | from patman import command |
| 11 | from patman import gitutil |
| 12 | from patman import terminal |
| 13 | from patman import tools |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 14 | |
| 15 | def FindCheckPatch(): |
Doug Anderson | 4baede0 | 2012-11-26 15:23:23 +0000 | [diff] [blame] | 16 | top_level = gitutil.GetTopLevel() |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 17 | try_list = [ |
| 18 | os.getcwd(), |
| 19 | os.path.join(os.getcwd(), '..', '..'), |
Doug Anderson | 4baede0 | 2012-11-26 15:23:23 +0000 | [diff] [blame] | 20 | os.path.join(top_level, 'tools'), |
| 21 | os.path.join(top_level, 'scripts'), |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 22 | '%s/bin' % os.getenv('HOME'), |
| 23 | ] |
| 24 | # Look in current dir |
| 25 | for path in try_list: |
| 26 | fname = os.path.join(path, 'checkpatch.pl') |
| 27 | if os.path.isfile(fname): |
| 28 | return fname |
| 29 | |
| 30 | # Look upwwards for a Chrome OS tree |
| 31 | while not os.path.ismount(path): |
| 32 | fname = os.path.join(path, 'src', 'third_party', 'kernel', 'files', |
| 33 | 'scripts', 'checkpatch.pl') |
| 34 | if os.path.isfile(fname): |
| 35 | return fname |
| 36 | path = os.path.dirname(path) |
Vadim Bendebury | c549f08 | 2013-01-09 16:00:10 +0000 | [diff] [blame] | 37 | |
Masahiro Yamada | 880828d | 2014-08-16 00:59:26 +0900 | [diff] [blame] | 38 | sys.exit('Cannot find checkpatch.pl - please put it in your ' + |
| 39 | '~/bin directory or use --no-check') |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 40 | |
| 41 | def CheckPatch(fname, verbose=False): |
| 42 | """Run checkpatch.pl on a file. |
| 43 | |
| 44 | Returns: |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 45 | namedtuple containing: |
| 46 | ok: False=failure, True=ok |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 47 | problems: List of problems, each a dict: |
| 48 | 'type'; error or warning |
| 49 | 'msg': text message |
| 50 | 'file' : filename |
| 51 | 'line': line number |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 52 | errors: Number of errors |
| 53 | warnings: Number of warnings |
| 54 | checks: Number of checks |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 55 | lines: Number of lines |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 56 | stdout: Full output of checkpatch |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 57 | """ |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 58 | fields = ['ok', 'problems', 'errors', 'warnings', 'checks', 'lines', |
| 59 | 'stdout'] |
| 60 | result = collections.namedtuple('CheckPatchResult', fields) |
| 61 | result.ok = False |
| 62 | result.errors, result.warning, result.checks = 0, 0, 0 |
| 63 | result.lines = 0 |
| 64 | result.problems = [] |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 65 | chk = FindCheckPatch() |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 66 | item = {} |
Simon Glass | c55e056 | 2016-07-25 18:59:00 -0600 | [diff] [blame] | 67 | result.stdout = command.Output(chk, '--no-tree', fname, |
| 68 | raise_on_error=False) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 69 | #pipe = subprocess.Popen(cmd, stdout=subprocess.PIPE) |
| 70 | #stdout, stderr = pipe.communicate() |
| 71 | |
| 72 | # total: 0 errors, 0 warnings, 159 lines checked |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 73 | # or: |
| 74 | # total: 0 errors, 2 warnings, 7 checks, 473 lines checked |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 75 | re_stats = re.compile('total: (\\d+) errors, (\d+) warnings, (\d+)') |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 76 | re_stats_full = re.compile('total: (\\d+) errors, (\d+) warnings, (\d+)' |
| 77 | ' checks, (\d+)') |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 78 | re_ok = re.compile('.*has no obvious style problems') |
| 79 | re_bad = re.compile('.*has style problems, please review') |
| 80 | re_error = re.compile('ERROR: (.*)') |
| 81 | re_warning = re.compile('WARNING: (.*)') |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 82 | re_check = re.compile('CHECK: (.*)') |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 83 | re_file = re.compile('#\d+: FILE: ([^:]*):(\d+):') |
| 84 | |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 85 | for line in result.stdout.splitlines(): |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 86 | if verbose: |
Paul Burton | c393134 | 2016-09-27 16:03:50 +0100 | [diff] [blame] | 87 | print(line) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 88 | |
| 89 | # A blank line indicates the end of a message |
| 90 | if not line and item: |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 91 | result.problems.append(item) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 92 | item = {} |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 93 | match = re_stats_full.match(line) |
| 94 | if not match: |
| 95 | match = re_stats.match(line) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 96 | if match: |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 97 | result.errors = int(match.group(1)) |
| 98 | result.warnings = int(match.group(2)) |
| 99 | if len(match.groups()) == 4: |
| 100 | result.checks = int(match.group(3)) |
| 101 | result.lines = int(match.group(4)) |
| 102 | else: |
| 103 | result.lines = int(match.group(3)) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 104 | elif re_ok.match(line): |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 105 | result.ok = True |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 106 | elif re_bad.match(line): |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 107 | result.ok = False |
| 108 | err_match = re_error.match(line) |
| 109 | warn_match = re_warning.match(line) |
| 110 | file_match = re_file.match(line) |
| 111 | check_match = re_check.match(line) |
| 112 | if err_match: |
| 113 | item['msg'] = err_match.group(1) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 114 | item['type'] = 'error' |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 115 | elif warn_match: |
| 116 | item['msg'] = warn_match.group(1) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 117 | item['type'] = 'warning' |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 118 | elif check_match: |
| 119 | item['msg'] = check_match.group(1) |
| 120 | item['type'] = 'check' |
| 121 | elif file_match: |
| 122 | item['file'] = file_match.group(1) |
| 123 | item['line'] = int(file_match.group(2)) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 124 | |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 125 | return result |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 126 | |
| 127 | def GetWarningMsg(col, msg_type, fname, line, msg): |
| 128 | '''Create a message for a given file/line |
| 129 | |
| 130 | Args: |
| 131 | msg_type: Message type ('error' or 'warning') |
| 132 | fname: Filename which reports the problem |
| 133 | line: Line number where it was noticed |
| 134 | msg: Message to report |
| 135 | ''' |
| 136 | if msg_type == 'warning': |
| 137 | msg_type = col.Color(col.YELLOW, msg_type) |
| 138 | elif msg_type == 'error': |
| 139 | msg_type = col.Color(col.RED, msg_type) |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 140 | elif msg_type == 'check': |
| 141 | msg_type = col.Color(col.MAGENTA, msg_type) |
Simon Glass | c004a5f | 2017-01-17 16:52:23 -0700 | [diff] [blame] | 142 | return '%s:%d: %s: %s\n' % (fname, line, msg_type, msg) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 143 | |
| 144 | def CheckPatches(verbose, args): |
| 145 | '''Run the checkpatch.pl script on each patch''' |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 146 | error_count, warning_count, check_count = 0, 0, 0 |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 147 | col = terminal.Color() |
| 148 | |
| 149 | for fname in args: |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 150 | result = CheckPatch(fname, verbose) |
| 151 | if not result.ok: |
| 152 | error_count += result.errors |
| 153 | warning_count += result.warnings |
| 154 | check_count += result.checks |
Paul Burton | c393134 | 2016-09-27 16:03:50 +0100 | [diff] [blame] | 155 | print('%d errors, %d warnings, %d checks for %s:' % (result.errors, |
| 156 | result.warnings, result.checks, col.Color(col.BLUE, fname))) |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 157 | if (len(result.problems) != result.errors + result.warnings + |
| 158 | result.checks): |
Paul Burton | c393134 | 2016-09-27 16:03:50 +0100 | [diff] [blame] | 159 | print("Internal error: some problems lost") |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 160 | for item in result.problems: |
Simon Glass | c004a5f | 2017-01-17 16:52:23 -0700 | [diff] [blame] | 161 | sys.stderr.write( |
| 162 | GetWarningMsg(col, item.get('type', '<unknown>'), |
Simon Glass | eb1526c | 2012-09-27 15:33:46 +0000 | [diff] [blame] | 163 | item.get('file', '<unknown>'), |
Paul Burton | c393134 | 2016-09-27 16:03:50 +0100 | [diff] [blame] | 164 | item.get('line', 0), item.get('msg', 'message'))) |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 165 | print |
Paul Burton | c393134 | 2016-09-27 16:03:50 +0100 | [diff] [blame] | 166 | #print(stdout) |
Simon Glass | 0495abf | 2013-03-26 13:09:39 +0000 | [diff] [blame] | 167 | if error_count or warning_count or check_count: |
| 168 | str = 'checkpatch.pl found %d error(s), %d warning(s), %d checks(s)' |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 169 | color = col.GREEN |
| 170 | if warning_count: |
| 171 | color = col.YELLOW |
| 172 | if error_count: |
| 173 | color = col.RED |
Paul Burton | c393134 | 2016-09-27 16:03:50 +0100 | [diff] [blame] | 174 | print(col.Color(color, str % (error_count, warning_count, check_count))) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 175 | return False |
| 176 | return True |