[PATCH 4/4] drm/i915: DDI: call intel_edp_drrs_enable() on encoder updates

Hans de Goede hdegoede at redhat.com
Wed Dec 19 10:55:41 UTC 2018


Hi,

On 19-12-18 10:55, Maarten Lankhorst wrote:
> Op 18-12-2018 om 23:29 schreef Hans de Goede:
>> Call intel_edp_drrs_enable() on encoder updates to make sure that we enable
>> DRRS (when applicable) on fastsets.
>>
>> Note calling intel_edp_drrs_enable() when PSR has already been enabled is a
>> no-op, so it is safe to do this on every encoder->update callback.
>>
>> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
>> ---
>>   drivers/gpu/drm/i915/intel_ddi.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
>> index 30f54c334dd7..b74f3c415fbb 100644
>> --- a/drivers/gpu/drm/i915/intel_ddi.c
>> +++ b/drivers/gpu/drm/i915/intel_ddi.c
>> @@ -3544,6 +3544,7 @@ static void intel_update_ddi_dp(struct intel_encoder *encoder,
>>   	struct intel_dp *intel_dp = enc_to_intel_dp(&encoder->base);
>>   
>>   	intel_psr_enable(intel_dp, crtc_state);
>> +	intel_edp_drrs_enable(intel_dp, crtc_state);
>>   }
>>   
>>   static void intel_update_ddi(struct intel_encoder *encoder,
> 
> I would merge patch 2 and 4, and rename the callback to pipe_update or something less generic than update, at least.
> 
> Other than that, looks good. So with those changes:
> 
> Reviewed-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>

Thank you, I agree with the suggested changes. I'm not sure if I will get
around to doing a v2 with the changes today.

> Can I convince you to review https://patchwork.freedesktop.org/series/54226/ ? Should be triggerable with
> 
> kms_atomic_transition --run-subtest plane-all-modeset-transition-fencing-internal-panels

Sure, but again I likely will not get around to this until tomorrow.

> At that point we should have FBC, PSR and DRRS working correctly with fastset. :)

Sounds good :)

Regards,

Hans



More information about the dri-devel mailing list