Commit 344c3278 authored by Jani Nikula's avatar Jani Nikula

drm/i915/debugfs: clean up LPSP capable

Clean up the LPSP capability printout. No functional changes.
Reviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/097390f12da8de57c9d52ad675309d0422dec8fb.1630327990.git.jani.nikula@intel.com
parent 3a5f9281
...@@ -2224,14 +2224,12 @@ static int i915_psr_status_show(struct seq_file *m, void *data) ...@@ -2224,14 +2224,12 @@ static int i915_psr_status_show(struct seq_file *m, void *data)
} }
DEFINE_SHOW_ATTRIBUTE(i915_psr_status); DEFINE_SHOW_ATTRIBUTE(i915_psr_status);
#define LPSP_CAPABLE(COND) (COND ? seq_puts(m, "LPSP: capable\n") : \
seq_puts(m, "LPSP: incapable\n"))
static int i915_lpsp_capability_show(struct seq_file *m, void *data) static int i915_lpsp_capability_show(struct seq_file *m, void *data)
{ {
struct drm_connector *connector = m->private; struct drm_connector *connector = m->private;
struct drm_i915_private *i915 = to_i915(connector->dev); struct drm_i915_private *i915 = to_i915(connector->dev);
struct intel_encoder *encoder; struct intel_encoder *encoder;
bool lpsp_capable = false;
encoder = intel_attached_encoder(to_intel_connector(connector)); encoder = intel_attached_encoder(to_intel_connector(connector));
if (!encoder) if (!encoder)
...@@ -2240,35 +2238,27 @@ static int i915_lpsp_capability_show(struct seq_file *m, void *data) ...@@ -2240,35 +2238,27 @@ static int i915_lpsp_capability_show(struct seq_file *m, void *data)
if (connector->status != connector_status_connected) if (connector->status != connector_status_connected)
return -ENODEV; return -ENODEV;
if (DISPLAY_VER(i915) >= 13) { if (DISPLAY_VER(i915) >= 13)
LPSP_CAPABLE(encoder->port <= PORT_B); lpsp_capable = encoder->port <= PORT_B;
return 0; else if (DISPLAY_VER(i915) >= 12)
}
switch (DISPLAY_VER(i915)) {
case 12:
/* /*
* Actually TGL can drive LPSP on port till DDI_C * Actually TGL can drive LPSP on port till DDI_C
* but there is no physical connected DDI_C on TGL sku's, * but there is no physical connected DDI_C on TGL sku's,
* even driver is not initilizing DDI_C port for gen12. * even driver is not initilizing DDI_C port for gen12.
*/ */
LPSP_CAPABLE(encoder->port <= PORT_B); lpsp_capable = encoder->port <= PORT_B;
break; else if (DISPLAY_VER(i915) == 11)
case 11: lpsp_capable = (connector->connector_type == DRM_MODE_CONNECTOR_DSI ||
LPSP_CAPABLE(connector->connector_type == DRM_MODE_CONNECTOR_DSI ||
connector->connector_type == DRM_MODE_CONNECTOR_eDP); connector->connector_type == DRM_MODE_CONNECTOR_eDP);
break; else if (IS_DISPLAY_VER(i915, 9, 10))
case 10: lpsp_capable = (encoder->port == PORT_A &&
case 9:
LPSP_CAPABLE(encoder->port == PORT_A &&
(connector->connector_type == DRM_MODE_CONNECTOR_DSI || (connector->connector_type == DRM_MODE_CONNECTOR_DSI ||
connector->connector_type == DRM_MODE_CONNECTOR_eDP || connector->connector_type == DRM_MODE_CONNECTOR_eDP ||
connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort)); connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort));
break; else if (IS_HASWELL(i915) || IS_BROADWELL(i915))
default: lpsp_capable = connector->connector_type == DRM_MODE_CONNECTOR_eDP;
if (IS_HASWELL(i915) || IS_BROADWELL(i915))
LPSP_CAPABLE(connector->connector_type == DRM_MODE_CONNECTOR_eDP); seq_printf(m, "LPSP: %s\n", lpsp_capable ? "capable" : "incapable");
}
return 0; return 0;
} }
......
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