On Tue, 25 Jul 2017 19:57:17 PDT (-0700), j.neuschae...@gmx.net wrote: > On Tue, Jul 11, 2017 at 06:31:30PM -0700, Palmer Dabbelt wrote: >> This patch contains all the build infastructure that actually enables >> the RISC-V port. This includes Makefiles, linker scripts, and Kconfig >> files. It also contains the only top-level change, which adds RISC-V to >> the list of architectures that need a sed run to produce the ARCH >> variable when building locally. >> >> Signed-off-by: Palmer Dabbelt <pal...@dabbelt.com> >> --- > [...] >> +config ISA_C >> + bool "Emit compressed instructions when building Linux" > > As a user, I'd prefer to have slightly more globally-recognisable names > than ISA_<single letter> for RISC-V instruction set architecture > extensions. A quick "git grep -A2 'config ISA'" shows the following > Kconfig symbols: > > * ISA, ISA_BUS_API, ISA_DMA_API, ISAPNP: > Settings related to the historic ISA bus. > * ISA_ARCOMPACT/ISA_ARCV2 (arch/arc), > ISA_M32R/ISA_M32R2/ISA_DSP_LEVEL2/ISA_DUAL_ISSUE (arch/m32r): > Instruction set options. > > Four out of the six instruction set options have ARC/M32R in the name, > and I think that makes things slightly more readable. Therefore I > humbly propose something longer, and with a hint of RISC-V in the name, > such as ISA_RVC. > > (Take this with a grain of salt, perhaps.)
Good timing: I was about to submit a v6 patch set. I'm cool with CONFIG_ISA_RVC and friends, do you mind submitting a patch?
