| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add .idea/ to .gitignore for the Clion IDE.
BRANCH=none
BUG=none
TEST=none
Change-Id: Ib15ae022f074d632785b222650a35f4dbb85c971
Signed-off-by: Tim Van Patten <timvp@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4316727
Reviewed-by: Rob Barnes <robbarnes@google.com>
Commit-Queue: Rob Barnes <robbarnes@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implement run_tests.sh --coverage by using vpython3 to install all the
necessary projects.
Since I added command line flags, pass any excess flags to pytest and
add some documentation on how to run to the comments.
BRANCH=None
BUG=None
TEST=zephyr/zmake/run_tests.sh --coverage
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Change-Id: Ib82271e414850079319769fa32ca4d6de04e67ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3989351
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Move the logic of the spi_flash_reg.c from common/ to a separate
directory. Also, move the tests. Update the build.mk system to still
build this in the same manner as well as CMake to use it as a library.
BRANCH=none
BUG=b:246839061
TEST=make BOARD=dooly
TEST=zmake build hoglin
TEST=./twister -T common
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: Icc6b4fce1619c2c6a86c040f30390b7788171a9d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3935025
Reviewed-by: Keith Short <keithshort@chromium.org>
Reviewed-by: Aaron Massey <aaronmassey@google.com>
Code-Coverage: Zoss <zoss-cl-coverage@prod.google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When you run a test using gdb, or just run it by hand in the platform/ec
dir, it leaves a flash.bin file laying around.
BRANCH=None
BUG=None
TEST=None
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Change-Id: I3cd5d251bc6074d6e413552266271e5f475068fe
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3824927
Reviewed-by: Al Semjonovs <asemjonovs@google.com>
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a wrapper script that helps invoke twister with the correct
configuration and defaults for running EC tests. Developers can simply
run `./twister` from `src/platform/ec` to run all Twister-compatible EC
tests, or specify specific tests to run by adding `-T path/to/testroot`
arguments. Other CLI args are passed straight through to Twister.
This CL also adds a testcase.yaml file to the `math` test, allowing it
to be run through Twister:
```
cd $CROS_WORKON_SRCROOT/src/platform/ec
./twister (or)
./twister -T zephyr/test/math
```
Twister's output directories, `twister-out` and `twister-out.N` have
been added to the `.gitignore`.
The `firmware_builder.py` CQ script now also calls Twister and can merge
in coverage reports.
BRANCH=None
BUG=None
TEST=./twister
Signed-off-by: Tristan Honscheid <honscheid@google.com>
Change-Id: If6ff10ffb76655912c96a66aec28d12dcebd1764
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3773766
Reviewed-by: Yuval Peress <peress@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add the optional commit message template file to .gitignore.
See https://crbug.com/gerrit/10963 for details.
BRANCH=none
BUG=none
TEST=none
Signed-off-by: Andrea Grandi <agrandi@google.com>
Change-Id: Ic8939bc7202c8a09c76e472b2884773ac34451ef
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3508713
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add instructions for running zmake unit tests outside the chroot and
generating coverage data.
BRANCH=None
BUG=b:219062927
TEST=Ran the commands
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Change-Id: Ib1e9a2ddba95d64c2a20973be444edff0e22c30a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3491404
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add .gdb_history to .gitignore.
BRANCH=None
BUG=None
TEST=None
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Change-Id: Ibb52970914fe8e68aa7b1bc6700a03aae3a6bc42
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3384481
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Using the clangd language server creates an index cache.
BRANCH=None
BUG=None
TEST=None
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Change-Id: Ic5c8739b9ab3b4f07b98a204932b0af3dd15206b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3270400
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Aaron Massey <aaronmassey@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add this file so that it doesn't complain for people who want to use
ctags.
BUG=none
BRANCH=none
TEST=see that 'git status' doesn't show 'ctags' anymore
Signed-off-by: Simon Glass <sjg@chromium.org>
Change-Id: Ia0e532d277c80243aed9fa834193ed9137078466
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2941806
Reviewed-by: Yuval Peress <peress@chromium.org>
Reviewed-by: Denis Brockus <dbrockus@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implement test coverage for BuildConfigs, including merging using
`|` operator, and running `popen_cmake`.
BUG=b:178642582
BRANCH=none
TEST=pytest
Cq-Depend: chromium:2654350
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I07d8e913af999dac4f66f904b9d53e974e57eb40
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2656515
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This copies zmake into platform/ec/zephyr/zmake, as explained in
go/zephyr-fewer-repos.
Follow-on CLs will be submitted to:
- Update the chromeos-base/zephyr-build-tools ebuild to reference this
directory instead of the one in zephyr-chrome.
- Remove the copy of zmake in zephyr-chrome.
Those interested in the git history of this code prior to it being
moved to platform/ec can find it here:
https://chromium.googlesource.com/chromiumos/platform/zephyr-chrome/+log/bacea2e3e62c41000e5bdb4ed6433f24386d14bf/util
BUG=b:177003034
BRANCH=none
TEST=emerge with new path (requires follow-on CL)
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: Ia957b3e35ce3b732968ebf8df603ef13298cc6b3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2618501
Reviewed-by: Yuval Peress <peress@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds a .default settings.json file for vscode, so that you
do not need to copy/paste form the ide-support.md doc. This also
makes it easier for people that don't known that the ide-support.md
doc exists.
This also adds the .vscode directory to the gitignores, so that local
customization/settings are not always in the git status.
BRANCH=none
BUG=none
TEST=Checked in Gitiles
TEST=Check that my settings.json is not in "git status".
TEST=Check that ec.code-workspace is not in "git status".
Change-Id: I4dd8cfcd79c291a6f4fb0d573df131f0cb0d9844
Signed-off-by: Craig Hesling <hesling@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2586041
Commit-Queue: Jett Rink <jettrink@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This allows for hiding local scripts and other artifacts from git.
BRANCH=none
BUG=none
TEST=git checkout -f cros/master
touch local/myscript
git status
# Nothing recommended
Change-Id: I5b2c80b3bf662e1134118e5f269fcecfd73b0cdc
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2116799
Tested-by: Craig Hesling <hesling@chromium.org>
Tested-by: George Engelbrecht <engeg@google.com>
Reviewed-by: Jett Rink <jettrink@chromium.org>
Commit-Queue: George Engelbrecht <engeg@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
emacs creates backup files ending in ~ and should be ignored by git.
BUG=none
BRANCH=none
TEST=git status no longer complains about *~ files.
Change-Id: I96043f89c1d51f8034a2427426ec07d9e2846363
Signed-off-by: Caveh Jalali <caveh@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1772525
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since the ec binaries are so tight on space, it would be nice to be
able to see just how much changes are affecting the size. This allows
users to easily do before/after comparisons.
Sample output:
build/sweetberry/RW/ec.RW.flat shrank by 44 bytes: (43828 to 43784)
build/twinkie/RO/ec.RO.flat shrank by 64 bytes: (46312 to 46248)
build/twinkie/RW/ec.RW.flat shrank by 40 bytes: (45900 to 45860)
build/wheatley/RW/ec.RW.flat shrank by 40 bytes: (102692 to 102652)
Compared 156 of 156 files.
81 files changed.
Total size change: -3100 bytes.
Average size change: -38 bytes.
BRANCH=none
BUG=none
TEST=make clobber buildall -j; make savesizes
[change some code that changes file sizes]
make clobber buildall -j ; make newsizes
[Shows size differences]
Change-Id: I48b440063eb6eb6c00900af3d0dfa075be6f9ec7
Signed-off-by: Martin Roth <martinroth@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/410860
Reviewed-by: Patrick Georgi <pgeorgi@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Internal builds may have private*/ subdirectories cloned from
repositories that we're not (yet) allowed to share
publicly (boo). Tell git not to complain about them every time I
run "git status".
BUG=none
BRANCH=none
TEST=manual
mkdir private-foo
touch private-foo/bar.txt
git status
Observe the lack of complaints about private-foo
Change-Id: I5281e3a533f9b1a548ced6f6716a388152c58776
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/341032
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It is quite annoying to see an error reported by buildall, but not
knowing what board(s) actually failed to build.
Create file for each board being built, put build progress information
in it, and remove it if the board build succeeded. Then, once the
build is completed see the failed boards in .failedboards/, if any,
and the states they failed at.
BRANCH=none
BUG=none
TEST=verified that the directory is empty after a successful buildall
run, and has files named after failed boards in case of build
failures.
Change-Id: I67eb1671cadf58d9f8feccebfcc860524f33c2a0
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/338883
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A new private repo has been added, so prevent git from
complaining about it.
BUG=chromium:507445
BRANCH=none
TEST=make build all; git status
Nothing breaks, "git status" stops pointing out the new
subdirectory.
Change-Id: Ibf1fbc7504fb6bbb9df7d6ddf0efda9a632e39b6
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/284874
Reviewed-by: Alec Berg <alecaberg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These are generated files used for code cross-reference in Emacs. Let's
ignore them so repo upload doesn't keep complaining.
BUG=none
BRANCH=ToT
TEST=manual
Leave these files in place, try "repo upload .", see that it stops
complaining about uncommitted files.
Change-Id: I9c0a7182050c0b50bd36e4b10091f9b2912f6596
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/192285
Reviewed-by: Randall Spangler <rspangler@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This enforces that "make buildall" runs at least once after the last
file change.
TEST=Try to upload without running "make buildall"
TEST=Change a file without re-running "make buildall", and try to
upload.
BUG=None
BRANCH=None
Change-Id: Ia4abb3c0e17cf4d559975574f398d74c7986c89f
Signed-off-by: Vic (Chun-Ju) Yang <victoryang@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/185116
Reviewed-by: Dave Parker <dparker@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With this, git doesn't complain about uncommited change if one uses
cscope.
BUG=none
TEST='git status' and see no uncommited change.
BRANCH=none
Change-Id: I30bb78e4d0a7541444ef1aced683335933a5aa73
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/49457
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If ctags is used, git complains about uncommitted 'tags'. Let's add it
to .gitignore.
BUG=None
TEST='git status' and doesn't see git complain about 'tags'.
BRANCH=none
Change-Id: I6026e83b1537eb61439f22a0b09dccb07283c820
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/42689
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since build output is in build/ we don't really need to have *.bin (and
others) excluded throughout the source tree. This change allows us to
check in .bin files without warnings.
BUG=none
BRANCH=snow,link
TEST=manual
$ git status
- see that we still have no stray files reported
$ git add test/test-matrix.bin
- see that there is no warning given now
Change-Id: Ia71c5c2b89c3f0eef5098c86c701cb34f666e164
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/34632
Reviewed-by: Randall Spangler <rspangler@chromium.org>
|
|
|
|
|
|
|
|
| |
Signed-off-by: Dave Parker <dparker@chromium.org>
BUG=None
TEST=Verify files under a "private" subdirectory are ignored by git.
Change-Id: I551effa5cf8967b787e40f1cc82f9c3f91d9fa09
|
|\
| |
| |
| |
| |
| |
| |
| | |
BUG=chrome-os-partner:7564
TEST=None
Conflicts:
.gitignore
|
|
|
|
|
|
|
|
| |
source files mainly done by Vincent.
Signed-off-by: Vincent Palatin <vpalatin@chromium.org>
Change-Id: Ic2d1becd400c9b4b4a14d4a243af1bdf77d9c1e2
|
|
|
|
|
|
|
|
|
|
| |
Added i8042 interface and matrix code. Demonstrate a key is pressed and passed
to host through the board-dependent matrix code.
BUG=None
TEST=make && make runtests
Change-Id: I6a9f5e621d9e93e5c16384afebf4d665000e81a6
|
|
|
|
|
|
|
|
|
|
|
| |
Build output is now in ./build
Fixed compiler warnings in ec_uartd, ec_console
BUG=none
TEST=make
Change-Id: I9a46ab6b9d4e912e59a60c669e95dc0c6f8485df
|
|
BUG=None
TEST=git status
Change-Id: I94d3929994b02dd90e41b4d87cc236d7611e27eb
|