✗ CI.checkpatch: warning for series starting with [CI,1/2] drm/i915: remove IS_LP()

Patchwork patchwork at emeril.freedesktop.org
Mon Sep 30 17:32:55 UTC 2024


== Series Details ==

Series: series starting with [CI,1/2] drm/i915: remove IS_LP()
URL   : https://patchwork.freedesktop.org/series/139305/
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 4b82b149fa848b66fb3fa5a57dd764a346db9251
Author: Jani Nikula <jani.nikula at intel.com>
Date:   Mon Sep 30 15:40:56 2024 +0300

    drm/i915/soc: stop using IS_GEN9_LP() and IS_GEN9_BC()
    
    Replace IS_GEN9_LP() and IS_GEN9_BC() with direct platform checks. This
    lets us remove their compat counterparts, as neither soc/ nor /display
    now no longer needs them.
    
    v2: Use !A && !B instead of !(A || B) (Ville)
    
    Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com> # v1
    Signed-off-by: Jani Nikula <jani.nikula at intel.com>
+ /mt/dim checkpatch 4e4d7873ac763aa0bd9207ea9ec2b89bb52a6fe1 drm-intel
ecc1d900089b drm/i915: remove IS_LP()
-:30: WARNING:LONG_LINE: line length of 109 exceeds 100 columns
#30: FILE: drivers/gpu/drm/i915/display/intel_display_device.h:153:
+#define HAS_SAGV(i915)			(DISPLAY_VER(i915) >= 9 && !IS_BROXTON(i915) && !IS_GEMINILAKE(i915))

-:30: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects?
#30: FILE: drivers/gpu/drm/i915/display/intel_display_device.h:153:
+#define HAS_SAGV(i915)			(DISPLAY_VER(i915) >= 9 && !IS_BROXTON(i915) && !IS_GEMINILAKE(i915))

-:79: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects?
#79: FILE: drivers/gpu/drm/i915/i915_drv.h:615:
+#define IS_GEN9_LP(i915)	(IS_BROXTON(i915) || IS_GEMINILAKE(i915))

-:80: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects?
#80: FILE: drivers/gpu/drm/i915/i915_drv.h:616:
+#define IS_GEN9_BC(i915)	(GRAPHICS_VER(i915) == 9 && !IS_GEN9_LP(i915))

total: 0 errors, 1 warnings, 3 checks, 78 lines checked
4b82b149fa84 drm/i915/soc: stop using IS_GEN9_LP() and IS_GEN9_BC()




More information about the Intel-xe mailing list