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