diff options
| author | Denis Brockus <dbrockus@google.com> | 2020-04-14 09:23:48 -0600 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-04-15 20:19:37 +0000 |
| commit | 2097501f72619ca189cc0a9ac39af3149c3f9315 (patch) | |
| tree | 0545a851887f46cf8dd3a0023c14845c56029798 /baseboard/zork/variant_dalboz.c | |
| parent | bb4a03953f571fc34d29160670ee68d3639f5f1b (diff) | |
| download | chrome-ec-2097501f72619ca189cc0a9ac39af3149c3f9315.tar.gz | |
TCPMv2: pd_set_suspend should not set tc state directly
During a cold boot, TASK_ID_CHIPSET calls pd_set_suspend.
This was setting the state in the TC state machine and
executed code in the state machine in an invalid task.
I changed this to use a deferred flag bit that gets set
in pd_set_suspend and is checked and handled in tc_run
and in the disabled state to reflect the current setting
of this flag bit
BUG=b:153994086,b:154120653,chromium:1052432
BRANCH=none
TEST=no AC attached, switch normal to dev-mode
Signed-off-by: Denis Brockus <dbrockus@google.com>
Change-Id: Ie04ef8f8c88f26e637a8b2547e1dc7da27e8b7b5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2149384
Commit-Queue: Denis Brockus <dbrockus@chromium.org>
Commit-Queue: Edward Hill <ecgh@chromium.org>
Tested-by: Denis Brockus <dbrockus@chromium.org>
Reviewed-by: Edward Hill <ecgh@chromium.org>
Auto-Submit: Denis Brockus <dbrockus@chromium.org>
Diffstat (limited to 'baseboard/zork/variant_dalboz.c')
0 files changed, 0 insertions, 0 deletions