On Fri, Apr 10, 2015 at 8:15 PM, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote: > > /** > + * intel_psr_single_frame_update - Single Frame Update > + * @dev: DRM device > + * > + * Some platforms support a single frame update feature that is used to > + * send and update only one frame on Remote Frame Buffer. > + * So far it is only implemented for Valleyview and Cherryview because > + * hardware requires this to be done before a page flip. > + */ > +void intel_psr_single_frame_update(struct drm_device *dev) While reading through frontbuffer code I realized that this function here doesn't filter flip_prepare events according to the frontbuffer bitmask. Which means we'll do a single-shot upload even when only another CRTC changes. Rodrigo, can you please follow-up with a patch to do that? Similar to how invalidate/flush only do something when it's about a frontbuffer on the psr CRTC. Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
