diff options
| author | Tom Rini <trini@konsulko.com> | 2023-01-16 23:46:49 +0300 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2023-01-18 20:46:13 +0300 |
| commit | 448e2b6327d0498d58506d6f4e4b2a325ab7cca0 (patch) | |
| tree | 2ed6c4e6ba1106d7f68e6a6d3424f76593bb3e2e /cmd | |
| parent | e8272cbd59afdc1617e8c0288ebd663fd743d4b5 (diff) | |
| download | u-boot-448e2b6327d0498d58506d6f4e4b2a325ab7cca0.tar.xz | |
event: Correct dependencies on the EVENT framework
The event framework is just that, a framework. Enabling it by itself
does nothing, so we shouldn't ask the user about it. Reword (and correct
typos) around this the option and help text. This also applies to
DM_EVENT and EVENT_DYNAMIC. Only EVENT_DEBUG and CMD_EVENT should be
visible to the user to select, when EVENT is selected.
With this, it's time to address the larger problems. When functionality
uses events, typically via EVENT_SPY, the appropriate framework then
must be select'd and NOT imply'd. As the functionality will cease to
work (and so, platforms will fail to boot) this is non-optional and
where select is appropriate. Audit the current users of EVENT_SPY to
have a more fine-grained approach to select'ing the framework where
used. Also ensure the current users of event_register and also select
EVENT_DYNAMIC.
Cc: AKASHI Takahiro <takahiro.akashi@linaro.org>
Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reported-by: Oliver Graute <Oliver.Graute@kococonnector.com>
Reported-by: Francesco Dolcini <francesco.dolcini@toradex.com>
Fixes: 7fe32b3442f0 ("event: Convert arch_cpu_init_dm() to use events")
Fixes: 42fdcebf859f ("event: Convert misc_init_f() to use events")
Fixes: c5ef2025579e ("dm: fix DM_EVENT dependencies")
Signed-off-by: Tom Rini <trini@konsulko.com>
Tested-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Fabio Estevam <festevam@denx.de>
Diffstat (limited to 'cmd')
| -rw-r--r-- | cmd/Kconfig | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/cmd/Kconfig b/cmd/Kconfig index b2aefae9cb..4fe2c75de2 100644 --- a/cmd/Kconfig +++ b/cmd/Kconfig @@ -2622,6 +2622,7 @@ config CMD_DIAG config CMD_EVENT bool "event - Show information about events" + depends on EVENT default y if EVENT_DEBUG help This enables the 'event' command which provides information about |