diff options
author | Randall Spangler <rspangler@chromium.org> | 2013-10-31 15:27:19 -0700 |
---|---|---|
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2013-11-01 17:13:11 +0000 |
commit | d7004207e5d809ff6bc4b6a0c6969fd3b277b886 (patch) | |
tree | dca2395867adeb5361e4ad47e5a4a57d09ab2816 /power | |
parent | 760ace07a5cc4b7dc5e8f2ec59068a84562be162 (diff) | |
download | chrome-ec-d7004207e5d809ff6bc4b6a0c6969fd3b277b886.tar.gz |
cleanup: Update more TODO comments
Add bug links, reword, or remove as applicable.
No code changes, just comments.
BUG=chrome-os-partner:18343
BRANCH=none
TEST=build all boards; pass unit tests
Change-Id: Id55dd530c10091d7ab9d0f942f750168fca793b4
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/175326
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
Diffstat (limited to 'power')
-rw-r--r-- | power/x86_common.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/power/x86_common.c b/power/x86_common.c index add033906e..d0b1088845 100644 --- a/power/x86_common.c +++ b/power/x86_common.c @@ -105,10 +105,10 @@ int x86_wait_signals(uint32_t want) return EC_ERROR_TIMEOUT; } /* - * TODO: should really shrink the remaining timeout if we woke - * up but didn't have all the signals we wanted. Also need to - * handle aborts if we're no longer in the same state we were - * when we started waiting. + * TODO(crosbug.com/p/23772): should really shrink the + * remaining timeout if we woke up but didn't have all the + * signals we wanted. Also need to handle aborts if we're no + * longer in the same state we were when we started waiting. */ } return EC_SUCCESS; @@ -207,9 +207,9 @@ int chipset_in_state(int state_mask) int need_mask = 0; /* - * TODO: what to do about state transitions? If the caller wants - * HARD_OFF|SOFT_OFF and we're in G3S5, we could still return - * non-zero. + * TODO(crosbug.com/p/23773): what to do about state transitions? If + * the caller wants HARD_OFF|SOFT_OFF and we're in G3S5, we could still + * return non-zero. */ switch (state) { case X86_G3: |