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

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

My perspective: We definitely need that.

Best regards,
Florian