diff options
| author | Simon Glass <sjg@chromium.org> | 2017-05-29 21:31:27 (GMT) |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2017-06-09 02:21:59 (GMT) |
| commit | db116cc8d095ec499ae748ea864fa0def54ca3d6 (patch) | |
| tree | 75115dace6e2dcf656fce1752914cbbac5925604 /tools | |
| parent | 04f7870635dff90ddc1b8465b5b8304c2d429a92 (diff) | |
| download | u-boot-db116cc8d095ec499ae748ea864fa0def54ca3d6.tar.xz | |
patman: Don't return the series in FixPatches()
There is no need for this function to return the same object that was
passed in. Drop the return value.
Signed-off-by: Simon Glass <sjg@chromium.org>
Tested-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/patman/patchstream.py | 1 | ||||
| -rwxr-xr-x | tools/patman/patman.py | 4 |
2 files changed, 2 insertions, 3 deletions
diff --git a/tools/patman/patchstream.py b/tools/patman/patchstream.py index 324c654..6098728 100644 --- a/tools/patman/patchstream.py +++ b/tools/patman/patchstream.py @@ -477,7 +477,6 @@ def FixPatches(series, fnames): print count += 1 print('Cleaned %d patches' % count) - return series def InsertCoverLetter(fname, series, count): """Inserts a cover letter with the required info into patch 0 diff --git a/tools/patman/patman.py b/tools/patman/patman.py index fdbee67..63da3f3 100755 --- a/tools/patman/patman.py +++ b/tools/patman/patman.py @@ -141,8 +141,8 @@ else: series) # Fix up the patch files to our liking, and insert the cover letter - series = patchstream.FixPatches(series, args) - if series and cover_fname and series.get('cover'): + patchstream.FixPatches(series, args) + if cover_fname and series.get('cover'): patchstream.InsertCoverLetter(cover_fname, series, options.count) # Do a few checks on the series |