[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Fix bigjoiner state readout

[Patchwork]

== Series Details ==

Series: drm/i915: Fix bigjoiner state readout
URL   : https://patchwork.freedesktop.org/series/94609/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
c2571575849c Revert "drm/i915/display: Disable audio, DRRS and PSR before planes"
36568f62f4b3 drm/i915: Disable all planes before modesetting any pipes
9be6c55e57d1 drm/i915: Extract intel_dp_use_bigjoiner()
a3fc21d5d6df drm/i915: Flatten hsw_crtc_compute_clock()
9f3183ee9429 drm/i915: s/pipe/transcoder/ when dealing with PIPECONF/TRANSCONF
6e8de16f0343 drm/i915: Introduce with_intel_display_power_if_enabled()
-:21: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects?
#21: FILE: drivers/gpu/drm/i915/display/intel_display_power.h:413:
+#define with_intel_display_power_if_enabled(i915, domain, wf) \
+	for ((wf) = intel_display_power_get_if_enabled((i915), (domain)); (wf); \
+	     intel_display_power_put_async((i915), (domain), (wf)), (wf) = 0)

-:21: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'domain' - possible side-effects?
#21: FILE: drivers/gpu/drm/i915/display/intel_display_power.h:413:
+#define with_intel_display_power_if_enabled(i915, domain, wf) \
+	for ((wf) = intel_display_power_get_if_enabled((i915), (domain)); (wf); \
+	     intel_display_power_put_async((i915), (domain), (wf)), (wf) = 0)

-:21: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects?
#21: FILE: drivers/gpu/drm/i915/display/intel_display_power.h:413:
+#define with_intel_display_power_if_enabled(i915, domain, wf) \
+	for ((wf) = intel_display_power_get_if_enabled((i915), (domain)); (wf); \
+	     intel_display_power_put_async((i915), (domain), (wf)), (wf) = 0)

total: 0 errors, 0 warnings, 3 checks, 10 lines checked
5adf1c1b4f21 drm/i915: Adjust intel_dsc_power_domain() calling convention
-:249: WARNING:LONG_LINE: line length of 103 exceeds 100 columns
#249: FILE: drivers/gpu/drm/i915/display/intel_vdsc.c:1141:
+		intel_de_write(dev_priv, dss_ctl1_reg(crtc, crtc_state->cpu_transcoder), dss_ctl1_val);

total: 0 errors, 1 warnings, 0 checks, 291 lines checked
d395994fdc73 drm/i915: Extract hsw_panel_transcoders()
f72fb22ecd92 drm/i915: Pimp HSW+ transcoder state readout
-:132: CHECK:SPACING: No space is necessary after a cast
#132: FILE: drivers/gpu/drm/i915/display/intel_display.c:5656:
+	cpu_transcoder = (enum transcoder) crtc->pipe;

total: 0 errors, 0 warnings, 1 checks, 178 lines checked
228d612ee392 drm/i915: Configure TRANSCONF just the once with bigjoiner
414928c8adbe drm/i915: Introduce intel_master_crtc()
4ac52967591b drm/i915: Simplify intel_crtc_copy_uapi_to_hw_state_nomodeset()
c2e6e3a6e446 drm/i915: Split PPS write from DSC enable
d1e188d0f92d drm/i915: Perform correct cpu_transcoder readout for bigjoiner
-:52: CHECK:SPACING: No space is necessary after a cast
#52: FILE: drivers/gpu/drm/i915/display/intel_display.c:5622:
+		power_domain = intel_dsc_power_domain(crtc, (enum transcoder) pipe);

-:101: CHECK:SPACING: No space is necessary after a cast
#101: FILE: drivers/gpu/drm/i915/display/intel_display.c:5725:
+		cpu_transcoder = (enum transcoder) crtc->pipe - 1;

total: 0 errors, 0 warnings, 2 checks, 88 lines checked
18329b38477a drm/i915: Reduce bigjoiner special casing
87e0b2b138a2 drm/i915: Nuke PIPE_CONFIG_QUIRK_BIGJOINER_SLAVE