diff options
| author | Ting Shen <phoenixshen@google.com> | 2022-08-15 15:42:16 +0800 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-08-15 16:17:20 +0000 |
| commit | e1d5970ad8ab0e80de6fc7f48b2c0bd7de4b882b (patch) | |
| tree | dddd27bfb29b2add9d36e2c73b6d38e081e0f005 /power | |
| parent | fd305e6809233a662c31dae8ca2953271830c958 (diff) | |
| download | chrome-ec-e1d5970ad8ab0e80de6fc7f48b2c0bd7de4b882b.tar.gz | |
power/mt8186: remove redundant msleep
The original purpose of msleep(500) line is to wait for pmic ready,
it was added in CL:3233783 ps30. After a few code changes, another
power_wait_mask_signals_timeout line was added to provide the same
functionality. Now we are waiting for pmic twice, we should remove the
duplicated msleep call.
BUG=b:237249144
TEST=manually
BRANCH=none
Signed-off-by: Ting Shen <phoenixshen@google.com>
Change-Id: I15e3930cf07c0ce6c225f61727e0566a0644e32a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3831159
Code-Coverage: Zoss <zoss-cl-coverage@prod.google.com>
Commit-Queue: Ting Shen <phoenixshen@chromium.org>
Auto-Submit: Ting Shen <phoenixshen@chromium.org>
Reviewed-by: Eric Yilun Lin <yllin@google.com>
Tested-by: Ting Shen <phoenixshen@chromium.org>
Diffstat (limited to 'power')
| -rw-r--r-- | power/mt8186.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/power/mt8186.c b/power/mt8186.c index 475ba769f9..7cf3309f7c 100644 --- a/power/mt8186.c +++ b/power/mt8186.c @@ -373,7 +373,6 @@ enum power_state power_handle_state(enum power_state state) /* Give up, go back to G3. */ return POWER_S5G3; - msleep(500); /* Call hooks now that rails are up */ hook_notify(HOOK_CHIPSET_STARTUP); /* |