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
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: Ibb2c8052bc4fb776c5e1c172eeb1f3faf356a147
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3052750
Commit-Queue: Josie Nordrum <josienordrum@google.com>
Reviewed-by: Josie Nordrum <josienordrum@google.com>

1 files changed, 2 insertions, 0 deletions

diff --git a/board/nocturne_fp/board.h b/board/nocturne_fp/board.h
index c30b4b76e0..6da68130df 100644
--- a/board/nocturne_fp/board.h
+++ b/board/nocturne_fp/board.h
@@ -186,6 +186,8 @@
 #include "gpio_signal.h"
 #include "board_rw.h"
 
+void slp_event(enum gpio_signal signal);
+
 #endif /* !__ASSEMBLER__ */
 
 #endif /* __BOARD_H */