Re: [Intel-gfx] [PATCH] drm/i915: Don't call intel_plane_restore() when the prop value didn't change

2014-09-02 Thread Damien Lespiau
On Mon, Sep 01, 2014 at 04:16:59PM +0100, Damien Lespiau wrote: Cc: Thomas Wood thomas.w...@intel.com Cc: Sonika Jindal sonika.jin...@intel.com Tested-by: Mika Kuoppala mika.kuopp...@linux.intel.com Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com Reviewed-by: Damien Lespiau

Re: [Intel-gfx] [PATCH] drm/i915: Don't call intel_plane_restore() when the prop value didn't change

2014-09-01 Thread Damien Lespiau
On Mon, Sep 01, 2014 at 06:08:25PM +0300, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com No point in calling intel_plane_restore() in .set_property() if the value didn't change. More importantly this papers over a bug where the current primary plane