== Series Details == Series: series starting with [1/7] drm/i915: Drop one more usless master_transcoder assignment URL : https://patchwork.freedesktop.org/series/82848/ State : warning == Summary == $ dim checkpatch origin/drm-tip 5e9c0a3f80bf drm/i915: Drop one more usless master_transcoder assignment 40e3f078c22a 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:330: +#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:330: +#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:330: +#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 ba8826254488 drm/i915: Adjust intel_dsc_power_domain() calling convention -:279: WARNING:LONG_LINE: line length of 104 exceeds 100 columns #279: FILE: drivers/gpu/drm/i915/display/intel_vdsc.c:1133: + intel_dsc_power_domain(crtc, old_crtc_state->cpu_transcoder)); total: 0 errors, 1 warnings, 0 checks, 245 lines checked 23aaa8770260 drm/i915: Extract hsw_panel_transcoders() 0de1b95e83f6 drm/i915: Pimp HSW+ transcoder state readout -:135: CHECK:SPACING: No space is necessary after a cast #135: FILE: drivers/gpu/drm/i915/display/intel_display.c:11111: + cpu_transcoder = (enum transcoder) crtc->pipe; total: 0 errors, 0 warnings, 1 checks, 187 lines checked d1ffd8987bbb drm/i915: cpu_transcoder readout for bigjoiner -:51: CHECK:SPACING: No space is necessary after a cast #51: FILE: drivers/gpu/drm/i915/display/intel_display.c:11073: + power_domain = intel_dsc_power_domain(crtc, (enum transcoder) pipe); -:87: CHECK:SPACING: No space is necessary after a cast #87: FILE: drivers/gpu/drm/i915/display/intel_display.c:11163: + cpu_transcoder = (enum transcoder) crtc->pipe - 1; total: 0 errors, 0 warnings, 2 checks, 75 lines checked d1051eed84f3 drm/i915: FIXMEs for bigjoiner readout -:10: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one -:38: WARNING:IF_0: Consider removing the code enclosed by this #if 0 and its #endif #38: FILE: drivers/gpu/drm/i915/display/intel_display.c:5859: +#if 0 total: 0 errors, 2 warnings, 0 checks, 45 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx