| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
find . -name '*.md' | xargs sed -i 's#refs/heads/master#HEAD#g'
find . -name '*.md' | xargs sed -i 's#+/master#+/HEAD#g'
BRANCH=none
BUG=none
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Iaac99ddc3463c8c7bb08038214e3eb22951db6a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3188552
Reviewed-by: Diana Z <dzigterman@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
sed -i 's#refs/heads/master#HEAD#g' *.md
sed -i 's#+/master#+/HEAD#g' *.md
BRANCH=none
BUG=none
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I1663701a16fd66b32d48358fa5a754247b9dbb5c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3188550
Reviewed-by: Edward Hill <ecgh@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The format tool was assuming the parentheses were a markdown link.
BRANCH=none
BUG=none
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Iea221310e87adaa9fbeaf046d634a57034f142c3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3153630
Reviewed-by: Craig Hesling <hesling@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mark some additional console commands as required for testing.
Add sections detailing CONFIG_LTO, CONFIG_CHIP_INIT_ROM_REGION, and
CONFIG_COMMON_GPIO_SHORTNAMES.
BUG=none
BRANCH=none
TEST=View output with gitiles
Signed-off-by: Keith Short <keithshort@chromium.org>
Change-Id: I58ac5bde1440ac6dae0caf9f8b6e2ff76bb2ff6f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3078043
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Correct some links between markdown documents local to the EC code.
BUG=none
BRANCH=none
TEST=verify links with gitiles
Signed-off-by: Keith Short <keithshort@chromium.org>
Change-Id: I8e31b7775d25e48f62180c4a754fc6a49ca0e8d9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3078042
Commit-Queue: Abe Levkoy <alevkoy@chromium.org>
Reviewed-by: Abe Levkoy <alevkoy@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Update SPI host interface config option for inclusive language.
BUG=b:163885307
BRANCH=none
TEST=compare_build.sh
Signed-off-by: Keith Short <keithshort@chromium.org>
Change-Id: I808d5960fa3e746626465bedc626a95e0f0aaa3f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3066271
Commit-Queue: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a document describing tips for reding the EC image size.
BUG=none
BRANCH=none
TEST=gittiles view of markdown
Signed-off-by: Keith Short <keithshort@chromium.org>
Change-Id: Ic639e069f911812181a10ee25b1bb5a17a8900a6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3059225
Commit-Queue: Diana Z <dzigterman@chromium.org>
Reviewed-by: Diana Z <dzigterman@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This changes the used port to JLinkRemoteServerCLExe's actual
default port of 19020, so that you don't need to always specify it.
BRANCH=none
BUG=none
TEST=JLinkRemoteServerCLExe # in background
make proj-bloonchipper -j
./util/flash_jlink.py --board bloonchipper
Signed-off-by: Craig Hesling <hesling@chromium.org>
Change-Id: I2a3c1a10e0b5c7b9cc62001a3a26508ea3db8199
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3055111
Reviewed-by: Josie Nordrum <josienordrum@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently keyboard_scan_config is defined by each board using
CONFIG_KEYBOARD_BOARD_CONFIG. This patch makes it defined as
__override hence removes CONFIG_KEYBOARD_BOARD_CONFIG.
BUG=None
BRANCH=None
TEST=buildall
Change-Id: I53a356741ba4d00e829ca59b74ee6dc704188728
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3044403
Tested-by: Gwendal Grignou <gwendal@chromium.org>
Commit-Queue: Gwendal Grignou <gwendal@chromium.org>
Reviewed-by: Gwendal Grignou <gwendal@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=b:175115925
TEST=servod --board=dragonclaw
dut-control fpmcu_slp_alt:on
dut-control fpmcu_slp_alt:off
Cq-Depend: chromium:2971145
Change-Id: I17174f73dc4817fa5fe169cbfb85489fb4437a5d
Signed-off-by: Craig Hesling <hesling@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2971827
Reviewed-by: Josie Nordrum <josienordrum@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename CONFIG_CROS_BOARD_INFO to CONFIG_CBI_EEPROM to make it clear
that the information comes from on-board EEPROM.
It sets up the groundwork for adding more options of CBI sources later.
BRANCH=None
BUG=b:186264627
TEST=make buildall -j
Signed-off-by: Philip Chen <philipchen@google.com>
Change-Id: I9a6feee0a8b35bbf29e445544243485507767ad8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2945792
Reviewed-by: Philip Chen <philipchen@chromium.org>
Commit-Queue: Philip Chen <philipchen@chromium.org>
Tested-by: Philip Chen <philipchen@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
At present there are no tests for zmake.py itself. Add a simple one to
provide some coverage.
Use this to test that the filtering does what we expect. This works by
using some pre-canned output for the RO and RW, running each file
through a separate 'cat' command in its own process, then comparing
the results obtained in the log.
BUG=b:177096315
BRANCH=none
TEST=cros workon --host start chromeos-base/zephyr-build-tools && \
sudo emerge chromeos-base/zephyr-build-tools && \
make -j16 runhosttests buildall && zmake testall && \
zephyr/firmware_builder.py --metrics /tmp/tmplt8ty8ci test ; echo $?
count=0 ; while pytest --verbose zephyr/zmake/tests/test_zmake.py ; do
count=$(($count + 1)) ; echo "PASSED $count times" ;
rm -rf .pytest_cache/ ; done
Change-Id: I9749d9533091ad5c54c298ba6268efcd91fd2bf4
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2863945
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=b:139532396
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Iab4df1b7d6876006326c15093cecb333f00d551b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2857474
Commit-Queue: Craig Hesling <hesling@chromium.org>
Reviewed-by: Craig Hesling <hesling@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
FPC 1025 imagery in the dragonclaw picture and FPC 1145 for
icetower. Note orientation for the sensor.
BUG=b:183747042
TEST=rendered with gittiles
Change-Id: I036e6cbe48c0c53f73ccb0d0157bfef884009184
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2824045
Tested-by: Aadit Maheshkumar Modi <modia@chromium.org>
Reviewed-by: Kevin Shelton <kmshelton@chromium.org>
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
Commit-Queue: Aadit Maheshkumar Modi <modia@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 477997408185b0d446d388239a53657de6367570.
Reason for revert: Introduces flakes to the CQ
BUG=chromium:1196696
TEST=none
Original change's description:
> Reland "zephyr: zmake: Add a test for zmake and output filtering"
>
> This reverts commit 3ea453e890f2f782e8195c94af5de00c8c754f74.
>
> Reason for revert: Trying again to see what error is produced
>
> Original change's description:
> > Revert "zephyr: zmake: Add a test for zmake and output filtering"
> >
> > This reverts commit ca160a3394c952f0732f66eb113b9735b5655ed2.
> >
> > Reason for revert: responsible for CQ failures (crbug.com/1198472)
> >
> > BUG=chromium:1198472
> > BRANCH=none
> > TEST=pytest zephyr/zmake/test
> >
> > Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
> > Change-Id: I273ee3e26309d595973ec8040a2d835169096670
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2824028
> > Reviewed-by: Sonny Rao <sonnyrao@chromium.org>
>
> Bug: chromium:1198472
> Change-Id: Icaae5c6698f6747b634952d03ae83668e937c089
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2822402
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
> Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
Bug: chromium:1198472
Change-Id: Iabbfdf2c7af82113d91f9aeba256024b7e59b4f8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2842363
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 3ea453e890f2f782e8195c94af5de00c8c754f74.
Reason for revert: Trying again to see what error is produced
Original change's description:
> Revert "zephyr: zmake: Add a test for zmake and output filtering"
>
> This reverts commit ca160a3394c952f0732f66eb113b9735b5655ed2.
>
> Reason for revert: responsible for CQ failures (crbug.com/1198472)
>
> BUG=chromium:1198472
> BRANCH=none
> TEST=pytest zephyr/zmake/test
>
> Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
> Change-Id: I273ee3e26309d595973ec8040a2d835169096670
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2824028
> Reviewed-by: Sonny Rao <sonnyrao@chromium.org>
Bug: chromium:1198472
Change-Id: Icaae5c6698f6747b634952d03ae83668e937c089
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2822402
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit ca160a3394c952f0732f66eb113b9735b5655ed2.
Reason for revert: responsible for CQ failures (crbug.com/1198472)
BUG=chromium:1198472
BRANCH=none
TEST=pytest zephyr/zmake/test
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I273ee3e26309d595973ec8040a2d835169096670
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2824028
Reviewed-by: Sonny Rao <sonnyrao@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
At present there are no tests for zmake.py itself. Add a simple one to
provide some coverage.
Use this to test that the filtering does what we expect. This works by
using some pre-canned output for the RO and RW, running each file
through a separate 'cat' command in its own process, then comparing
the results obtained in the log.
In creating this test, unexpected behaviour was found with the
image-size logging (Memory, FLASH:, etc), in that they end up on the
ERROR level, if previous output has appeared on that level. This is
because the default log level changes to whatever the last log output
was. So fix this at the same time.
For now we don't check the DEBUG level, since it has a bug.
BUG=b:177096315
BRANCH=none
TEST=(cd zephyr/zmake/ && python3 -m pytest .)
Signed-off-by: Simon Glass <sjg@chromium.org>
Cq-Depend: chromium:2819138
Change-Id: Iaa777815b577ab814eb2e69d34a3047ae57f0b45
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2807486
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now we are shipping more and more detachables, we should have a
general document about the detachable base.
BUG=b:182884941
TEST=Preview at
https://chromium.googlesource.com/chromiumos/platform/ec/+/82d2822/docs/hammer.md
BRANCH=None
Signed-off-by: Ting Shen <phoenixshen@google.com>
Change-Id: I2745e054b802872be57457f41bf26696b8f6a113
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2772293
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>
Commit-Queue: Ting Shen <phoenixshen@chromium.org>
Tested-by: Ting Shen <phoenixshen@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When running on a distribution it seems better to use the standard
install procedure if possible, with pip3 reserved for things that are
not available. If users have an old distribution, they will presumably
fall back to using pip3 anyway.
Update to use 'apt-get install' in the docs.
BUG=b:181253613
BRANCH=none
TEST=check it in gitiles
Signed-off-by: Simon Glass <sjg@chromium.org>
Change-Id: I539518a3009e73cb948145ca68f868fa5588909d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2809779
Reviewed-by: Yuval Peress <peress@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add some information from the care & feeding guide.
BUG=b:181253613
BRANCH=none
TEST=check it in gitiles
Change-Id: I1ad507cd87359dbfc6e2b89dcfd4c5d4ea77b2ca
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2807485
Reviewed-by: Yuval Peress <peress@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is confusing when browsing since it has no title. Add one.
Also move the 'alternatives considered' to the bottom while we are
here, since that is confusing for people who are less interested in
the motivation for shimming and just want to know what to do.
BUG=b:181253613
BRANCH=none
TEST=check it in gitiles
Signed-off-by: Simon Glass <sjg@chromium.org>
Change-Id: I73a4716278396dfdf0661d6ced67b3d6a9785165
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2807487
Reviewed-by: Yuval Peress <peress@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To make it easier for people to get started with Zephyr, link in the
existing docs.
BUG=b:181253613
BRANCH=none
TEST=check it in gitiles
Signed-off-by: Simon Glass <sjg@chromium.org>
Change-Id: I68f0deb52a64147e705e53410699c372fd1e69b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2807484
Reviewed-by: Yuval Peress <peress@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For host based tests, add instructions on emerging and running gdb.
BRANCH=none
BUG=b:178484932
TEST=doc change only
Change-Id: I5c99d2699134d732e4596a0ce9acffb39c22bda6
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2805225
Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
Commit-Queue: Simon Glass <sjg@chromium.org>
Tested-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added a new zmake sub-command `coverage`, which builds all projects
with coverage, runs unit tests, and creates a html coverage report.
BUG=b:183007888
TEST=sudo emerge zephyr-build-tools && \
zmake coverage build/ztest-coverage
BRANCH=none
Change-Id: Idb6af59c223ece00d3eb09982778cb1b500d8db4
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2794925
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Yuval Peress <peress@chromium.org>
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Populating sensor information inside the diagram for
dragonclaw/ icetower.
BUG=b:183747042
TEST=rendered with gittiles
Change-Id: Iac0873c3e77c221f2e9528ae36d2d6f21ded8474
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2795071
Tested-by: Aadit Maheshkumar Modi <modia@chromium.org>
Reviewed-by: Yicheng Li <yichengli@chromium.org>
Commit-Queue: Yicheng Li <yichengli@chromium.org>
Commit-Queue: Aadit Maheshkumar Modi <modia@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Create a new ninja target 'all.libraries' which is everything that
zephyr.elf depends on, but doesn't actually link the zephyr.elf binary.
This allows creating coverage info for boards that run on SOCs that are
too small for real coverage runs.
Enabled the HAS_COVERAGE_SUPPORT for all boards, so that the
all.libraries target will build with coverage enabled.
HAS_COVERAGE_SUPPORT can't be set in prj files, as it is not directly
user assignable. This is probably somewhat of a misuse of this flag, but
nothing depends on it except for CONFIG_COVERAGE.
Tweaked the instructions to exclude some more directories that are not
useful code, like tests and generate headers.
BUG=b:183007888
TEST=Ran commands in code_coverage.md
BRANCH=none
Change-Id: I6cf1b889b6b3883f7a76d1ae4957805bfa6d7b96
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2786067
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Update the location of the firmwaretestmanual.html link, which must have
moved on partnersite.
BUG=b:183612768
BRANCH=None
TEST=Visual inspection.
Signed-off-by: Stuart Langley <slangley@chromium.org>
Change-Id: I3ca4e8f9282b583624e5ca35237de41c51ba067d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2789809
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The reset cause will influence the initialization flow. We define some
of initial flow of the reset cause for the following development.
This CL include the following:
1. Add check_reset_cause() which sets the system reset flag.
2. Add chip_bbram_status_check() to clear the error status & show the
error message.
3. Add CONFIG_BOARD_RESET_AFTER_POWER_ON feature.
4. Define the initialize flow for reset cause.
BRANCH=none
BUG=b:178101173
TEST=check the following reset cause by 'sysinfo'
1. power-up
2. reset-pin reset
3. soft by 'reboot' console command
4. watchdog by 'waitms 2000'
Signed-off-by: Wealian Liao <whliao@nuvoton.corp-partner.google.com>
Change-Id: I515868d8cda4544fdbe782210b0108b4dda0d8cc
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2731180
Reviewed-by: Keith Short <keithshort@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Building a zephyr ztest with zmake (say that 3 times fast) creates many
symlinks, and makes it hard to merge the coverage output. The
util/normalize_symlinks.py tool takes a lcov info file as input and
replaces symlinks with their actual paths in all SF: lines.
BUG=b:183007888
TEST=Ran the commands in the md file.
BRANCH=none
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Change-Id: Ic8f9aedec30051d7474aa77ba2fb406e0746a719
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2770508
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
Tested-by: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a developer-only utility for converting a gpio.inc file to DTS.
I hope this gets deleted someday when we aren't doing PoCs anymore.
BUG=none
BRANCH=none
TEST=compile/run instructions at top of file, BOARD=lazor
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I354c8917bac0b82f904e07c3e029806ecb67d6d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2752795
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add documentation on how to bring up new Zephyr OS devices for devices
and platforms which already have a CrOS EC OS implementation.
Based this on Lazor bringup.
BUG=none
BRANCH=none
TEST=view in gitiles
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I1d7466bacdd35dc4c5f02e708e87ee748e097236
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2752794
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a guide to make it easier for people to add a new Kconfig.
BUG=b:181253613
BRANCH=none
TEST=none
Signed-off-by: Simon Glass <sjg@chromium.org>
Change-Id: I2714c02f3541a07c18cfa1d4a40e6b75fb2aa4fa
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2749762
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Denis Brockus <dbrockus@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Revised go/zephyr-shim-how-to, and added in Markdown form to our
existing shim documentation.
Large documentation CL... hoping to non-minor concerns in follow-up
CLs if possible.
BUG=b:181253613
BRANCH=none
TEST=view in gitiles
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: Ife666bf06ef76b05b4d3d877049d160706441514
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2737245
Commit-Queue: Simon Glass <sjg@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://crrev.com/c/2689101 is needed in order to configure the sleep
GPIOs correctly on icetower, so the change was cherry-picked to the
release branch to perform the measurement.
BRANCH=none
BUG=b:180445334
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I608a725c6f4c474365814b95a39ed365b2ea1eee
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2724876
Commit-Queue: Craig Hesling <hesling@chromium.org>
Reviewed-by: Craig Hesling <hesling@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
model.yaml files no longer have to be created by hand; instead the
Chrome OS Project Configuration tools will generate it.
BRANCH=none
BUG=b:180410925
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Id7eb218b7b908ed77dec33a2b7479bbf5a72fc4c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2721139
Reviewed-by: Craig Hesling <hesling@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=none
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Ib9c7ec288c9efabb454580c6bf6028ee6a0a1373
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2721087
Commit-Queue: Craig Hesling <hesling@chromium.org>
Reviewed-by: Craig Hesling <hesling@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now that the full source-out policy has been implemented in the TCPMv2
DPM, modify the previous note about implementation.
BRANCH=None
BUG=b:141690755
TEST=view in Chrome with Markdown Preview Plus
Signed-off-by: Diana Z <dzigterman@chromium.org>
Change-Id: Ic74c780841c42e1e1d88306d366a00bf0fadb3c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2701892
Reviewed-by: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=b:179789305
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I2bfe78a25bc148fb2a9242a83b3fad5f97c89ed8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2688118
Reviewed-by: Craig Hesling <hesling@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=b:179789305
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Id5bdb84ef94c2585ee3f3036a85b6ce5f0c5662d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2688117
Reviewed-by: Craig Hesling <hesling@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
FRS ports which require 3A should also be considered in the source-out
policy. This updates the diagram to include FRS and includes FRS in the
example flow. Additionally, it changes the policy for re-allocation
on detach from "oldest port" to "lowest number port" since that policy
requires less flash space.
BRANCH=None
BUG=b:141690755
TEST=verify markdown renders in Chrome with Markdown Preview Plus
Signed-off-by: Diana Z <dzigterman@chromium.org>
Change-Id: I14e359696fa7dc15ae349cade6fc723e7998d941
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2602597
Reviewed-by: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The FPMCU console UART has been renamed to raw_fpmcu_console_uart_pty in
https://crrev.com/c/2637968. The same CL added "fpmcu_pp3300" as an
alias for "spi1_vref".
BRANCH=none
BUG=b:177331210
TEST=On dragonclaw v0.2 with segger jlink:
./test/run_device_tests.py --board bloonchipper -t aes
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: If616421a55a6ecdfcf589b1c1b663820890b01ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2638448
Reviewed-by: Craig Hesling <hesling@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=b:178648877
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I0ac5581ba7bc512234d40dbf34222422afa9c725
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2650551
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These instructions clarify how to use J-Link/J-Trace to flash the FPMCU
dev boards.
BRANCH=none
BUG=b:178124518
TEST=view in gitiles
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I73e0774e0721001f229e3a608262a64129d2ab44
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2643588
Reviewed-by: Josie Nordrum <josienordrum@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a new section to the Ztest documentation providing details of
what sorts of EC unit tests are not good candidates for porting
to Ztest.
BUG=b:172240633, b:172240757
BRANCH=None
TEST=N/A
Signed-off-by: Paul Fagerburg <pfagerburg@google.com>
Change-Id: I3010dfb338e8e9c835c55b10a79a2e4dce703f96
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2648397
Tested-by: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Paul Fagerburg <pfagerburg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=b:172240633
BRANCH=none
TEST=build for both EC and Ztest:
`TEST_LIST_HOST=accel_cal make runhosttests`
`zmake configure --test -B build/accel_cal zephyr/test/accel_cal`
Signed-off-by: Paul Fagerburg <pfagerburg@google.com>
Change-Id: Ie787403b36d775d4185cafad8ca134561a97794b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2645198
Tested-by: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Paul Fagerburg <pfagerburg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change refactors test functionality in test_util.h to better
accomomdate zTests. This is done by:
* Removing the shim version of test_util.h. This was causing a
conflict that made it harder to tell what's being used. This
involved migrating some needed code over:
- Defining different TASK_PARAMS.
- Defining test_pass for Zephyr tests.
* Creating a macro (DECLARE_EC_TEST) that will automatically
handle creating the individual test functions for both
platform and Zephyr tests.
* Creating a macro (TEST_MAIN) that will automatically handle
creating the main test entry function. This use to be
test_main(void) for Zephyr and run_test(int, char**) for
platform/ec. To do this we'll be removing the int, char**
arguments from platform/ec. This may result in some tests
having to be refactored, but overall should improve the
test codebase as tests should remain deterministic (i.e.
not depend on any outside arguments/parameters).
* Creating some common ztest_ function/macros that will
allow writing platform/ec tests in a zephyr like style.
see test/base32.c for an example.
* Update the type of __shared_mem_buf to match Zephyr. This
was causing an issue now with the full test_util.h in
zephyr/test/system/.
BRANCH=none
BUG=b:168032590
TEST=make runhosttests
TEST=zmake configure --test -B build/host/base32 zephyr/test/base32
Signed-off-by: Yuval Peress <peress@chromium.org>
Change-Id: I72173a3e94c7df09a2966e7ffeb9f5668d030f29
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2634401
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In Zephyr CONFIG_FLASH_SIZE is a Kconfig value that is used
throughout. The issue is that the units don't match. In
Zephyr the value is in KiB instead of bytes. This refactor
simply renames CONFIG_FLASH_SIZE in platform/ec to include
the unit (via _BYTES).
BRANCH=none
BUG=b:174873770
TEST=make buildall
be generated by the build instead of per board
Signed-off-by: Yuval Peress <peress@chromium.org>
Change-Id: I44bf3c7a20fcf62aaa9ae15715be78db4210f384
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2627638
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adding diagram for Icetower in CQ.
BUG=b:171259291
TEST=rendered with gittiles
Change-Id: I3a1d10deb10e19aaa4552e9c04addd4853ea0067
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2593947
Tested-by: Aadit Maheshkumar Modi <modia@chromium.org>
Reviewed-by: Yicheng Li <yichengli@chromium.org>
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
Commit-Queue: Aadit Maheshkumar Modi <modia@chromium.org>
|