Re: [PATCH v2 1/5] drm/i915: Set csc coefficients in update_pipe_size.

From: Jesse Barnes <jbarnes@virtuousgeek.org>
To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	intel-gfx@lists.freedesktop.org
Cc: Daniel Stone <daniels@collabora.com>
Subject: Re: [PATCH v2 1/5] drm/i915: Set csc coefficients in update_pipe_size.
Date: Thu, 10 Sep 2015 12:27:33 -0700	[thread overview]
Message-ID: <55F1D9A5.9080107@virtuousgeek.org> (raw)
In-Reply-To: <55F1D90F.5000708@virtuousgeek.org>

On 09/10/2015 12:25 PM, Jesse Barnes wrote:
> On 08/27/2015 06:44 AM, Maarten Lankhorst wrote:
>> This might not have been set during boot, and when we preserve
>> the initial mode this can result in a black screen.
>>
>> Cc: Daniel Stone <daniels@collabora.com>
>> Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>> ---
>>  drivers/gpu/drm/i915/intel_display.c | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
>> index 364104dee32f..7b5dfe2f7b2d 100644
>> --- a/drivers/gpu/drm/i915/intel_display.c
>> +++ b/drivers/gpu/drm/i915/intel_display.c
>> @@ -3314,6 +3314,9 @@ static void intel_update_pipe_size(struct intel_crtc *crtc)
>>  	if (!i915.fastboot)
>>  		return;
>>  
>> +	if (HAS_DDI(dev))
>> +		intel_set_pipe_csc(&crtc->base);
>> +
>>  	/*
>>  	 * Update pipe size and adjust fitter if needed: the reason for this is
>>  	 * that in compute_mode_changes we check the native mode (not the pfit
>>
> 
> Maybe there's a better place to put this later?

Nevermind, I see the function gets renamed later; much clearer.

Jesse

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx