✗ CI.checkpatch: warning for Separate PVC Device ID patches (rev2)

[Patchwork]

== Series Details ==

Series: Separate PVC Device ID patches (rev2)
URL   : https://patchwork.freedesktop.org/series/127580/
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
6030b24c1386b00de8187b5fb987e283a57b372a
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 8e1fe363126484ffbe2ab9724cf524e4b12dd3e4
Author: Rodrigo Vivi <rodrigo.vivi at intel.com>
Date:   Fri Dec 8 14:44:09 2023 -0500

    drm/xe: Add PVC's PCI device IDs
    
    This patch introduces Ponte Vecchio support in Xe driver.
    
    Please note that besides this patch, likely the force_probe
    still needs to be used in order to actually enable the
    support for PVC.
    
    This patch was separated from the rest so we can ensure
    compliance with DRM uAPI rules on compute platforms, where
    there must be an open-source compiler to be shared among
    different vendors.
    
    v2: Add more info about the missing compliance (Lucas)
    
    Cc: Lucas De Marchi <lucas.demarchi at intel.com>
    Cc: Thomas Hellström <thomas.hellstrom at linux.intel.com>
    Cc: Oded Gabbay <ogabbay at kernel.org>
    Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
+ /mt/dim checkpatch 1fc141d1f445b8e5ab4ff14645b0482dd98460b9 drm-intel
f5a19b5f8 Revert "drm/xe/pvc: Add pci ids for PVC"
-:6: WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'fe2e5f2eed85acbbf03f411a4830ce6cbbaaf179', maybe rebased or not pulled?
#6: 
This reverts commit fe2e5f2eed85acbbf03f411a4830ce6cbbaaf179.

total: 0 errors, 1 warnings, 0 checks, 8 lines checked
ae5468ddb fixup! drm/xe: Introduce a new DRM driver for Intel GPUs
-:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

total: 0 errors, 1 warnings, 0 checks, 19 lines checked
8e1fe3631 drm/xe: Add PVC's PCI device IDs
-:36: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#36: FILE: include/drm/xe_pciids.h:186:
+#define XE_PVC_IDS(MACRO__, ...)		\
+	MACRO__(0x0B69, ## __VA_ARGS__),	\
+	MACRO__(0x0B6E, ## __VA_ARGS__),	\
+	MACRO__(0x0BD5, ## __VA_ARGS__),	\
+	MACRO__(0x0BD6, ## __VA_ARGS__),	\
+	MACRO__(0x0BD7, ## __VA_ARGS__),	\
+	MACRO__(0x0BD8, ## __VA_ARGS__),	\
+	MACRO__(0x0BD9, ## __VA_ARGS__),	\
+	MACRO__(0x0BDA, ## __VA_ARGS__),	\
+	MACRO__(0x0BDB, ## __VA_ARGS__),	\
+	MACRO__(0x0BE0, ## __VA_ARGS__),	\
+	MACRO__(0x0BE1, ## __VA_ARGS__),	\
+	MACRO__(0x0BE5, ## __VA_ARGS__)

-:36: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'MACRO__' - possible side-effects?
#36: FILE: include/drm/xe_pciids.h:186:
+#define XE_PVC_IDS(MACRO__, ...)		\
+	MACRO__(0x0B69, ## __VA_ARGS__),	\
+	MACRO__(0x0B6E, ## __VA_ARGS__),	\
+	MACRO__(0x0BD5, ## __VA_ARGS__),	\
+	MACRO__(0x0BD6, ## __VA_ARGS__),	\
+	MACRO__(0x0BD7, ## __VA_ARGS__),	\
+	MACRO__(0x0BD8, ## __VA_ARGS__),	\
+	MACRO__(0x0BD9, ## __VA_ARGS__),	\
+	MACRO__(0x0BDA, ## __VA_ARGS__),	\
+	MACRO__(0x0BDB, ## __VA_ARGS__),	\
+	MACRO__(0x0BE0, ## __VA_ARGS__),	\
+	MACRO__(0x0BE1, ## __VA_ARGS__),	\
+	MACRO__(0x0BE5, ## __VA_ARGS__)

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