✗ CI.checkpatch: warning for drm/xe: Add psmi support (rev5)

[Patchwork]

== Series Details ==

Series: drm/xe: Add psmi support (rev5)
URL   : https://patchwork.freedesktop.org/series/151729/
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
553439844b6500767ce8aef522cfe9fbb7ece541
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 95a061730bdd463437456a92d1da7bfad9006d72
Author: Lucas De Marchi <lucas.demarchi at intel.com>
Date:   Thu Aug 21 09:18:07 2025 -0700

    drm/xe/configfs: Dump custom settings when binding
    
    Device configuration using configfs could be prepared long time prior
    the driver load. Currently all the xe configfs entries are for things
    that are important to have in the log if a non-default value is being
    used. Add a info-level message about that with the individual entries
    that are different than the default.
    
    Based on previous patch by Michal Wajdeczko.
    
    Cc: Michal Wajdeczko <michal.wajdeczko at intel.com>
    Cc: John Harrison <John.C.Harrison at Intel.com>
    Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar at intel.com>
    Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
+ /mt/dim checkpatch 633a37934680cad76b742422d1b955d4cdde5189 drm-intel
52a993d0356f drm/xe/psmi: Add GuC flag to enable PSMI
f2918bbe7b94 drm/xe/psmi: Add debugfs interface for PSMI
-:111: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#111: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 373 lines checked
9f963ae3fa56 drm/xe/rtp: Add match for psmi
5be7b6fa0487 drm/xe/psmi: Add Wa_14020001231
-:35: WARNING:LONG_LINE: line length of 105 exceeds 100 columns
#35: FILE: drivers/gpu/drm/xe/xe_guc_ads.c:364:
+				 GUC_WORKAROUND_KLV_DISABLE_PSMI_INTERRUPTS_AT_C6_ENTRY_RESTORE_AT_EXIT);

total: 0 errors, 1 warnings, 0 checks, 27 lines checked
098893b9e171 drm/xe/psmi: Add Wa_16023683509
16807a63f0fe drm/xe/configfs: Simplify kernel doc
3aa01d4e16d5 drm/xe/configfs: Allow to enable PSMI
8e870b16e895 drm/xe/configfs: Use guard() for dev->lock
e212dafea394 drm/xe/configfs: Use tree-like output in documentation
46b64520b9f1 drm/xe/configfs: Improve documentation steps
3489982f9831 drm/xe/configfs: Minor fixes to documentation
95a061730bdd drm/xe/configfs: Dump custom settings when binding
-:30: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'attr_' - possible side-effects?
#30: FILE: drivers/gpu/drm/xe/xe_configfs.c:448:
+#define PRI_CUSTOM_ATTR(fmt_, attr_) do { \
+		if (dev->config.attr_ != device_defaults.attr_) \
+			pci_info(pdev, "configfs: " __stringify(attr_) " = " fmt_ "\n", \
+				 dev->config.attr_); \
+	} while (0)

-:30: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'attr_' may be better as '(attr_)' to avoid precedence issues
#30: FILE: drivers/gpu/drm/xe/xe_configfs.c:448:
+#define PRI_CUSTOM_ATTR(fmt_, attr_) do { \
+		if (dev->config.attr_ != device_defaults.attr_) \
+			pci_info(pdev, "configfs: " __stringify(attr_) " = " fmt_ "\n", \
+				 dev->config.attr_); \
+	} while (0)

total: 0 errors, 0 warnings, 2 checks, 74 lines checked