✗ CI.checkpatch: warning for drm/xe: Create and use XE_DEVICE_WA infrastructure (rev3)

[Patchwork]

== Series Details ==

Series: drm/xe: Create and use XE_DEVICE_WA infrastructure (rev3)
URL   : https://patchwork.freedesktop.org/series/150777/
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
f8ff75ae1d2127635239b134695774ed4045d05b
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 7fe9bf800d8b93a83ba2e65dce0d56717119ebe4
Author: Matt Atwood <matthew.s.atwood at intel.com>
Date:   Wed Jun 25 13:07:50 2025 -0700

    drm/xe: disable wa_15015404425 for PTL B0
    
    This workaround only applies to PTL Compute Die A0. The reality of
    modern platforms is we're Multi Chip Packages with logic spread across
    multiple dies. Because this information is not available during PCI
    probe it becomes a bit more complicated.
    
    This workaround needs to be applied on PTL until we prove that we are
    not Compute Die A0 stepping without reading any MMIOs. So use the new
    XE_DEVICE_WA infrastructure to apply early, until we can determine our
    stepping.
    
    There are at least two ways to determine Compute Die stepping. This
    patch uses the Media GT stepping to map to Compute Die stepping, in this
    case Compute and Media dies step synchronously.
    
    Since we're using the Media GT information to determine Compute Die
    stepping, use the XE_WA and the oob infrastructure to come back and
    toggle the workaround off when we know its safe, and after GT init.
    
    v2: rename SoC to device, avoid null pointer dereference, update commit
    message.
    v3: rebase
    
    Signed-off-by: Matt Atwood <matthew.s.atwood at intel.com>
+ /mt/dim checkpatch 74f3ed8bab4e5ef51f1c266e8c7ffd1f2a052a73 drm-intel
5c7590e4f4c1 drm/xe: add xe_device_wa infrastructure
-:42: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#42: 
new file mode 100644

-:73: CHECK:BOOL_COMPARISON: Using comparison to false is error prone
#73: FILE: drivers/gpu/drm/xe/xe_gen_wa_oob.c:99:
+			if (device == false)

-:86: CHECK:BOOL_COMPARISON: Using comparison to false is error prone
#86: FILE: drivers/gpu/drm/xe/xe_gen_wa_oob.c:123:
+	if (device == false)

-:99: WARNING:LINE_SPACING: Missing a blank line after declarations
#99: FILE: drivers/gpu/drm/xe/xe_gen_wa_oob.c:166:
+	char *device = strstr(args[ARGS_CHEADER].fn, "device_");
+	if (device == NULL)

-:99: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!device"
#99: FILE: drivers/gpu/drm/xe/xe_gen_wa_oob.c:166:
+	if (device == NULL)

-:104: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "device"
#104: FILE: drivers/gpu/drm/xe/xe_gen_wa_oob.c:171:
+	bool prefix = device != NULL ? true : false;

total: 0 errors, 2 warnings, 4 checks, 78 lines checked
1d8bb22e3bd3 drm/xe: Add infrastructure for Device OOB workarounds
-:83: WARNING:LONG_LINE: line length of 113 exceeds 100 columns
#83: FILE: drivers/gpu/drm/xe/xe_rtp.h:425:
+	struct xe_gt * :	(struct xe_rtp_process_ctx){ { (void *)(arg__) }, XE_RTP_PROCESS_TYPE_GT },	\

-:84: WARNING:LONG_LINE: line length of 111 exceeds 100 columns
#84: FILE: drivers/gpu/drm/xe/xe_rtp.h:426:
+	struct xe_device * :	(struct xe_rtp_process_ctx){ { (void *)(arg__) }, XE_RTP_PROCESS_TYPE_DEVICE })

-:222: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'xe__' - possible side-effects?
#222: FILE: drivers/gpu/drm/xe/xe_wa.h:43:
+#define XE_DEVICE_WA(xe__, id__) ({					\
+	xe_assert(xe__, (xe__)->oob_initialized);			\
+	test_bit(XE_DEVICE_WA_OOB_ ## id__, (xe__)->oob);			\
+})

total: 0 errors, 2 warnings, 1 checks, 171 lines checked
7a556a75f0fc drm/xe: Move Wa_15015404425 to use the new XE_DEVICE_WA macro
79780f653b66 drm/xe: extend Wa_15015404425 to apply to PTL
7fe9bf800d8b drm/xe: disable wa_15015404425 for PTL B0
-:48: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "xe->tiles->media_gt"
#48: FILE: drivers/gpu/drm/xe/xe_pci.c:902:
+	if (xe->tiles->media_gt != NULL &&

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