[PATCH v2 1/1] dm: fix DM_EVENT dependencies
Heinrich Schuchardt
heinrich.schuchardt at canonical.com
Sat May 7 22:39:01 CEST 2022
CONFIG_DM_EVENT without CONFIG_EVENT is non-functional.
Let CONFIG_DM_EVENT depend on CONFIG_EVENT.
Remove superfluous stub in include/event.h.
Fixes: 5b896ed5856f ("event: Add events for device probe/remove")
Reported-by: Jan Kiszka <jan.kiszka at siemens.com>
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
---
v2:
Let DM_EVENT depend on EVENT instead of selecting it.
---
drivers/core/Kconfig | 3 +--
include/event.h | 7 +------
2 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/core/Kconfig b/drivers/core/Kconfig
index 408a8d8e28..85154afa99 100644
--- a/drivers/core/Kconfig
+++ b/drivers/core/Kconfig
@@ -89,8 +89,7 @@ config DM_DEVICE_REMOVE
config DM_EVENT
bool "Support events with driver model"
- depends on DM
- imply EVENT
+ depends on DM && EVENT
default y if SANDBOX
help
This enables support for generating events related to driver model
diff --git a/include/event.h b/include/event.h
index 62e72a7bd3..7765f07250 100644
--- a/include/event.h
+++ b/include/event.h
@@ -144,7 +144,6 @@ int event_register(const char *id, enum event_t type, event_handler_t func,
/** event_show_spy_list( - Show a list of event spies */
void event_show_spy_list(void);
-#if CONFIG_IS_ENABLED(EVENT)
/**
* event_notify() - notify spies about an event
*
@@ -159,6 +158,7 @@ void event_show_spy_list(void);
*/
int event_notify(enum event_t type, void *data, int size);
+#if CONFIG_IS_ENABLED(EVENT)
/**
* event_notify_null() - notify spies about an event
*
@@ -169,11 +169,6 @@ int event_notify(enum event_t type, void *data, int size);
*/
int event_notify_null(enum event_t type);
#else
-static inline int event_notify(enum event_t type, void *data, int size)
-{
- return 0;
-}
-
static inline int event_notify_null(enum event_t type)
{
return 0;
--
2.34.1
More information about the U-Boot
mailing list