diff options
author | Jack Rosenthal <jrosenth@chromium.org> | 2021-11-04 12:11:58 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-11-05 04:22:34 +0000 |
commit | 252457d4b21f46889eebad61d4c0a65331919cec (patch) | |
tree | 01856c4d31d710b20e85a74c8d7b5836e35c3b98 /cts/timer | |
parent | 08f5a1e6fc2c9467230444ac9b582dcf4d9f0068 (diff) | |
download | chrome-ec-stabilize-14528.B-ish.tar.gz |
ish: Trim down the release branchstabilize-wristpin-14469.59.B-ishstabilize-voshyr-14637.B-ishstabilize-quickfix-14695.187.B-ishstabilize-quickfix-14695.124.B-ishstabilize-quickfix-14526.91.B-ishstabilize-14695.85.B-ishstabilize-14695.107.B-ishstabilize-14682.B-ishstabilize-14633.B-ishstabilize-14616.B-ishstabilize-14589.B-ishstabilize-14588.98.B-ishstabilize-14588.14.B-ishstabilize-14588.123.B-ishstabilize-14536.B-ishstabilize-14532.B-ishstabilize-14528.B-ishstabilize-14526.89.B-ishstabilize-14526.84.B-ishstabilize-14526.73.B-ishstabilize-14526.67.B-ishstabilize-14526.57.B-ishstabilize-14498.B-ishstabilize-14496.B-ishstabilize-14477.B-ishstabilize-14469.9.B-ishstabilize-14469.8.B-ishstabilize-14469.58.B-ishstabilize-14469.41.B-ishstabilize-14442.B-ishstabilize-14438.B-ishstabilize-14411.B-ishstabilize-14396.B-ishstabilize-14395.B-ishstabilize-14388.62.B-ishstabilize-14388.61.B-ishstabilize-14388.52.B-ishstabilize-14385.B-ishstabilize-14345.B-ishstabilize-14336.B-ishstabilize-14333.B-ishrelease-R99-14469.B-ishrelease-R98-14388.B-ishrelease-R102-14695.B-ishrelease-R101-14588.B-ishrelease-R100-14526.B-ishfirmware-cherry-14454.B-ishfirmware-brya-14505.B-ishfirmware-brya-14505.71.B-ishfactory-kukui-14374.B-ishfactory-guybrush-14600.B-ishfactory-cherry-14455.B-ishfactory-brya-14517.B-ish
In the interest of making long-term branch maintenance incur as little
technical debt on us as possible, we should not maintain any files on
the branch we are not actually using.
This has the added effect of making it extremely clear when merging CLs
from the main branch when changes have the possibility to affect us.
The follow-on CL adds a convenience script to actually pull updates from
the main branch and generate a CL for the update.
BUG=b:204206272
BRANCH=ish
TEST=make BOARD=arcada_ish && make BOARD=drallion_ish
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I17e4694c38219b5a0823e0a3e55a28d1348f4b18
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3262038
Reviewed-by: Jett Rink <jettrink@chromium.org>
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
Diffstat (limited to 'cts/timer')
-rw-r--r-- | cts/timer/cts.testlist | 19 | ||||
-rw-r--r-- | cts/timer/dut.c | 30 | ||||
-rw-r--r-- | cts/timer/th.c | 62 |
3 files changed, 0 insertions, 111 deletions
diff --git a/cts/timer/cts.testlist b/cts/timer/cts.testlist deleted file mode 100644 index 9b5da0d6c9..0000000000 --- a/cts/timer/cts.testlist +++ /dev/null @@ -1,19 +0,0 @@ -/* Copyright 2016 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - */ - -/* - * Test timer accuracy - * - * After sync, DUT and TH start counting down one second. After one second, - * DUT raises GPIO level, which triggers an interrupt on TH. TH determines - * whether the test passes or not based on how much more or less time elapsed - * than one second. - * - * Requirements: - * - Sync connection - * - GPIO_OUTPUT connection for sending notification from DUT - * - Calibrated TH timer - */ -CTS_TEST(timer_calibration_test,,,,) diff --git a/cts/timer/dut.c b/cts/timer/dut.c deleted file mode 100644 index 96d7c5a3cf..0000000000 --- a/cts/timer/dut.c +++ /dev/null @@ -1,30 +0,0 @@ -/* Copyright 2016 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - */ - -#include "common.h" -#include "cts_common.h" -#include "gpio.h" -#include "task.h" -#include "timer.h" -#include "watchdog.h" - -static enum cts_rc timer_calibration_test(void) -{ - gpio_set_flags(GPIO_OUTPUT_TEST, GPIO_ODR_HIGH); - - sync(); - sleep(1); - gpio_set_level(GPIO_OUTPUT_TEST, 0); - - return CTS_RC_SUCCESS; -} - -#include "cts_testlist.h" - -void cts_task(void) -{ - cts_main_loop(tests, "Timer"); - task_wait_event(-1); -} diff --git a/cts/timer/th.c b/cts/timer/th.c deleted file mode 100644 index e82cac71ab..0000000000 --- a/cts/timer/th.c +++ /dev/null @@ -1,62 +0,0 @@ -/* Copyright 2016 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - */ - -#include "common.h" -#include "cts_common.h" -#include "gpio.h" -#include "registers.h" -#include "task.h" -#include "timer.h" -#include "watchdog.h" - -/* - * Interrupt handler - * - * DUT is supposed to trigger an interrupt when it's done counting down, - * causing this function to be invoked. - */ -void cts_irq(enum gpio_signal signal) -{ - /* Wake up the CTS task */ - task_wake(TASK_ID_CTS); -} - -static enum cts_rc timer_calibration_test(void) -{ - /* Error margin: +/-2 msec (0.2% for one second) */ - const int32_t margin = 2 * MSEC; - int32_t elapsed, delta; - timestamp_t t0, t1; - - gpio_enable_interrupt(GPIO_CTS_NOTIFY); - interrupt_enable(); - - sync(); - t0 = get_time(); - /* Wait for interrupt */ - task_wait_event(-1); - t1 = get_time(); - - elapsed = (int32_t)(t1.val - t0.val); - delta = elapsed - SECOND; - if (delta < -margin) { - CPRINTS("DUT clock runs too fast: %+d usec", delta); - return CTS_RC_FAILURE; - } - if (margin < delta) { - CPRINTS("DUT clock runs too slow: %+d usec", delta); - return CTS_RC_FAILURE; - } - - return CTS_RC_SUCCESS; -} - -#include "cts_testlist.h" - -void cts_task(void) -{ - cts_main_loop(tests, "Timer"); - task_wait_event(-1); -} |