commit db116cc8d095ec499ae748ea864fa0def54ca3d6
author Simon Glass <sjg@chromium.org> Mon May 29 15:31:27 2017 -0600
committer Simon Glass <sjg@chromium.org> Thu Jun 08 20:21:59 2017 -0600
tree 75115dace6e2dcf656fce1752914cbbac5925604
parent 04f7870635dff90ddc1b8465b5b8304c2d429a92

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>

tools/patman/patchstream.py

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 @@
             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

tools/patman/patman.py

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 @@
                 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