[PATCH 1/2] riscv: Rework riscv_cpu_probe for current event macros

Heinrich Schuchardt heinrich.schuchardt at canonical.com
Tue Sep 5 10:58:31 CEST 2023


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 <trini at 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.

Reviewed-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>




More information about the U-Boot mailing list