From 6bd030ecd978a8b619da261510490f27f68c4e6c Mon Sep 17 00:00:00 2001 From: Daisuke Nojiri Date: Wed, 3 Apr 2019 11:03:23 -0700 Subject: ec.tasklist: Consolidate duplicate comments It's simply a bad idea to describe a macro in multiple locations. It'll make it hard to change. It'll be difficult to keep all locations in sync. This patch replaces the comment duplicated in all ec.tasklist with a pointer to the CONFIG_TASK_LIST definition. The macro will be described in a single place (just like all/most other macros). Signed-off-by: Daisuke Nojiri BUG=none BRANCH=none TEST=buildall Change-Id: Id658b9d68e742e4334c692b804d9c98c8de21313 Reviewed-on: https://chromium-review.googlesource.com/1551579 Commit-Ready: ChromeOS CL Exonerator Bot Tested-by: Daisuke Nojiri Reviewed-by: Daisuke Nojiri Reviewed-by: Jett Rink --- board/scarlet/ec.tasklist | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) (limited to 'board/scarlet') diff --git a/board/scarlet/ec.tasklist b/board/scarlet/ec.tasklist index 26fc8d26a8..8a3aedf71c 100644 --- a/board/scarlet/ec.tasklist +++ b/board/scarlet/ec.tasklist @@ -4,17 +4,7 @@ */ /** - * List of enabled tasks in the priority order - * - * The first one has the lowest priority. - * - * For each task, use the macro TASK_ALWAYS(n, r, d, s) for base tasks and - * TASK_NOTEST(n, r, d, s) for tasks that can be excluded in test binaries, - * where : - * 'n' is the name of the task - * 'r' is the main routine of the task - * 'd' is an opaque parameter passed to the routine at startup - * 's' is the stack size in bytes; must be a multiple of 8 + * See CONFIG_TASK_LIST in config.h for details. */ #define CONFIG_TASK_LIST \ TASK_ALWAYS(HOOKS, hook_task, NULL, LARGER_TASK_STACK_SIZE) \ -- cgit v1.2.1