drm/i915: annotate maybe unused but set intel_crtc_state variables
authorJani Nikula <jani.nikula@intel.com>
Fri, 26 May 2023 16:38:06 +0000 (19:38 +0300)
committerJani Nikula <jani.nikula@intel.com>
Wed, 7 Jun 2023 10:25:48 +0000 (13:25 +0300)
Prepare for re-enabling -Wunused-but-set-variable.

for_each_new_intel_crtc_in_state() requires passing in a struct
intel_crtc_state pointer, which it uses, but in a few places this leads
to warning about unused but set variables. Annotate them with
__maybe_unused.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Jouni Högander <jouni.hogander@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/eb041f426bc3d76ef7a0ea906f99367cbf439b1a.1685119007.git.jani.nikula@intel.com
drivers/gpu/drm/i915/display/intel_display.c
drivers/gpu/drm/i915/display/skl_watermark.c

index b847c1c753703f0b1b24a13a552691552e8424a0..34447d96fe5703748189125241a5a0cdd2d3145c 100644 (file)
@@ -5715,7 +5715,7 @@ static int intel_atomic_check_planes(struct intel_atomic_state *state)
 
 static int intel_atomic_check_crtcs(struct intel_atomic_state *state)
 {
-       struct intel_crtc_state *crtc_state;
+       struct intel_crtc_state __maybe_unused *crtc_state;
        struct intel_crtc *crtc;
        int i;
 
index d1245c847f1cb35710f5a5d2fb291779d369787c..063929a42a42f543decb97f6385070eb79f06365 100644 (file)
@@ -2900,7 +2900,7 @@ static int
 skl_compute_wm(struct intel_atomic_state *state)
 {
        struct intel_crtc *crtc;
-       struct intel_crtc_state *new_crtc_state;
+       struct intel_crtc_state __maybe_unused *new_crtc_state;
        int ret, i;
 
        for_each_new_intel_crtc_in_state(state, crtc, new_crtc_state, i) {