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