✗ CI.checkpatch: warning for drm/xe: Add assert for XE_WA() usage

Patchwork patchwork at emeril.freedesktop.org
Tue Jul 16 14:47:59 UTC 2024


== Series Details ==

Series: drm/xe: Add assert for XE_WA() usage
URL   : https://patchwork.freedesktop.org/series/136146/
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
51ce9f6cd981d42d7467409d7dbc559a450abc1e
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit bd12d143796af45ba4e1d1af81d5324a42f8adb1
Author: Lucas De Marchi <lucas.demarchi at intel.com>
Date:   Tue Jul 16 07:37:20 2024 -0700

    drm/xe: Add assert for XE_WA() usage
    
    It's not always safe to call XE_WA() in the driver initialization. Add a
    xe_gt_assert() so this doesn't go unnoticed.
    
    Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
+ /mt/dim checkpatch e6cf7ddccb36f44102032b3c803d978ef5c8016c drm-intel
bd12d143796a drm/xe: Add assert for XE_WA() usage
-:66: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'gt__' - possible side-effects?
#66: FILE: drivers/gpu/drm/xe/xe_wa.h:30:
+#define XE_WA(gt__, id__) ({						\
+	xe_gt_assert(gt__, (gt__)->wa_active.oob_initialized);		\
+	test_bit(XE_WA_OOB_ ## id__, (gt__)->wa_active.oob);		\
+})

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




More information about the Intel-xe mailing list