Commit 5244221e authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915: Dump the mode for the crtc just the once

No point in repeating the crtc mode for each cloned encoder.
Just print it once, and avoid using multiple lines for it.
And while at let's polish the fixed mode print to fit on
one line as well.
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20191129185434.25549-6-ville.syrjala@linux.intel.comReviewed-by: default avatarRamalingam C <ramalingam.c@intel.com>
parent 64754233
...@@ -2388,29 +2388,16 @@ static void intel_encoder_info(struct seq_file *m, ...@@ -2388,29 +2388,16 @@ static void intel_encoder_info(struct seq_file *m,
seq_printf(m, "\tencoder %d: type: %s, connectors:\n", seq_printf(m, "\tencoder %d: type: %s, connectors:\n",
encoder->base.base.id, encoder->base.name); encoder->base.base.id, encoder->base.name);
for_each_connector_on_encoder(dev, &encoder->base, connector) { for_each_connector_on_encoder(dev, &encoder->base, connector)
seq_printf(m, "\t\tconnector %d: type: %s, status: %s", seq_printf(m, "\t\tconnector %d: type: %s\n",
connector->base.base.id, connector->base.name, connector->base.base.id, connector->base.name);
drm_get_connector_status_name(connector->base.status));
if (connector->base.status == connector_status_connected) {
const struct intel_crtc_state *crtc_state =
to_intel_crtc_state(crtc->base.state);
seq_printf(m, ", mode:\n");
intel_seq_print_mode(m, 2, &crtc_state->hw.mode);
} else {
seq_putc(m, '\n');
}
}
} }
static void intel_panel_info(struct seq_file *m, struct intel_panel *panel) static void intel_panel_info(struct seq_file *m, struct intel_panel *panel)
{ {
struct drm_display_mode *mode = panel->fixed_mode; const struct drm_display_mode *mode = panel->fixed_mode;
seq_printf(m, "\tfixed mode:\n"); seq_printf(m, "\tfixed mode: " DRM_MODE_FMT "\n", DRM_MODE_ARG(mode));
intel_seq_print_mode(m, 2, mode);
} }
static void intel_hdcp_info(struct seq_file *m, static void intel_hdcp_info(struct seq_file *m,
...@@ -2651,8 +2638,12 @@ static void intel_crtc_info(struct seq_file *m, struct intel_crtc *crtc) ...@@ -2651,8 +2638,12 @@ static void intel_crtc_info(struct seq_file *m, struct intel_crtc *crtc)
yesno(crtc_state->dither), crtc_state->pipe_bpp); yesno(crtc_state->dither), crtc_state->pipe_bpp);
if (crtc_state->hw.active) { if (crtc_state->hw.active) {
const struct drm_display_mode *mode =
&crtc_state->hw.mode;
struct intel_encoder *encoder; struct intel_encoder *encoder;
seq_printf(m, "\tmode: " DRM_MODE_FMT "\n", DRM_MODE_ARG(mode));
for_each_encoder_on_crtc(&dev_priv->drm, &crtc->base, encoder) for_each_encoder_on_crtc(&dev_priv->drm, &crtc->base, encoder)
intel_encoder_info(m, crtc, encoder); intel_encoder_info(m, crtc, encoder);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment