summaryrefslogtreecommitdiff
path: root/docs
Commit message (Collapse)AuthorAgeFilesLines
* 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-ishJack Rosenthal2021-11-0591-17425/+0
| | | | | | | | | | | | | | | | | | | | | | 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>
* docs: replace "master" with "HEAD" in URLsTom Hughes2021-09-2811-18/+18
| | | | | | | | | | | | | | 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>
* docs/fingerprint: replace master with HEAD in URLsTom Hughes2021-09-286-60/+60
| | | | | | | | | | | | | | 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>
* docs/fingerprint: Fix linkTom Hughes2021-09-101-2/+2
| | | | | | | | | | | | | 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>
* docs: Update Reducing EC Image SizeKeith Short2021-08-061-3/+98
| | | | | | | | | | | | | | | | 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>
* docs: Fix link referencesKeith Short2021-08-062-4/+4
| | | | | | | | | | | | | | 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>
* COIL: Change host interface option to CONFIG_HOSTCMD_SHIKeith Short2021-08-032-1/+10
| | | | | | | | | | | | | | 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>
* docs: Add document on reducing EC image sizeKeith Short2021-08-032-0/+365
| | | | | | | | | | | | | | 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>
* util/flash_jlink: Change JLinkRemoteServer default portCraig Hesling2021-07-271-1/+3
| | | | | | | | | | | | | | | | 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>
* keyboard: Use __override for keyboard_scan_configDaisuke Nojiri2021-07-231-2/+2
| | | | | | | | | | | | | | | | | 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>
* fingerprint: Rename sleep lines to be more genericCraig Hesling2021-07-011-4/+4
| | | | | | | | | | | | | | 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>
* config: Rename CONFIG_CROS_BOARD_INFOPhilip Chen2021-06-161-1/+1
| | | | | | | | | | | | | | | | | | 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>
* zephyr: zmake: Add a test for zmake and output filteringSimon Glass2021-05-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* docs/fingerprint: Add TPM Seed design docTom Hughes2021-04-283-2/+326
| | | | | | | | | | | | 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>
* Cleanup v .1/ v . 2 diagram to match current stateAadit Modi2021-04-274-8/+2
| | | | | | | | | | | | | | | 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>
* Revert "Reland "zephyr: zmake: Add a test for zmake and output filtering""Jack Rosenthal2021-04-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* Reland "zephyr: zmake: Add a test for zmake and output filtering"Simon Glass2021-04-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* Revert "zephyr: zmake: Add a test for zmake and output filtering"Jack Rosenthal2021-04-131-1/+1
| | | | | | | | | | | | | | | 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>
* zephyr: zmake: Add a test for zmake and output filteringSimon Glass2021-04-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* hammer: create a public hammer care & feeding docTing Shen2021-04-091-0/+199
| | | | | | | | | | | | | | | | | 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>
* zephyr: zmake: Avoid using pip3 for installSimon Glass2021-04-071-3/+2
| | | | | | | | | | | | | | | | | | 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>
* zephyr: Add build instructionsSimon Glass2021-04-071-0/+165
| | | | | | | | | | | | | 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>
* zephyr: doc: Add a title to the shimming docSimon Glass2021-04-061-47/+49
| | | | | | | | | | | | | | | | | 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>
* zephyr: doc: Add zephyr docs to sitemapSimon Glass2021-04-061-0/+7
| | | | | | | | | | | | | | 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>
* ec: Add instructions on debugging testsJeremy Bettis2021-04-061-0/+17
| | | | | | | | | | | | | | | | 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>
* zephyr: Add zmake command coverage.Jeremy Bettis2021-04-021-29/+12
| | | | | | | | | | | | | | | | | | | 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>
* Sensors in fingerprint CQ Prototype EnvironmentAadit Modi2021-03-312-0/+0
| | | | | | | | | | | | | | | 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>
* zephyr: Generate initial coverage for every boardJeremy Bettis2021-03-301-2/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* [platform/ec/docs] - Update link on partnersite.Stuart Langley2021-03-301-1/+1
| | | | | | | | | | | | | | 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>
* zephyr: npcx: add reset cause checkWealian Liao2021-03-241-0/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* zephyr: Add util to normalize symlinks from lcovJeremy Bettis2021-03-181-0/+13
| | | | | | | | | | | | | | | | | | 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>
* zephyr: add a utility to convert gpio.inc to device treeJack Rosenthal2021-03-111-0/+5
| | | | | | | | | | | | | | | 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>
* docs: zephyr: add some documentation on PoC device bringupJack Rosenthal2021-03-112-0/+50
| | | | | | | | | | | | | | | | 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>
* zephyr: Add a step-by-step guideSimon Glass2021-03-111-0/+42
| | | | | | | | | | | | | | 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>
* zephyr: extend the shim documentation with go/zephyr-shim-how-toJack Rosenthal2021-03-041-0/+166
| | | | | | | | | | | | | | | | | | 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>
* docs/fingerprint: Update power numbers for dartmonkeyTom Hughes2021-02-271-2/+23
| | | | | | | | | | | | | | | | 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>
* docs/fingerprint: Add Chrome OS Project Configuration detailsTom Hughes2021-02-261-1/+16
| | | | | | | | | | | | | | 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>
* docs/fingerprint: Add information on fingerprint SKUsTom Hughes2021-02-261-0/+12
| | | | | | | | | | | | 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>
* Docs: Update source-out policy noteDiana Z2021-02-191-2/+3
| | | | | | | | | | | | | | 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>
* docs/fingerprint: Update power numbers for dragonclaw/bloonchipperTom Hughes2021-02-121-19/+26
| | | | | | | | | | | 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>
* docs/fingerprint: Update power numbers for icetower/dartmonkeyTom Hughes2021-02-121-11/+17
| | | | | | | | | | | 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>
* Docs: Update TCPMv2 source policy to include FRSDiana Z2021-02-064-37/+63
| | | | | | | | | | | | | | | | | 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>
* Update FPMCU console UART name and power railTom Hughes2021-01-291-1/+1
| | | | | | | | | | | | | | | | 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>
* docs: Run mdformat on all .md filesTom Hughes2021-01-2832-989/+1031
| | | | | | | | | | | 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>
* docs: Add instructions on using the flash_jlink.py scriptTom Hughes2021-01-262-14/+95
| | | | | | | | | | | | | | 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>
* docs: add porting considerations for ZtestPaul Fagerburg2021-01-261-0/+29
| | | | | | | | | | | | | | | | | 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>
* test: port accel_cal to ZtestPaul Fagerburg2021-01-221-6/+21
| | | | | | | | | | | | | | | 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>
* common:test: refactor test_util.h to accommodate ZephyrYuval Peress2021-01-211-33/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* Refactor CONFIG_FLASH_SIZE to CONFIG_FLASH_SIZE_BYTESYuval Peress2021-01-151-1/+1
| | | | | | | | | | | | | | | | | | | | | 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>
* Icetower in CQ Prototype EnvironmentAadit Modi2021-01-052-0/+6
| | | | | | | | | | | | | | 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>