| Message ID | 20210717044424.17478-1-rdunlap@infradead.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | riscv: add rv32 and rv64 randconfig build targets | expand |
Hi, Any comments on this patch? thanks. On 7/16/21 9:44 PM, Randy Dunlap wrote: > Add the ability to do randconfig build targets for both > rv32 and rv64. > > Based on a similar patch by Michael Ellerman for PowerPC. > > Usage: > make ARCH=riscv rv32_randconfig > or > make ARCH=riscv rv64_randconfig > > Cc: Michael Ellerman <mpe@ellerman.id.au> > Cc: Paul Walmsley <paul.walmsley@sifive.com> > Cc: Palmer Dabbelt <palmer@dabbelt.com> > Cc: Albert Ou <aou@eecs.berkeley.edu> > Cc: linux-riscv@lists.infradead.org > > arch/riscv/Makefile | 10 ++++++++++ > arch/riscv/configs/32-bit.config | 2 ++ > arch/riscv/configs/64-bit.config | 2 ++ > 3 files changed, 14 insertions(+) > > --- linux-next-20210715.orig/arch/riscv/Makefile > +++ linux-next-20210715/arch/riscv/Makefile > @@ -132,3 +132,13 @@ zinstall install: > > archclean: > $(Q)$(MAKE) $(clean)=$(boot) > + > +PHONY += rv32_randconfig > +rv32_randconfig: > + $(Q)$(MAKE) KCONFIG_ALLCONFIG=$(srctree)/arch/riscv/configs/32-bit.config \ > + -f $(srctree)/Makefile randconfig > + > +PHONY += rv64_randconfig > +rv64_randconfig: > + $(Q)$(MAKE) KCONFIG_ALLCONFIG=$(srctree)/arch/riscv/configs/64-bit.config \ > + -f $(srctree)/Makefile randconfig > --- /dev/null > +++ linux-next-20210715/arch/riscv/configs/32-bit.config > @@ -0,0 +1,2 @@ > +CONFIG_ARCH_RV32I=y > +CONFIG_32BIT=y > --- /dev/null > +++ linux-next-20210715/arch/riscv/configs/64-bit.config > @@ -0,0 +1,2 @@ > +CONFIG_ARCH_RV64I=y > +CONFIG_64BIT=y >
On Thu, 12 Aug 2021 07:29:30 PDT (-0700), rdunlap@infradead.org wrote: > Hi, > Any comments on this patch? Sorry, I must have missed this one. It looks great, it's just that I don't see your Signed-off-by on the original patch. > thanks. > > On 7/16/21 9:44 PM, Randy Dunlap wrote: >> Add the ability to do randconfig build targets for both >> rv32 and rv64. >> >> Based on a similar patch by Michael Ellerman for PowerPC. >> >> Usage: >> make ARCH=riscv rv32_randconfig >> or >> make ARCH=riscv rv64_randconfig >> >> Cc: Michael Ellerman <mpe@ellerman.id.au> >> Cc: Paul Walmsley <paul.walmsley@sifive.com> >> Cc: Palmer Dabbelt <palmer@dabbelt.com> >> Cc: Albert Ou <aou@eecs.berkeley.edu> >> Cc: linux-riscv@lists.infradead.org >> >> arch/riscv/Makefile | 10 ++++++++++ >> arch/riscv/configs/32-bit.config | 2 ++ >> arch/riscv/configs/64-bit.config | 2 ++ >> 3 files changed, 14 insertions(+) >> >> --- linux-next-20210715.orig/arch/riscv/Makefile >> +++ linux-next-20210715/arch/riscv/Makefile >> @@ -132,3 +132,13 @@ zinstall install: >> >> archclean: >> $(Q)$(MAKE) $(clean)=$(boot) >> + >> +PHONY += rv32_randconfig >> +rv32_randconfig: >> + $(Q)$(MAKE) KCONFIG_ALLCONFIG=$(srctree)/arch/riscv/configs/32-bit.config \ >> + -f $(srctree)/Makefile randconfig >> + >> +PHONY += rv64_randconfig >> +rv64_randconfig: >> + $(Q)$(MAKE) KCONFIG_ALLCONFIG=$(srctree)/arch/riscv/configs/64-bit.config \ >> + -f $(srctree)/Makefile randconfig >> --- /dev/null >> +++ linux-next-20210715/arch/riscv/configs/32-bit.config >> @@ -0,0 +1,2 @@ >> +CONFIG_ARCH_RV32I=y >> +CONFIG_32BIT=y >> --- /dev/null >> +++ linux-next-20210715/arch/riscv/configs/64-bit.config >> @@ -0,0 +1,2 @@ >> +CONFIG_ARCH_RV64I=y >> +CONFIG_64BIT=y >>
On 9/10/21 11:06 PM, Palmer Dabbelt wrote: > On Thu, 12 Aug 2021 07:29:30 PDT (-0700), rdunlap@infradead.org wrote: >> Hi, >> Any comments on this patch? > > Sorry, I must have missed this one. It looks great, it's just that I don't see your Signed-off-by on the original patch. > >> thanks. Hi Palmer, I sent a v2 with the missing S-o-b: https://lore.kernel.org/all/20210912034538.19404-1-rdunlap@infradead.org/
--- linux-next-20210715.orig/arch/riscv/Makefile +++ linux-next-20210715/arch/riscv/Makefile @@ -132,3 +132,13 @@ zinstall install: archclean: $(Q)$(MAKE) $(clean)=$(boot) + +PHONY += rv32_randconfig +rv32_randconfig: + $(Q)$(MAKE) KCONFIG_ALLCONFIG=$(srctree)/arch/riscv/configs/32-bit.config \ + -f $(srctree)/Makefile randconfig + +PHONY += rv64_randconfig +rv64_randconfig: + $(Q)$(MAKE) KCONFIG_ALLCONFIG=$(srctree)/arch/riscv/configs/64-bit.config \ + -f $(srctree)/Makefile randconfig --- /dev/null +++ linux-next-20210715/arch/riscv/configs/32-bit.config @@ -0,0 +1,2 @@ +CONFIG_ARCH_RV32I=y +CONFIG_32BIT=y --- /dev/null +++ linux-next-20210715/arch/riscv/configs/64-bit.config @@ -0,0 +1,2 @@ +CONFIG_ARCH_RV64I=y +CONFIG_64BIT=y