[PATCH] riscv: add base extensions to enum riscv_isa_ext_id

Vineet Gupta vineetg at rivosinc.com
Thu Dec 22 15:13:56 PST 2022



On 12/22/22 14:53, Conor Dooley wrote:
> Hi Vineet,
>
> On Thu, Dec 22, 2022 at 02:41:04PM -0800, Vineet Gupta wrote:
>> This allows for using the enum in general to refer to any extension.
> What is the point of this patch? It's hard to understand the rationale
> without having any user, especially...

Right I'm trying to build a cpu feature enable API which required 
handling of all extensions.
Ideally yes this needs to go there. But it seemed more like a cleanup 
type patch to warrant a standalone submission.
But I guess I can wait for dust to settle.


>
>> +	RISCV_ISA_EXT_s = ('s' - 'a'),
>> +	RISCV_ISA_EXT_u = ('u' - 'a'),
> ... as it is diametrically opposed to another patchset which deletes
> this enum, with a rationale.
> https://lore.kernel.org/linux-riscv/20221204174632.3677-5-jszhang@kernel.org/
>
> +CC Jisheng.
>
> Also, please run get_maintainer.pl, there's a bunch of people touching
> this area at the moment & CCing them would be great.

Ok sure.

> Thanks,
> Conor.
>
> btw, is the rivos list intentionally on CC?

Yep.



More information about the linux-riscv mailing list