Re: [Intel-gfx] [PATCH] drm/atomic-helper: Add option to update planes only on active crtc

2015-09-08 Thread Laurent Pinchart
Hi Daniel, Thank you for the patch. On Tuesday 08 September 2015 12:02:07 Daniel Vetter wrote: > With drivers supporting runtime pm it's generally not a good idea to > touch the hardware when it's off. Add an option to the commit_planes > helper to support this case. > > Note that the helpers

Re: [Intel-gfx] [PATCH] drm/atomic-helper: Add option to update planes only on active crtc

2015-09-08 Thread Daniel Vetter
On Tue, Sep 08, 2015 at 01:10:58PM +0200, Thierry Reding wrote: > On Tue, Sep 08, 2015 at 12:02:07PM +0200, Daniel Vetter wrote: > > With drivers supporting runtime pm it's generally not a good idea to > > touch the hardware when it's off. Add an option to the commit_planes > > helper to support

[Intel-gfx] [PATCH] drm/atomic-helper: Add option to update planes only on active crtc

2015-09-08 Thread Daniel Vetter
With drivers supporting runtime pm it's generally not a good idea to touch the hardware when it's off. Add an option to the commit_planes helper to support this case. Note that the helpers already add all planes on a crtc when a modeset happens, hence plane updates will not be lost if drivers set

Re: [Intel-gfx] [PATCH] drm/atomic-helper: Add option to update planes only on active crtc

2015-09-08 Thread Thierry Reding
On Tue, Sep 08, 2015 at 12:02:07PM +0200, Daniel Vetter wrote: > With drivers supporting runtime pm it's generally not a good idea to > touch the hardware when it's off. Add an option to the commit_planes > helper to support this case. > > Note that the helpers already add all planes on a crtc

[Intel-gfx] [PATCH] drm/atomic-helper: Add option to update planes only on active crtc

2015-08-26 Thread Daniel Vetter
With drivers supporting runtime pm it's generally not a good idea to touch the hardware when it's off. Add an option to the commit_planes helper to support this case. Note that the helpers already add all planes on a crtc when a modeset happens, hence plane updates will not be lost if drivers set