[PATCH 1/1] dm: fix DM_EVENT dependencies

Heinrich Schuchardt heinrich.schuchardt at canonical.com
Wed May 4 08:06:12 CEST 2022



On 5/3/22 21:35, Tom Rini wrote:
> On Tue, May 03, 2022 at 09:18:33PM +0200, Heinrich Schuchardt wrote:
> 
>> CONFIG_DM_EVENT without CONFIG_EVENT is non-functional. Select instead
>> of imply 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>
>> ---
>>   drivers/core/Kconfig | 2 +-
>>   include/event.h      | 7 +------
>>   2 files changed, 2 insertions(+), 7 deletions(-)
> 
> I don't think it makes sense to have two symbols at all.  It's part of
> DM, it should always be DM_EVENT I suppose.

The event EVT_MISC_INIT_F is used outside of the driver model.

arch/sandbox/cpu/start.c:127:EVENT_SPY(EVT_MISC_INIT_F, 
sandbox_misc_init_f);
board/google/chromebook_coral/coral.c:60:EVENT_SPY(EVT_MISC_INIT_F, 
coral_check_ll_boot);
board/keymile/kmcent2/kmcent2.c:213:EVENT_SPY(EVT_MISC_INIT_F, 
kmcent2_misc_init_f);
board/keymile/pg-wcom-ls102xa/pg-wcom-ls102xa.c:120:EVENT_SPY(EVT_MISC_INIT_F, 
pg_wcom_misc_init_f);

Best regards

Heinrich


More information about the U-Boot mailing list