Re: [PATCH linux-evl 0/2] Add RISC-V support to EVL

On 7/8/26 08:07, Jan Kiszka wrote:
> On 08.07.26 07:17, Tobias Schaffner wrote:
>> On 7/8/26 00:30, Florian Bezdeka wrote:
>>> On Fri, 2025-10-10 at 12:10 +0200, Tobias Schaffner wrote:
>>>> Hi all,
>>>>
>>>> this series introduces an initial EVL port for RISC-V.
>>>>
>>>> While some tests are still failing, I’d like to share this early
>>>> version with the community to gather feedback, reviews, and hopefully
>>>> some assistance in refining the port.
>>>>
>>>> The patches have been tested in combination with the Dovetail RISC-V
>>>> port on top of v6.12-evl1-rebase.
>>>>
>>>> Looking forward to your feedback and suggestions.
>>>>
>>>
>>> I tried to apply this series today, and run into the following issues
>>> reported by my tooling.
>>>
>>> Patch 1 is fine, patch 2 has some checkpatch findings.
>>>
>>> A couple of debatable warnings like:
>>>
>>> WARNING: Misplaced SPDX-License-Identifier tag - use line 1 instead
>>> #58: FILE: arch/riscv/include/uapi/asm/evl/fptest.h:2:
>>> + * SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note
>>>
>>> WARNING: unnecessary whitespace before a quoted newline
>>> #95: FILE: arch/riscv/include/uapi/asm/evl/fptest.h:34:
>>> +
>>>
>>> In addition to a serious one:
>>>
>>> ERROR: Missing Signed-off-by: line by nominal patch author 'shannmu
>>> <[email protected]>'
>>
>> I did not get any responses from this email anymore. I can try to reach
>> out to him one last time, but what do we do about it if he will not
>> reply?
>>
> 
> If I read that patch correctly, it's majority is taken from the
> corresponding Xenomai 3 logic. How much is unique contribution, how much
> was your work, Tobias?

I added the evl_set_fpregs/evl_check_fpregs content which is a 
translation from what Gilles Chanteperdrix 
<[email protected]> did for the x86 and arm architectures 
with some isa enablement bits added.

This should be the initial version by shanmu where I added this on top:
https://github.com/shannmu/linux/blob/xenomai4/wip/dovetail-riscv/evl_port/arch/riscv/include/asm/evl/fptest.h

Best,
Tobias

> Jan
> 
>>> My perspective: We definitely need that.
>>>
>>> Best regards,
>>> Florian
>>
>>
>