Hi Heinrich, On Tue, Sep 05, 2023 at 10:58:31AM +0200, Heinrich Schuchardt wrote: > On 05.09.23 05:01, Leo Liang wrote: > > Hi Tom, > > > > On Mon, Sep 04, 2023 at 03:06:34PM -0400, Tom Rini wrote: > > > This function should now be a EVENT_SPY_SIMPLE call, update it. > > > > > > Signed-off-by: Tom Rini <tr...@konsulko.com> > > > --- > > > arch/riscv/cpu/cpu.c | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/arch/riscv/cpu/cpu.c b/arch/riscv/cpu/cpu.c > > > index d64aa330f206..cfe9fdc9df55 100644 > > > --- a/arch/riscv/cpu/cpu.c > > > +++ b/arch/riscv/cpu/cpu.c > > > @@ -66,7 +66,7 @@ static inline bool supports_extension(char ext) > > > #endif /* CONFIG_CPU */ > > > } > > > -static int riscv_cpu_probe(void *ctx, struct event *event) > > > +static int riscv_cpu_probe(void) > > > { > > > #ifdef CONFIG_CPU > > > int ret; > > > @@ -79,7 +79,7 @@ static int riscv_cpu_probe(void *ctx, struct event > > > *event) > > > return 0; > > > } > > > -EVENT_SPY(EVT_DM_POST_INIT_R, riscv_cpu_probe); > > > +EVENT_SPY_SIMPLE(EVT_DM_POST_INIT_R, riscv_cpu_probe); > > > > I cannot seem to find macro "EVENT_SPY_SIMPLE". > > Is there anything that should be added? > > Or did I miss anything? > > This patch is only applicable to origin/next. The definition there is in > include/event.h:311. >
Thanks for the pointer! Best regards, Leo > Reviewed-by: Heinrich Schuchardt <heinrich.schucha...@canonical.com> >