diff options
| author | Daisuke Nojiri <dnojiri@chromium.org> | 2019-04-03 11:03:23 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2019-04-08 22:30:32 -0700 |
| commit | 6bd030ecd978a8b619da261510490f27f68c4e6c (patch) | |
| tree | 4cd5827d1842384d477eb7f9d68c05ec7906c2ec /fuzz | |
| parent | b4cca02d9dcc5c7fa42813ae819933fd5ae9f43c (diff) | |
| download | chrome-ec-6bd030ecd978a8b619da261510490f27f68c4e6c.tar.gz | |
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 <dnojiri@chromium.org>
BUG=none
BRANCH=none
TEST=buildall
Change-Id: Id658b9d68e742e4334c692b804d9c98c8de21313
Reviewed-on: https://chromium-review.googlesource.com/1551579
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>
Diffstat (limited to 'fuzz')
| -rw-r--r-- | fuzz/cr50_fuzz.tasklist | 10 | ||||
| -rw-r--r-- | fuzz/host_command_fuzz.tasklist | 10 | ||||
| -rw-r--r-- | fuzz/usb_pd_fuzz.tasklist | 10 |
3 files changed, 3 insertions, 27 deletions
diff --git a/fuzz/cr50_fuzz.tasklist b/fuzz/cr50_fuzz.tasklist index de4df33e13..24870f2abb 100644 --- a/fuzz/cr50_fuzz.tasklist +++ b/fuzz/cr50_fuzz.tasklist @@ -4,14 +4,6 @@ */ /** - * List of enabled tasks in the priority order - * - * The first one has the lowest priority. - * - * For each task, use the macro TASK_TEST(n, r, d, s) where : - * 'n' in the name of the task - * 'r' in the main routine of the task - * 'd' in 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_TEST_TASK_LIST diff --git a/fuzz/host_command_fuzz.tasklist b/fuzz/host_command_fuzz.tasklist index de4df33e13..24870f2abb 100644 --- a/fuzz/host_command_fuzz.tasklist +++ b/fuzz/host_command_fuzz.tasklist @@ -4,14 +4,6 @@ */ /** - * List of enabled tasks in the priority order - * - * The first one has the lowest priority. - * - * For each task, use the macro TASK_TEST(n, r, d, s) where : - * 'n' in the name of the task - * 'r' in the main routine of the task - * 'd' in 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_TEST_TASK_LIST diff --git a/fuzz/usb_pd_fuzz.tasklist b/fuzz/usb_pd_fuzz.tasklist index 3d7a91254b..6edeac2f98 100644 --- a/fuzz/usb_pd_fuzz.tasklist +++ b/fuzz/usb_pd_fuzz.tasklist @@ -4,15 +4,7 @@ */ /** - * List of enabled tasks in the priority order - * - * The first one has the lowest priority. - * - * For each task, use the macro TASK_TEST(n, r, d, s) where : - * 'n' in the name of the task - * 'r' in the main routine of the task - * 'd' in 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_TEST_TASK_LIST \ TASK_TEST(PD_C0, pd_task, NULL, LARGER_TASK_STACK_SIZE) \ |