diff options
author | Craig Hesling <hesling@chromium.org> | 2021-07-25 17:06:09 -0400 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-08-09 18:45:02 +0000 |
commit | 4b507515187aa031d0d3fe4ae4af5fa0c44e1ab8 (patch) | |
tree | f4e383dc846930c3e76dc65a080f4de235f7a7f0 /board/nocturne_fp/board_rw.h | |
parent | 580dad3d9956d55565ed5db9e5de9851df9c437b (diff) | |
download | chrome-ec-4b507515187aa031d0d3fe4ae4af5fa0c44e1ab8.tar.gz |
dartmonkey: Reintroduce sleep lines to RO
Change https://crrev.com/c/2657778 removed the sleep lines from RO
in an effort to minimize the RO complexity.
Most notably, this isolated the deep-sleep/low-power-idle
logic to RW only.
Unfortunately, the sleep lines also control whether the SPI host interface
is listening, which allows it to ignore spurious communication.
It seems safer to reinstate the the sleep line with low power idle
active and directly disable CONFIG_LOW_POWER_IDLE in subsequent CL.
We reinstate the sleep line gpio logic from the following:
crrev.com/c1b5095aa8404709bb447bd7a58f262d7d471a01/board/nocturne_fp/board.c
This is the parent commit to the CL that refactors the sleep lines.
Considering dartmonkey has a sleep line modification for dev boards,
we keep this in RW only. Since nearly all functionality would need
to be conditioned and communicated between RO and RW, I decided to
just create a clean break between RO and RW board init. The original
board.c no longer spans both RO and RW, there are exclusive board_ro.c
and board_rw.c files.
BRANCH=none
BUG=b:178746753, b:195718112
TEST=make -j tests-dartmonkey
TEST=# Connect servo_micro and J-Link to an icetower board.
make proj-dartmonkey -j
sudo servod --board=icetower
./util/flash_jlink.py --board=dartmonkey --image=./build/dartmonkey/ec.bin
# Unplug J-Link and unplug/replug servo connector.
dut-control fpmcu_slp:off fpmcu_slp_alt:off
dut-control pp3300_dx_mcu_mw # Should be more than 40mw
dut-control fpmcu_slp:on fpmcu_slp_alt:off
dut-control pp3300_dx_mcu_mw # Should be less than 10mw
dut-control fpmcu_slp:off fpmcu_slp_alt:on
dut-control pp3300_dx_mcu_mw # Should be less than 10mw
dut-control fpmcu_slp:on fpmcu_slp_alt:on
dut-control pp3300_dx_mcu_mw # Should be less than 10mw
dut-control fpmcu_slp:off fpmcu_slp_alt:off
minicom -D$(dut-control -o raw_fpmcu_console_uart_pty)
> reboot ro
# Ctrl-A Q
# RO does not have the code to adjust to SLP_ALT_DEV_L, thus
# it will use SLP_ALT_L, which is 0. This means it will always
# think sleep is asserted.
dut-control fpmcu_slp:off fpmcu_slp_alt:off
dut-control pp3300_dx_mcu_mw # Should be less than 10mw
minicom -D$(dut-control -o raw_fpmcu_console_uart_pty)
> gpioget
# Should see SLP_L=1, SLP_ALT_L=0, and SLP_ALT_DEV_L=1
> reboot
> fpenroll
> fpmatch
# Ctrl-A Q
Signed-off-by: Craig Hesling <hesling@chromium.org>
Change-Id: I6183a4afa61817c552da5272f7d66a5c58d96f66
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3078824
Reviewed-by: Josie Nordrum <josienordrum@google.com>
Diffstat (limited to 'board/nocturne_fp/board_rw.h')
-rw-r--r-- | board/nocturne_fp/board_rw.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/board/nocturne_fp/board_rw.h b/board/nocturne_fp/board_rw.h index 29e304f115..6ef7cc29b4 100644 --- a/board/nocturne_fp/board_rw.h +++ b/board/nocturne_fp/board_rw.h @@ -7,11 +7,8 @@ #define __CROS_EC_BOARD_NOCTURNE_FP_BOARD_RW_H void fps_event(enum gpio_signal signal); -void slp_event(enum gpio_signal signal); /* Defined in ro_workarounds.c */ void wp_event(enum gpio_signal signal); -void board_init_rw(void); - #endif /* __CROS_EC_BOARD_NOCTURNE_FP_BOARD_RW_H */ |