From 7143a51e0e4bb1eb1c9130017c29781ec6eff75e Mon Sep 17 00:00:00 2001 From: Keith Short Date: Wed, 29 May 2019 11:20:39 -0600 Subject: cr50: Remove user presence check message. Coreboot on Wilco now uses phyiscal presense checks from the Cr50 to confirm user actions. When booting to the recovery screen, the Cr50 console is flooded with "Presence: 0" messages. Demote the message to DBG images only and only if the user presense was detected. BUG=b:133266946 BRANCH=cr50 TEST=Build DBG and non DBG CR50 images on Wilco. Confirm the message flood is gone at the recovery screen. Change-Id: Ie6b08a386e9e479e61a6764dad8d4a3c9aa57f05 Signed-off-by: Keith Short Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1638483 Reviewed-by: Vadim Bendebury (cherry picked from commit 93a3f30e55e2ac5304a7cb66f24a76d51127e242) Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1657313 Tested-by: Vadim Bendebury Commit-Queue: Vadim Bendebury (cherry picked from commit c6de101996111514bcaa9f4b70cdfe8f56016ee7) Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1705728 (cherry picked from commit 6c3bce231b27364e3b82af3d1df1bbc8ea1a84ab) --- board/cr50/u2f.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/board/cr50/u2f.c b/board/cr50/u2f.c index a9989aa3fb..bf0bbb8791 100644 --- a/board/cr50/u2f.c +++ b/board/cr50/u2f.c @@ -43,7 +43,10 @@ enum touch_state pop_check_presence(int consume) int recent = ((last_press.val > 0) && ((get_time().val - last_press.val) < PRESENCE_TIMEOUT)); - CPRINTS("Presence:%d", recent); +#ifdef CR50_DEV + if (recent) + CPRINTS("User presence: consumed %d", consume); +#endif if (consume) last_press.val = 0; -- cgit v1.2.1