Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 10 Sep 2021 23:06:24 -0700 (PDT)
Subject:     Re: [PATCH] riscv: add rv32 and rv64 randconfig build targets
In-Reply-To: <25894c79-120f-8885-34cd-2e9dec0017ba@infradead.org>
CC:     linux-kernel@vger.kernel.org, mpe@ellerman.id.au,
        Paul Walmsley <paul.walmsley@sifive.com>,
        aou@eecs.berkeley.edu, linux-riscv@lists.infradead.org
From:   Palmer Dabbelt <palmer@dabbelt.com>
To:     rdunlap@infradead.org
Message-ID: <mhng-a1bcb222-57a9-4ef5-82bc-bed70d1a7e38@palmerdabbelt-glaptop>
Mime-Version: 1.0 (MHng)
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk

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
>>