[PATCH] lib: sbi: Add missing sscrind and sscfg extensions in sbi_hart_ext[]

[Clément Léger]

On 20/03/2024 15:57, Andreas Schwab wrote:
> On Mär 20 2024, Clément Léger wrote:
> 
>> The sbi_hart_ext[] array is missing these two extensions ids. It is
>> expected that this array contains all the extensions declaration at the
>> same index of the SBI_HART_EXT_* define. Without this, when adding a new
>> extension, there is a mismatch between ids and extension names and it
>> can even display corrupted extension names.
>>
>> Signed-off-by: Clément Léger <cleger at rivosinc.com>
>>
>> ---
>>  lib/sbi/sbi_hart.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/lib/sbi/sbi_hart.c b/lib/sbi/sbi_hart.c
>> index 3d13694..d68e2a0 100644
>> --- a/lib/sbi/sbi_hart.c
>> +++ b/lib/sbi/sbi_hart.c
>> @@ -666,6 +666,8 @@ const struct sbi_hart_ext_data sbi_hart_ext[] = {
>>  	__SBI_HART_EXT_DATA(sdtrig, SBI_HART_EXT_SDTRIG),
>>  	__SBI_HART_EXT_DATA(smcsrind, SBI_HART_EXT_SMCSRIND),
>>  	__SBI_HART_EXT_DATA(smcdeleg, SBI_HART_EXT_SMCDELEG),
>> +	__SBI_HART_EXT_DATA(sscsrind, SBI_HART_EXT_SSCSRIND),
>> +	__SBI_HART_EXT_DATA(ssccfg, SBI_HART_EXT_SSCCFG),
>>  };
> 
> Please add a static_assert to verify that the size of sbi_hart_ext
> agrees with SBI_HART_EXT_MAX.

Yes sure.

Regards,

Clément