summaryrefslogtreecommitdiff
path: root/include
Commit message (Collapse)AuthorAgeFilesLines
* include/config.h: Format with clang-formatJack Rosenthal2022-06-281-269/+227
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: I69b033fd16f138d78ad024093968bd2236ea4f8d Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730232 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/port80.h: Format with clang-formatJack Rosenthal2022-06-281-4/+4
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: I5f6a2d5318d8c984a24a22cf4f5b743ff8bea33f Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730379 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/cros_version.h: Format with clang-formatJack Rosenthal2022-06-281-3/+3
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: I53ceccc33a80ad48246c8d5aba72640fa67be5a4 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730236 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* Revert "trng: Rename rand to trng_rand"Pin-yen Lin2022-06-281-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit a6b0b3554f59cc9b0c4aae9bff7dff075f2089a9. Reason for revert: This CL breaks ec-utils-test package and affects CQ Original change's description: > trng: Rename rand to trng_rand > > The declaration for rand conflicts with the standard library declaration > so rename it from "rand" to "trng_rand". This has the benefit of making > it obvious when we're using the true random number generator. > > For consistency, this also renames init_trng/exit_trng to > trng_init/trng_exit. > > BRANCH=none > BUG=b:234181908 > TEST=./util/compare_build.sh -b all -j 120 > => MATCH > > Signed-off-by: Tom Hughes <tomhughes@chromium.org> > Change-Id: Ic3305a91263c45786c051eaa5b3689e7464aa0ab > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3712032 > Reviewed-by: Bobby Casey <bobbycasey@google.com> > Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Bug: b:234181908,b:237344361 Change-Id: Iceae55ad9ff0e6aa98aebd474f92a98e12c287e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3726597 Tested-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Pin-yen Lin <treapking@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Owners-Override: Pin-yen Lin <treapking@chromium.org> Commit-Queue: Pin-yen Lin <treapking@chromium.org> Tested-by: Pin-yen Lin <treapking@chromium.org> Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
* include/usb_pd_tbt.h: Format with clang-formatJack Rosenthal2022-06-281-1/+1
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: I67e5873406379e0d25b963e1fb70d699cd2aaf61 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730442 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/mock/tcpc_mock.h: Format with clang-formatJack Rosenthal2022-06-281-2/+1
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: Id5cdfc07bb16204d7d655bba48b2eb2d5b74b8ba Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730361 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/mock/fp_sensor_mock.h: Format with clang-formatJack Rosenthal2022-06-281-14/+15
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: I160cb8173bd6fabcba022e99d173526f8f91cf8a Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730357 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/charge_state_v1.h: Format with clang-formatJack Rosenthal2022-06-281-16/+14
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: I23ee50668dad6c9ff6ef445a7e0570e3c64047bf Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730223 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/crypto_api.h: Format with clang-formatJack Rosenthal2022-06-281-2/+2
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: I8aeea495fd280b3470dd01221e9977e00581bbea Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730237 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/task_filter.h: Format with clang-formatJack Rosenthal2022-06-281-1/+0
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: I4cdfb11747141934aa0a71c057d14c64865f7b27 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730417 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/dptf.h: Format with clang-formatJack Rosenthal2022-06-281-5/+5
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: Idc5d29926272f7c2ccd2f7eef1e2eb4257126e02 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730243 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/driver/mag_bmm150.h: Format with clang-formatJack Rosenthal2022-06-281-75/+71
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: Ic3d9ae47d89bdd2ab537680468241c07a8580b35 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730249 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/keyboard_8042.h: Format with clang-formatJack Rosenthal2022-06-281-1/+1
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: If7f0a58f87f56f1b4916e8aecb39dfd7a06f8a8e Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730294 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/usb_charge.h: Format with clang-formatJack Rosenthal2022-06-281-11/+11
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: Ifc8e4d020bbe41adfcb80dd5a83375fb0f1f561f Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730429 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/charge_state_v2.h: Format with clang-formatJack Rosenthal2022-06-281-2/+2
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: If35dbe7cab1bed2d744538624ba4ad1cf0471691 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730224 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/sha256.h: Format with clang-formatJack Rosenthal2022-06-281-2/+2
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: If558fecc0ee88def67289b0cc472d53ecbc7fca6 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730404 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* trng: Rename rand to trng_randTom Hughes2022-06-281-7/+5
| | | | | | | | | | | | | | | | | | | | The declaration for rand conflicts with the standard library declaration so rename it from "rand" to "trng_rand". This has the benefit of making it obvious when we're using the true random number generator. For consistency, this also renames init_trng/exit_trng to trng_init/trng_exit. BRANCH=none BUG=b:234181908 TEST=./util/compare_build.sh -b all -j 120 => MATCH Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: Ic3305a91263c45786c051eaa5b3689e7464aa0ab Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3712032 Reviewed-by: Bobby Casey <bobbycasey@google.com> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* include/lpc.h: Format with clang-formatJack Rosenthal2022-06-281-1/+1
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: If90187409f944f0d8fc95a8badfb287802a3a9e2 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730347 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/driver/retimer/bb_retimer.h: Format with clang-formatJack Rosenthal2022-06-271-33/+33
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: Ifdf62770a7960d25b08941704789c33dd4d78df2 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730251 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* include/power/icelake.h: Format with clang-formatJack Rosenthal2022-06-271-5/+5
| | | | | | | | | | | BUG=b:236386294 BRANCH=none TEST=none Change-Id: I0f905ac3f279a91510ef18eb8234931ad976e912 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3730386 Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
* zephyr: tests: Add Battery Capability Data Block struct definitionTristan Honscheid2022-06-271-0/+28
| | | | | | | | | | | | | | | Add a struct that models the fields of a USB-PD Battery Capability Data Block per USB Power Delivery Specification Revision 3.0, Version 1.1, table 6-47 BUG=b:223452169 BRANCH=None TEST=zmake test test-drivers Signed-off-by: Tristan Honscheid <honscheid@google.com> Change-Id: I31510c994a710b9d9e788c40e9be9942df265c8b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3715697 Reviewed-by: Abe Levkoy <alevkoy@chromium.org>
* zephyr: tests: Add USB-PD rev 3.0 extended msg support to emulTristan Honscheid2022-06-231-5/+7
| | | | | | | | | | | | | | | | | | | This CL adds functionality to the port partner emulator common code for sending PD revision 3 extended messages. * Add a new function for sending extended messages * Refactor the allocator function into one for extended messages and another for standard messages * Update set_header function to fill the extended bit properly BUG=b:223452169 BRANCH=None TEST=zmake test test-drivers Signed-off-by: Tristan Honscheid <honscheid@google.com> Change-Id: Ieadfd523a8ab0686a2ba88f2fe1e575d0e9db481 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3706077 Reviewed-by: Abe Levkoy <alevkoy@chromium.org>
* USB Mux: Add feature flag for AP mux settingDiana Z2022-06-221-0/+4
| | | | | | | | | | | | | | | Add a new feature flag to indicate when the AP is permitted to perform mux sets on a board. BRANCH=None BUG=b:208882941 TEST=on nipperkin with DNS patch, ensure feature flag is reported Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: I62c236ee43fc634f9d25d96918ce64bdc8cb2341 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3712025 Reviewed-by: Abe Levkoy <alevkoy@chromium.org> Commit-Queue: Abe Levkoy <alevkoy@chromium.org>
* zephyr:test: Add tests for EC_CMD_HOST_EVENT commandAl Semjonovs2022-06-221-0/+7
| | | | | | | | | | | | | | | | Validate permutations of EC_CMD_HOST_EVENT. BUG=b:236161217 BRANCH=none TEST=zmake test test-drivers and shuffled with arg -test="host_cmd_host_event_commands::*" Signed-off-by: Al Semjonovs <asemjonovs@google.com> Change-Id: Id8776922c5b0a09d786339a8e46c1a72e2109695 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3716793 Reviewed-by: Aaron Massey <aaronmassey@google.com> Commit-Queue: Aaron Massey <aaronmassey@google.com>
* hook: Add HOOK_POWER_SUPPLY_CHANGEDaisuke Nojiri2022-06-222-0/+7
| | | | | | | | | | | | | | | | | This patch adds HOOK_POWER_SUPPLY_CHANGE. It triggers when there is a change in the active charge port. BUG=None BRANCH=None TEST=buildall TEST=zephyr/firmware_builder.py --metrics /tmp/tmpb1k9vctu --code-coverage build Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org> Change-Id: I77fda265c323e4101c8cc95ed15be714b8989c85 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3716790 Commit-Queue: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
* power: Drop support for Braswell power sequencingJack Rosenthal2022-06-201-3/+0
| | | | | | | | | | | | | | We don't have any supported Braswell boards remaining, delete the unused code. BUG=none BRANCH=none TEST=make buildall Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: Ica4ce5c53e7e8a5b727ae23e9bcd3fbc411a594f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3708434 Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
* charge_manager: Make chgsup print moreDaisuke Nojiri2022-06-191-1/+39
| | | | | | | | | | | | | | | | | | | | | | | | | This patch makes chgsup command print all available charge port information. > chgsup Port --Supplier-- Prio -Avalable Power- P0 PD 1 3000mA 15000mV USBC_1_5A 4 500mA 5000mV VBUS 4 500mA 5000mV *P2 DEDICATED 0 7700mA 19500mV Left safe mode Override port = -1 BUG=None BRANCH=None TEST=See above Change-Id: I3531b445f8d79c910150db643363aa84b74aee94 Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3710279 Reviewed-by: Keith Short <keithshort@chromium.org>
* charge_state: Remove unnecessary macro guardsDaisuke Nojiri2022-06-151-3/+1
| | | | | | | | | | | | | | | | | | Declarations don't needed to be guarded unless there is a conflicting declaration. This patch removes guards for charge_get_percent and board_get_battery_soc in charge_state.h. This patch allows the aforementioned declarations to be used in unit tests without declaring CONFIG_CHARGER or CONFIG_BATTERY. BUG=None BRANCH=None TEST=buildall Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org> Change-Id: Iee833210e00b34df874c89cacfc49d45253b5600 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3704269 Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
* test: Relocate test_mockable for better compatibilityDaisuke Nojiri2022-06-151-16/+16
| | | | | | | | | | | | | | | | test_mockable macros are placed after config.h. This causes header files included indirectly by common.h not to be able to use the macros. This patch places test_mockable macros before config.h in common.h so that the macros will be defined for aforementioned header files. BUG=None BRANCH=None TEST=buildall Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org> Change-Id: I05716931bc7aedd3171569189ae228d2e9052b13 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3704267 Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
* chipset: introduce CONFIG_AP_POWER_CONTROLPeter Marheine2022-06-154-9/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | Because the new Zephyr power sequencing support is gated on a different config option than HAS_TASK_CHIPSET which code has historically assumed implies there is an AP present, it is now easy to introduce bugs when code that applies in both configurations uses the old option test. This change decouples the presence of an AP from HAS_TASK_CHIPSET, introducing a new CONFIG_AP_POWER_CONTROL symbol that is derived from the power sequencing config options. All existing applicable users of HAS_TASK_CHIPSET are changed to use the new symbol, fixing several callers which would not behave correctly under Zephyr with the new power sequencing code. The duplicate stub implementations of functions provided by Zephyr's chipset_api are removed, because they already appear in the header that declares those functions. BUG=b:233681784 TEST=make buildall, zmake testall BRANCH=none Signed-off-by: Peter Marheine <pmarheine@chromium.org> Change-Id: I282da30839ca52fcc88c6f9dea2bd00d4811b976 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3670735 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Remove ec_version.h dependency on common/system.cTristan Honscheid2022-06-131-0/+2
| | | | | | | | | | | | | | | | | | | | | | common/system.c includes ec_version.h to access its definition of CROS_FWID_MISSING_STR. However, this is just a constant placeholder for cases when an actual version string is not available. It does not need to placed into the auto-generated ec_version.h header. * Remove the code that adds the CROS_FWID_MISSING_STR definition in zmake * Place the #define macro into cros_version.h, which is included by common/system.c already. * Update zmake unit tests BRANCH=None BUG=None TEST=zmake test test-drivers Signed-off-by: Tristan Honscheid <honscheid@google.com> Change-Id: I8bb8406823e7860162157c1baaa21440608f3eea Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3656257 Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* chargesplash: Change host command number to 0x00A5Jack Rosenthal2022-06-101-1/+1
| | | | | | | | | | | | | | | EC_CMD_CHARGESPLASH is using the same number as EC_CMD_HOST_EVENT. Assign an unused number to EC_CMD_CHARGESPLASH. BUG=b:235531569 BRANCH=none TEST=CQ Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: I085991ffe42c102149cb0e7a876522c76642c50e Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3699312 Commit-Queue: Diana Z <dzigterman@chromium.org> Reviewed-by: Diana Z <dzigterman@chromium.org>
* RGBKBD: Add TLC59116F driverjohnwc_yeh2022-06-101-0/+1
| | | | | | | | | | | | | | TLC59116F is a LED driver which can control 16 LEDs. BUG=b:223526803 BRANCH=None TEST=make BOARD=mithrax Signed-off-by: johnwc_yeh <johnwc_yeh@compal.corp-partner.google.com> Change-Id: I24865c5af796cf918af1f361ba1a1f87bf5923f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3670621 Reviewed-by: Ko Ko <ko_ko@compal.corp-partner.google.com> Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
* timer.h: Avoid redefining clock_t when compiling with libcTom Hughes2022-06-091-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | Move the clock_t typedef to the "builtin" directory, which the directory containing headers that mirror those in the C standard library. The "builtin" directory can then be excluded when building with the standard library. include/timer.h:31:14: error: typedef redefinition with different types ('long' vs 'unsigned long') typedef long clock_t; ^ /usr/armv7m-cros-eabi/include/sys/types.h:107:19: note: previous definition is here typedef _CLOCK_T_ clock_t; ^ BRANCH=none BUG=b:234181908 TEST=./util/compare_build.sh -b all -j 120 => MATCH Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: Ib2381664535781035b367b414c5f00639ce9b9af Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3691958 Reviewed-by: Edward Hill <ecgh@chromium.org>
* it8xxx2: The "M" extension is disabled by defaultDino Li2022-06-091-0/+6
| | | | | | | | | | | | | | | | | | | | | There is a mul instruction bug. The bug may cause instructions of writing back CPU GPR (e.g mv a0,s2) which following the mul instruction to fail. This patch disables the 'M' extension and overwrite integer multiplication and division arithmetic library routines with using hardware multiplication and division and nop instructions. This will ensure that there is no write back GPR instruction to follow mul instruction to avoid the bug. BUG=b:235297478 BRANCH=asurada,cherry,icarus TEST=- buildall - The "M" extension is disabled on cherry image (-march=rv32iac) Signed-off-by: Dino Li <Dino.Li@ite.com.tw> Change-Id: I39b34a91dd77d975b78b6756494691c6b28dc42d Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3690042 Reviewed-by: Eric Yilun Lin <yllin@google.com>
* config: riscv: add CONFIG_RISCV_EXTENSION_M configurationDino Li2022-06-091-0/+6
| | | | | | | | | | | | | | This makes the 'M' extension to be configurable. BUG=b:235297478 BRANCH=asurada,cherry,icarus TEST=- buildall - ISA for cherry images includes "M" extension (-march=rv32imac) Signed-off-by: Dino Li <Dino.Li@ite.com.tw> Change-Id: I56957a6767378121443659a170ca33896ada67ce Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3690041 Reviewed-by: Eric Yilun Lin <yllin@google.com>
* include/host_command.h: Fix C++ compilation errorTom Hughes2022-06-081-1/+1
| | | | | | | | | | | | | | | | | | include/host_command.h:277:41: error: invalid suffix on literal; C++11 requires a space between literal and identifier [-Wreserved-user-defined-literal] __attribute__((section(".rodata.hcmds."EXPANDSTR(0x0000, command)))) \ ^ BRANCH=none BUG=b:234181908 TEST=./util/compare_build.sh -b all -j 120 => MATCH Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: I7428c6bf5fa6aa0c9440ae44d665f7d744d23c00 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3691834 Reviewed-by: Ting Shen <phoenixshen@chromium.org>
* include/console.h: Use _STATIC_ASSERT for C++ compatibilityTom Hughes2022-06-081-1/+1
| | | | | | | | | | | | BRANCH=none BUG=b:234181908 TEST=./util/compare_build.sh -b all -j 120 => MATCH Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: I92ffb946b6b0cc6953fc12b7221267b5bf3abc91 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3689785 Reviewed-by: Diana Z <dzigterman@chromium.org>
* RGBKBD: Set scale per keyDaisuke Nojiri2022-06-072-5/+20
| | | | | | | | | | | | | | | | | Currently, rgbkbd_set_scale can only set a scale of all keys. This patch makes it set a scale of an individual key. rgbkbd_reset_scale will replace the previous rgbkbd_set_scale. BUG=b:233986174, b:234530397 BRANCH=None TEST=Run ectool rgbkbd scale on Vell. TEST=Run ectool rgbkbd scale on Taniks. TEST=Run rgb all, demo, reset, scale, command in EC console. Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org> Change-Id: I34e6b3e41c40510eca148fcf879e8caaf57317be Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3689778 Reviewed-by: Parth Malkan <parthmalkan@google.com>
* RGBKBD: Add demo sub-command to EC_CMD_RGBKBDDaisuke Nojiri2022-06-072-8/+12
| | | | | | | | | | | BUG=None BRANCH=None TEST=Vell Change-Id: Ie486b9ae6731622882b21d5273f79280770bbd17 Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3657839 Reviewed-by: Parth Malkan <parthmalkan@google.com>
* mkbp_info: Provide run time check volume buttons functionKnox Chiou2022-06-071-0/+22
| | | | | | | | | | | | | | | | | Define a volume button override function for board specific usage, so the devices whom share the firmware could config volume buttons by run time. BUG=b:234175434 BRANCH=trogdor TEST=emerge-trogdor chromeos-ec & ectool mkbpget buttons on lazor. Signed-off-by: Knox Chiou <knoxchiou@google.com> Change-Id: I18cd25c25183f8149650ed2302a476242b2680e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3674845 Commit-Queue: Wai-Hong Tam <waihong@google.com> Tested-by: Knox Chiou <knoxchiou@chromium.org> Reviewed-by: Wai-Hong Tam <waihong@google.com>
* ANX7483: Add EQ tuning interfaceDiana Z2022-06-071-0/+34
| | | | | | | | | | | | | | Add an API which may be used to adjust the EQ settings of pins on the retimer individually. BRANCH=None BUG=b:230694492 TEST=zmake testall Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: I26ddb01cb3d84ef074545d05a5836ad6e385ed92 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3689783 Reviewed-by: Peter Marheine <pmarheine@chromium.org>
* zephyr: shim: add single charge task supportFabio Baltieri2022-06-032-9/+20
| | | | | | | | | | | | | | | | | | | | | | | | Add an option to run the charger code in a single task. This is enabled by default in Zephyr builds, but explicitly disabled for a couple of platforms that currently have task dependent board code. Should be enough to start testing the configuration, will work on handling the remaining boards (corsola and nissa) later. BRANCH=none BUG=b:226411332 TEST=make buildall TEST=zmake testall TEST=cq dry run TEST=run on Brya, connect/disconnect usb devices Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com> Change-Id: I75a63f7b0a9545e6c824114de7f81b71924e0789 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3663748 Reviewed-by: Keith Short <keithshort@chromium.org> Reviewed-by: Diana Z <dzigterman@chromium.org> Reviewed-by: Aaron Massey <aaronmassey@google.com>
* zephyr: LED: Define battery-level property as int rangeParth Malkan2022-06-031-6/+18
| | | | | | | | | | | | | | | | Currently battery-level property is defined using enums. This patch changes this to use int range instead. BRANCH=None BUG=b:233953617 TEST=Manual test on Lazor by modifying nodes to include battery-level range, use battfake to set different levels and observe LED behavior. Cq-Depend: chromium:3669089, chromium:3669090 Signed-off-by: Parth Malkan <parthmalkan@google.com> Change-Id: Idfe516addfe20d5141e5106afd338b49f10a6349 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3669088 Reviewed-by: Wai-Hong Tam <waihong@google.com>
* USB Mux: Set an event when AP mux sets completeDiana Z2022-06-021-0/+2
| | | | | | | | | | | | | | | | | The AP will need to know when its selected mux has finished setting. For now, most boards have at most one mux and one retimer, and so add events for each of those. BRANCH=None BUG=b:208882941,b:155889939 TEST=on nipperkin, set muxes via the AP console and view the corresponding events are set Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: I52fcfe861f80e60bd01ab4bd610edb7cadf1e049 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3687080 Commit-Queue: Abe Levkoy <alevkoy@chromium.org> Reviewed-by: Abe Levkoy <alevkoy@chromium.org>
* TCPMv2: Add a host command to allow the AP to set muxesDiana Z2022-06-021-0/+8
| | | | | | | | | | | | | | | Add a sub-command onto the TYPEC_CONTROL command in order to permit the AP to set muxes and retimers. BRANCH=None BUG=b:208882941,b:155889939 TEST=on nipperkin, ensure muxes can be changed from ectool Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: I08ed46474693ec09b1db155b469b83d4acec8fbb Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3687079 Commit-Queue: Abe Levkoy <alevkoy@chromium.org> Reviewed-by: Abe Levkoy <alevkoy@chromium.org>
* USB mux: Allow single muxes or retimers to be setDiana Z2022-06-021-0/+12
| | | | | | | | | | | | | | | | | The AP may want to only set a single mux or retimer in a chain, so create an interface to allow this to happen. Existing APIs will default to setting all muxes in the chain, as they did before. BRANCH=None BUG=b:208882941,b:155889939 TEST=make -j buildall, on nipperkin ensure a monitor still works normally Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: Iebfa55d9ad1b89c9ea0f626728c9f2028b490ac1 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3687077 Commit-Queue: Abe Levkoy <alevkoy@chromium.org> Reviewed-by: Abe Levkoy <alevkoy@chromium.org>
* USB mux: Add CONFIG for AP driven mux controlDiana Z2022-06-021-0/+3
| | | | | | | | | | | | | | Add a CONFIG option to support the AP directing mux and retimer sets. BRANCH=None BUG=b:208882941,b:155889939 TEST=make -j buildall Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: Ic42e256126f68684bbfdd27654f41c0e5290caf9 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3687076 Commit-Queue: Keith Short <keithshort@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org>
* task: change task_set_event return type to voidFabio Baltieri2022-06-022-3/+2
| | | | | | | | | | | | | | | No implementation does nothing other than return 0 on task_set_event(), change the return type to void so that the API is clear. BRANCH=none BUG=none TEST=cq dry run Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com> Change-Id: I88b20f53aa6dffa291ce78f4776888b21985f57f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3683769 Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Keith Short <keithshort@chromium.org>
* ec: Change EC_SPI_RX_READY host commandSam Hurst2022-05-311-2/+6
| | | | | | | | | | | | | | | | The EC_SPI_RX_READY macro has been changed so that MSB is low if it is placed on the SDO lin when the CS goes low. BUG=b:216831450 BRANCH=none TEST=zmake testall Verified on scope that SHI_SDO does not pulse high during AP Shutdown Signed-off-by: Sam Hurst <shurst@google.com> Change-Id: I851633855e1c2bcb91f98ca81612cbdea8330ee0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3668647 Reviewed-by: Wai-Hong Tam <waihong@google.com>
View Lorry Controller Status