On 5/13/25 16:32, Abdelrahman Fekry wrote:
The test file for the IR decoder used single-line comments at the top to document its purpose and licensing, which is inconsistent with the style used throughout the Linux kernel.
in this patch i converted the file header to a proper multi-line comment block (/*) that aligns with standard kernel practices. This improves readability, consistency across selftests, and ensures the license and documentation are clearly visible in a familiar format.
No functional changes have been made.
Signed-off-by: Abdelrahman Fekry Abdelrahmanfekry375@gmail.com
Did you run checkpatch on this patch? I am seeing
CHECK: From:/Signed-off-by: email comments mismatch: 'From: Abdelrahman Fekry abdelrahmanfekry375@gmail.com' != 'Signed-off-by: Abdelrahman Fekry Abdelrahmanfekry375@gmail.com'
Fix these as well.
ERROR: trailing whitespace #136: FILE: tools/testing/selftests/ir/ir_loopback.c:3: + * $
ERROR: trailing whitespace #137: FILE: tools/testing/selftests/ir/ir_loopback.c:4: + * Selftest for IR decoder $
Please fix it and send v2.
thanks, -- Shuah