[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <fc881ae2-4803-5859-f9c1-52f1a1381f6f@roeck-us.net>
Date: Tue, 16 Oct 2018 06:18:49 -0700
From: Guenter Roeck <linux@...ck-us.net>
To: Palmer Dabbelt <palmer@...ive.com>, alankao@...estech.com
Cc: zong@...estech.com, nickhu@...estech.com, greentime@...estech.com,
linux-kernel@...r.kernel.org, vincentc@...estech.com,
linux-riscv@...ts.infradead.org, Christoph Hellwig <hch@....de>
Subject: Re: [PATCH v8 3/5] Cleanup ISA string setting
On 10/15/2018 03:34 PM, Palmer Dabbelt wrote:
> On Sun, 14 Oct 2018 16:46:40 PDT (-0700), alankao@...estech.com wrote:
>> Hi Guenter,
>>
>> On Sat, Oct 13, 2018 at 05:02:05PM -0700, Guenter Roeck wrote:
>>> Hi,
>>>
>>> With this patch in -next applied, I get the following error
>>> when building riscv:defconfig.
>>
>> I guess what you meant was this commit:
>> commit 0a5bee05fc67ce2044b8d91a18abbf14042da531
>>
>> The modification in Makefile was
>>
>> -endif
>> -
>> -KBUILD_AFLAGS += -march=$(KBUILD_MARCH)$(KBUILD_ARCH_A)fd$(KBUILD_ARCH_C)
>> +# ISA string setting
>> +riscv-march-$(CONFIG_ARCH_RV32I) := rv32im
>> +riscv-march-$(CONFIG_ARCH_RV64I) := rv64im
>> +riscv-march-$(CONFIG_RISCV_ISA_A) := $(riscv-march-y)a
>> +riscv-march-y := fd
>> +riscv-march-$(CONFIG_RISCV_ISA_C) := $(riscv-march-y)c
>>
>> and the ":= fd" line was wrong. It should have been
>>
>> +riscv-march-y := $(riscv-march-y)fd
>>
>>>
>>>
>>> What am I missing ? Does RISC-V now require a new non-upstream compiler ?
>>> Or a special version of make ? I am currently using GNU Make 4.1.
>>
>> So I sent the v8 to fix this problem.
>> Sorry for the inconvinience.
>>
>>>
>>> Thanks,
>>> Guenter
>>>
>>> ---
>>
>> And Palmer, it seems that the patch that causes this build error has been
>> pulled into linux-next. Any idea how to fix that one?
>
> If I run into the new version then I'll overwrite in in my for-next branch. I don't see one, though...
>
The series is on lore, lkml, and patchwork. Patchwork link to v8 of this patch:
https://lore.kernel.org/patchwork/patch/997407/
Guenter
Powered by blists - more mailing lists