[Intel-gfx] [PATCH 2/2] drm/i915: Add missing parameter to intel_dp_set_drrs_state documentation.

Maarten Lankhorst maarten.lankhorst at linux.intel.com
Thu Aug 25 11:29:02 UTC 2016


Op 25-08-16 om 11:49 schreef Chris Wilson:
> On Thu, Aug 25, 2016 at 11:07:02AM +0200, Maarten Lankhorst wrote:
>> Signed-off-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
>> ---
>>  drivers/gpu/drm/i915/intel_dp.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
>> index a3c7dd8fd406..df0773afb7f5 100644
>> --- a/drivers/gpu/drm/i915/intel_dp.c
>> +++ b/drivers/gpu/drm/i915/intel_dp.c
>> @@ -5049,6 +5049,7 @@ static void intel_dp_pps_init(struct drm_device *dev,
>>  /**
>>   * intel_dp_set_drrs_state - program registers for RR switch to take effect
>>   * @dev: DRM device
>> + * @crtc_state: a pointer to the active intel_crtc_state
>>   * @refresh_rate: RR to be programmed
>>   *
>>   * This function gets called when refresh rate (RR) has to be changed from
> Which is the active state? The one currently in use, or the one going to
> be in use?

I wrote active, because it can be called from a work item with the current state or when disabling with the old crtc state.

~Maarten



More information about the Intel-gfx mailing list