| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
BRANCH=none
TEST=make buildall -j
Change-Id: I6b1cd6968f38eeb96198153a124b11affea2b0a2
Signed-off-by: Parth Malkan <parthmalkan@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3205483
Reviewed-by: Wai-Hong Tam <waihong@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The `size` field of the cros flash driver was from the days of old where
we weren't too familiar with DT_ options. Consolidate the binding to
make use of the register's size.
BRANCH=none
BUG=b:202295086
TEST=zmake testall
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I86c312c0d7e7dc798833b47df333cb2499062aaf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3208912
Reviewed-by: Wai-Hong Tam <waihong@google.com>
Commit-Queue: Wai-Hong Tam <waihong@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
CONFIG_RO_HEAD_ROOM and CONFIG_RW_HEAD_ROOM are always defined as 0, so
remove the values and associated code in order to simplify the code and
improve readability.
BRANCH=none
BUG=b:172020503
TEST=./util/compare_build.sh -b all
=> MATCH
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Ie11d23befda674cc15dda9a2d66b9c43ea22d49e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3205489
Reviewed-by: Ting Shen <phoenixshen@chromium.org>
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
"enum ec_led_colors" was used here instead of "enum led_color".
Fixes the following clang warning:
error: implicit conversion from enumeration type
'enum led_color' to different enumeration type 'enum ec_led_colors'
[-Werror,-Wenum-conversion]
This is the same fix as commit 036cda45bba9fca0137add1d8693a264ba686d71.
BRANCH=none
BUG=b:172020503
TEST=make CC=arm-none-eabi-clang BOARD=dratini
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Ide9188bef86b44041cf214405aee9461a3fbd917
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3205488
Reviewed-by: Eric Yilun Lin <yllin@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When building with clang, it reports:
common/usbc/usb_prl_sm.c:2304:44: error: 'prl_tx_states' causes a
section type conflict with 'flag_bit_names'
static __const_data const struct usb_state prl_tx_states[] = {
^
common/usbc/usb_prl_sm.c:131:37: note: declared here
static __const_data struct bit_name flag_bit_names[] = {
^
common/usbc/usb_prl_sm.c:2346:44: error: 'prl_hr_states' causes a
section type conflict with 'flag_bit_names'
static __const_data const struct usb_state prl_hr_states[] = {
^
common/usbc/usb_prl_sm.c:131:37: note: declared here
static __const_data struct bit_name flag_bit_names[] = {
^
The issue is that flag_bit_names is not const. When making
flag_bit_names const, we must also make the argument to print_bits const
to fix the additional warning:
common/usbc/usb_prl_sm.c:169:56: error: passing 'const struct bit_name
[11]' to parameter of type 'struct bit_name *' discards qualifiers
[-Werror,-Wincompatible-pointer-types-discards-qualifiers]
print_bits(group, set_or_clear ? "Set" : "Clr", flag, flag_bit_names,
^~~~~~~~~~~~~~
common/usbc/usb_prl_sm.c:149:28: note: passing argument to parameter 'names' here
struct bit_name *names,
BRANCH=none
BUG=b:172020503
TEST=make CC=arm-none-eabi-clang BOARD=cret
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I95d28f53755dd23fe288ae70055e09fcf44511ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3205487
Reviewed-by: Edward Hill <ecgh@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The interrupt seems to be very sensitive to cpu load and when running
for coverage in gitlab, it often times out. Change to skip this test
in coverage builds, since it is unlikely to add any coverage for zephyr
used code anyway.
BRANCH=None
BUG=None
TEST=None
Change-Id: Iefe0da6b925cc820f764c1ad5a79187b8f09885f
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3208910
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
Commit-Queue: Abe Levkoy <alevkoy@chromium.org>
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Abe Levkoy <alevkoy@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit removes the prompt for HAS_TASK_* Kconfigs.
They are enabled automatically when corresponding feature is enabled.
BRANCH=main
BUG=b:198777314
TEST=Verify that all tasks are created correctly. USB PD tasks
should match amount or usb-c ports count.
Change-Id: Ia69f9762cb8953df99d12b772f607d26df1fe379
Signed-off-by: Michał Barnaś <mb@semihalf.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3197726
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The port number passed via the I2C control host command may be different
than the one used in ZephyrEC so convert it based on the "remote-port"
property in DTS.
BUG=b:201408931
BRANCH=none
TEST=zmake testall
Signed-off-by: Dawid Niedzwiecki <dn@semihalf.com>
Change-Id: I71e05df2eb4f76b8c93948a97f47193aaaa6e25e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3205251
Reviewed-by: Yuval Peress <peress@google.com>
Commit-Queue: Dawid Niedzwiecki <dawidn@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=b:187189596
BRANCH=none
TEST=verify LEDs on asurada are working
Signed-off-by: Denis Brockus <dbrockus@google.com>
Change-Id: I9bb42099ff58932d06c8e48087f8aa44665b0f57
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3205482
Tested-by: Denis Brockus <dbrockus@chromium.org>
Auto-Submit: Denis Brockus <dbrockus@chromium.org>
Reviewed-by: Yuval Peress <peress@google.com>
Commit-Queue: Yuval Peress <peress@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add CONFIG_PLATFORM_EC_LED_ONOFF_STATES and
CONFIG_PLATFORM_EC_LED_ONOFF_STATES_BAT_LOW
BUG=none
BRANCH=none
TEST=make buildall
TEST=zmake testall
Signed-off-by: Denis Brockus <dbrockus@google.com>
Change-Id: Iaa858b09774b4caf5fe0a2f2bba4e72532990921
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3205481
Tested-by: Denis Brockus <dbrockus@chromium.org>
Auto-Submit: Denis Brockus <dbrockus@chromium.org>
Reviewed-by: Yuval Peress <peress@google.com>
Commit-Queue: Yuval Peress <peress@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Enable the Zephyr Kconfig for CBI from strapping pins.
BRANCH=None
BUG=b:186264627, b:193807794
TEST=Built herobrine_npcx9 Zephyr image. Verified the cbi command:
uart:~$ cbi
[59342.269700 CBI Reading board info]
CBI_VERSION: 0x0000
TOTAL_SIZE: 20
BOARD_VERSION: 0 (0x0)
OEM_ID: (Error 1)
MODEL_ID: (Error 1)
SKU_ID: 0 (0x0)
FW_CONFIG: (Error 1)
PCB_SUPPLIER: (Error 1)
SSFC: (Error 1)
REWORK_ID: (Error 1)
43 42 49 a8 00 00 14 00 00 04 00 00 00 00 02 04 |CBI.............|
00 00 00 00 |.... |
Change-Id: I4a6acbc7a1fdeb92e3e25a622108b210c999524d
Signed-off-by: Wai-Hong Tam <waihong@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3204192
Reviewed-by: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implement a default led_set_brightness, should be good enough to replace
all existing pwm leds currently defined:
atlas, nocturne: 2 leds with independent channels
volteer, boldar, trondo: 2 leds with sidesel
terrador, todor, waddledee, wheelie: 1 led
brya, kano, liara, waddledoo: 2 leds but only amber/white
grunt:1 led with only blue/amber
BRANCH=none
BUG=b:177452529
TEST=build and run on volteer
TEST=gdb disassemble/s led_set_brightness
Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
Change-Id: I24cb10574889ccfbc7c9fddc70ceb81d1e8b2170
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3207147
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These are not needed in few boards, make them default to zero.
BRANCH=none
BUG=b:177452529
TEST=build and gdb check led_color_map
Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
Change-Id: I83cd38d9438ee0d8d99ae187075c6f7f34791a17
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3207146
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This changes the PWM_CHANNEL_BY_IDX macro to correctly handle pwm-leds
entries with less than 3 phandles, defaulting the corresponding channels
to PWM_LED_NO_CHANNEL.
This is necessary for devices with only two LED channels.
BRANCH=none
BUG=b:200974848
TEST=build only, compared the resulting pwm_leds structure
Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
Change-Id: I719ebfb085e60cc2ef27c3508c2b73ae85d3f7ac
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3205318
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a unit test for `st_write_data_with_mask()`. Also adds a new mock
i2c write function and adds the ability to return fake i2c read values
from the read mock.
BUG=b:200589041
BRANCH=None
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Tristan Honscheid <honscheid@google.com>
Cq-Depend: chromium:3203150
Change-Id: I9d7a5673417f3e62d28536bf2a871d57d7ff5cfb
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3203591
Reviewed-by: Yuval Peress <peress@google.com>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a unit test for `st_raw_read_n_noinc()` in `stm_mems_common.c` and
a clarifying comment to `test_st_raw_read_n()`
BUG=b:200589041
BRANCH=None
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Tristan Honscheid <honscheid@google.com>
Change-Id: I55bfacaf139283705765ad236807d9b871d77429
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3203630
Reviewed-by: Yuval Peress <peress@google.com>
Reviewed-by: Aaron Massey <aaronmassey@google.com>
Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Some of the IOEXes are needed early than the task is initialized,
those IOEXes will be initialized at the board level. This CL skips
re-initializing the IOEXes from HOOK function if already initialized.
BUG=none
BRANCH=none
TEST=Tested on ADL RVP, re-initialization of IOEX is skipped
Change-Id: I3062c46992a5578bf8f0f0bc2613b6815c28a616
Signed-off-by: Vijay Hiremath <vijay.p.hiremath@intel.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3183330
Reviewed-by: Keith Short <keithshort@chromium.org>
Reviewed-by: Deepti Deshatty <deepti.deshatty@intel.corp-partner.google.com>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Intel RVP requires GPIOs from few of the IO expanders to configure pins
pre-task, but the current code initializes the IO expanders after the
HOOK task starts. This CL allows board to configure the I2C peripherals
pre-task.
BUG=none
BRANCH=none
TEST=Tested on ADL-RVP, able to initialize IOEX pre-task
Change-Id: I827e22153a98a058d04da0139e302a6ec0276b43
Signed-off-by: Vijay Hiremath <vijay.p.hiremath@intel.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3183329
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In Intel ADL+_RVP, the keyboard is scanned by a discrete I/O expander IC
and it is connected to EC via the I2C interface. EC needs to initialize
the IC via I2C transaction before the task scheduling starts. It may
cause the system panic if the EC's I2C driver is implemented by
task-event-based.
With this CL, if any of the I2C ports that need pre-task I2C transaction
are defined as bitbang ports along with regular I2C ports will switch to
the bitbang (GPIO) mode before the task starts and will again switch
back to event based I2C upon task initialization.
BRANCH=none
BUG=b:199374643
TEST=issue an I2C transaction before the task scheduling starts; make
sure the transaction success without panic on npcx9_evb.
TEST=pass "make buildall"
Signed-off-by: Jun Lin <CHLin56@nuvoton.com>
Change-Id: I65460d2b612328f25bce60561bbb82995dd1cfdf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3161869
Tested-by: Vijay Hiremath <vijay.p.hiremath@intel.com>
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The usb_mux test use proxy mux driver to be between usb_mux API and
usb_mux driver. Test is able to instrument calls made to the drivers.
All public API functions of usb_mux are tested.
BUG=b:184857076
BRANCH=none
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Tomasz Michalec <tm@semihalf.com>
Change-Id: I5f68e860b4afc633bce5d165e15851328207b916
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3152256
Reviewed-by: Yuval Peress <peress@google.com>
Tested-by: Tomasz Michalec <tmichalec@google.com>
Commit-Queue: Tomasz Michalec <tmichalec@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We need to increase this limit to accommodate an upcoming test that uses
more than 5 parameters to a mocked function.
BUG=b:200589041
BRANCH=None
TEST=zmake configure --test zephyr/test/drivers
Change-Id: I57f908fb7abb7c4c148758216bf00ff317e9c46a
Signed-off-by: Tristan Honscheid <honscheid@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3203150
Reviewed-by: Aaron Massey <aaronmassey@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
"enum ec_led_colors" was used here instead of "enum led_color".
Fixes the following clang warning:
error: implicit conversion from enumeration type
'enum led_color' to different enumeration type 'enum ec_led_colors'
[-Werror,-Wenum-conversion]
This is the same fix as commit 533640b6d2ce1189b8c0559ec69e1d64202d2c8b,
but this should fix all instances. These usages were found with:
git grep --name-only 'enum led_color' | xargs grep ec_led_colors
As indicated by the TEST line, all boards match with this change, except
for bloog since it uses the incorrect enum in two places.
BRANCH=none
BUG=b:172020503
TEST=CC=arm-none-eabi-clang BOARD=<affected board>
TEST=./util/compare_build.sh -b all -j 120
=> All match except bloog
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I75cff3769d5b3b82b9218bfb8584f449fb6a781f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3203936
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When building with clang:
In file included from board/ambassador/board.c:30:
include/power/cometlake-discrete.h:11:9: error:
'__CROS_EC_COMETLAKE_DISCRETE_H' is used as a header guard here,
followed by
include/power/cometlake-discrete.h:12:9: note:
'__CROS_EC_COMETLATE_DISCRETE_H'
is defined here; did you mean '__CROS_EC_COMETLAKE_DISCRETE_H'?
BRANCH=none
BUG=b:172020503
TEST=make CC=arm-none-eabi-clang BOARD=ambassador
TEST=./util/compare_build.sh -b all -j 70
=> MATCH
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I4cda48bc44720ee7ee59e558bf00f26059e58345
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3203934
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Building with clang:
driver/led/lm3630a.c:23:19: error: unused function 'lm3630a_read'
[-Werror,-Wunused-function]
static inline int lm3630a_read(uint8_t reg, int *val)
BRANCH=none
BUG=b:172020503
TEST=make CC=arm-none-eabi-clang BOARD=aleena
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: If4a26261d41f470c2cb86208e3c1eaada0b2ff3e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3202999
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
clang reports that the enum types do not match:
chip/npcx/lpc.c:967:9: error: implicit conversion from enumeration type
'enum ec_error_list' to different enumeration type 'enum ec_status'
[-Werror,-Wenum-conversion]
return EC_SUCCESS;
common/power_button_x86.c:523:9: error: implicit conversion from enumeration
type 'enum ec_error_list' to different enumeration type 'enum ec_status'
[-Werror,-Wenum-conversion]
return EC_SUCCESS;
BRANCH=none
BUG=b:172020503
TEST=make CC=arm-none-eabi-clang BOARD=aleena
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I25d1526d58e1e8558153592be7174a8448c130f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3202998
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When compiling with clang, it reports the following error:
board/zinger/runtime.c:271:2: error: non-ASM statement in naked function is not
supported
cpu_reset();
^
board/zinger/runtime.c:258:43: note: attribute is here
void exception_panic(void) __attribute__((naked));
This is a partial revert of commit
28c034e69e6610602ef592b89476ce48e7ce58e8. Instead of replacing the "b
cpu_reset" with a C call to cpu_reset(), change to "bl cpu_reset", which
is the same thing.
The code generated by gcc for exception_panic is the same before and
after this change:
00000000 <exception_panic>:
0: 4b04 ldr r3, [pc, #16] ; (14 <exception_panic+0x14>)
2: 1c18 adds r0, r3, #0
4: 466b mov r3, sp
6: 6999 ldr r1, [r3, #24]
8: 695a ldr r2, [r3, #20]
a: f7ff fffe bl 0 <debug_printf>
e: f7ff fffe bl 0 <exception_panic>
12: 46c0 nop ; (mov r8, r8)
14: 00000000 .word 0x00000000
BRANCH=none
BUG=b:172020503, b:172343310
TEST=make CC=arm-none-eabi-clang BOARD=zinger -j
TEST=make buildall -j
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I590ba4d2caabc77f04288d9f566286e87d08df02
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3198205
Reviewed-by: Eric Yilun Lin <yllin@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There was a recent breakage where one job failed because of a git clone
timeout. But more importantly, we should make sure all the jobs are
using the same commit.
Change CI jobs to use the gitlab cache to keep the non-ec git repos so
only the first job (seed_cache) will do the git clone, and the others
will just pull the files from the cache.
The cache key is the ec commit sha that is being built.
The total build time after this is 40m, and the builds before this were
37-39m, so it doesn't seem to be a change.
BUG=None
BRANCH=None
TEST=Pushed to coverage branch on gitlab:
https://gitlab.com/zephyr-ec/ec/-/pipelines/382237078
Change-Id: Ic596464fbf90c2964075b4f8f3e6bdc77bde531b
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3205632
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A few tests were not declared as static functions, fix that.
BRANCH=none
BUG=b:201602829
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I83052ab0f72b9803ff4f06e3459cc819e6744dd1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3201917
Reviewed-by: Wai-Hong Tam <waihong@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Test the remaining (failure/uncommon) code paths in init(). This includes
a mocking of the system_jumped_late function which was already stubbed
to always return false. This mock will later need to be cleaned up with
a better mock to avoid the custom struct added.
BRANCH=none
BUG=b:201602829
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I1be53b0a15850d8b97b7b5a96ccef764f0782816
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3201916
Reviewed-by: Denis Brockus <dbrockus@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Verify all the code paths in get_vbus_voltage (including calculated
mV value).
BRANCH=none
BUG=b:201819565
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I508880a8af30c2d546655424b289d521f3c22626
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3201915
Reviewed-by: Fabio Baltieri <fabiobaltieri@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Verify all the code paths on the discharge_on_ac API
BRANCH=none
BUG=b:201819565
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: Id9b4163ad42a7bcb14bdac425ed9af5d29d1d1d7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3201914
Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adding support for this register improves code coverage for the existing
init function. No additional tests are needed at this point.
BRANCH=none
BUG=b:201819565
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I9e19372501775940f1a23d05317ba3971212083a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3201913
Reviewed-by: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=b:201602829
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I3296c94fd2b591a37349f08c3cd29c92ce5831b9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3201912
Reviewed-by: Aaron Massey <aaronmassey@google.com>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Verify setting the prochot amps as well as edge conditions
BRANCH=none
BUG=b:201602829
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I12cd413172bfdf395a1c60ef7e487b33687202bc
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3200055
Reviewed-by: Aaron Massey <aaronmassey@google.com>
Reviewed-by: Abe Levkoy <alevkoy@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Verify setting the prochot amps as well as edge conditions
BRANCH=none
BUG=b:201602829
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I306adf1d74eea65a8522c61e5ed71cf8deacfe2a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3200054
Reviewed-by: Wai-Hong Tam <waihong@google.com>
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=b:201602829
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I85f80ad1d8233c1fbefff9ab2124c4c39a3d5062
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3198688
Reviewed-by: Aaron Massey <aaronmassey@google.com>
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=b:201602829
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: I7e4ed6327c3393f64153f4383002a2c9c38f57b1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3198687
Reviewed-by: Abe Levkoy <alevkoy@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
| |
BRANCH=none
BUG=b:201602829
TEST=zmake configure --test zephyr/test/drivers
Signed-off-by: Yuval Peress <peress@google.com>
Change-Id: Ic2481c0de52268cc65475de8e79f1db7834358d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3198686
Reviewed-by: Sam Hurst <shurst@google.com>
Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To make the device bootable without battery, board_rt1718s_init()
enables sink path (GPIO1) at boot, but also incorrectly enables sink
path on subsequent low-power-mode exits.
Fix this incorrect behavior: board_rt1718s_init should keep the previous
GPIO state on LPM exit.
BUG=none
TEST=verify that C1 sink path does not turn on by default when replug
BRANCH=main
Signed-off-by: Ting Shen <phoenixshen@google.com>
Change-Id: I8ae8f65e101fc66f828168237c136ca180189c08
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3195204
Reviewed-by: Eric Yilun Lin <yllin@google.com>
Commit-Queue: Ting Shen <phoenixshen@chromium.org>
Tested-by: Ting Shen <phoenixshen@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=b:197494540, b:201544471, b:196475862
BRANCH=none
TEST=verify keyboard top row key work as intended.
Signed-off-by: Zick Wei <zick.wei@quanta.corp-partner.google.com>
Change-Id: I7a0b1573b323f09f8e2ec8b0188dee5cb498af81
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3181440
Reviewed-by: Rob Barnes <robbarnes@google.com>
Reviewed-by: Devin Lu <Devin.Lu@quantatw.com>
Commit-Queue: Rob Barnes <robbarnes@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
SEQ_EC_ALL_SYS_PG is commonly named as PG_EC_ALL_SYS_PWRGD.
Removing it to reduce the redundant declarations needed in
zephyr.
BUG=b:200975143
BRANCH=main
TEST=make buildall -j; boot up brya
Change-Id: Ie266460dfe9c61dbbb97e1d1b714824929c5a098
Signed-off-by: Divya Sasidharan <divya.s.sasidharan@intel.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3180807
Reviewed-by: Vijay Hiremath <vijay.p.hiremath@intel.com>
Reviewed-by: caveh jalali <caveh@chromium.org>
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Split task_ctx structure to separate const data from mutable.
This saves a lot of ram and flash memory.
BRANCH=main
BUG=b:200693667
TEST=Compile and flash firmware for lazor.
Check that RAM and flash have more free bytes.
EC should work correctly.
Change-Id: Ia0706e9e1521f6fbdf8f5d299e83ba2940094091
Signed-off-by: Michał Barnaś <mb@semihalf.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3202197
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Removed gpio_data array to use config instead.
This structure was redundant and could be removed to save memory.
BRANCH=main
BUG=b:200693667
TEST=Compile and flash firmware for lazor.
Check that RAM and flash have more free bytes.
EC should work correctly.
Change-Id: I7aad3c255d1285ec453e164323e5ff03fa16c614
Signed-off-by: Michał Barnaś <mb@semihalf.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3178865
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix lid accel address and matrix.
BUG=b:197200940
BRANCH=none
TEST=Use "ectool motionsense lid_angle" to check lid angle correct.
Signed-off-by: David Huang <david.huang@quanta.corp-partner.google.com>
Change-Id: I474999818d7eeca3214b609997035c72b48d0a29
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3192748
Reviewed-by: Boris Mittelberg <bmbm@google.com>
Commit-Queue: Boris Mittelberg <bmbm@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
clang warns:
board/fluffy/board.c:263:2: error: variable 'c0' is used uninitialized whenever
switch default is taken [-Werror,-Wsometimes-uninitialized]
default:
^~~~~~~
board/fluffy/board.c:269:17: note: uninitialized use occurs here
gpio_set_level(c0, val & BIT(0));
^~
board/fluffy/board.c:240:2: note: variable 'c0' is declared here
enum gpio_signal c0;
^
board/fluffy/board.c:263:2: error: variable 'c1' is used uninitialized whenever
switch default is taken [-Werror,-Wsometimes-uninitialized]
default:
^~~~~~~
board/fluffy/board.c:270:17: note: uninitialized use occurs here
gpio_set_level(c1, val & BIT(1));
^~
board/fluffy/board.c:241:2: note: variable 'c1' is declared here
enum gpio_signal c1;
^
board/fluffy/board.c:263:2: error: variable 'c2' is used uninitialized whenever
switch default is taken [-Werror,-Wsometimes-uninitialized]
default:
^~~~~~~
board/fluffy/board.c:271:17: note: uninitialized use occurs here
gpio_set_level(c2, val & BIT(2));
^~
board/fluffy/board.c:242:2: note: variable 'c2' is declared here
enum gpio_signal c2;
^
BRANCH=none
BUG=b:172020503
TEST=none
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Ie05c31a9f6be8e92d0a8904fe6c20e38ced09e67
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3198202
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Just preserving the build at ToT. Do not cherry pick.
BRANCH=none
BUG=none
TEST=make BOARD=krane -j
Before:
4180 bytes in flash and 11268 bytes in RAM still available on krane RO
80 bytes in flash and 8244 bytes in RAM still available on krane RW
After:
4308 bytes in flash and 11268 bytes in RAM still available on krane RO
304 bytes in flash and 8244 bytes in RAM still available on krane RW
Change-Id: I1d433999560ca9e176d1c420cda6dadb29ec7509
Signed-off-by: Vijay Hiremath <vijay.p.hiremath@intel.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3200517
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Initialize vivaldi keyboard.
BUG=b:201340797
BRANCH=keeby
TEST=make BOARD=corori
Signed-off-by: Josh Tsai <josh_tsai@compal.corp-partner.google.com>
Change-Id: I9fd160327f455ec7f78e878c07fbf3e5de66cae7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3201830
Reviewed-by: SamSP Liu <samsp_liu@compal.corp-partner.google.com>
Reviewed-by: Elmo Lan <elmo_lan@compal.corp-partner.google.com>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Commit-Queue: Aseda Aboagye <aaboagye@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
common/base_state.c:65:9: error: implicit conversion from enumeration type
'enum ec_error_list' to different enumeration type 'enum ec_status'
[-Werror,-Wenum-conversion]
return EC_SUCCESS;
BRANCH=none
BUG=b:172020503
TEST=make CC=arm-none-eabi-clang BOARD=kakadu
TEST=make buildall
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I7ad0b349270d00a1156462aa32ad19f342b6db61
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3198204
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
clang warns:
ip/stm32/usb_hid_keyboard.c:378:21: error: unused variable 'hid_current_buf'
[-Werror,-Wunused-variable]
static volatile int hid_current_buf;
^
BRANCH=none
BUG=b:172020503
TEST=make buildall -j
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I88b424cbc8930bf2d7d6c0856d81104cbd4d3561
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3198203
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Everything upgraded and working with v2.7. In the overlays,
asurada-zephyr, trogdor-zephyr, and volteer-zephyr all use Zephyr v2.7
as well.
BUG=b:198824039
BRANCH=none
TEST=zmake buildall
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I91296976e473e82abbbae76e754e8eb350ebd423
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3200526
Reviewed-by: Yuval Peress <peress@google.com>
|