diff options
author | Jesse Barnes <jbarnes@virtuousgeek.org> | 2013-04-02 13:03:55 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-04-02 14:25:39 -0400 |
commit | 22f9fe50597094970a9bcdf2464fc3b356fbedb6 (patch) | |
tree | 68da3839169867caeb7680a641fb90aa52ba1902 | |
parent | deb18211a110c102d32b3e9ed866bd7d25e0f8d5 (diff) |
drm/i915: fix DDI get_hw_state return value
If we couldn't find a pipe we shouldn't return true. This might be even
better as a WARN though, since it should be impossible to have the port
enabled without a pipe selected.
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
-rw-r--r-- | drivers/gpu/drm/i915/intel_ddi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c index 6c6b0124f17d..22524cb6903b 100644 --- a/drivers/gpu/drm/i915/intel_ddi.c +++ b/drivers/gpu/drm/i915/intel_ddi.c | |||
@@ -1150,7 +1150,7 @@ bool intel_ddi_get_hw_state(struct intel_encoder *encoder, | |||
1150 | 1150 | ||
1151 | DRM_DEBUG_KMS("No pipe for ddi port %i found\n", port); | 1151 | DRM_DEBUG_KMS("No pipe for ddi port %i found\n", port); |
1152 | 1152 | ||
1153 | return true; | 1153 | return false; |
1154 | } | 1154 | } |
1155 | 1155 | ||
1156 | static uint32_t intel_ddi_get_crtc_pll(struct drm_i915_private *dev_priv, | 1156 | static uint32_t intel_ddi_get_crtc_pll(struct drm_i915_private *dev_priv, |