[PATCH 07/12] drm/i915: Use drm_printer more extensively in intel_crtc_state_dump()
Rodrigo Vivi
rodrigo.vivi at intel.com
Thu Feb 22 21:57:21 UTC 2024
On Thu, Feb 15, 2024 at 06:40:50PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> Convert all the direct drm_dbg_kms() stuff in intel_crtc_state_dump()
> over to drm_printf() since we now have the drm_printer around.
looking all this I ask myself if it is really a good idea
to move from the debug helpers to the printf... we need
to keep coming back to where the printer was defined to
know what level we are printing...
well, since the printer is already there, well, let's
use
Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
but with some bad gut feeling about the whole change...
>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
> .../drm/i915/display/intel_crtc_state_dump.c | 185 ++++++++----------
> 1 file changed, 87 insertions(+), 98 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c b/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> index 59d2b3d39951..b34fb0e6e46d 100644
> --- a/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> +++ b/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> @@ -211,43 +211,40 @@ void intel_crtc_state_dump(const struct intel_crtc_state *pipe_config,
>
> p = drm_dbg_printer(&i915->drm, DRM_UT_KMS, NULL);
>
> - drm_dbg_kms(&i915->drm, "[CRTC:%d:%s] enable: %s [%s]\n",
> - crtc->base.base.id, crtc->base.name,
> - str_yes_no(pipe_config->hw.enable), context);
> + drm_printf(&p, "[CRTC:%d:%s] enable: %s [%s]\n",
> + crtc->base.base.id, crtc->base.name,
> + str_yes_no(pipe_config->hw.enable), context);
>
> if (!pipe_config->hw.enable)
> goto dump_planes;
>
> snprintf_output_types(buf, sizeof(buf), pipe_config->output_types);
> - drm_dbg_kms(&i915->drm,
> - "active: %s, output_types: %s (0x%x), output format: %s, sink format: %s\n",
> - str_yes_no(pipe_config->hw.active),
> - buf, pipe_config->output_types,
> - intel_output_format_name(pipe_config->output_format),
> - intel_output_format_name(pipe_config->sink_format));
> + drm_printf(&p, "active: %s, output_types: %s (0x%x), output format: %s, sink format: %s\n",
> + str_yes_no(pipe_config->hw.active),
> + buf, pipe_config->output_types,
> + intel_output_format_name(pipe_config->output_format),
> + intel_output_format_name(pipe_config->sink_format));
>
> - drm_dbg_kms(&i915->drm,
> - "cpu_transcoder: %s, pipe bpp: %i, dithering: %i\n",
> - transcoder_name(pipe_config->cpu_transcoder),
> - pipe_config->pipe_bpp, pipe_config->dither);
> + drm_printf(&p, "cpu_transcoder: %s, pipe bpp: %i, dithering: %i\n",
> + transcoder_name(pipe_config->cpu_transcoder),
> + pipe_config->pipe_bpp, pipe_config->dither);
>
> - drm_dbg_kms(&i915->drm, "MST master transcoder: %s\n",
> - transcoder_name(pipe_config->mst_master_transcoder));
> + drm_printf(&p, "MST master transcoder: %s\n",
> + transcoder_name(pipe_config->mst_master_transcoder));
>
> - drm_dbg_kms(&i915->drm,
> - "port sync: master transcoder: %s, slave transcoder bitmask = 0x%x\n",
> - transcoder_name(pipe_config->master_transcoder),
> - pipe_config->sync_mode_slaves_mask);
> + drm_printf(&p, "port sync: master transcoder: %s, slave transcoder bitmask = 0x%x\n",
> + transcoder_name(pipe_config->master_transcoder),
> + pipe_config->sync_mode_slaves_mask);
>
> - drm_dbg_kms(&i915->drm, "bigjoiner: %s, pipes: 0x%x\n",
> - intel_crtc_is_bigjoiner_slave(pipe_config) ? "slave" :
> - intel_crtc_is_bigjoiner_master(pipe_config) ? "master" : "no",
> - pipe_config->bigjoiner_pipes);
> + drm_printf(&p, "bigjoiner: %s, pipes: 0x%x\n",
> + intel_crtc_is_bigjoiner_slave(pipe_config) ? "slave" :
> + intel_crtc_is_bigjoiner_master(pipe_config) ? "master" : "no",
> + pipe_config->bigjoiner_pipes);
>
> - drm_dbg_kms(&i915->drm, "splitter: %s, link count %d, overlap %d\n",
> - str_enabled_disabled(pipe_config->splitter.enable),
> - pipe_config->splitter.link_count,
> - pipe_config->splitter.pixel_overlap);
> + drm_printf(&p, "splitter: %s, link count %d, overlap %d\n",
> + str_enabled_disabled(pipe_config->splitter.enable),
> + pipe_config->splitter.link_count,
> + pipe_config->splitter.pixel_overlap);
>
> if (pipe_config->has_pch_encoder)
> intel_dump_m_n_config(pipe_config, "fdi",
> @@ -261,32 +258,30 @@ void intel_crtc_state_dump(const struct intel_crtc_state *pipe_config,
> intel_dump_m_n_config(pipe_config, "dp m2_n2",
> pipe_config->lane_count,
> &pipe_config->dp_m2_n2);
> - drm_dbg_kms(&i915->drm, "fec: %s, enhanced framing: %s\n",
> - str_enabled_disabled(pipe_config->fec_enable),
> - str_enabled_disabled(pipe_config->enhanced_framing));
> + drm_printf(&p, "fec: %s, enhanced framing: %s\n",
> + str_enabled_disabled(pipe_config->fec_enable),
> + str_enabled_disabled(pipe_config->enhanced_framing));
>
> - drm_dbg_kms(&i915->drm, "sdp split: %s\n",
> - str_enabled_disabled(pipe_config->sdp_split_enable));
> + drm_printf(&p, "sdp split: %s\n",
> + str_enabled_disabled(pipe_config->sdp_split_enable));
>
> - drm_dbg_kms(&i915->drm, "psr: %s, psr2: %s, panel replay: %s, selective fetch: %s\n",
> - str_enabled_disabled(pipe_config->has_psr),
> - str_enabled_disabled(pipe_config->has_psr2),
> - str_enabled_disabled(pipe_config->has_panel_replay),
> - str_enabled_disabled(pipe_config->enable_psr2_sel_fetch));
> + drm_printf(&p, "psr: %s, psr2: %s, panel replay: %s, selective fetch: %s\n",
> + str_enabled_disabled(pipe_config->has_psr),
> + str_enabled_disabled(pipe_config->has_psr2),
> + str_enabled_disabled(pipe_config->has_panel_replay),
> + str_enabled_disabled(pipe_config->enable_psr2_sel_fetch));
> }
>
> - drm_dbg_kms(&i915->drm, "framestart delay: %d, MSA timing delay: %d\n",
> - pipe_config->framestart_delay, pipe_config->msa_timing_delay);
> + drm_printf(&p, "framestart delay: %d, MSA timing delay: %d\n",
> + pipe_config->framestart_delay, pipe_config->msa_timing_delay);
>
> - drm_dbg_kms(&i915->drm,
> - "audio: %i, infoframes: %i, infoframes enabled: 0x%x\n",
> - pipe_config->has_audio, pipe_config->has_infoframe,
> - pipe_config->infoframes.enable);
> + drm_printf(&p, "audio: %i, infoframes: %i, infoframes enabled: 0x%x\n",
> + pipe_config->has_audio, pipe_config->has_infoframe,
> + pipe_config->infoframes.enable);
>
> if (pipe_config->infoframes.enable &
> intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GENERAL_CONTROL))
> - drm_dbg_kms(&i915->drm, "GCP: 0x%x\n",
> - pipe_config->infoframes.gcp);
> + drm_printf(&p, "GCP: 0x%x\n", pipe_config->infoframes.gcp);
> if (pipe_config->infoframes.enable &
> intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_AVI))
> intel_dump_infoframe(i915, &pipe_config->infoframes.avi);
> @@ -310,75 +305,69 @@ void intel_crtc_state_dump(const struct intel_crtc_state *pipe_config,
> intel_dump_buffer(i915, "ELD: ", pipe_config->eld,
> drm_eld_size(pipe_config->eld));
>
> - drm_dbg_kms(&i915->drm, "vrr: %s, vmin: %d, vmax: %d, pipeline full: %d, guardband: %d flipline: %d, vmin vblank: %d, vmax vblank: %d\n",
> - str_yes_no(pipe_config->vrr.enable),
> - pipe_config->vrr.vmin, pipe_config->vrr.vmax,
> - pipe_config->vrr.pipeline_full, pipe_config->vrr.guardband,
> - pipe_config->vrr.flipline,
> - intel_vrr_vmin_vblank_start(pipe_config),
> - intel_vrr_vmax_vblank_start(pipe_config));
> + drm_printf(&p, "vrr: %s, vmin: %d, vmax: %d, pipeline full: %d, guardband: %d flipline: %d, vmin vblank: %d, vmax vblank: %d\n",
> + str_yes_no(pipe_config->vrr.enable),
> + pipe_config->vrr.vmin, pipe_config->vrr.vmax,
> + pipe_config->vrr.pipeline_full, pipe_config->vrr.guardband,
> + pipe_config->vrr.flipline,
> + intel_vrr_vmin_vblank_start(pipe_config),
> + intel_vrr_vmax_vblank_start(pipe_config));
>
> - drm_dbg_kms(&i915->drm, "requested mode: " DRM_MODE_FMT "\n",
> - DRM_MODE_ARG(&pipe_config->hw.mode));
> - drm_dbg_kms(&i915->drm, "adjusted mode: " DRM_MODE_FMT "\n",
> - DRM_MODE_ARG(&pipe_config->hw.adjusted_mode));
> + drm_printf(&p, "requested mode: " DRM_MODE_FMT "\n",
> + DRM_MODE_ARG(&pipe_config->hw.mode));
> + drm_printf(&p, "adjusted mode: " DRM_MODE_FMT "\n",
> + DRM_MODE_ARG(&pipe_config->hw.adjusted_mode));
> intel_dump_crtc_timings(i915, &pipe_config->hw.adjusted_mode);
> - drm_dbg_kms(&i915->drm, "pipe mode: " DRM_MODE_FMT "\n",
> - DRM_MODE_ARG(&pipe_config->hw.pipe_mode));
> + drm_printf(&p, "pipe mode: " DRM_MODE_FMT "\n",
> + DRM_MODE_ARG(&pipe_config->hw.pipe_mode));
> intel_dump_crtc_timings(i915, &pipe_config->hw.pipe_mode);
> - drm_dbg_kms(&i915->drm,
> - "port clock: %d, pipe src: " DRM_RECT_FMT ", pixel rate %d\n",
> - pipe_config->port_clock, DRM_RECT_ARG(&pipe_config->pipe_src),
> - pipe_config->pixel_rate);
> + drm_printf(&p, "port clock: %d, pipe src: " DRM_RECT_FMT ", pixel rate %d\n",
> + pipe_config->port_clock, DRM_RECT_ARG(&pipe_config->pipe_src),
> + pipe_config->pixel_rate);
>
> - drm_dbg_kms(&i915->drm, "linetime: %d, ips linetime: %d\n",
> - pipe_config->linetime, pipe_config->ips_linetime);
> + drm_printf(&p, "linetime: %d, ips linetime: %d\n",
> + pipe_config->linetime, pipe_config->ips_linetime);
>
> if (DISPLAY_VER(i915) >= 9)
> - drm_dbg_kms(&i915->drm,
> - "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d, scaling_filter: %d\n",
> - crtc->num_scalers,
> - pipe_config->scaler_state.scaler_users,
> - pipe_config->scaler_state.scaler_id,
> - pipe_config->hw.scaling_filter);
> + drm_printf(&p, "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d, scaling_filter: %d\n",
> + crtc->num_scalers,
> + pipe_config->scaler_state.scaler_users,
> + pipe_config->scaler_state.scaler_id,
> + pipe_config->hw.scaling_filter);
>
> if (HAS_GMCH(i915))
> - drm_dbg_kms(&i915->drm,
> - "gmch pfit: control: 0x%08x, ratios: 0x%08x, lvds border: 0x%08x\n",
> - pipe_config->gmch_pfit.control,
> - pipe_config->gmch_pfit.pgm_ratios,
> - pipe_config->gmch_pfit.lvds_border_bits);
> + drm_printf(&p, "gmch pfit: control: 0x%08x, ratios: 0x%08x, lvds border: 0x%08x\n",
> + pipe_config->gmch_pfit.control,
> + pipe_config->gmch_pfit.pgm_ratios,
> + pipe_config->gmch_pfit.lvds_border_bits);
> else
> - drm_dbg_kms(&i915->drm,
> - "pch pfit: " DRM_RECT_FMT ", %s, force thru: %s\n",
> - DRM_RECT_ARG(&pipe_config->pch_pfit.dst),
> - str_enabled_disabled(pipe_config->pch_pfit.enabled),
> - str_yes_no(pipe_config->pch_pfit.force_thru));
> + drm_printf(&p, "pch pfit: " DRM_RECT_FMT ", %s, force thru: %s\n",
> + DRM_RECT_ARG(&pipe_config->pch_pfit.dst),
> + str_enabled_disabled(pipe_config->pch_pfit.enabled),
> + str_yes_no(pipe_config->pch_pfit.force_thru));
>
> - drm_dbg_kms(&i915->drm, "ips: %i, double wide: %i, drrs: %i\n",
> - pipe_config->ips_enabled, pipe_config->double_wide,
> - pipe_config->has_drrs);
> + drm_printf(&p, "ips: %i, double wide: %i, drrs: %i\n",
> + pipe_config->ips_enabled, pipe_config->double_wide,
> + pipe_config->has_drrs);
>
> intel_dpll_dump_hw_state(i915, &p, &pipe_config->dpll_hw_state);
>
> if (IS_CHERRYVIEW(i915))
> - drm_dbg_kms(&i915->drm,
> - "cgm_mode: 0x%x gamma_mode: 0x%x gamma_enable: %d csc_enable: %d\n",
> - pipe_config->cgm_mode, pipe_config->gamma_mode,
> - pipe_config->gamma_enable, pipe_config->csc_enable);
> + drm_printf(&p, "cgm_mode: 0x%x gamma_mode: 0x%x gamma_enable: %d csc_enable: %d\n",
> + pipe_config->cgm_mode, pipe_config->gamma_mode,
> + pipe_config->gamma_enable, pipe_config->csc_enable);
> else
> - drm_dbg_kms(&i915->drm,
> - "csc_mode: 0x%x gamma_mode: 0x%x gamma_enable: %d csc_enable: %d\n",
> - pipe_config->csc_mode, pipe_config->gamma_mode,
> - pipe_config->gamma_enable, pipe_config->csc_enable);
> + drm_printf(&p, "csc_mode: 0x%x gamma_mode: 0x%x gamma_enable: %d csc_enable: %d\n",
> + pipe_config->csc_mode, pipe_config->gamma_mode,
> + pipe_config->gamma_enable, pipe_config->csc_enable);
>
> - drm_dbg_kms(&i915->drm, "pre csc lut: %s%d entries, post csc lut: %d entries\n",
> - pipe_config->pre_csc_lut && pipe_config->pre_csc_lut ==
> - i915->display.color.glk_linear_degamma_lut ? "(linear) " : "",
> - pipe_config->pre_csc_lut ?
> - drm_color_lut_size(pipe_config->pre_csc_lut) : 0,
> - pipe_config->post_csc_lut ?
> - drm_color_lut_size(pipe_config->post_csc_lut) : 0);
> + drm_printf(&p, "pre csc lut: %s%d entries, post csc lut: %d entries\n",
> + pipe_config->pre_csc_lut && pipe_config->pre_csc_lut ==
> + i915->display.color.glk_linear_degamma_lut ? "(linear) " : "",
> + pipe_config->pre_csc_lut ?
> + drm_color_lut_size(pipe_config->pre_csc_lut) : 0,
> + pipe_config->post_csc_lut ?
> + drm_color_lut_size(pipe_config->post_csc_lut) : 0);
>
> if (DISPLAY_VER(i915) >= 11)
> ilk_dump_csc(i915, "output csc", &pipe_config->output_csc);
> --
> 2.43.0
>
More information about the Intel-gfx
mailing list