Re: [PATCH 2/2] drm/i915: Add missing parameter to intel_dp_set_drrs_state documentation.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxx>
>> ---
>>  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

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




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux