Hi Cyan, On Fri, 16 May 2025 at 05:07, Cyan Yang <cyan.yang@xxxxxxxxxx> wrote: > Add SiFive vendor extension support to the kernel with the target of > "xsfvqmaccdod" and "xsfvqmaccqoq". > > Signed-off-by: Cyan Yang <cyan.yang@xxxxxxxxxx> Thanks for your patch, which is now commit 2d147d77ae6e96c1 ("riscv: Add SiFive xsfvqmaccdod and xsfvqmaccqoq vendor extensions") in riscv/for-next. > --- a/arch/riscv/Kconfig.vendor > +++ b/arch/riscv/Kconfig.vendor > @@ -16,6 +16,19 @@ config RISCV_ISA_VENDOR_EXT_ANDES > If you don't know what to do here, say Y. > endmenu > > +menu "SiFive" > +config RISCV_ISA_VENDOR_EXT_SIFIVE > + bool "SiFive vendor extension support" > + select RISCV_ISA_VENDOR_EXT > + default y I guess this has no dependency on ARCH_SIFIVE and does not default to ARCH_SIFIVE because this extension can be present on non-Sifive SoCs, too? Probably I should have asked this when the other RISCV_ISA_VENDOR_EXT_* were introduced, but at least for ANDES I already know the answer. > + help > + Say N here if you want to disable all SiFive vendor extension > + support. This will cause any SiFive vendor extensions that are > + requested by hardware probing to be ignored. > + > + If you don't know what to do here, say Y. > +endmenu > + > menu "T-Head" > config RISCV_ISA_VENDOR_EXT_THEAD > bool "T-Head vendor extension support" Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds