Attachment 59505 Details for Bug 48170 – properly compute dp dithering v2

[patch] properly compute dp dithering v2

0001-properly-compute-dithering-for-user-created-modes-v2.patch (text/plain), 3.36 KB, created by Daniel Vetter on 2012-04-05 01:01:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Daniel Vetter

Created: 2012-04-05 01:01:02 UTC

Size: 3.36 KB

patch

obsolete

>From 2de4953cbb8507339988914709b57b43f84370c1 Mon Sep 17 00:00:00 2001
>From: Daniel Vetter <daniel.vetter@ffwll.ch>
>Date: Wed, 4 Apr 2012 20:57:15 +0200
>Subject: [PATCH] properly compute dithering for user-created modes v2
>
>---
> drivers/gpu/drm/i915/intel_dp.c |   50 ++++++++++++++++++++++++++++-----------
> 1 files changed, 36 insertions(+), 14 deletions(-)
>
>diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
>index 110552f..e0517c5 100644
>--- a/drivers/gpu/drm/i915/intel_dp.c
>+++ b/drivers/gpu/drm/i915/intel_dp.c
>@@ -219,14 +219,38 @@ intel_dp_max_data_rate(int max_link_clock, int max_lanes)
> 	return (max_link_clock * max_lanes * 8) / 10;
> }
> 
>+static bool
>+intel_dp_adjust_dithering(struct intel_dp *intel_dp,
>+			  struct drm_display_mode *mode,
>+			  struct drm_display_mode *adjusted_mode)
>+{
>+	int max_link_clock = intel_dp_link_clock(intel_dp_max_link_bw(intel_dp));
>+	int max_lanes = intel_dp_max_lane_count(intel_dp);
>+	int max_rate, mode_rate;
>+
>+	mode_rate = intel_dp_link_required(mode->clock, 24);
>+	max_rate = intel_dp_max_data_rate(max_link_clock, max_lanes);
>+
>+	if (mode_rate > max_rate) {
>+			mode_rate = intel_dp_link_required(mode->clock, 18);
>+			if (mode_rate > max_rate)
>+				return false;
>+
>+			if (adjusted_mode)
>+				adjusted_mode->private_flags
>+					|= INTEL_MODE_DP_FORCE_6BPC;
>+
>+			return true;
>+	}
>+
>+	return true;
>+}
>+
> static int
> intel_dp_mode_valid(struct drm_connector *connector,
> 		    struct drm_display_mode *mode)
> {
> 	struct intel_dp *intel_dp = intel_attached_dp(connector);
>-	int max_link_clock = intel_dp_link_clock(intel_dp_max_link_bw(intel_dp));
>-	int max_lanes = intel_dp_max_lane_count(intel_dp);
>-	int max_rate, mode_rate;
> 
> 	if (is_edp(intel_dp) && intel_dp->panel_fixed_mode) {
> 		if (mode->hdisplay > intel_dp->panel_fixed_mode->hdisplay)
>@@ -236,16 +260,8 @@ intel_dp_mode_valid(struct drm_connector *connector,
> 			return MODE_PANEL;
> 	}
> 
>-	mode_rate = intel_dp_link_required(mode->clock, 24);
>-	max_rate = intel_dp_max_data_rate(max_link_clock, max_lanes);
>-
>-	if (mode_rate > max_rate) {
>-			mode_rate = intel_dp_link_required(mode->clock, 18);
>-			if (mode_rate > max_rate)
>-				return MODE_CLOCK_HIGH;
>-			else
>-				mode->private_flags |= INTEL_MODE_DP_FORCE_6BPC;
>-	}
>+	if (!intel_dp_adjust_dithering(intel_dp, mode, NULL))
>+		return MODE_CLOCK_HIGH;
> 
> 	if (mode->clock < 10000)
> 		return MODE_CLOCK_LOW;
>@@ -672,7 +688,7 @@ intel_dp_mode_fixup(struct drm_encoder *encoder, struct drm_display_mode *mode,
> 	int lane_count, clock;
> 	int max_lane_count = intel_dp_max_lane_count(intel_dp);
> 	int max_clock = intel_dp_max_link_bw(intel_dp) == DP_LINK_BW_2_7 ? 1 : 0;
>-	int bpp = mode->private_flags & INTEL_MODE_DP_FORCE_6BPC ? 18 : 24;
>+	int bpp;
> 	static int bws[2] = { DP_LINK_BW_1_62, DP_LINK_BW_2_7 };
> 
> 	if (is_edp(intel_dp) && intel_dp->panel_fixed_mode) {
>@@ -686,6 +702,12 @@ intel_dp_mode_fixup(struct drm_encoder *encoder, struct drm_display_mode *mode,
> 		mode->clock = intel_dp->panel_fixed_mode->clock;
> 	}
> 
>+	if (!intel_dp_adjust_dithering(intel_dp, mode, adjusted_mode))
>+		return false;
>+
>+	bpp = mode->private_flags & INTEL_MODE_DP_FORCE_6BPC ? 18 : 24;
>+	printk("dp mode_fixup, bpp: %i\n", bpp);
>+
> 	for (lane_count = 1; lane_count <= max_lane_count; lane_count <<= 1) {
> 		for (clock = 0; clock <= max_clock; clock++) {
> 			int link_avail = intel_dp_max_data_rate(intel_dp_link_clock(bws[clock]), lane_count);
>-- 
>1.7.9.1
>

Actions: View | Splinter Review

Attachments on bug 48170: 59360 | 59363 | 59365 | 59367 | 59428 | 59481 | 59486 | 59504 | 59505 | 59683 | 59708

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.