✗ CI.checkpatch: warning for drm/i915/display: use x100 version for full version and release (rev2)

[Patchwork]

== Series Details ==

Series: drm/i915/display: use x100 version for full version and release (rev2)
URL   : https://patchwork.freedesktop.org/series/140455/
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
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit e984cf40c056aaa70f5f1f0e09aa63018b6c0327
Author: Jani Nikula <jani.nikula at intel.com>
Date:   Mon Oct 28 22:12:07 2024 +0200

    drm/i915/display: use x100 version for full version and release
    
    Use x100, or ver * 100 + rel, versions for full IP version checks,
    similar to what xe driver does:
    
    - Replace IP_VER(14, 1) inline with 1401, etc.
    
    - Convert DISPLAY_VER_FULL() to DISPLAY_VERx100()
    
    - Convert IS_DISPLAY_VER_FULL() to IS_DISPLAY_VERx100()
    
    - Convert IS_DISPLAY_VER_STEP() to IS_DISPLAY_VERx100_STEP()
    
    This makes ver.rel versions easier to use, follows the xe driver
    pattern, and drops the dependency on the IP_VER() macro.
    
    v2: Rebase, drop IP_VER() from xe compat headers
    
    Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
    Acked-by: Matt Roper <matthew.d.roper at intel.com>
    Signed-off-by: Jani Nikula <jani.nikula at intel.com>
+ /mt/dim checkpatch 405e02c22d91503cc6c3d54f675ae25df62db9df drm-intel
e984cf40c056 drm/i915/display: use x100 version for full version and release
-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects?
#127: FILE: drivers/gpu/drm/i915/display/intel_display_device.h:171:
+#define IS_DISPLAY_VERx100(__i915, from, until) ( \
+	BUILD_BUG_ON_ZERO((from) < 200) + \
+	(DISPLAY_VERx100(__i915) >= (from) && \
+	 DISPLAY_VERx100(__i915) <= (until)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'from' - possible side-effects?
#127: FILE: drivers/gpu/drm/i915/display/intel_display_device.h:171:
+#define IS_DISPLAY_VERx100(__i915, from, until) ( \
+	BUILD_BUG_ON_ZERO((from) < 200) + \
+	(DISPLAY_VERx100(__i915) >= (from) && \
+	 DISPLAY_VERx100(__i915) <= (until)))

-:148: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects?
#148: FILE: drivers/gpu/drm/i915/display/intel_display_device.h:191:
+#define IS_DISPLAY_VERx100_STEP(__i915, ipver, from, until) \
+	(IS_DISPLAY_VERx100((__i915), (ipver), (ipver)) && \
 	 IS_DISPLAY_STEP((__i915), (from), (until)))

-:148: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'ipver' - possible side-effects?
#148: FILE: drivers/gpu/drm/i915/display/intel_display_device.h:191:
+#define IS_DISPLAY_VERx100_STEP(__i915, ipver, from, until) \
+	(IS_DISPLAY_VERx100((__i915), (ipver), (ipver)) && \
 	 IS_DISPLAY_STEP((__i915), (from), (until)))

-:158: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects?
#158: FILE: drivers/gpu/drm/i915/display/intel_display_device.h:199:
+#define DISPLAY_VERx100(i915)	(DISPLAY_RUNTIME_INFO(i915)->ip.ver * 100 + \
+				 DISPLAY_RUNTIME_INFO(i915)->ip.rel)

total: 0 errors, 0 warnings, 5 checks, 223 lines checked