From 10627870feec074b4241deca8b33494215fd7244 Mon Sep 17 00:00:00 2001 From: ChromeOS Developer Date: Tue, 4 Feb 2014 13:01:23 -0800 Subject: [PATCH] Only call button handler if keyboard protocol task defined This fixes a breakage when building tests. BUG=None BRANCH=None TEST=Manual. Run "make BOARD=clapper tests" and "emerge-clapper chromeos-ec" (having cros_worked on it) Change-Id: Icdfa655b7fc246b103111f957d9c3f9e7f49c736 Signed-off-by: Dave Parker Reviewed-on: https://chromium-review.googlesource.com/184931 Reviewed-by: Bernie Thompson Reviewed-by: Randall Spangler (cherry picked from commit 18baa15cc10d1d0906a47fa60ab3fb1eb2cf484e) Reviewed-on: https://chromium-review.googlesource.com/185244 Tested-by: Randall Spangler Reviewed-by: Bill Richardson Commit-Queue: Randall Spangler --- common/button.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/common/button.c b/common/button.c index 2e48102dd3..32f2bd255b 100644 --- a/common/button.c +++ b/common/button.c @@ -77,8 +77,10 @@ static void button_change_deferred(void) CPRINTF("[%T Button '%s' was %s]\n", buttons[i].name, new_pressed ? "pressed" : "released"); +#ifdef HAS_TASK_KEYPROTO keyboard_update_button(buttons[i].type, new_pressed); +#endif } /* Clear the debounce time to stop checking it */