[Intel-xe] ✗ CI.checkpatch: warning for drm/xe: Add mocs kunit (rev14)

[Patchwork]

== Series Details ==

Series: drm/xe: Add mocs kunit (rev14)
URL   : https://patchwork.freedesktop.org/series/124926/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
63c2b6b160bca2df6efc7bc4cea6f442097d7854
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 769fbd21328fe5c8ce7215ff60cbadd3c5375141
Author: Ruthuvikas Ravikumar <ruthuvikas.ravikumar at intel.com>
Date:   Thu Nov 16 00:33:21 2023 +0530

    drm/xe: Add mocs kunit
    
    This kunit verifies the hardware values of mocs and
    l3cc registers with the KMD programmed values.
    
    v13: Remove ret after forcewake.
    
    v11: Add KUNIT_ASSERT_EQ_MSG for Forcewake.
    
    v9/v10: Add Forcewake Fail.
    
    v8: Remove xe_bo.h and xe_pm.h
        Remove mocs and l3cc from live_mocs.
        Pull debug and err msg for mocs/l3cc out of if else block.
        Add HAS_LNCF_MOCS.
    
    v7: correct checkpath
    
    v6: Change ssize_t type.
        Change forcewake domain to XE_FW_GT.
        Update change of MOCS registers are multicast on Xe_HP and beyond
        patch.
    
    v5: Release forcewake.
        Remove single statement braces.
        Fix debug statements.
    
    v4: Drop stratch and vaddr.
        Fix debug statements.
        Fix indentation.
    
    v3: Fix checkpath.
    
    v2: Fix checkpath.
    
    Cc: Aravind Iddamsetty <aravind.iddamsetty at intel.com>
    Cc: Mathew D Roper <matthew.d.roper at intel.com>
    Reviewed-by: Mathew D Roper <matthew.d.roper at intel.com>
    Signed-off-by: Ruthuvikas Ravikumar <ruthuvikas.ravikumar at intel.com>
+ /mt/dim checkpatch 3b8183b7efad3d97ab6cf401f3fc0d24b30b6d3d drm-intel
769fbd213 drm/xe: Add mocs kunit
Traceback (most recent call last):
  File "scripts/spdxcheck.py", line 6, in <module>
    from ply import lex, yacc
ModuleNotFoundError: No module named 'ply'
Traceback (most recent call last):
  File "scripts/spdxcheck.py", line 6, in <module>
    from ply import lex, yacc
ModuleNotFoundError: No module named 'ply'
Traceback (most recent call last):
  File "scripts/spdxcheck.py", line 6, in <module>
    from ply import lex, yacc
ModuleNotFoundError: No module named 'ply'
-:57: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#57: 
new file mode 100644

-:123: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#123: FILE: drivers/gpu/drm/xe/tests/xe_mocs.c:62:
+		mocs_dbg(&gt_to_xe(gt)->drm, "%d 0x%x 0x%x 0x%x\n", i,
+			XELP_LNCFCMOCS(i).addr, reg_val, l3cc);

-:126: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#126: FILE: drivers/gpu/drm/xe/tests/xe_mocs.c:65:
+			KUNIT_FAIL(test, "l3cc reg 0x%x has incorrect val.\n",
+				XELP_LNCFCMOCS(i).addr);

-:133: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#133: FILE: drivers/gpu/drm/xe/tests/xe_mocs.c:72:
+static void read_mocs_table(struct xe_gt *gt,
+			      const struct xe_mocs_info *info)

-:153: CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#153: FILE: drivers/gpu/drm/xe/tests/xe_mocs.c:92:
+	     i++) {
+

-:159: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#159: FILE: drivers/gpu/drm/xe/tests/xe_mocs.c:98:
+		mocs_dbg(&gt_to_xe(gt)->drm, "%d 0x%x 0x%x 0x%x\n", i,
+			XELP_GLOBAL_MOCS(i).addr, reg_val, mocs);

-:162: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#162: FILE: drivers/gpu/drm/xe/tests/xe_mocs.c:101:
+			KUNIT_FAIL(test, "mocs reg 0x%x has incorrect val.\n",
+				XELP_GLOBAL_MOCS(i).addr);

total: 0 errors, 1 warnings, 6 checks, 182 lines checked