New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 624520 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Feature



Sign in to add a comment

EC: Merge target tests into EC CTS

Project Member Reported by aaboagye@chromium.org, Jun 29 2016

Issue description

Right now, we build target tests for many boards. However, these target tests are not being run on actual DUTs. With EC CTS on the way, I think we should merge the target tests to somewhere within the EC CTS.
 
Owner: dnojiri@chromium.org
Status: Assigned (was: Untriaged)
Daisuke will be maintaining CTS after Chris leaves.  I'm guessing the merge won't happen until after that.
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 12 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/36eb325519a46bddaa1a7567c6224fd0e28fd1c4

commit 36eb325519a46bddaa1a7567c6224fd0e28fd1c4
Author: Daisuke Nojiri <dnojiri@chromium.org>
Date: Fri Jul 08 19:42:45 2016

Add pingpong to CTS

cts/pingpong/cts.tasklist contains tasks run only for pingpong.

BUG=chromium:624520
BRANCH=none
TEST=Ran the followings:
make buildall
make CTS_MODULE=gpio BOARD=nucleo-f072rb
make CTS_MODULE=pingpong BOARD=nucleo-f072rb
make CTS_MODULE=gpio BOARD=stm32l476g-eval
make CTS_MODULE=pingpong BOARD=stm32l476g-eval

Change-Id: I5180c60aed5f7d41363c502df539ea9b25ff5d13
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/359446
Reviewed-by: Chris Chen <twothreecc@google.com>

[add] https://crrev.com/36eb325519a46bddaa1a7567c6224fd0e28fd1c4/cts/pingpong/cts.tasklist
[add] https://crrev.com/36eb325519a46bddaa1a7567c6224fd0e28fd1c4/cts/pingpong/dut.c
[add] https://crrev.com/36eb325519a46bddaa1a7567c6224fd0e28fd1c4/cts/pingpong/th.c

Project Member

Comment 3 by bugdroid1@chromium.org, Jul 12 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/afa53e3950abd4727eab9b14275b4805495767b7

commit afa53e3950abd4727eab9b14275b4805495767b7
Author: Daisuke Nojiri <dnojiri@chromium.org>
Date: Fri Jul 08 19:42:45 2016

Add cts.tasklist

cts.tasklist contains tasks run only for CTS. These tasks are added to the
tasks registered in ec.tasklist with higher priority. This design allows
board directories to be free from CTS stuff.

cts.tasklist can be placed in each suite directory (cts/suite/cts.tasklist).
If a suite does not define its own cts.tasklist, the common list is used
(i.e. cts/cts.tasklist).

BUG=chromium:624520
BRANCH=none
TEST=Ran the followings:
make buildall
make CTS_MODULE=gpio BOARD=nucleo-f072rb
make CTS_MODULE=gpio BOARD=stm32l476g-eval

Change-Id: Ibb242297ee10a397a8fcb6ff73d8cbc560daa885
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/359445
Reviewed-by: Chris Chen <twothreecc@google.com>

[add] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/cts/cts.tasklist
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/core/cortex-m0/task.c
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/common/build.mk
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/board/stm32l476g-eval/ec.tasklist
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/board/nucleo-f072rb/ec.tasklist
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/core/cortex-m/task.c
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/include/task_id.h
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/Makefile
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/include/config.h
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/Makefile.toolchain
[modify] https://crrev.com/afa53e3950abd4727eab9b14275b4805495767b7/core/host/task.c

Project Member

Comment 4 by bugdroid1@chromium.org, Jul 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/9c1e181e16e67f3f8c00b07588b1df9c21be9ed8

commit 9c1e181e16e67f3f8c00b07588b1df9c21be9ed8
Author: Daisuke Nojiri <dnojiri@chromium.org>
Date: Thu Jul 14 21:12:21 2016

cts: Add interrupt test

It's imported from test/interrupt.c and adjusted to CTS.

BUG=chromium:624520
BRANCH=none
TEST=make buildall. Test passed on stm32l476-geval and nucleo-f072rb.

Change-Id: Ie948d284cebad60d97aab1512bb9e3af8838004e
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/360660
Reviewed-by: Chris Chen <twothreecc@google.com>

[add] https://crrev.com/9c1e181e16e67f3f8c00b07588b1df9c21be9ed8/cts/interrupt/dut.c
[add] https://crrev.com/9c1e181e16e67f3f8c00b07588b1df9c21be9ed8/cts/interrupt/th.c

Project Member

Comment 5 by bugdroid1@chromium.org, Jul 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/a7d454f48b19b6cb2dca76151cb3faa050df24d1

commit a7d454f48b19b6cb2dca76151cb3faa050df24d1
Author: Daisuke Nojiri <dnojiri@chromium.org>
Date: Thu Jul 14 21:12:21 2016

cts: Add hook test

It's imported from test/hooks.c and adjusted to CTS.

BUG=chromium:624520
BRANCH=none
TEST=make buildall. Test passed on stm32l476-geval and nucleo-f072rb.

Change-Id: I70673f2c0f8316a2b1fd9472eeb7db350fdc2d84
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/360631
Reviewed-by: Chris Chen <twothreecc@google.com>

[add] https://crrev.com/a7d454f48b19b6cb2dca76151cb3faa050df24d1/cts/hook/dut.c
[add] https://crrev.com/a7d454f48b19b6cb2dca76151cb3faa050df24d1/cts/hook/th.c

Project Member

Comment 6 by bugdroid1@chromium.org, Jul 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/8eb3ad19b4a4439d94e6013ca73d84698a5a6641

commit 8eb3ad19b4a4439d94e6013ca73d84698a5a6641
Author: Daisuke Nojiri <dnojiri@chromium.org>
Date: Thu Jul 14 21:12:21 2016

cts: Add mutext test

It's imported from test/mutex.c and adjusted to CTS.

BUG=chromium:624520
BRANCH=none
TEST=Test passed on stm32l476-geval and nucleo-f072rb. make -j buildall

Change-Id: I8cab0541ecbb1daa26b4d728fbd3e45e903ee512
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/360600
Reviewed-by: Chris Chen <twothreecc@google.com>

[add] https://crrev.com/8eb3ad19b4a4439d94e6013ca73d84698a5a6641/cts/mutex/cts.tasklist
[add] https://crrev.com/8eb3ad19b4a4439d94e6013ca73d84698a5a6641/cts/mutex/dut.c
[add] https://crrev.com/8eb3ad19b4a4439d94e6013ca73d84698a5a6641/cts/mutex/th.c

Project Member

Comment 7 by bugdroid1@chromium.org, Jul 29 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/b517067a418e1551f7c70cc32840f10845003934

commit b517067a418e1551f7c70cc32840f10845003934
Author: Daisuke Nojiri <dnojiri@chromium.org>
Date: Mon Jul 18 22:12:32 2016

cts: Add timer test

The timer test checks the accuracy of the internal timer. After sync,
DUT and TH start counting down one second. After one second, DUT raises
GPIO level.  TH determines whether the test passes or not based on how
much more or less time elapsed than one second, assuming its clock is
calibrated.

This test takes advantage of TH running on a bare chip. If the host
were measuring (instead of TH), the timing would be affected by many
software and hardware layers (e.g. UART drivers on DUT and host,
python interpreter, etc.).

BUG=chromium:624520
BRANCH=none
TEST=cts.py --module timer && cts.py --module gpio && make buildall

Change-Id: I535e7772b4d93f1f5d248506f7ea167429a50174
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/361384

[add] https://crrev.com/b517067a418e1551f7c70cc32840f10845003934/cts/timer/cts.testlist
[add] https://crrev.com/b517067a418e1551f7c70cc32840f10845003934/cts/timer/dut.c
[modify] https://crrev.com/b517067a418e1551f7c70cc32840f10845003934/board/stm32l476g-eval/board.c
[modify] https://crrev.com/b517067a418e1551f7c70cc32840f10845003934/board/stm32l476g-eval/gpio.inc
[modify] https://crrev.com/b517067a418e1551f7c70cc32840f10845003934/cts/build.mk
[add] https://crrev.com/b517067a418e1551f7c70cc32840f10845003934/cts/timer/th.c

Sign in to add a comment