From db116cc8d095ec499ae748ea864fa0def54ca3d6 Mon Sep 17 00:00:00 2001
From: Simon Glass <sjg@chromium.org>
Date: Mon, 29 May 2017 15:31:27 -0600
Subject: [PATCH] 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 | 1 -
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 324c65442f4..6098728aa11 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 fdbee67f559..63da3f32009 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
--
GitLab