tools/patman/test.py

   1 # -*- coding: utf-8 -*-
   2 # SPDX-License-Identifier: GPL-2.0+
   3 #
   4 # Copyright (c) 2011 The Chromium OS Authors.
   5 #
   6
   7 import os
   8 import tempfile
   9 import unittest
  10
  11 from patman import checkpatch
  12 from patman import gitutil
  13 from patman import patchstream
  14 from patman import series
  15 from patman import commit
  16
  17
  18 class TestPatch(unittest.TestCase):
  19     """Test this program
  20
  21     TODO: Write tests for the rest of the functionality
  22     """
  23
  24     def testBasic(self):
  25         """Test basic filter operation"""
  26         data='''
  27
  28 From 656c9a8c31fa65859d924cd21da920d6ba537fad Mon Sep 17 00:00:00 2001
  29 From: Simon Glass <sjg@chromium.org>
  30 Date: Thu, 28 Apr 2011 09:58:51 -0700
  31 Subject: [PATCH (resend) 3/7] Tegra2: Add more clock support
  32
  33 This adds functions to enable/disable clocks and reset to on-chip peripherals.
  34
  35 cmd/pci.c:152:11: warning: format ‘%llx’ expects argument of type
  36    ‘long long unsigned int’, but argument 3 has type
  37    ‘u64 {aka long unsigned int}’ [-Wformat=]
  38
  39 BUG=chromium-os:13875
  40 TEST=build U-Boot for Seaboard, boot
  41
  42 Change-Id: I80fe1d0c0b7dd10aa58ce5bb1d9290b6664d5413
  43
  44 Review URL: http://codereview.chromium.org/6900006
  45
  46 Signed-off-by: Simon Glass <sjg@chromium.org>
  47 ---
  48  arch/arm/cpu/armv7/tegra2/Makefile         |    2 +-
  49  arch/arm/cpu/armv7/tegra2/ap20.c           |   57 ++----
  50  arch/arm/cpu/armv7/tegra2/clock.c          |  163 +++++++++++++++++
  51 '''
  52         expected='''Message-Id: <19991231235959.0.I80fe1d0c0b7dd10aa58ce5bb1d9290b6664d5413@changeid>
  53
  54
  55 From 656c9a8c31fa65859d924cd21da920d6ba537fad Mon Sep 17 00:00:00 2001
  56 From: Simon Glass <sjg@chromium.org>
  57 Date: Thu, 28 Apr 2011 09:58:51 -0700
  58 Subject: [PATCH (resend) 3/7] Tegra2: Add more clock support
  59
  60 This adds functions to enable/disable clocks and reset to on-chip peripherals.
  61
  62 cmd/pci.c:152:11: warning: format ‘%llx’ expects argument of type
  63    ‘long long unsigned int’, but argument 3 has type
  64    ‘u64 {aka long unsigned int}’ [-Wformat=]
  65
  66 Signed-off-by: Simon Glass <sjg@chromium.org>
  67 ---
  68
  69  arch/arm/cpu/armv7/tegra2/Makefile         |    2 +-
  70  arch/arm/cpu/armv7/tegra2/ap20.c           |   57 ++----
  71  arch/arm/cpu/armv7/tegra2/clock.c          |  163 +++++++++++++++++
  72 '''
  73         out = ''
  74         inhandle, inname = tempfile.mkstemp()
  75         infd = os.fdopen(inhandle, 'w', encoding='utf-8')
  76         infd.write(data)
  77         infd.close()
  78
  79         exphandle, expname = tempfile.mkstemp()
  80         expfd = os.fdopen(exphandle, 'w', encoding='utf-8')
  81         expfd.write(expected)
  82         expfd.close()
  83
  84         # Normally by the time we call FixPatch we've already collected
  85         # metadata.  Here, we haven't, but at least fake up something.
  86         # Set the "count" to -1 which tells FixPatch to use a bogus/fixed
  87         # time for generating the Message-Id.
  88         com = commit.Commit('')
  89         com.change_id = 'I80fe1d0c0b7dd10aa58ce5bb1d9290b6664d5413'
  90         com.count = -1
  91
  92         patchstream.FixPatch(None, inname, series.Series(), com)
  93
  94         rc = os.system('diff -u %s %s' % (inname, expname))
  95         self.assertEqual(rc, 0)
  96
  97         os.remove(inname)
  98         os.remove(expname)
  99
 100     def GetData(self, data_type):
 101         data='''From 4924887af52713cabea78420eff03badea8f0035 Mon Sep 17 00:00:00 2001
 102 From: Simon Glass <sjg@chromium.org>
 103 Date: Thu, 7 Apr 2011 10:14:41 -0700
 104 Subject: [PATCH 1/4] Add microsecond boot time measurement
 105
 106 This defines the basics of a new boot time measurement feature. This allows
 107 logging of very accurate time measurements as the boot proceeds, by using
 108 an available microsecond counter.
 109
 110 %s
 111 ---
 112  README              |   11 ++++++++
 113  MAINTAINERS         |    3 ++
 114  common/bootstage.c  |   50 ++++++++++++++++++++++++++++++++++++
 115  include/bootstage.h |   71 +++++++++++++++++++++++++++++++++++++++++++++++++++
 116  include/common.h    |    8 ++++++
 117  5 files changed, 141 insertions(+), 0 deletions(-)
 118  create mode 100644 common/bootstage.c
 119  create mode 100644 include/bootstage.h
 120
 121 diff --git a/README b/README
 122 index 6f3748d..f9e4e65 100644
 123 --- a/README
 124 +++ b/README
 125 @@ -2026,6 +2026,17 @@ The following options need to be configured:
 126                 example, some LED's) on your board. At the moment,
 127                 the following checkpoints are implemented:
 128
 129 +- Time boot progress
 130 +               CONFIG_BOOTSTAGE
 131 +
 132 +               Define this option to enable microsecond boot stage timing
 133 +               on supported platforms. For this to work your platform
 134 +               needs to define a function timer_get_us() which returns the
 135 +               number of microseconds since reset. This would normally
 136 +               be done in your SOC or board timer.c file.
 137 +
 138 +               You can add calls to bootstage_mark() to set time markers.
 139 +
 140  - Standalone program support:
 141                 CONFIG_STANDALONE_LOAD_ADDR
 142
 143 diff --git a/MAINTAINERS b/MAINTAINERS
 144 index b167b028ec..beb7dc634f 100644
 145 --- a/MAINTAINERS
 146 +++ b/MAINTAINERS
 147 @@ -474,3 +474,8 @@ S:  Maintained
 148  T:     git git://git.denx.de/u-boot.git
 149  F:     *
 150  F:     */
 151 +
 152 +BOOTSTAGE
 153 +M:     Simon Glass <sjg@chromium.org>
 154 +L:     u-boot@lists.denx.de
 155 +F:     common/bootstage.c
 156 diff --git a/common/bootstage.c b/common/bootstage.c
 157 new file mode 100644
 158 index 0000000..2234c87
 159 --- /dev/null
 160 +++ b/common/bootstage.c
 161 @@ -0,0 +1,37 @@
 162 +%s
 163 +/*
 164 + * Copyright (c) 2011, Google Inc. All rights reserved.
 165 + *
 166 + */
 167 +
 168 +/*
 169 + * This module records the progress of boot and arbitrary commands, and
 170 + * permits accurate timestamping of each. The records can optionally be
 171 + * passed to kernel in the ATAGs
 172 + */
 173 +
 174 +#include <common.h>
 175 +
 176 +struct bootstage_record {
 177 +       u32 time_us;
 178 +       const char *name;
 179 +};
 180 +
 181 +static struct bootstage_record record[BOOTSTAGE_COUNT];
 182 +
 183 +u32 bootstage_mark(enum bootstage_id id, const char *name)
 184 +{
 185 +       struct bootstage_record *rec = &record[id];
 186 +
 187 +       /* Only record the first event for each */
 188 +%sif (!rec->name) {
 189 +               rec->time_us = (u32)timer_get_us();
 190 +               rec->name = name;
 191 +       }
 192 +       if (!rec->name &&
 193 +       %ssomething_else) {
 194 +               rec->time_us = (u32)timer_get_us();
 195 +               rec->name = name;
 196 +       }
 197 +%sreturn rec->time_us;
 198 +}
 199 --
 200 1.7.3.1
 201 '''
 202         signoff = 'Signed-off-by: Simon Glass <sjg@chromium.org>\n'
 203         license = '// SPDX-License-Identifier: GPL-2.0+'
 204         tab = ' '
 205         indent = '    '
 206         if data_type == 'good':
 207             pass
 208         elif data_type == 'no-signoff':
 209             signoff = ''
 210         elif data_type == 'no-license':
 211             license = ''
 212         elif data_type == 'spaces':
 213             tab = '   '
 214         elif data_type == 'indent':
 215             indent = tab
 216         else:
 217             print('not implemented')
 218         return data % (signoff, license, tab, indent, tab)
 219
 220     def SetupData(self, data_type):
 221         inhandle, inname = tempfile.mkstemp()
 222         infd = os.fdopen(inhandle, 'w')
 223         data = self.GetData(data_type)
 224         infd.write(data)
 225         infd.close()
 226         return inname
 227
 228     def testGood(self):
 229         """Test checkpatch operation"""
 230         inf = self.SetupData('good')
 231         result = checkpatch.CheckPatch(inf)
 232         self.assertEqual(result.ok, True)
 233         self.assertEqual(result.problems, [])
 234         self.assertEqual(result.errors, 0)
 235         self.assertEqual(result.warnings, 0)
 236         self.assertEqual(result.checks, 0)
 237         self.assertEqual(result.lines, 62)
 238         os.remove(inf)
 239
 240     def testNoSignoff(self):
 241         inf = self.SetupData('no-signoff')
 242         result = checkpatch.CheckPatch(inf)
 243         self.assertEqual(result.ok, False)
 244         self.assertEqual(len(result.problems), 1)
 245         self.assertEqual(result.errors, 1)
 246         self.assertEqual(result.warnings, 0)
 247         self.assertEqual(result.checks, 0)
 248         self.assertEqual(result.lines, 62)
 249         os.remove(inf)
 250
 251     def testNoLicense(self):
 252         inf = self.SetupData('no-license')
 253         result = checkpatch.CheckPatch(inf)
 254         self.assertEqual(result.ok, False)
 255         self.assertEqual(len(result.problems), 1)
 256         self.assertEqual(result.errors, 0)
 257         self.assertEqual(result.warnings, 1)
 258         self.assertEqual(result.checks, 0)
 259         self.assertEqual(result.lines, 62)
 260         os.remove(inf)
 261
 262     def testSpaces(self):
 263         inf = self.SetupData('spaces')
 264         result = checkpatch.CheckPatch(inf)
 265         self.assertEqual(result.ok, False)
 266         self.assertEqual(len(result.problems), 3)
 267         self.assertEqual(result.errors, 0)
 268         self.assertEqual(result.warnings, 3)
 269         self.assertEqual(result.checks, 0)
 270         self.assertEqual(result.lines, 62)
 271         os.remove(inf)
 272
 273     def testIndent(self):
 274         inf = self.SetupData('indent')
 275         result = checkpatch.CheckPatch(inf)
 276         self.assertEqual(result.ok, False)
 277         self.assertEqual(len(result.problems), 1)
 278         self.assertEqual(result.errors, 0)
 279         self.assertEqual(result.warnings, 0)
 280         self.assertEqual(result.checks, 1)
 281         self.assertEqual(result.lines, 62)
 282         os.remove(inf)
 283
 284
 285 if __name__ == "__main__":
 286     unittest.main()
 287     gitutil.RunTests()