✗ CI.checkpatch: warning for series starting with [1/2] drm/xe/vf: Store the VF GuC version in the compatibility struct

[Patchwork]

== Series Details ==

Series: series starting with [1/2] drm/xe/vf: Store the VF GuC version in the compatibility struct
URL   : https://patchwork.freedesktop.org/series/145667/
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
22f9cda3436b4fe965b5c5f31d2f2c1bcb483189
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 13c2fd66677bb42dc013ce4f4dbaf1cdbf1f5428
Author: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
Date:   Fri Feb 28 12:02:46 2025 -0800

    drm/xe/vf: Fix guc_info debugfs for VFs
    
    There are 2 issues with the guc_info debugfs when using it from a VF:
    
    1 - We report a GuC binary path and a GuC release version as wanted, but
        the VF actually wants a compatibility version and no specific binary
    
    2 - We attempt to read a bunch of registers that the VFs doesn't have
        access to.
    
    Fix #1 by setting the VF expected value during handshake and #2 by not
    accessing those registers if VF. The output of the debugfs changes from:
    
    GuC firmware: xe/ptl_guc_70.38.1.bin
            status: PRELOADED
            wanted release version 70.38.1
            found compatibility version 1.18.1
    
    to:
    
    GuC firmware: Loaded by PF
            status: PRELOADED
            wanted compatibility version 1.2.0
            found compatibility version 1.18.1
    
    Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
    Cc: Michal Wajdeczko <michal.wajdeczko at intel.com>
    Cc: Lukasz Laguna <lukasz.laguna at intel.com>
+ /mt/dim checkpatch ca9d01b66c70413a62a1a28fbf8709d2e8440c1c drm-intel
c97ea3e44d4d drm/xe/vf: Store the VF GuC version in the compatibility struct
-:247: WARNING:LONG_LINE: line length of 105 exceeds 100 columns
#247: FILE: drivers/gpu/drm/xe/xe_gt_sriov_vf.c:1091:
+	struct xe_uc_fw_version *guc_version = &gt->uc.guc.fw.versions.found[XE_UC_FW_VER_COMPATIBILITY];

total: 0 errors, 1 warnings, 0 checks, 305 lines checked
13c2fd66677b drm/xe/vf: Fix guc_info debugfs for VFs
-:22: WARNING:BAD_SIGN_OFF: Use a single space after To:
#22: 
to:

-:22: ERROR:BAD_SIGN_OFF: Unrecognized email address: ''
#22: 
to:

-:80: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#80: FILE: drivers/gpu/drm/xe/xe_guc.c:1528:
+		drm_printf(p, "\tBootrom status = 0x%x\n",
+			REG_FIELD_GET(GS_BOOTROM_MASK, status));

-:82: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#82: FILE: drivers/gpu/drm/xe/xe_guc.c:1530:
+		drm_printf(p, "\tuKernel status = 0x%x\n",
+			REG_FIELD_GET(GS_UKERNEL_MASK, status));

-:84: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#84: FILE: drivers/gpu/drm/xe/xe_guc.c:1532:
+		drm_printf(p, "\tMIA Core status = 0x%x\n",
+			REG_FIELD_GET(GS_MIA_MASK, status));

-:86: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#86: FILE: drivers/gpu/drm/xe/xe_guc.c:1534:
+		drm_printf(p, "\tLog level = %d\n",
+			xe_guc_log_get_level(&guc->log));

-:91: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#91: FILE: drivers/gpu/drm/xe/xe_guc.c:1539:
+			drm_printf(p, "\t%2d: \t0x%x\n",
+				i, xe_mmio_read32(&gt->mmio, SOFT_SCRATCH(i)));

total: 1 errors, 1 warnings, 5 checks, 94 lines checked