patman: Rename test.py to test_checkpatch.py

These tests check checkpatch.pl operation and can server as our tests for
the U-Boot-specific updates to that script. Rename the file and update
comments to indicate this.

Signed-off-by: Simon Glass <sjg@chromium.org>

1 files changed, 0 insertions, 287 deletions

diff --git a/tools/patman/test.py b/tools/patman/test.py
deleted file mode 100644
index e7f709e34c7..00000000000
--- a/tools/patman/test.py
+++ /dev/null
@@ -1,287 +0,0 @@
-# -*- coding: utf-8 -*-
-# SPDX-License-Identifier: GPL-2.0+
-#
-# Copyright (c) 2011 The Chromium OS Authors.
-#
-
-import os
-import tempfile
-import unittest
-
-from patman import checkpatch
-from patman import gitutil
-from patman import patchstream
-from patman import series
-from patman import commit
-
-
-class TestPatch(unittest.TestCase):
-    """Test this program
-
-    TODO: Write tests for the rest of the functionality
-    """
-
-    def testBasic(self):
-        """Test basic filter operation"""
-        data='''
-
-From 656c9a8c31fa65859d924cd21da920d6ba537fad Mon Sep 17 00:00:00 2001
-From: Simon Glass <sjg@chromium.org>
-Date: Thu, 28 Apr 2011 09:58:51 -0700
-Subject: [PATCH (resend) 3/7] Tegra2: Add more clock support
-
-This adds functions to enable/disable clocks and reset to on-chip peripherals.
-
-cmd/pci.c:152:11: warning: format ‘%llx’ expects argument of type
-   ‘long long unsigned int’, but argument 3 has type
-   ‘u64 {aka long unsigned int}’ [-Wformat=]
-
-BUG=chromium-os:13875
-TEST=build U-Boot for Seaboard, boot
-
-Change-Id: I80fe1d0c0b7dd10aa58ce5bb1d9290b6664d5413
-
-Review URL: http://codereview.chromium.org/6900006
-
-Signed-off-by: Simon Glass <sjg@chromium.org>
----
- arch/arm/cpu/armv7/tegra2/Makefile         |    2 +-
- arch/arm/cpu/armv7/tegra2/ap20.c           |   57 ++----
- arch/arm/cpu/armv7/tegra2/clock.c          |  163 +++++++++++++++++
-'''
-        expected='''Message-Id: <19991231235959.0.I80fe1d0c0b7dd10aa58ce5bb1d9290b6664d5413@changeid>
-
-
-From 656c9a8c31fa65859d924cd21da920d6ba537fad Mon Sep 17 00:00:00 2001
-From: Simon Glass <sjg@chromium.org>
-Date: Thu, 28 Apr 2011 09:58:51 -0700
-Subject: [PATCH (resend) 3/7] Tegra2: Add more clock support
-
-This adds functions to enable/disable clocks and reset to on-chip peripherals.
-
-cmd/pci.c:152:11: warning: format ‘%llx’ expects argument of type
-   ‘long long unsigned int’, but argument 3 has type
-   ‘u64 {aka long unsigned int}’ [-Wformat=]
-
-Signed-off-by: Simon Glass <sjg@chromium.org>
----
-
- arch/arm/cpu/armv7/tegra2/Makefile         |    2 +-
- arch/arm/cpu/armv7/tegra2/ap20.c           |   57 ++----
- arch/arm/cpu/armv7/tegra2/clock.c          |  163 +++++++++++++++++
-'''
-        out = ''
-        inhandle, inname = tempfile.mkstemp()
-        infd = os.fdopen(inhandle, 'w', encoding='utf-8')
-        infd.write(data)
-        infd.close()
-
-        exphandle, expname = tempfile.mkstemp()
-        expfd = os.fdopen(exphandle, 'w', encoding='utf-8')
-        expfd.write(expected)
-        expfd.close()
-
-        # Normally by the time we call FixPatch we've already collected
-        # metadata.  Here, we haven't, but at least fake up something.
-        # Set the "count" to -1 which tells FixPatch to use a bogus/fixed
-        # time for generating the Message-Id.
-        com = commit.Commit('')
-        com.change_id = 'I80fe1d0c0b7dd10aa58ce5bb1d9290b6664d5413'
-        com.count = -1
-
-        patchstream.FixPatch(None, inname, series.Series(), com)
-
-        rc = os.system('diff -u %s %s' % (inname, expname))
-        self.assertEqual(rc, 0)
-
-        os.remove(inname)
-        os.remove(expname)
-
-    def GetData(self, data_type):
-        data='''From 4924887af52713cabea78420eff03badea8f0035 Mon Sep 17 00:00:00 2001
-From: Simon Glass <sjg@chromium.org>
-Date: Thu, 7 Apr 2011 10:14:41 -0700
-Subject: [PATCH 1/4] Add microsecond boot time measurement
-
-This defines the basics of a new boot time measurement feature. This allows
-logging of very accurate time measurements as the boot proceeds, by using
-an available microsecond counter.
-
-%s
----
- README              |   11 ++++++++
- MAINTAINERS         |    3 ++
- common/bootstage.c  |   50 ++++++++++++++++++++++++++++++++++++
- include/bootstage.h |   71 +++++++++++++++++++++++++++++++++++++++++++++++++++
- include/common.h    |    8 ++++++
- 5 files changed, 141 insertions(+), 0 deletions(-)
- create mode 100644 common/bootstage.c
- create mode 100644 include/bootstage.h
-
-diff --git a/README b/README
-index 6f3748d..f9e4e65 100644
---- a/README
-+++ b/README
-@@ -2026,6 +2026,17 @@ The following options need to be configured:
- 		example, some LED's) on your board. At the moment,
- 		the following checkpoints are implemented:
-
-+- Time boot progress
-+		CONFIG_BOOTSTAGE
-+
-+		Define this option to enable microsecond boot stage timing
-+		on supported platforms. For this to work your platform
-+		needs to define a function timer_get_us() which returns the
-+		number of microseconds since reset. This would normally
-+		be done in your SOC or board timer.c file.
-+
-+		You can add calls to bootstage_mark() to set time markers.
-+
- - Standalone program support:
- 		CONFIG_STANDALONE_LOAD_ADDR
-
-diff --git a/MAINTAINERS b/MAINTAINERS
-index b167b028ec..beb7dc634f 100644
---- a/MAINTAINERS
-+++ b/MAINTAINERS
-@@ -474,3 +474,8 @@ S:	Maintained
- T:	git git://git.denx.de/u-boot.git
- F:	*
- F:	*/
-+
-+BOOTSTAGE
-+M:	Simon Glass <sjg@chromium.org>
-+L:	u-boot@lists.denx.de
-+F:	common/bootstage.c
-diff --git a/common/bootstage.c b/common/bootstage.c
-new file mode 100644
-index 0000000..2234c87
---- /dev/null
-+++ b/common/bootstage.c
-@@ -0,0 +1,37 @@
-+%s
-+/*
-+ * Copyright (c) 2011, Google Inc. All rights reserved.
-+ *
-+ */
-+
-+/*
-+ * This module records the progress of boot and arbitrary commands, and
-+ * permits accurate timestamping of each. The records can optionally be
-+ * passed to kernel in the ATAGs
-+ */
-+
-+#include <common.h>
-+
-+struct bootstage_record {
-+	u32 time_us;
-+	const char *name;
-+};
-+
-+static struct bootstage_record record[BOOTSTAGE_COUNT];
-+
-+u32 bootstage_mark(enum bootstage_id id, const char *name)
-+{
-+	struct bootstage_record *rec = &record[id];
-+
-+	/* Only record the first event for each */
-+%sif (!rec->name) {
-+		rec->time_us = (u32)timer_get_us();
-+		rec->name = name;
-+	}
-+	if (!rec->name &&
-+	%ssomething_else) {
-+		rec->time_us = (u32)timer_get_us();
-+		rec->name = name;
-+	}
-+%sreturn rec->time_us;
-+}
---
-1.7.3.1
-'''
-        signoff = 'Signed-off-by: Simon Glass <sjg@chromium.org>\n'
-        license = '// SPDX-License-Identifier: GPL-2.0+'
-        tab = '	'
-        indent = '    '
-        if data_type == 'good':
-            pass
-        elif data_type == 'no-signoff':
-            signoff = ''
-        elif data_type == 'no-license':
-            license = ''
-        elif data_type == 'spaces':
-            tab = '   '
-        elif data_type == 'indent':
-            indent = tab
-        else:
-            print('not implemented')
-        return data % (signoff, license, tab, indent, tab)
-
-    def SetupData(self, data_type):
-        inhandle, inname = tempfile.mkstemp()
-        infd = os.fdopen(inhandle, 'w')
-        data = self.GetData(data_type)
-        infd.write(data)
-        infd.close()
-        return inname
-
-    def testGood(self):
-        """Test checkpatch operation"""
-        inf = self.SetupData('good')
-        result = checkpatch.CheckPatch(inf)
-        self.assertEqual(result.ok, True)
-        self.assertEqual(result.problems, [])
-        self.assertEqual(result.errors, 0)
-        self.assertEqual(result.warnings, 0)
-        self.assertEqual(result.checks, 0)
-        self.assertEqual(result.lines, 62)
-        os.remove(inf)
-
-    def testNoSignoff(self):
-        inf = self.SetupData('no-signoff')
-        result = checkpatch.CheckPatch(inf)
-        self.assertEqual(result.ok, False)
-        self.assertEqual(len(result.problems), 1)
-        self.assertEqual(result.errors, 1)
-        self.assertEqual(result.warnings, 0)
-        self.assertEqual(result.checks, 0)
-        self.assertEqual(result.lines, 62)
-        os.remove(inf)
-
-    def testNoLicense(self):
-        inf = self.SetupData('no-license')
-        result = checkpatch.CheckPatch(inf)
-        self.assertEqual(result.ok, False)
-        self.assertEqual(len(result.problems), 1)
-        self.assertEqual(result.errors, 0)
-        self.assertEqual(result.warnings, 1)
-        self.assertEqual(result.checks, 0)
-        self.assertEqual(result.lines, 62)
-        os.remove(inf)
-
-    def testSpaces(self):
-        inf = self.SetupData('spaces')
-        result = checkpatch.CheckPatch(inf)
-        self.assertEqual(result.ok, False)
-        self.assertEqual(len(result.problems), 3)
-        self.assertEqual(result.errors, 0)
-        self.assertEqual(result.warnings, 3)
-        self.assertEqual(result.checks, 0)
-        self.assertEqual(result.lines, 62)
-        os.remove(inf)
-
-    def testIndent(self):
-        inf = self.SetupData('indent')
-        result = checkpatch.CheckPatch(inf)
-        self.assertEqual(result.ok, False)
-        self.assertEqual(len(result.problems), 1)
-        self.assertEqual(result.errors, 0)
-        self.assertEqual(result.warnings, 0)
-        self.assertEqual(result.checks, 1)
-        self.assertEqual(result.lines, 62)
-        os.remove(inf)
-
-
-if __name__ == "__main__":
-    unittest.main()
-    gitutil.RunTests()