diff options
author | Denis Brockus <dbrockus@chromium.org> | 2019-06-19 10:14:34 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-06-20 15:49:10 +0000 |
commit | b4c70fd9371ecc133161d513c32d7546ccf00c5c (patch) | |
tree | d191882f86678033e59089922efa052138d516f9 /chip/ish/ipc_heci.c | |
parent | 308592140b792872d9235073b151f290a45ea514 (diff) | |
download | chrome-ec-b4c70fd9371ecc133161d513c32d7546ccf00c5c.tar.gz |
ish: power management temp code is really needed
Adjust the comments for the ISH VNN_REQ that indicate code can
be removed once power management is put in place.
The ipc_heci.c instances are needed to make sure write power is
enabled for the heci interface so we can receive the message.
The system_state_subsys.c instances disable and re-enable power
based on message flow request to suspend/resume.
I am going to leave the code as is and adjust the comments a little.
BUG=b:135279314
BRANCH=none
TEST=make buildall -j
Change-Id: I3467e7005ce466bcb06c57a0590834f42aea9ddb
Signed-off-by: Denis Brockus <dbrockus@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1667929
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Diffstat (limited to 'chip/ish/ipc_heci.c')
-rw-r--r-- | chip/ish/ipc_heci.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/chip/ish/ipc_heci.c b/chip/ish/ipc_heci.c index 9e62c33520..cce3298f01 100644 --- a/chip/ish/ipc_heci.c +++ b/chip/ish/ipc_heci.c @@ -448,10 +448,7 @@ static void ipc_host2ish_isr(void) uint32_t pimr = IPC_PIMR; /* - * TODO(b/122364080): Remove this code once proper power management is - * in place for ISH. - * - * Ensure that the host IPC power is requested after getting an + * Ensure that the host IPC write power is requested after getting an * interrupt otherwise the resume message will never get delivered (via * host ipc communication). Resume is where we would like to restore all * power settings, but that is too late for this power request. @@ -687,10 +684,7 @@ void ipc_mng_task(void) ipc_handle_t handle; /* - * TODO(b/122364080): Remove this code once proper power management is - * in place for ISH. - * - * Ensure that power for host IPCs is requested and ack'ed + * Ensure that power for host IPC writes is requested and ack'ed */ if (IS_ENABLED(CHIP_FAMILY_ISH5)) { PMU_VNN_REQ = VNN_REQ_IPC_HOST_WRITE & ~PMU_VNN_REQ; |