Message ID | f154a5c58d0a51286696b3d3b1407409067458ee.1655297182.git.jani.nikula@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | drm/i915/display: split out verifation, compare and dump from intel_display.c | expand |
On Wed, Jun 15, 2022 at 03:48:01PM +0300, Jani Nikula wrote: > Declutter intel_display.c by splitting out pipe config dumping to a > separate file. > > Signed-off-by: Jani Nikula <jani.nikula@intel.com> > --- > drivers/gpu/drm/i915/Makefile | 1 + > drivers/gpu/drm/i915/display/intel_display.c | 311 +---------------- > drivers/gpu/drm/i915/display/intel_display.h | 3 - > .../drm/i915/display/intel_modeset_verify.c | 5 +- > .../drm/i915/display/intel_pipe_config_dump.c | 314 ++++++++++++++++++ > .../drm/i915/display/intel_pipe_config_dump.h | 16 + > 6 files changed, 338 insertions(+), 312 deletions(-) > create mode 100644 drivers/gpu/drm/i915/display/intel_pipe_config_dump.c > create mode 100644 drivers/gpu/drm/i915/display/intel_pipe_config_dump.h > > diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile > index 8b4e5c59ee70..d69881506dc7 100644 > --- a/drivers/gpu/drm/i915/Makefile > +++ b/drivers/gpu/drm/i915/Makefile > @@ -247,6 +247,7 @@ i915-y += \ > display/intel_pch_display.o \ > display/intel_pch_refclk.o \ > display/intel_pipe_config_compare.o \ > + display/intel_pipe_config_dump.o \ I would like to get rid of the pipe_config naming here. Just foo_state_dump() I guess would be good. That would sort of match the core state_dump() midlayer, which we may want to a) demidlayer and b) start to use at some point. > display/intel_plane_initial.o \ > display/intel_psr.o \ > display/intel_quirks.o \ > diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c > index 093ba6bde105..8741ae5e6108 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.c > +++ b/drivers/gpu/drm/i915/display/intel_display.c > @@ -106,6 +106,7 @@ > #include "intel_pch_refclk.h" > #include "intel_pcode.h" > #include "intel_pipe_config_compare.h" > +#include "intel_pipe_config_dump.h" > #include "intel_pipe_crc.h" > #include "intel_plane_initial.h" > #include "intel_pm.h" > @@ -5059,310 +5060,6 @@ compute_baseline_pipe_bpp(struct intel_atomic_state *state, > return 0; > } > > -static void intel_dump_crtc_timings(struct drm_i915_private *i915, > - const struct drm_display_mode *mode) > -{ > - drm_dbg_kms(&i915->drm, "crtc timings: %d %d %d %d %d %d %d %d %d, " > - "type: 0x%x flags: 0x%x\n", > - mode->crtc_clock, > - mode->crtc_hdisplay, mode->crtc_hsync_start, > - mode->crtc_hsync_end, mode->crtc_htotal, > - mode->crtc_vdisplay, mode->crtc_vsync_start, > - mode->crtc_vsync_end, mode->crtc_vtotal, > - mode->type, mode->flags); > -} > - > -static void > -intel_dump_m_n_config(const struct intel_crtc_state *pipe_config, > - const char *id, unsigned int lane_count, > - const struct intel_link_m_n *m_n) > -{ > - struct drm_i915_private *i915 = to_i915(pipe_config->uapi.crtc->dev); > - > - drm_dbg_kms(&i915->drm, > - "%s: lanes: %i; data_m: %u, data_n: %u, link_m: %u, link_n: %u, tu: %u\n", > - id, lane_count, > - m_n->data_m, m_n->data_n, > - m_n->link_m, m_n->link_n, m_n->tu); > -} > - > -static void > -intel_dump_infoframe(struct drm_i915_private *dev_priv, > - const union hdmi_infoframe *frame) > -{ > - if (!drm_debug_enabled(DRM_UT_KMS)) > - return; > - > - hdmi_infoframe_log(KERN_DEBUG, dev_priv->drm.dev, frame); > -} > - > -static void > -intel_dump_dp_vsc_sdp(struct drm_i915_private *dev_priv, > - const struct drm_dp_vsc_sdp *vsc) > -{ > - if (!drm_debug_enabled(DRM_UT_KMS)) > - return; > - > - drm_dp_vsc_sdp_log(KERN_DEBUG, dev_priv->drm.dev, vsc); > -} > - > -#define OUTPUT_TYPE(x) [INTEL_OUTPUT_ ## x] = #x > - > -static const char * const output_type_str[] = { > - OUTPUT_TYPE(UNUSED), > - OUTPUT_TYPE(ANALOG), > - OUTPUT_TYPE(DVO), > - OUTPUT_TYPE(SDVO), > - OUTPUT_TYPE(LVDS), > - OUTPUT_TYPE(TVOUT), > - OUTPUT_TYPE(HDMI), > - OUTPUT_TYPE(DP), > - OUTPUT_TYPE(EDP), > - OUTPUT_TYPE(DSI), > - OUTPUT_TYPE(DDI), > - OUTPUT_TYPE(DP_MST), > -}; > - > -#undef OUTPUT_TYPE > - > -static void snprintf_output_types(char *buf, size_t len, > - unsigned int output_types) > -{ > - char *str = buf; > - int i; > - > - str[0] = '\0'; > - > - for (i = 0; i < ARRAY_SIZE(output_type_str); i++) { > - int r; > - > - if ((output_types & BIT(i)) == 0) > - continue; > - > - r = snprintf(str, len, "%s%s", > - str != buf ? "," : "", output_type_str[i]); > - if (r >= len) > - break; > - str += r; > - len -= r; > - > - output_types &= ~BIT(i); > - } > - > - WARN_ON_ONCE(output_types != 0); > -} > - > -static const char * const output_format_str[] = { > - [INTEL_OUTPUT_FORMAT_RGB] = "RGB", > - [INTEL_OUTPUT_FORMAT_YCBCR420] = "YCBCR4:2:0", > - [INTEL_OUTPUT_FORMAT_YCBCR444] = "YCBCR4:4:4", > -}; > - > -static const char *output_formats(enum intel_output_format format) > -{ > - if (format >= ARRAY_SIZE(output_format_str)) > - return "invalid"; > - return output_format_str[format]; > -} > - > -static void intel_dump_plane_state(const struct intel_plane_state *plane_state) > -{ > - struct intel_plane *plane = to_intel_plane(plane_state->uapi.plane); > - struct drm_i915_private *i915 = to_i915(plane->base.dev); > - const struct drm_framebuffer *fb = plane_state->hw.fb; > - > - if (!fb) { > - drm_dbg_kms(&i915->drm, > - "[PLANE:%d:%s] fb: [NOFB], visible: %s\n", > - plane->base.base.id, plane->base.name, > - str_yes_no(plane_state->uapi.visible)); > - return; > - } > - > - drm_dbg_kms(&i915->drm, > - "[PLANE:%d:%s] fb: [FB:%d] %ux%u format = %p4cc modifier = 0x%llx, visible: %s\n", > - plane->base.base.id, plane->base.name, > - fb->base.id, fb->width, fb->height, &fb->format->format, > - fb->modifier, str_yes_no(plane_state->uapi.visible)); > - drm_dbg_kms(&i915->drm, "\trotation: 0x%x, scaler: %d\n", > - plane_state->hw.rotation, plane_state->scaler_id); > - if (plane_state->uapi.visible) > - drm_dbg_kms(&i915->drm, > - "\tsrc: " DRM_RECT_FP_FMT " dst: " DRM_RECT_FMT "\n", > - DRM_RECT_FP_ARG(&plane_state->uapi.src), > - DRM_RECT_ARG(&plane_state->uapi.dst)); > -} > - > -void intel_dump_pipe_config(const struct intel_crtc_state *pipe_config, > - struct intel_atomic_state *state, > - const char *context) > -{ > - struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc); > - struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); > - const struct intel_plane_state *plane_state; > - struct intel_plane *plane; > - char buf[64]; > - int i; > - > - drm_dbg_kms(&dev_priv->drm, "[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(&dev_priv->drm, > - "active: %s, output_types: %s (0x%x), output format: %s\n", > - str_yes_no(pipe_config->hw.active), > - buf, pipe_config->output_types, > - output_formats(pipe_config->output_format)); > - > - drm_dbg_kms(&dev_priv->drm, > - "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(&dev_priv->drm, "MST master transcoder: %s\n", > - transcoder_name(pipe_config->mst_master_transcoder)); > - > - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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); > - > - if (pipe_config->has_pch_encoder) > - intel_dump_m_n_config(pipe_config, "fdi", > - pipe_config->fdi_lanes, > - &pipe_config->fdi_m_n); > - > - if (intel_crtc_has_dp_encoder(pipe_config)) { > - intel_dump_m_n_config(pipe_config, "dp m_n", > - pipe_config->lane_count, > - &pipe_config->dp_m_n); > - intel_dump_m_n_config(pipe_config, "dp m2_n2", > - pipe_config->lane_count, > - &pipe_config->dp_m2_n2); > - } > - > - drm_dbg_kms(&dev_priv->drm, "framestart delay: %d, MSA timing delay: %d\n", > - pipe_config->framestart_delay, pipe_config->msa_timing_delay); > - > - drm_dbg_kms(&dev_priv->drm, > - "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(&dev_priv->drm, "GCP: 0x%x\n", > - pipe_config->infoframes.gcp); > - if (pipe_config->infoframes.enable & > - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_AVI)) > - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.avi); > - if (pipe_config->infoframes.enable & > - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_SPD)) > - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.spd); > - if (pipe_config->infoframes.enable & > - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_VENDOR)) > - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.hdmi); > - if (pipe_config->infoframes.enable & > - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_DRM)) > - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); > - if (pipe_config->infoframes.enable & > - intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GAMUT_METADATA)) > - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); > - if (pipe_config->infoframes.enable & > - intel_hdmi_infoframe_enable(DP_SDP_VSC)) > - intel_dump_dp_vsc_sdp(dev_priv, &pipe_config->infoframes.vsc); > - > - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "requested mode: " DRM_MODE_FMT "\n", > - DRM_MODE_ARG(&pipe_config->hw.mode)); > - drm_dbg_kms(&dev_priv->drm, "adjusted mode: " DRM_MODE_FMT "\n", > - DRM_MODE_ARG(&pipe_config->hw.adjusted_mode)); > - intel_dump_crtc_timings(dev_priv, &pipe_config->hw.adjusted_mode); > - drm_dbg_kms(&dev_priv->drm, "pipe mode: " DRM_MODE_FMT "\n", > - DRM_MODE_ARG(&pipe_config->hw.pipe_mode)); > - intel_dump_crtc_timings(dev_priv, &pipe_config->hw.pipe_mode); > - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "linetime: %d, ips linetime: %d\n", > - pipe_config->linetime, pipe_config->ips_linetime); > - > - if (DISPLAY_VER(dev_priv) >= 9) > - drm_dbg_kms(&dev_priv->drm, > - "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d\n", > - crtc->num_scalers, > - pipe_config->scaler_state.scaler_users, > - pipe_config->scaler_state.scaler_id); > - > - if (HAS_GMCH(dev_priv)) > - drm_dbg_kms(&dev_priv->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); > - else > - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "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(dev_priv, &pipe_config->dpll_hw_state); > - > - if (IS_CHERRYVIEW(dev_priv)) > - drm_dbg_kms(&dev_priv->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); > - else > - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "degamma lut: %d entries, gamma lut: %d entries\n", > - pipe_config->hw.degamma_lut ? > - drm_color_lut_size(pipe_config->hw.degamma_lut) : 0, > - pipe_config->hw.gamma_lut ? > - drm_color_lut_size(pipe_config->hw.gamma_lut) : 0); > - > -dump_planes: > - if (!state) > - return; > - > - for_each_new_intel_plane_in_state(state, plane, plane_state, i) { > - if (plane->pipe == crtc->pipe) > - intel_dump_plane_state(plane_state); > - } > -} > - > static bool check_digital_port_conflicts(struct intel_atomic_state *state) > { > struct drm_device *dev = state->base.dev; > @@ -6823,7 +6520,7 @@ static int intel_atomic_check(struct drm_device *dev, > !new_crtc_state->update_pipe) > continue; > > - intel_dump_pipe_config(new_crtc_state, state, > + intel_pipe_config_dump(new_crtc_state, state, > intel_crtc_needs_modeset(new_crtc_state) ? > "[modeset]" : "[fastset]"); > } > @@ -6840,7 +6537,7 @@ static int intel_atomic_check(struct drm_device *dev, > */ > for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state, > new_crtc_state, i) > - intel_dump_pipe_config(new_crtc_state, state, "[failed]"); > + intel_pipe_config_dump(new_crtc_state, state, "[failed]"); > > return ret; > } > @@ -9320,7 +9017,7 @@ intel_modeset_setup_hw_state(struct drm_device *dev, > to_intel_crtc_state(crtc->base.state); > > intel_sanitize_crtc(crtc, ctx); > - intel_dump_pipe_config(crtc_state, NULL, "[setup_hw_state]"); > + intel_pipe_config_dump(crtc_state, NULL, "[setup_hw_state]"); > } > > intel_modeset_update_connector_atomic_state(dev); > diff --git a/drivers/gpu/drm/i915/display/intel_display.h b/drivers/gpu/drm/i915/display/intel_display.h > index e827c84ece56..68477eb82049 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.h > +++ b/drivers/gpu/drm/i915/display/intel_display.h > @@ -560,9 +560,6 @@ bool intel_crtc_is_bigjoiner_master(const struct intel_crtc_state *crtc_state); > u8 intel_crtc_bigjoiner_slave_pipes(const struct intel_crtc_state *crtc_state); > struct intel_crtc *intel_master_crtc(const struct intel_crtc_state *crtc_state); > bool intel_crtc_get_pipe_config(struct intel_crtc_state *crtc_state); > -void intel_dump_pipe_config(const struct intel_crtc_state *pipe_config, > - struct intel_atomic_state *state, > - const char *context); > > void intel_plane_destroy(struct drm_plane *plane); > void intel_enable_transcoder(const struct intel_crtc_state *new_crtc_state); > diff --git a/drivers/gpu/drm/i915/display/intel_modeset_verify.c b/drivers/gpu/drm/i915/display/intel_modeset_verify.c > index fd752c61d854..de034ccef289 100644 > --- a/drivers/gpu/drm/i915/display/intel_modeset_verify.c > +++ b/drivers/gpu/drm/i915/display/intel_modeset_verify.c > @@ -15,6 +15,7 @@ > #include "intel_fdi.h" > #include "intel_modeset_verify.h" > #include "intel_pipe_config_compare.h" > +#include "intel_pipe_config_dump.h" > #include "intel_pm.h" > #include "intel_snps_phy.h" > > @@ -217,8 +218,8 @@ verify_crtc_state(struct intel_crtc *crtc, > if (!intel_pipe_config_compare(new_crtc_state, > pipe_config, false)) { > I915_STATE_WARN(1, "pipe state doesn't match!\n"); > - intel_dump_pipe_config(pipe_config, NULL, "[hw state]"); > - intel_dump_pipe_config(new_crtc_state, NULL, "[sw state]"); > + intel_pipe_config_dump(pipe_config, NULL, "[hw state]"); > + intel_pipe_config_dump(new_crtc_state, NULL, "[sw state]"); > } > } > > diff --git a/drivers/gpu/drm/i915/display/intel_pipe_config_dump.c b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.c > new file mode 100644 > index 000000000000..c290cf009407 > --- /dev/null > +++ b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.c > @@ -0,0 +1,314 @@ > +// SPDX-License-Identifier: MIT > +/* > + * Copyright © 2022 Intel Corporation > + */ > + > +#include "i915_drv.h" > +#include "intel_display_types.h" > +#include "intel_hdmi.h" > +#include "intel_pipe_config_dump.h" > +#include "intel_vrr.h" > + > +static void intel_dump_crtc_timings(struct drm_i915_private *i915, > + const struct drm_display_mode *mode) > +{ > + drm_dbg_kms(&i915->drm, "crtc timings: %d %d %d %d %d %d %d %d %d, " > + "type: 0x%x flags: 0x%x\n", > + mode->crtc_clock, > + mode->crtc_hdisplay, mode->crtc_hsync_start, > + mode->crtc_hsync_end, mode->crtc_htotal, > + mode->crtc_vdisplay, mode->crtc_vsync_start, > + mode->crtc_vsync_end, mode->crtc_vtotal, > + mode->type, mode->flags); > +} > + > +static void > +intel_dump_m_n_config(const struct intel_crtc_state *pipe_config, > + const char *id, unsigned int lane_count, > + const struct intel_link_m_n *m_n) > +{ > + struct drm_i915_private *i915 = to_i915(pipe_config->uapi.crtc->dev); > + > + drm_dbg_kms(&i915->drm, > + "%s: lanes: %i; data_m: %u, data_n: %u, link_m: %u, link_n: %u, tu: %u\n", > + id, lane_count, > + m_n->data_m, m_n->data_n, > + m_n->link_m, m_n->link_n, m_n->tu); > +} > + > +static void > +intel_dump_infoframe(struct drm_i915_private *dev_priv, > + const union hdmi_infoframe *frame) > +{ > + if (!drm_debug_enabled(DRM_UT_KMS)) > + return; > + > + hdmi_infoframe_log(KERN_DEBUG, dev_priv->drm.dev, frame); > +} > + > +static void > +intel_dump_dp_vsc_sdp(struct drm_i915_private *dev_priv, > + const struct drm_dp_vsc_sdp *vsc) > +{ > + if (!drm_debug_enabled(DRM_UT_KMS)) > + return; > + > + drm_dp_vsc_sdp_log(KERN_DEBUG, dev_priv->drm.dev, vsc); > +} > + > +#define OUTPUT_TYPE(x) [INTEL_OUTPUT_ ## x] = #x > + > +static const char * const output_type_str[] = { > + OUTPUT_TYPE(UNUSED), > + OUTPUT_TYPE(ANALOG), > + OUTPUT_TYPE(DVO), > + OUTPUT_TYPE(SDVO), > + OUTPUT_TYPE(LVDS), > + OUTPUT_TYPE(TVOUT), > + OUTPUT_TYPE(HDMI), > + OUTPUT_TYPE(DP), > + OUTPUT_TYPE(EDP), > + OUTPUT_TYPE(DSI), > + OUTPUT_TYPE(DDI), > + OUTPUT_TYPE(DP_MST), > +}; > + > +#undef OUTPUT_TYPE > + > +static void snprintf_output_types(char *buf, size_t len, > + unsigned int output_types) > +{ > + char *str = buf; > + int i; > + > + str[0] = '\0'; > + > + for (i = 0; i < ARRAY_SIZE(output_type_str); i++) { > + int r; > + > + if ((output_types & BIT(i)) == 0) > + continue; > + > + r = snprintf(str, len, "%s%s", > + str != buf ? "," : "", output_type_str[i]); > + if (r >= len) > + break; > + str += r; > + len -= r; > + > + output_types &= ~BIT(i); > + } > + > + WARN_ON_ONCE(output_types != 0); > +} > + > +static const char * const output_format_str[] = { > + [INTEL_OUTPUT_FORMAT_RGB] = "RGB", > + [INTEL_OUTPUT_FORMAT_YCBCR420] = "YCBCR4:2:0", > + [INTEL_OUTPUT_FORMAT_YCBCR444] = "YCBCR4:4:4", > +}; > + > +static const char *output_formats(enum intel_output_format format) > +{ > + if (format >= ARRAY_SIZE(output_format_str)) > + return "invalid"; > + return output_format_str[format]; > +} > + > +static void intel_dump_plane_state(const struct intel_plane_state *plane_state) > +{ > + struct intel_plane *plane = to_intel_plane(plane_state->uapi.plane); > + struct drm_i915_private *i915 = to_i915(plane->base.dev); > + const struct drm_framebuffer *fb = plane_state->hw.fb; > + > + if (!fb) { > + drm_dbg_kms(&i915->drm, > + "[PLANE:%d:%s] fb: [NOFB], visible: %s\n", > + plane->base.base.id, plane->base.name, > + str_yes_no(plane_state->uapi.visible)); > + return; > + } > + > + drm_dbg_kms(&i915->drm, > + "[PLANE:%d:%s] fb: [FB:%d] %ux%u format = %p4cc modifier = 0x%llx, visible: %s\n", > + plane->base.base.id, plane->base.name, > + fb->base.id, fb->width, fb->height, &fb->format->format, > + fb->modifier, str_yes_no(plane_state->uapi.visible)); > + drm_dbg_kms(&i915->drm, "\trotation: 0x%x, scaler: %d\n", > + plane_state->hw.rotation, plane_state->scaler_id); > + if (plane_state->uapi.visible) > + drm_dbg_kms(&i915->drm, > + "\tsrc: " DRM_RECT_FP_FMT " dst: " DRM_RECT_FMT "\n", > + DRM_RECT_FP_ARG(&plane_state->uapi.src), > + DRM_RECT_ARG(&plane_state->uapi.dst)); > +} > + > +void intel_pipe_config_dump(const struct intel_crtc_state *pipe_config, > + struct intel_atomic_state *state, > + const char *context) > +{ > + struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc); > + struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); > + const struct intel_plane_state *plane_state; > + struct intel_plane *plane; > + char buf[64]; > + int i; > + > + drm_dbg_kms(&dev_priv->drm, "[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(&dev_priv->drm, > + "active: %s, output_types: %s (0x%x), output format: %s\n", > + str_yes_no(pipe_config->hw.active), > + buf, pipe_config->output_types, > + output_formats(pipe_config->output_format)); > + > + drm_dbg_kms(&dev_priv->drm, > + "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(&dev_priv->drm, "MST master transcoder: %s\n", > + transcoder_name(pipe_config->mst_master_transcoder)); > + > + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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); > + > + if (pipe_config->has_pch_encoder) > + intel_dump_m_n_config(pipe_config, "fdi", > + pipe_config->fdi_lanes, > + &pipe_config->fdi_m_n); > + > + if (intel_crtc_has_dp_encoder(pipe_config)) { > + intel_dump_m_n_config(pipe_config, "dp m_n", > + pipe_config->lane_count, > + &pipe_config->dp_m_n); > + intel_dump_m_n_config(pipe_config, "dp m2_n2", > + pipe_config->lane_count, > + &pipe_config->dp_m2_n2); > + } > + > + drm_dbg_kms(&dev_priv->drm, "framestart delay: %d, MSA timing delay: %d\n", > + pipe_config->framestart_delay, pipe_config->msa_timing_delay); > + > + drm_dbg_kms(&dev_priv->drm, > + "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(&dev_priv->drm, "GCP: 0x%x\n", > + pipe_config->infoframes.gcp); > + if (pipe_config->infoframes.enable & > + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_AVI)) > + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.avi); > + if (pipe_config->infoframes.enable & > + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_SPD)) > + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.spd); > + if (pipe_config->infoframes.enable & > + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_VENDOR)) > + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.hdmi); > + if (pipe_config->infoframes.enable & > + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_DRM)) > + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); > + if (pipe_config->infoframes.enable & > + intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GAMUT_METADATA)) > + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); > + if (pipe_config->infoframes.enable & > + intel_hdmi_infoframe_enable(DP_SDP_VSC)) > + intel_dump_dp_vsc_sdp(dev_priv, &pipe_config->infoframes.vsc); > + > + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "requested mode: " DRM_MODE_FMT "\n", > + DRM_MODE_ARG(&pipe_config->hw.mode)); > + drm_dbg_kms(&dev_priv->drm, "adjusted mode: " DRM_MODE_FMT "\n", > + DRM_MODE_ARG(&pipe_config->hw.adjusted_mode)); > + intel_dump_crtc_timings(dev_priv, &pipe_config->hw.adjusted_mode); > + drm_dbg_kms(&dev_priv->drm, "pipe mode: " DRM_MODE_FMT "\n", > + DRM_MODE_ARG(&pipe_config->hw.pipe_mode)); > + intel_dump_crtc_timings(dev_priv, &pipe_config->hw.pipe_mode); > + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "linetime: %d, ips linetime: %d\n", > + pipe_config->linetime, pipe_config->ips_linetime); > + > + if (DISPLAY_VER(dev_priv) >= 9) > + drm_dbg_kms(&dev_priv->drm, > + "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d\n", > + crtc->num_scalers, > + pipe_config->scaler_state.scaler_users, > + pipe_config->scaler_state.scaler_id); > + > + if (HAS_GMCH(dev_priv)) > + drm_dbg_kms(&dev_priv->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); > + else > + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "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(dev_priv, &pipe_config->dpll_hw_state); > + > + if (IS_CHERRYVIEW(dev_priv)) > + drm_dbg_kms(&dev_priv->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); > + else > + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "degamma lut: %d entries, gamma lut: %d entries\n", > + pipe_config->hw.degamma_lut ? > + drm_color_lut_size(pipe_config->hw.degamma_lut) : 0, > + pipe_config->hw.gamma_lut ? > + drm_color_lut_size(pipe_config->hw.gamma_lut) : 0); > + > +dump_planes: > + if (!state) > + return; > + > + for_each_new_intel_plane_in_state(state, plane, plane_state, i) { > + if (plane->pipe == crtc->pipe) > + intel_dump_plane_state(plane_state); > + } > +} > diff --git a/drivers/gpu/drm/i915/display/intel_pipe_config_dump.h b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.h > new file mode 100644 > index 000000000000..20bbe11f0527 > --- /dev/null > +++ b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.h > @@ -0,0 +1,16 @@ > +/* SPDX-License-Identifier: MIT */ > +/* > + * Copyright © 2022 Intel Corporation > + */ > + > +#ifndef __INTEL_PIPE_CONFIG_DUMP_H__ > +#define __INTEL_PIPE_CONFIG_DUMP_H__ > + > +struct intel_crtc_state; > +struct intel_atomic_state; > + > +void intel_pipe_config_dump(const struct intel_crtc_state *pipe_config, > + struct intel_atomic_state *state, > + const char *context); > + > +#endif /* __INTEL_PIPE_CONFIG_DUMP_H__ */ > -- > 2.30.2
On Wed, 15 Jun 2022, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote: > On Wed, Jun 15, 2022 at 03:48:01PM +0300, Jani Nikula wrote: >> Declutter intel_display.c by splitting out pipe config dumping to a >> separate file. >> >> Signed-off-by: Jani Nikula <jani.nikula@intel.com> >> --- >> drivers/gpu/drm/i915/Makefile | 1 + >> drivers/gpu/drm/i915/display/intel_display.c | 311 +---------------- >> drivers/gpu/drm/i915/display/intel_display.h | 3 - >> .../drm/i915/display/intel_modeset_verify.c | 5 +- >> .../drm/i915/display/intel_pipe_config_dump.c | 314 ++++++++++++++++++ >> .../drm/i915/display/intel_pipe_config_dump.h | 16 + >> 6 files changed, 338 insertions(+), 312 deletions(-) >> create mode 100644 drivers/gpu/drm/i915/display/intel_pipe_config_dump.c >> create mode 100644 drivers/gpu/drm/i915/display/intel_pipe_config_dump.h >> >> diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile >> index 8b4e5c59ee70..d69881506dc7 100644 >> --- a/drivers/gpu/drm/i915/Makefile >> +++ b/drivers/gpu/drm/i915/Makefile >> @@ -247,6 +247,7 @@ i915-y += \ >> display/intel_pch_display.o \ >> display/intel_pch_refclk.o \ >> display/intel_pipe_config_compare.o \ >> + display/intel_pipe_config_dump.o \ > > I would like to get rid of the pipe_config naming here. Just > foo_state_dump() I guess would be good. That would sort of match > the core state_dump() midlayer, which we may want to a) demidlayer > and b) start to use at some point. I knew the naming would be a point of contention, but I was hoping for a concrete suggestion. Other than foo, that is. ;) BR, Jani. > >> display/intel_plane_initial.o \ >> display/intel_psr.o \ >> display/intel_quirks.o \ >> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c >> index 093ba6bde105..8741ae5e6108 100644 >> --- a/drivers/gpu/drm/i915/display/intel_display.c >> +++ b/drivers/gpu/drm/i915/display/intel_display.c >> @@ -106,6 +106,7 @@ >> #include "intel_pch_refclk.h" >> #include "intel_pcode.h" >> #include "intel_pipe_config_compare.h" >> +#include "intel_pipe_config_dump.h" >> #include "intel_pipe_crc.h" >> #include "intel_plane_initial.h" >> #include "intel_pm.h" >> @@ -5059,310 +5060,6 @@ compute_baseline_pipe_bpp(struct intel_atomic_state *state, >> return 0; >> } >> >> -static void intel_dump_crtc_timings(struct drm_i915_private *i915, >> - const struct drm_display_mode *mode) >> -{ >> - drm_dbg_kms(&i915->drm, "crtc timings: %d %d %d %d %d %d %d %d %d, " >> - "type: 0x%x flags: 0x%x\n", >> - mode->crtc_clock, >> - mode->crtc_hdisplay, mode->crtc_hsync_start, >> - mode->crtc_hsync_end, mode->crtc_htotal, >> - mode->crtc_vdisplay, mode->crtc_vsync_start, >> - mode->crtc_vsync_end, mode->crtc_vtotal, >> - mode->type, mode->flags); >> -} >> - >> -static void >> -intel_dump_m_n_config(const struct intel_crtc_state *pipe_config, >> - const char *id, unsigned int lane_count, >> - const struct intel_link_m_n *m_n) >> -{ >> - struct drm_i915_private *i915 = to_i915(pipe_config->uapi.crtc->dev); >> - >> - drm_dbg_kms(&i915->drm, >> - "%s: lanes: %i; data_m: %u, data_n: %u, link_m: %u, link_n: %u, tu: %u\n", >> - id, lane_count, >> - m_n->data_m, m_n->data_n, >> - m_n->link_m, m_n->link_n, m_n->tu); >> -} >> - >> -static void >> -intel_dump_infoframe(struct drm_i915_private *dev_priv, >> - const union hdmi_infoframe *frame) >> -{ >> - if (!drm_debug_enabled(DRM_UT_KMS)) >> - return; >> - >> - hdmi_infoframe_log(KERN_DEBUG, dev_priv->drm.dev, frame); >> -} >> - >> -static void >> -intel_dump_dp_vsc_sdp(struct drm_i915_private *dev_priv, >> - const struct drm_dp_vsc_sdp *vsc) >> -{ >> - if (!drm_debug_enabled(DRM_UT_KMS)) >> - return; >> - >> - drm_dp_vsc_sdp_log(KERN_DEBUG, dev_priv->drm.dev, vsc); >> -} >> - >> -#define OUTPUT_TYPE(x) [INTEL_OUTPUT_ ## x] = #x >> - >> -static const char * const output_type_str[] = { >> - OUTPUT_TYPE(UNUSED), >> - OUTPUT_TYPE(ANALOG), >> - OUTPUT_TYPE(DVO), >> - OUTPUT_TYPE(SDVO), >> - OUTPUT_TYPE(LVDS), >> - OUTPUT_TYPE(TVOUT), >> - OUTPUT_TYPE(HDMI), >> - OUTPUT_TYPE(DP), >> - OUTPUT_TYPE(EDP), >> - OUTPUT_TYPE(DSI), >> - OUTPUT_TYPE(DDI), >> - OUTPUT_TYPE(DP_MST), >> -}; >> - >> -#undef OUTPUT_TYPE >> - >> -static void snprintf_output_types(char *buf, size_t len, >> - unsigned int output_types) >> -{ >> - char *str = buf; >> - int i; >> - >> - str[0] = '\0'; >> - >> - for (i = 0; i < ARRAY_SIZE(output_type_str); i++) { >> - int r; >> - >> - if ((output_types & BIT(i)) == 0) >> - continue; >> - >> - r = snprintf(str, len, "%s%s", >> - str != buf ? "," : "", output_type_str[i]); >> - if (r >= len) >> - break; >> - str += r; >> - len -= r; >> - >> - output_types &= ~BIT(i); >> - } >> - >> - WARN_ON_ONCE(output_types != 0); >> -} >> - >> -static const char * const output_format_str[] = { >> - [INTEL_OUTPUT_FORMAT_RGB] = "RGB", >> - [INTEL_OUTPUT_FORMAT_YCBCR420] = "YCBCR4:2:0", >> - [INTEL_OUTPUT_FORMAT_YCBCR444] = "YCBCR4:4:4", >> -}; >> - >> -static const char *output_formats(enum intel_output_format format) >> -{ >> - if (format >= ARRAY_SIZE(output_format_str)) >> - return "invalid"; >> - return output_format_str[format]; >> -} >> - >> -static void intel_dump_plane_state(const struct intel_plane_state *plane_state) >> -{ >> - struct intel_plane *plane = to_intel_plane(plane_state->uapi.plane); >> - struct drm_i915_private *i915 = to_i915(plane->base.dev); >> - const struct drm_framebuffer *fb = plane_state->hw.fb; >> - >> - if (!fb) { >> - drm_dbg_kms(&i915->drm, >> - "[PLANE:%d:%s] fb: [NOFB], visible: %s\n", >> - plane->base.base.id, plane->base.name, >> - str_yes_no(plane_state->uapi.visible)); >> - return; >> - } >> - >> - drm_dbg_kms(&i915->drm, >> - "[PLANE:%d:%s] fb: [FB:%d] %ux%u format = %p4cc modifier = 0x%llx, visible: %s\n", >> - plane->base.base.id, plane->base.name, >> - fb->base.id, fb->width, fb->height, &fb->format->format, >> - fb->modifier, str_yes_no(plane_state->uapi.visible)); >> - drm_dbg_kms(&i915->drm, "\trotation: 0x%x, scaler: %d\n", >> - plane_state->hw.rotation, plane_state->scaler_id); >> - if (plane_state->uapi.visible) >> - drm_dbg_kms(&i915->drm, >> - "\tsrc: " DRM_RECT_FP_FMT " dst: " DRM_RECT_FMT "\n", >> - DRM_RECT_FP_ARG(&plane_state->uapi.src), >> - DRM_RECT_ARG(&plane_state->uapi.dst)); >> -} >> - >> -void intel_dump_pipe_config(const struct intel_crtc_state *pipe_config, >> - struct intel_atomic_state *state, >> - const char *context) >> -{ >> - struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc); >> - struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); >> - const struct intel_plane_state *plane_state; >> - struct intel_plane *plane; >> - char buf[64]; >> - int i; >> - >> - drm_dbg_kms(&dev_priv->drm, "[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(&dev_priv->drm, >> - "active: %s, output_types: %s (0x%x), output format: %s\n", >> - str_yes_no(pipe_config->hw.active), >> - buf, pipe_config->output_types, >> - output_formats(pipe_config->output_format)); >> - >> - drm_dbg_kms(&dev_priv->drm, >> - "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(&dev_priv->drm, "MST master transcoder: %s\n", >> - transcoder_name(pipe_config->mst_master_transcoder)); >> - >> - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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); >> - >> - if (pipe_config->has_pch_encoder) >> - intel_dump_m_n_config(pipe_config, "fdi", >> - pipe_config->fdi_lanes, >> - &pipe_config->fdi_m_n); >> - >> - if (intel_crtc_has_dp_encoder(pipe_config)) { >> - intel_dump_m_n_config(pipe_config, "dp m_n", >> - pipe_config->lane_count, >> - &pipe_config->dp_m_n); >> - intel_dump_m_n_config(pipe_config, "dp m2_n2", >> - pipe_config->lane_count, >> - &pipe_config->dp_m2_n2); >> - } >> - >> - drm_dbg_kms(&dev_priv->drm, "framestart delay: %d, MSA timing delay: %d\n", >> - pipe_config->framestart_delay, pipe_config->msa_timing_delay); >> - >> - drm_dbg_kms(&dev_priv->drm, >> - "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(&dev_priv->drm, "GCP: 0x%x\n", >> - pipe_config->infoframes.gcp); >> - if (pipe_config->infoframes.enable & >> - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_AVI)) >> - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.avi); >> - if (pipe_config->infoframes.enable & >> - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_SPD)) >> - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.spd); >> - if (pipe_config->infoframes.enable & >> - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_VENDOR)) >> - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.hdmi); >> - if (pipe_config->infoframes.enable & >> - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_DRM)) >> - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); >> - if (pipe_config->infoframes.enable & >> - intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GAMUT_METADATA)) >> - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); >> - if (pipe_config->infoframes.enable & >> - intel_hdmi_infoframe_enable(DP_SDP_VSC)) >> - intel_dump_dp_vsc_sdp(dev_priv, &pipe_config->infoframes.vsc); >> - >> - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "requested mode: " DRM_MODE_FMT "\n", >> - DRM_MODE_ARG(&pipe_config->hw.mode)); >> - drm_dbg_kms(&dev_priv->drm, "adjusted mode: " DRM_MODE_FMT "\n", >> - DRM_MODE_ARG(&pipe_config->hw.adjusted_mode)); >> - intel_dump_crtc_timings(dev_priv, &pipe_config->hw.adjusted_mode); >> - drm_dbg_kms(&dev_priv->drm, "pipe mode: " DRM_MODE_FMT "\n", >> - DRM_MODE_ARG(&pipe_config->hw.pipe_mode)); >> - intel_dump_crtc_timings(dev_priv, &pipe_config->hw.pipe_mode); >> - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "linetime: %d, ips linetime: %d\n", >> - pipe_config->linetime, pipe_config->ips_linetime); >> - >> - if (DISPLAY_VER(dev_priv) >= 9) >> - drm_dbg_kms(&dev_priv->drm, >> - "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d\n", >> - crtc->num_scalers, >> - pipe_config->scaler_state.scaler_users, >> - pipe_config->scaler_state.scaler_id); >> - >> - if (HAS_GMCH(dev_priv)) >> - drm_dbg_kms(&dev_priv->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); >> - else >> - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "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(dev_priv, &pipe_config->dpll_hw_state); >> - >> - if (IS_CHERRYVIEW(dev_priv)) >> - drm_dbg_kms(&dev_priv->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); >> - else >> - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "degamma lut: %d entries, gamma lut: %d entries\n", >> - pipe_config->hw.degamma_lut ? >> - drm_color_lut_size(pipe_config->hw.degamma_lut) : 0, >> - pipe_config->hw.gamma_lut ? >> - drm_color_lut_size(pipe_config->hw.gamma_lut) : 0); >> - >> -dump_planes: >> - if (!state) >> - return; >> - >> - for_each_new_intel_plane_in_state(state, plane, plane_state, i) { >> - if (plane->pipe == crtc->pipe) >> - intel_dump_plane_state(plane_state); >> - } >> -} >> - >> static bool check_digital_port_conflicts(struct intel_atomic_state *state) >> { >> struct drm_device *dev = state->base.dev; >> @@ -6823,7 +6520,7 @@ static int intel_atomic_check(struct drm_device *dev, >> !new_crtc_state->update_pipe) >> continue; >> >> - intel_dump_pipe_config(new_crtc_state, state, >> + intel_pipe_config_dump(new_crtc_state, state, >> intel_crtc_needs_modeset(new_crtc_state) ? >> "[modeset]" : "[fastset]"); >> } >> @@ -6840,7 +6537,7 @@ static int intel_atomic_check(struct drm_device *dev, >> */ >> for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state, >> new_crtc_state, i) >> - intel_dump_pipe_config(new_crtc_state, state, "[failed]"); >> + intel_pipe_config_dump(new_crtc_state, state, "[failed]"); >> >> return ret; >> } >> @@ -9320,7 +9017,7 @@ intel_modeset_setup_hw_state(struct drm_device *dev, >> to_intel_crtc_state(crtc->base.state); >> >> intel_sanitize_crtc(crtc, ctx); >> - intel_dump_pipe_config(crtc_state, NULL, "[setup_hw_state]"); >> + intel_pipe_config_dump(crtc_state, NULL, "[setup_hw_state]"); >> } >> >> intel_modeset_update_connector_atomic_state(dev); >> diff --git a/drivers/gpu/drm/i915/display/intel_display.h b/drivers/gpu/drm/i915/display/intel_display.h >> index e827c84ece56..68477eb82049 100644 >> --- a/drivers/gpu/drm/i915/display/intel_display.h >> +++ b/drivers/gpu/drm/i915/display/intel_display.h >> @@ -560,9 +560,6 @@ bool intel_crtc_is_bigjoiner_master(const struct intel_crtc_state *crtc_state); >> u8 intel_crtc_bigjoiner_slave_pipes(const struct intel_crtc_state *crtc_state); >> struct intel_crtc *intel_master_crtc(const struct intel_crtc_state *crtc_state); >> bool intel_crtc_get_pipe_config(struct intel_crtc_state *crtc_state); >> -void intel_dump_pipe_config(const struct intel_crtc_state *pipe_config, >> - struct intel_atomic_state *state, >> - const char *context); >> >> void intel_plane_destroy(struct drm_plane *plane); >> void intel_enable_transcoder(const struct intel_crtc_state *new_crtc_state); >> diff --git a/drivers/gpu/drm/i915/display/intel_modeset_verify.c b/drivers/gpu/drm/i915/display/intel_modeset_verify.c >> index fd752c61d854..de034ccef289 100644 >> --- a/drivers/gpu/drm/i915/display/intel_modeset_verify.c >> +++ b/drivers/gpu/drm/i915/display/intel_modeset_verify.c >> @@ -15,6 +15,7 @@ >> #include "intel_fdi.h" >> #include "intel_modeset_verify.h" >> #include "intel_pipe_config_compare.h" >> +#include "intel_pipe_config_dump.h" >> #include "intel_pm.h" >> #include "intel_snps_phy.h" >> >> @@ -217,8 +218,8 @@ verify_crtc_state(struct intel_crtc *crtc, >> if (!intel_pipe_config_compare(new_crtc_state, >> pipe_config, false)) { >> I915_STATE_WARN(1, "pipe state doesn't match!\n"); >> - intel_dump_pipe_config(pipe_config, NULL, "[hw state]"); >> - intel_dump_pipe_config(new_crtc_state, NULL, "[sw state]"); >> + intel_pipe_config_dump(pipe_config, NULL, "[hw state]"); >> + intel_pipe_config_dump(new_crtc_state, NULL, "[sw state]"); >> } >> } >> >> diff --git a/drivers/gpu/drm/i915/display/intel_pipe_config_dump.c b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.c >> new file mode 100644 >> index 000000000000..c290cf009407 >> --- /dev/null >> +++ b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.c >> @@ -0,0 +1,314 @@ >> +// SPDX-License-Identifier: MIT >> +/* >> + * Copyright © 2022 Intel Corporation >> + */ >> + >> +#include "i915_drv.h" >> +#include "intel_display_types.h" >> +#include "intel_hdmi.h" >> +#include "intel_pipe_config_dump.h" >> +#include "intel_vrr.h" >> + >> +static void intel_dump_crtc_timings(struct drm_i915_private *i915, >> + const struct drm_display_mode *mode) >> +{ >> + drm_dbg_kms(&i915->drm, "crtc timings: %d %d %d %d %d %d %d %d %d, " >> + "type: 0x%x flags: 0x%x\n", >> + mode->crtc_clock, >> + mode->crtc_hdisplay, mode->crtc_hsync_start, >> + mode->crtc_hsync_end, mode->crtc_htotal, >> + mode->crtc_vdisplay, mode->crtc_vsync_start, >> + mode->crtc_vsync_end, mode->crtc_vtotal, >> + mode->type, mode->flags); >> +} >> + >> +static void >> +intel_dump_m_n_config(const struct intel_crtc_state *pipe_config, >> + const char *id, unsigned int lane_count, >> + const struct intel_link_m_n *m_n) >> +{ >> + struct drm_i915_private *i915 = to_i915(pipe_config->uapi.crtc->dev); >> + >> + drm_dbg_kms(&i915->drm, >> + "%s: lanes: %i; data_m: %u, data_n: %u, link_m: %u, link_n: %u, tu: %u\n", >> + id, lane_count, >> + m_n->data_m, m_n->data_n, >> + m_n->link_m, m_n->link_n, m_n->tu); >> +} >> + >> +static void >> +intel_dump_infoframe(struct drm_i915_private *dev_priv, >> + const union hdmi_infoframe *frame) >> +{ >> + if (!drm_debug_enabled(DRM_UT_KMS)) >> + return; >> + >> + hdmi_infoframe_log(KERN_DEBUG, dev_priv->drm.dev, frame); >> +} >> + >> +static void >> +intel_dump_dp_vsc_sdp(struct drm_i915_private *dev_priv, >> + const struct drm_dp_vsc_sdp *vsc) >> +{ >> + if (!drm_debug_enabled(DRM_UT_KMS)) >> + return; >> + >> + drm_dp_vsc_sdp_log(KERN_DEBUG, dev_priv->drm.dev, vsc); >> +} >> + >> +#define OUTPUT_TYPE(x) [INTEL_OUTPUT_ ## x] = #x >> + >> +static const char * const output_type_str[] = { >> + OUTPUT_TYPE(UNUSED), >> + OUTPUT_TYPE(ANALOG), >> + OUTPUT_TYPE(DVO), >> + OUTPUT_TYPE(SDVO), >> + OUTPUT_TYPE(LVDS), >> + OUTPUT_TYPE(TVOUT), >> + OUTPUT_TYPE(HDMI), >> + OUTPUT_TYPE(DP), >> + OUTPUT_TYPE(EDP), >> + OUTPUT_TYPE(DSI), >> + OUTPUT_TYPE(DDI), >> + OUTPUT_TYPE(DP_MST), >> +}; >> + >> +#undef OUTPUT_TYPE >> + >> +static void snprintf_output_types(char *buf, size_t len, >> + unsigned int output_types) >> +{ >> + char *str = buf; >> + int i; >> + >> + str[0] = '\0'; >> + >> + for (i = 0; i < ARRAY_SIZE(output_type_str); i++) { >> + int r; >> + >> + if ((output_types & BIT(i)) == 0) >> + continue; >> + >> + r = snprintf(str, len, "%s%s", >> + str != buf ? "," : "", output_type_str[i]); >> + if (r >= len) >> + break; >> + str += r; >> + len -= r; >> + >> + output_types &= ~BIT(i); >> + } >> + >> + WARN_ON_ONCE(output_types != 0); >> +} >> + >> +static const char * const output_format_str[] = { >> + [INTEL_OUTPUT_FORMAT_RGB] = "RGB", >> + [INTEL_OUTPUT_FORMAT_YCBCR420] = "YCBCR4:2:0", >> + [INTEL_OUTPUT_FORMAT_YCBCR444] = "YCBCR4:4:4", >> +}; >> + >> +static const char *output_formats(enum intel_output_format format) >> +{ >> + if (format >= ARRAY_SIZE(output_format_str)) >> + return "invalid"; >> + return output_format_str[format]; >> +} >> + >> +static void intel_dump_plane_state(const struct intel_plane_state *plane_state) >> +{ >> + struct intel_plane *plane = to_intel_plane(plane_state->uapi.plane); >> + struct drm_i915_private *i915 = to_i915(plane->base.dev); >> + const struct drm_framebuffer *fb = plane_state->hw.fb; >> + >> + if (!fb) { >> + drm_dbg_kms(&i915->drm, >> + "[PLANE:%d:%s] fb: [NOFB], visible: %s\n", >> + plane->base.base.id, plane->base.name, >> + str_yes_no(plane_state->uapi.visible)); >> + return; >> + } >> + >> + drm_dbg_kms(&i915->drm, >> + "[PLANE:%d:%s] fb: [FB:%d] %ux%u format = %p4cc modifier = 0x%llx, visible: %s\n", >> + plane->base.base.id, plane->base.name, >> + fb->base.id, fb->width, fb->height, &fb->format->format, >> + fb->modifier, str_yes_no(plane_state->uapi.visible)); >> + drm_dbg_kms(&i915->drm, "\trotation: 0x%x, scaler: %d\n", >> + plane_state->hw.rotation, plane_state->scaler_id); >> + if (plane_state->uapi.visible) >> + drm_dbg_kms(&i915->drm, >> + "\tsrc: " DRM_RECT_FP_FMT " dst: " DRM_RECT_FMT "\n", >> + DRM_RECT_FP_ARG(&plane_state->uapi.src), >> + DRM_RECT_ARG(&plane_state->uapi.dst)); >> +} >> + >> +void intel_pipe_config_dump(const struct intel_crtc_state *pipe_config, >> + struct intel_atomic_state *state, >> + const char *context) >> +{ >> + struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc); >> + struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); >> + const struct intel_plane_state *plane_state; >> + struct intel_plane *plane; >> + char buf[64]; >> + int i; >> + >> + drm_dbg_kms(&dev_priv->drm, "[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(&dev_priv->drm, >> + "active: %s, output_types: %s (0x%x), output format: %s\n", >> + str_yes_no(pipe_config->hw.active), >> + buf, pipe_config->output_types, >> + output_formats(pipe_config->output_format)); >> + >> + drm_dbg_kms(&dev_priv->drm, >> + "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(&dev_priv->drm, "MST master transcoder: %s\n", >> + transcoder_name(pipe_config->mst_master_transcoder)); >> + >> + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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); >> + >> + if (pipe_config->has_pch_encoder) >> + intel_dump_m_n_config(pipe_config, "fdi", >> + pipe_config->fdi_lanes, >> + &pipe_config->fdi_m_n); >> + >> + if (intel_crtc_has_dp_encoder(pipe_config)) { >> + intel_dump_m_n_config(pipe_config, "dp m_n", >> + pipe_config->lane_count, >> + &pipe_config->dp_m_n); >> + intel_dump_m_n_config(pipe_config, "dp m2_n2", >> + pipe_config->lane_count, >> + &pipe_config->dp_m2_n2); >> + } >> + >> + drm_dbg_kms(&dev_priv->drm, "framestart delay: %d, MSA timing delay: %d\n", >> + pipe_config->framestart_delay, pipe_config->msa_timing_delay); >> + >> + drm_dbg_kms(&dev_priv->drm, >> + "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(&dev_priv->drm, "GCP: 0x%x\n", >> + pipe_config->infoframes.gcp); >> + if (pipe_config->infoframes.enable & >> + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_AVI)) >> + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.avi); >> + if (pipe_config->infoframes.enable & >> + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_SPD)) >> + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.spd); >> + if (pipe_config->infoframes.enable & >> + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_VENDOR)) >> + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.hdmi); >> + if (pipe_config->infoframes.enable & >> + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_DRM)) >> + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); >> + if (pipe_config->infoframes.enable & >> + intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GAMUT_METADATA)) >> + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); >> + if (pipe_config->infoframes.enable & >> + intel_hdmi_infoframe_enable(DP_SDP_VSC)) >> + intel_dump_dp_vsc_sdp(dev_priv, &pipe_config->infoframes.vsc); >> + >> + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "requested mode: " DRM_MODE_FMT "\n", >> + DRM_MODE_ARG(&pipe_config->hw.mode)); >> + drm_dbg_kms(&dev_priv->drm, "adjusted mode: " DRM_MODE_FMT "\n", >> + DRM_MODE_ARG(&pipe_config->hw.adjusted_mode)); >> + intel_dump_crtc_timings(dev_priv, &pipe_config->hw.adjusted_mode); >> + drm_dbg_kms(&dev_priv->drm, "pipe mode: " DRM_MODE_FMT "\n", >> + DRM_MODE_ARG(&pipe_config->hw.pipe_mode)); >> + intel_dump_crtc_timings(dev_priv, &pipe_config->hw.pipe_mode); >> + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "linetime: %d, ips linetime: %d\n", >> + pipe_config->linetime, pipe_config->ips_linetime); >> + >> + if (DISPLAY_VER(dev_priv) >= 9) >> + drm_dbg_kms(&dev_priv->drm, >> + "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d\n", >> + crtc->num_scalers, >> + pipe_config->scaler_state.scaler_users, >> + pipe_config->scaler_state.scaler_id); >> + >> + if (HAS_GMCH(dev_priv)) >> + drm_dbg_kms(&dev_priv->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); >> + else >> + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "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(dev_priv, &pipe_config->dpll_hw_state); >> + >> + if (IS_CHERRYVIEW(dev_priv)) >> + drm_dbg_kms(&dev_priv->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); >> + else >> + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "degamma lut: %d entries, gamma lut: %d entries\n", >> + pipe_config->hw.degamma_lut ? >> + drm_color_lut_size(pipe_config->hw.degamma_lut) : 0, >> + pipe_config->hw.gamma_lut ? >> + drm_color_lut_size(pipe_config->hw.gamma_lut) : 0); >> + >> +dump_planes: >> + if (!state) >> + return; >> + >> + for_each_new_intel_plane_in_state(state, plane, plane_state, i) { >> + if (plane->pipe == crtc->pipe) >> + intel_dump_plane_state(plane_state); >> + } >> +} >> diff --git a/drivers/gpu/drm/i915/display/intel_pipe_config_dump.h b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.h >> new file mode 100644 >> index 000000000000..20bbe11f0527 >> --- /dev/null >> +++ b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.h >> @@ -0,0 +1,16 @@ >> +/* SPDX-License-Identifier: MIT */ >> +/* >> + * Copyright © 2022 Intel Corporation >> + */ >> + >> +#ifndef __INTEL_PIPE_CONFIG_DUMP_H__ >> +#define __INTEL_PIPE_CONFIG_DUMP_H__ >> + >> +struct intel_crtc_state; >> +struct intel_atomic_state; >> + >> +void intel_pipe_config_dump(const struct intel_crtc_state *pipe_config, >> + struct intel_atomic_state *state, >> + const char *context); >> + >> +#endif /* __INTEL_PIPE_CONFIG_DUMP_H__ */ >> -- >> 2.30.2
diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile index 8b4e5c59ee70..d69881506dc7 100644 --- a/drivers/gpu/drm/i915/Makefile +++ b/drivers/gpu/drm/i915/Makefile @@ -247,6 +247,7 @@ i915-y += \ display/intel_pch_display.o \ display/intel_pch_refclk.o \ display/intel_pipe_config_compare.o \ + display/intel_pipe_config_dump.o \ display/intel_plane_initial.o \ display/intel_psr.o \ display/intel_quirks.o \ diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c index 093ba6bde105..8741ae5e6108 100644 --- a/drivers/gpu/drm/i915/display/intel_display.c +++ b/drivers/gpu/drm/i915/display/intel_display.c @@ -106,6 +106,7 @@ #include "intel_pch_refclk.h" #include "intel_pcode.h" #include "intel_pipe_config_compare.h" +#include "intel_pipe_config_dump.h" #include "intel_pipe_crc.h" #include "intel_plane_initial.h" #include "intel_pm.h" @@ -5059,310 +5060,6 @@ compute_baseline_pipe_bpp(struct intel_atomic_state *state, return 0; } -static void intel_dump_crtc_timings(struct drm_i915_private *i915, - const struct drm_display_mode *mode) -{ - drm_dbg_kms(&i915->drm, "crtc timings: %d %d %d %d %d %d %d %d %d, " - "type: 0x%x flags: 0x%x\n", - mode->crtc_clock, - mode->crtc_hdisplay, mode->crtc_hsync_start, - mode->crtc_hsync_end, mode->crtc_htotal, - mode->crtc_vdisplay, mode->crtc_vsync_start, - mode->crtc_vsync_end, mode->crtc_vtotal, - mode->type, mode->flags); -} - -static void -intel_dump_m_n_config(const struct intel_crtc_state *pipe_config, - const char *id, unsigned int lane_count, - const struct intel_link_m_n *m_n) -{ - struct drm_i915_private *i915 = to_i915(pipe_config->uapi.crtc->dev); - - drm_dbg_kms(&i915->drm, - "%s: lanes: %i; data_m: %u, data_n: %u, link_m: %u, link_n: %u, tu: %u\n", - id, lane_count, - m_n->data_m, m_n->data_n, - m_n->link_m, m_n->link_n, m_n->tu); -} - -static void -intel_dump_infoframe(struct drm_i915_private *dev_priv, - const union hdmi_infoframe *frame) -{ - if (!drm_debug_enabled(DRM_UT_KMS)) - return; - - hdmi_infoframe_log(KERN_DEBUG, dev_priv->drm.dev, frame); -} - -static void -intel_dump_dp_vsc_sdp(struct drm_i915_private *dev_priv, - const struct drm_dp_vsc_sdp *vsc) -{ - if (!drm_debug_enabled(DRM_UT_KMS)) - return; - - drm_dp_vsc_sdp_log(KERN_DEBUG, dev_priv->drm.dev, vsc); -} - -#define OUTPUT_TYPE(x) [INTEL_OUTPUT_ ## x] = #x - -static const char * const output_type_str[] = { - OUTPUT_TYPE(UNUSED), - OUTPUT_TYPE(ANALOG), - OUTPUT_TYPE(DVO), - OUTPUT_TYPE(SDVO), - OUTPUT_TYPE(LVDS), - OUTPUT_TYPE(TVOUT), - OUTPUT_TYPE(HDMI), - OUTPUT_TYPE(DP), - OUTPUT_TYPE(EDP), - OUTPUT_TYPE(DSI), - OUTPUT_TYPE(DDI), - OUTPUT_TYPE(DP_MST), -}; - -#undef OUTPUT_TYPE - -static void snprintf_output_types(char *buf, size_t len, - unsigned int output_types) -{ - char *str = buf; - int i; - - str[0] = '\0'; - - for (i = 0; i < ARRAY_SIZE(output_type_str); i++) { - int r; - - if ((output_types & BIT(i)) == 0) - continue; - - r = snprintf(str, len, "%s%s", - str != buf ? "," : "", output_type_str[i]); - if (r >= len) - break; - str += r; - len -= r; - - output_types &= ~BIT(i); - } - - WARN_ON_ONCE(output_types != 0); -} - -static const char * const output_format_str[] = { - [INTEL_OUTPUT_FORMAT_RGB] = "RGB", - [INTEL_OUTPUT_FORMAT_YCBCR420] = "YCBCR4:2:0", - [INTEL_OUTPUT_FORMAT_YCBCR444] = "YCBCR4:4:4", -}; - -static const char *output_formats(enum intel_output_format format) -{ - if (format >= ARRAY_SIZE(output_format_str)) - return "invalid"; - return output_format_str[format]; -} - -static void intel_dump_plane_state(const struct intel_plane_state *plane_state) -{ - struct intel_plane *plane = to_intel_plane(plane_state->uapi.plane); - struct drm_i915_private *i915 = to_i915(plane->base.dev); - const struct drm_framebuffer *fb = plane_state->hw.fb; - - if (!fb) { - drm_dbg_kms(&i915->drm, - "[PLANE:%d:%s] fb: [NOFB], visible: %s\n", - plane->base.base.id, plane->base.name, - str_yes_no(plane_state->uapi.visible)); - return; - } - - drm_dbg_kms(&i915->drm, - "[PLANE:%d:%s] fb: [FB:%d] %ux%u format = %p4cc modifier = 0x%llx, visible: %s\n", - plane->base.base.id, plane->base.name, - fb->base.id, fb->width, fb->height, &fb->format->format, - fb->modifier, str_yes_no(plane_state->uapi.visible)); - drm_dbg_kms(&i915->drm, "\trotation: 0x%x, scaler: %d\n", - plane_state->hw.rotation, plane_state->scaler_id); - if (plane_state->uapi.visible) - drm_dbg_kms(&i915->drm, - "\tsrc: " DRM_RECT_FP_FMT " dst: " DRM_RECT_FMT "\n", - DRM_RECT_FP_ARG(&plane_state->uapi.src), - DRM_RECT_ARG(&plane_state->uapi.dst)); -} - -void intel_dump_pipe_config(const struct intel_crtc_state *pipe_config, - struct intel_atomic_state *state, - const char *context) -{ - struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc); - struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); - const struct intel_plane_state *plane_state; - struct intel_plane *plane; - char buf[64]; - int i; - - drm_dbg_kms(&dev_priv->drm, "[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(&dev_priv->drm, - "active: %s, output_types: %s (0x%x), output format: %s\n", - str_yes_no(pipe_config->hw.active), - buf, pipe_config->output_types, - output_formats(pipe_config->output_format)); - - drm_dbg_kms(&dev_priv->drm, - "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(&dev_priv->drm, "MST master transcoder: %s\n", - transcoder_name(pipe_config->mst_master_transcoder)); - - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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); - - if (pipe_config->has_pch_encoder) - intel_dump_m_n_config(pipe_config, "fdi", - pipe_config->fdi_lanes, - &pipe_config->fdi_m_n); - - if (intel_crtc_has_dp_encoder(pipe_config)) { - intel_dump_m_n_config(pipe_config, "dp m_n", - pipe_config->lane_count, - &pipe_config->dp_m_n); - intel_dump_m_n_config(pipe_config, "dp m2_n2", - pipe_config->lane_count, - &pipe_config->dp_m2_n2); - } - - drm_dbg_kms(&dev_priv->drm, "framestart delay: %d, MSA timing delay: %d\n", - pipe_config->framestart_delay, pipe_config->msa_timing_delay); - - drm_dbg_kms(&dev_priv->drm, - "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(&dev_priv->drm, "GCP: 0x%x\n", - pipe_config->infoframes.gcp); - if (pipe_config->infoframes.enable & - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_AVI)) - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.avi); - if (pipe_config->infoframes.enable & - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_SPD)) - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.spd); - if (pipe_config->infoframes.enable & - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_VENDOR)) - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.hdmi); - if (pipe_config->infoframes.enable & - intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_DRM)) - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); - if (pipe_config->infoframes.enable & - intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GAMUT_METADATA)) - intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); - if (pipe_config->infoframes.enable & - intel_hdmi_infoframe_enable(DP_SDP_VSC)) - intel_dump_dp_vsc_sdp(dev_priv, &pipe_config->infoframes.vsc); - - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "requested mode: " DRM_MODE_FMT "\n", - DRM_MODE_ARG(&pipe_config->hw.mode)); - drm_dbg_kms(&dev_priv->drm, "adjusted mode: " DRM_MODE_FMT "\n", - DRM_MODE_ARG(&pipe_config->hw.adjusted_mode)); - intel_dump_crtc_timings(dev_priv, &pipe_config->hw.adjusted_mode); - drm_dbg_kms(&dev_priv->drm, "pipe mode: " DRM_MODE_FMT "\n", - DRM_MODE_ARG(&pipe_config->hw.pipe_mode)); - intel_dump_crtc_timings(dev_priv, &pipe_config->hw.pipe_mode); - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "linetime: %d, ips linetime: %d\n", - pipe_config->linetime, pipe_config->ips_linetime); - - if (DISPLAY_VER(dev_priv) >= 9) - drm_dbg_kms(&dev_priv->drm, - "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d\n", - crtc->num_scalers, - pipe_config->scaler_state.scaler_users, - pipe_config->scaler_state.scaler_id); - - if (HAS_GMCH(dev_priv)) - drm_dbg_kms(&dev_priv->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); - else - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "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(dev_priv, &pipe_config->dpll_hw_state); - - if (IS_CHERRYVIEW(dev_priv)) - drm_dbg_kms(&dev_priv->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); - else - drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "degamma lut: %d entries, gamma lut: %d entries\n", - pipe_config->hw.degamma_lut ? - drm_color_lut_size(pipe_config->hw.degamma_lut) : 0, - pipe_config->hw.gamma_lut ? - drm_color_lut_size(pipe_config->hw.gamma_lut) : 0); - -dump_planes: - if (!state) - return; - - for_each_new_intel_plane_in_state(state, plane, plane_state, i) { - if (plane->pipe == crtc->pipe) - intel_dump_plane_state(plane_state); - } -} - static bool check_digital_port_conflicts(struct intel_atomic_state *state) { struct drm_device *dev = state->base.dev; @@ -6823,7 +6520,7 @@ static int intel_atomic_check(struct drm_device *dev, !new_crtc_state->update_pipe) continue; - intel_dump_pipe_config(new_crtc_state, state, + intel_pipe_config_dump(new_crtc_state, state, intel_crtc_needs_modeset(new_crtc_state) ? "[modeset]" : "[fastset]"); } @@ -6840,7 +6537,7 @@ static int intel_atomic_check(struct drm_device *dev, */ for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state, new_crtc_state, i) - intel_dump_pipe_config(new_crtc_state, state, "[failed]"); + intel_pipe_config_dump(new_crtc_state, state, "[failed]"); return ret; } @@ -9320,7 +9017,7 @@ intel_modeset_setup_hw_state(struct drm_device *dev, to_intel_crtc_state(crtc->base.state); intel_sanitize_crtc(crtc, ctx); - intel_dump_pipe_config(crtc_state, NULL, "[setup_hw_state]"); + intel_pipe_config_dump(crtc_state, NULL, "[setup_hw_state]"); } intel_modeset_update_connector_atomic_state(dev); diff --git a/drivers/gpu/drm/i915/display/intel_display.h b/drivers/gpu/drm/i915/display/intel_display.h index e827c84ece56..68477eb82049 100644 --- a/drivers/gpu/drm/i915/display/intel_display.h +++ b/drivers/gpu/drm/i915/display/intel_display.h @@ -560,9 +560,6 @@ bool intel_crtc_is_bigjoiner_master(const struct intel_crtc_state *crtc_state); u8 intel_crtc_bigjoiner_slave_pipes(const struct intel_crtc_state *crtc_state); struct intel_crtc *intel_master_crtc(const struct intel_crtc_state *crtc_state); bool intel_crtc_get_pipe_config(struct intel_crtc_state *crtc_state); -void intel_dump_pipe_config(const struct intel_crtc_state *pipe_config, - struct intel_atomic_state *state, - const char *context); void intel_plane_destroy(struct drm_plane *plane); void intel_enable_transcoder(const struct intel_crtc_state *new_crtc_state); diff --git a/drivers/gpu/drm/i915/display/intel_modeset_verify.c b/drivers/gpu/drm/i915/display/intel_modeset_verify.c index fd752c61d854..de034ccef289 100644 --- a/drivers/gpu/drm/i915/display/intel_modeset_verify.c +++ b/drivers/gpu/drm/i915/display/intel_modeset_verify.c @@ -15,6 +15,7 @@ #include "intel_fdi.h" #include "intel_modeset_verify.h" #include "intel_pipe_config_compare.h" +#include "intel_pipe_config_dump.h" #include "intel_pm.h" #include "intel_snps_phy.h" @@ -217,8 +218,8 @@ verify_crtc_state(struct intel_crtc *crtc, if (!intel_pipe_config_compare(new_crtc_state, pipe_config, false)) { I915_STATE_WARN(1, "pipe state doesn't match!\n"); - intel_dump_pipe_config(pipe_config, NULL, "[hw state]"); - intel_dump_pipe_config(new_crtc_state, NULL, "[sw state]"); + intel_pipe_config_dump(pipe_config, NULL, "[hw state]"); + intel_pipe_config_dump(new_crtc_state, NULL, "[sw state]"); } } diff --git a/drivers/gpu/drm/i915/display/intel_pipe_config_dump.c b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.c new file mode 100644 index 000000000000..c290cf009407 --- /dev/null +++ b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.c @@ -0,0 +1,314 @@ +// SPDX-License-Identifier: MIT +/* + * Copyright © 2022 Intel Corporation + */ + +#include "i915_drv.h" +#include "intel_display_types.h" +#include "intel_hdmi.h" +#include "intel_pipe_config_dump.h" +#include "intel_vrr.h" + +static void intel_dump_crtc_timings(struct drm_i915_private *i915, + const struct drm_display_mode *mode) +{ + drm_dbg_kms(&i915->drm, "crtc timings: %d %d %d %d %d %d %d %d %d, " + "type: 0x%x flags: 0x%x\n", + mode->crtc_clock, + mode->crtc_hdisplay, mode->crtc_hsync_start, + mode->crtc_hsync_end, mode->crtc_htotal, + mode->crtc_vdisplay, mode->crtc_vsync_start, + mode->crtc_vsync_end, mode->crtc_vtotal, + mode->type, mode->flags); +} + +static void +intel_dump_m_n_config(const struct intel_crtc_state *pipe_config, + const char *id, unsigned int lane_count, + const struct intel_link_m_n *m_n) +{ + struct drm_i915_private *i915 = to_i915(pipe_config->uapi.crtc->dev); + + drm_dbg_kms(&i915->drm, + "%s: lanes: %i; data_m: %u, data_n: %u, link_m: %u, link_n: %u, tu: %u\n", + id, lane_count, + m_n->data_m, m_n->data_n, + m_n->link_m, m_n->link_n, m_n->tu); +} + +static void +intel_dump_infoframe(struct drm_i915_private *dev_priv, + const union hdmi_infoframe *frame) +{ + if (!drm_debug_enabled(DRM_UT_KMS)) + return; + + hdmi_infoframe_log(KERN_DEBUG, dev_priv->drm.dev, frame); +} + +static void +intel_dump_dp_vsc_sdp(struct drm_i915_private *dev_priv, + const struct drm_dp_vsc_sdp *vsc) +{ + if (!drm_debug_enabled(DRM_UT_KMS)) + return; + + drm_dp_vsc_sdp_log(KERN_DEBUG, dev_priv->drm.dev, vsc); +} + +#define OUTPUT_TYPE(x) [INTEL_OUTPUT_ ## x] = #x + +static const char * const output_type_str[] = { + OUTPUT_TYPE(UNUSED), + OUTPUT_TYPE(ANALOG), + OUTPUT_TYPE(DVO), + OUTPUT_TYPE(SDVO), + OUTPUT_TYPE(LVDS), + OUTPUT_TYPE(TVOUT), + OUTPUT_TYPE(HDMI), + OUTPUT_TYPE(DP), + OUTPUT_TYPE(EDP), + OUTPUT_TYPE(DSI), + OUTPUT_TYPE(DDI), + OUTPUT_TYPE(DP_MST), +}; + +#undef OUTPUT_TYPE + +static void snprintf_output_types(char *buf, size_t len, + unsigned int output_types) +{ + char *str = buf; + int i; + + str[0] = '\0'; + + for (i = 0; i < ARRAY_SIZE(output_type_str); i++) { + int r; + + if ((output_types & BIT(i)) == 0) + continue; + + r = snprintf(str, len, "%s%s", + str != buf ? "," : "", output_type_str[i]); + if (r >= len) + break; + str += r; + len -= r; + + output_types &= ~BIT(i); + } + + WARN_ON_ONCE(output_types != 0); +} + +static const char * const output_format_str[] = { + [INTEL_OUTPUT_FORMAT_RGB] = "RGB", + [INTEL_OUTPUT_FORMAT_YCBCR420] = "YCBCR4:2:0", + [INTEL_OUTPUT_FORMAT_YCBCR444] = "YCBCR4:4:4", +}; + +static const char *output_formats(enum intel_output_format format) +{ + if (format >= ARRAY_SIZE(output_format_str)) + return "invalid"; + return output_format_str[format]; +} + +static void intel_dump_plane_state(const struct intel_plane_state *plane_state) +{ + struct intel_plane *plane = to_intel_plane(plane_state->uapi.plane); + struct drm_i915_private *i915 = to_i915(plane->base.dev); + const struct drm_framebuffer *fb = plane_state->hw.fb; + + if (!fb) { + drm_dbg_kms(&i915->drm, + "[PLANE:%d:%s] fb: [NOFB], visible: %s\n", + plane->base.base.id, plane->base.name, + str_yes_no(plane_state->uapi.visible)); + return; + } + + drm_dbg_kms(&i915->drm, + "[PLANE:%d:%s] fb: [FB:%d] %ux%u format = %p4cc modifier = 0x%llx, visible: %s\n", + plane->base.base.id, plane->base.name, + fb->base.id, fb->width, fb->height, &fb->format->format, + fb->modifier, str_yes_no(plane_state->uapi.visible)); + drm_dbg_kms(&i915->drm, "\trotation: 0x%x, scaler: %d\n", + plane_state->hw.rotation, plane_state->scaler_id); + if (plane_state->uapi.visible) + drm_dbg_kms(&i915->drm, + "\tsrc: " DRM_RECT_FP_FMT " dst: " DRM_RECT_FMT "\n", + DRM_RECT_FP_ARG(&plane_state->uapi.src), + DRM_RECT_ARG(&plane_state->uapi.dst)); +} + +void intel_pipe_config_dump(const struct intel_crtc_state *pipe_config, + struct intel_atomic_state *state, + const char *context) +{ + struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc); + struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); + const struct intel_plane_state *plane_state; + struct intel_plane *plane; + char buf[64]; + int i; + + drm_dbg_kms(&dev_priv->drm, "[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(&dev_priv->drm, + "active: %s, output_types: %s (0x%x), output format: %s\n", + str_yes_no(pipe_config->hw.active), + buf, pipe_config->output_types, + output_formats(pipe_config->output_format)); + + drm_dbg_kms(&dev_priv->drm, + "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(&dev_priv->drm, "MST master transcoder: %s\n", + transcoder_name(pipe_config->mst_master_transcoder)); + + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->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); + + if (pipe_config->has_pch_encoder) + intel_dump_m_n_config(pipe_config, "fdi", + pipe_config->fdi_lanes, + &pipe_config->fdi_m_n); + + if (intel_crtc_has_dp_encoder(pipe_config)) { + intel_dump_m_n_config(pipe_config, "dp m_n", + pipe_config->lane_count, + &pipe_config->dp_m_n); + intel_dump_m_n_config(pipe_config, "dp m2_n2", + pipe_config->lane_count, + &pipe_config->dp_m2_n2); + } + + drm_dbg_kms(&dev_priv->drm, "framestart delay: %d, MSA timing delay: %d\n", + pipe_config->framestart_delay, pipe_config->msa_timing_delay); + + drm_dbg_kms(&dev_priv->drm, + "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(&dev_priv->drm, "GCP: 0x%x\n", + pipe_config->infoframes.gcp); + if (pipe_config->infoframes.enable & + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_AVI)) + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.avi); + if (pipe_config->infoframes.enable & + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_SPD)) + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.spd); + if (pipe_config->infoframes.enable & + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_VENDOR)) + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.hdmi); + if (pipe_config->infoframes.enable & + intel_hdmi_infoframe_enable(HDMI_INFOFRAME_TYPE_DRM)) + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); + if (pipe_config->infoframes.enable & + intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GAMUT_METADATA)) + intel_dump_infoframe(dev_priv, &pipe_config->infoframes.drm); + if (pipe_config->infoframes.enable & + intel_hdmi_infoframe_enable(DP_SDP_VSC)) + intel_dump_dp_vsc_sdp(dev_priv, &pipe_config->infoframes.vsc); + + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "requested mode: " DRM_MODE_FMT "\n", + DRM_MODE_ARG(&pipe_config->hw.mode)); + drm_dbg_kms(&dev_priv->drm, "adjusted mode: " DRM_MODE_FMT "\n", + DRM_MODE_ARG(&pipe_config->hw.adjusted_mode)); + intel_dump_crtc_timings(dev_priv, &pipe_config->hw.adjusted_mode); + drm_dbg_kms(&dev_priv->drm, "pipe mode: " DRM_MODE_FMT "\n", + DRM_MODE_ARG(&pipe_config->hw.pipe_mode)); + intel_dump_crtc_timings(dev_priv, &pipe_config->hw.pipe_mode); + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "linetime: %d, ips linetime: %d\n", + pipe_config->linetime, pipe_config->ips_linetime); + + if (DISPLAY_VER(dev_priv) >= 9) + drm_dbg_kms(&dev_priv->drm, + "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d\n", + crtc->num_scalers, + pipe_config->scaler_state.scaler_users, + pipe_config->scaler_state.scaler_id); + + if (HAS_GMCH(dev_priv)) + drm_dbg_kms(&dev_priv->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); + else + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "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(dev_priv, &pipe_config->dpll_hw_state); + + if (IS_CHERRYVIEW(dev_priv)) + drm_dbg_kms(&dev_priv->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); + else + drm_dbg_kms(&dev_priv->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_dbg_kms(&dev_priv->drm, "degamma lut: %d entries, gamma lut: %d entries\n", + pipe_config->hw.degamma_lut ? + drm_color_lut_size(pipe_config->hw.degamma_lut) : 0, + pipe_config->hw.gamma_lut ? + drm_color_lut_size(pipe_config->hw.gamma_lut) : 0); + +dump_planes: + if (!state) + return; + + for_each_new_intel_plane_in_state(state, plane, plane_state, i) { + if (plane->pipe == crtc->pipe) + intel_dump_plane_state(plane_state); + } +} diff --git a/drivers/gpu/drm/i915/display/intel_pipe_config_dump.h b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.h new file mode 100644 index 000000000000..20bbe11f0527 --- /dev/null +++ b/drivers/gpu/drm/i915/display/intel_pipe_config_dump.h @@ -0,0 +1,16 @@ +/* SPDX-License-Identifier: MIT */ +/* + * Copyright © 2022 Intel Corporation + */ + +#ifndef __INTEL_PIPE_CONFIG_DUMP_H__ +#define __INTEL_PIPE_CONFIG_DUMP_H__ + +struct intel_crtc_state; +struct intel_atomic_state; + +void intel_pipe_config_dump(const struct intel_crtc_state *pipe_config, + struct intel_atomic_state *state, + const char *context); + +#endif /* __INTEL_PIPE_CONFIG_DUMP_H__ */
Declutter intel_display.c by splitting out pipe config dumping to a separate file. Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/Makefile | 1 + drivers/gpu/drm/i915/display/intel_display.c | 311 +---------------- drivers/gpu/drm/i915/display/intel_display.h | 3 - .../drm/i915/display/intel_modeset_verify.c | 5 +- .../drm/i915/display/intel_pipe_config_dump.c | 314 ++++++++++++++++++ .../drm/i915/display/intel_pipe_config_dump.h | 16 + 6 files changed, 338 insertions(+), 312 deletions(-) create mode 100644 drivers/gpu/drm/i915/display/intel_pipe_config_dump.c create mode 100644 drivers/gpu/drm/i915/display/intel_pipe_config_dump.h