summaryrefslogtreecommitdiff
path: root/board/pit
diff options
context:
space:
mode:
authorRandall Spangler <rspangler@chromium.org>2013-09-13 10:42:18 -0700
committerchrome-internal-fetch <chrome-internal-fetch@google.com>2013-09-14 00:32:09 +0000
commitfa76d68ce9b59b82ef69ed297613c67dd6725964 (patch)
tree5483b3f34101f1fbeccd7ead436e5a6a60f3a479 /board/pit
parent40f4d61266192f2ec439c943be7a945232efd388 (diff)
downloadchrome-ec-fa76d68ce9b59b82ef69ed297613c67dd6725964.tar.gz
Fix task priorities for console and hostcmd tasks
The console task should be higher priority than the host command task, since that allows debugging problems with host commands. The keyboard scanning task should be higher priority than both of them, since it's extremely latency-sensitive. As currently written, long-running host commands such as I2C passthru can interfere with keyboard scanning. BUG=chrome-os-partner:22681 BRANCH=none (potentially affects pit, but apparently not noticeably) TEST=type bursts of 6-8 characters quickly while doing a flash update of the EC; should not drop characters. Change-Id: I48db014053750a5f1fae5d06df34768975bb8297 Reviewed-on: https://chromium-review.googlesource.com/169334 Tested-by: Randall Spangler <rspangler@chromium.org> Reviewed-by: Bill Richardson <wfrichar@chromium.org> Commit-Queue: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'board/pit')
-rw-r--r--board/pit/ec.tasklist4
1 files changed, 2 insertions, 2 deletions
diff --git a/board/pit/ec.tasklist b/board/pit/ec.tasklist
index 4cf8e75954..448879d105 100644
--- a/board/pit/ec.tasklist
+++ b/board/pit/ec.tasklist
@@ -19,7 +19,7 @@
TASK_NOTEST(VBOOTHASH, vboot_hash_task, NULL, TASK_STACK_SIZE) \
TASK_NOTEST(POWERLED, power_led_task, NULL, 256) \
TASK_ALWAYS(CHARGER, charger_task, NULL, TASK_STACK_SIZE) \
- TASK_NOTEST(KEYSCAN, keyboard_scan_task, NULL, TASK_STACK_SIZE) \
TASK_NOTEST(CHIPSET, chipset_task, NULL, TASK_STACK_SIZE) \
+ TASK_ALWAYS(HOSTCMD, host_command_task, NULL, TASK_STACK_SIZE) \
TASK_ALWAYS(CONSOLE, console_task, NULL, TASK_STACK_SIZE) \
- TASK_ALWAYS(HOSTCMD, host_command_task, NULL, TASK_STACK_SIZE)
+ TASK_NOTEST(KEYSCAN, keyboard_scan_task, NULL, TASK_STACK_SIZE)