✗ CI.checkpatch: warning for Fixes for VF GuC version (rev3)

Patchwork patchwork at emeril.freedesktop.org
Thu Jun 5 00:12:20 UTC 2025


== Series Details ==

Series: Fixes for VF GuC version (rev3)
URL   : https://patchwork.freedesktop.org/series/149613/
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
202708c00696422fd217223bb679a353a5936e23
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit e81ddb9ce242ab2fef9f11e4f1154a732af1b327
Author: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
Date:   Tue Jun 3 16:54:37 2025 -0700

    drm/xe/vf: Store the GuC FW info in guc->fw
    
    The GuC compatibility version that we read from the CSS header in
    native/PF and the GuC VF version that we get when a VF handshakes with
    the GuC are the same version number, so we should store it into the same
    structure. This makes all the checks based on the compatibility version
    automatically work for VFs without having to copy the value over.
    
    For completion, also copy the wanted version and set the path to a known
    string to indicate that the FW is PF-loaded. This way all the info will
    be coherent when dumped from debugfs.
    
    v2: several code cleanups and style changes (Michal), rebase on
    bootstrap changes.
    
    v3: s/min/wanted/, clarify that handshake must happen before we can get
    the VF versions (Michal)
    
    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>
    Reviewed-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
+ /mt/dim checkpatch 9becf23dcd5721a2ddb80f01d4964a1657117f11 drm-intel
2bc3c493ee20 drm/xe/uc: Prepare uc_fw_version for storing the VF ABI version
d754c54ca8c8 drm/xe/vf: Boostrap all GTs immediately after MMIO init
8857e82ed4aa drm/xe/vf: Use uc_fw_version to store the negotiated GuC ABI
e81ddb9ce242 drm/xe/vf: Store the GuC FW info in guc->fw
-:68: WARNING:RETURN_VOID: void function return statements are not generally useful
#68: FILE: drivers/gpu/drm/xe/xe_gt_sriov_vf.c:294:
+	return;
+}

total: 0 errors, 1 warnings, 0 checks, 133 lines checked




More information about the Intel-xe mailing list