[Intel-gfx] [PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-27 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 05:51:46PM -0400, Rob Clark wrote: > On Wed, Aug 26, 2015 at 3:49 PM, Daniel Vetter > wrote: > > Very strictly speaking this is possible if you have special hw and > > genlocked CRTCs. In general switching a plane between two active CRTC > > just won't work so well and is

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-27 Thread Maarten Lankhorst
Op 26-08-15 om 21:49 schreef Daniel Vetter: > Very strictly speaking this is possible if you have special hw and > genlocked CRTCs. In general switching a plane between two active CRTC > just won't work so well and is probably not tested at all. Just forbid > it. > > I've put this into the core

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Daniel Vetter
Very strictly speaking this is possible if you have special hw and genlocked CRTCs. In general switching a plane between two active CRTC just won't work so well and is probably not tested at all. Just forbid it. I've put this into the core since right now no helper or driver copes with it, no

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Ville Syrjälä
On Wed, Aug 26, 2015 at 01:38:36PM -0400, Rob Clark wrote: > On Wed, Aug 26, 2015 at 12:30 PM, Ville Syrjälä > wrote: > > On Wed, Aug 26, 2015 at 12:07:30PM -0400, Rob Clark wrote: > >> On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark wrote: > >> > On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Ville Syrjälä
On Wed, Aug 26, 2015 at 12:07:30PM -0400, Rob Clark wrote: > On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark wrote: > > On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter > > wrote: > >> Very strictly speaking this is possible if you have special hw and > >> genlocked CRTCs. In general switching a

[Intel-gfx] [PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Ville Syrjälä
On Wed, Aug 26, 2015 at 05:41:23PM +0200, Daniel Vetter wrote: > Very strictly speaking this is possible if you have special hw and > genlocked CRTCs. In general switching a plane between two active CRTC > just won't work so well and is probably not tested at all. Just forbid > it. > > I've put

[Intel-gfx] [PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 3:49 PM, Daniel Vetter wrote: > Very strictly speaking this is possible if you have special hw and > genlocked CRTCs. In general switching a plane between two active CRTC > just won't work so well and is probably not tested at all. Just forbid > it. > > I've put this into

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Daniel Vetter
Very strictly speaking this is possible if you have special hw and genlocked CRTCs. In general switching a plane between two active CRTC just won't work so well and is probably not tested at all. Just forbid it. I've put this into the core since I really couldn't come up with a case where we

[Intel-gfx] [PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Daniel Stone
On Wed, 2015-08-26 at 18:53 +0300, Ville Syrjälä wrote: > On Wed, Aug 26, 2015 at 05:41:23PM +0200, Daniel Vetter wrote: > > Very strictly speaking this is possible if you have special hw and > > genlocked CRTCs. In general switching a plane between two active > > CRTC > > just won't work so

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 1:53 PM, Ville Syrjälä wrote: > On Wed, Aug 26, 2015 at 01:38:36PM -0400, Rob Clark wrote: >> On Wed, Aug 26, 2015 at 12:30 PM, Ville Syrjälä >> wrote: >> > On Wed, Aug 26, 2015 at 12:07:30PM -0400, Rob Clark wrote: >> >> On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 12:30 PM, Ville Syrjälä wrote: > On Wed, Aug 26, 2015 at 12:07:30PM -0400, Rob Clark wrote: >> On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark wrote: >> > On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter > > ffwll.ch> wrote: >> >> Very strictly speaking this is possible if

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark wrote: > On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter > wrote: >> Very strictly speaking this is possible if you have special hw and >> genlocked CRTCs. In general switching a plane between two active CRTC >> just won't work so well and is probably

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter wrote: > Very strictly speaking this is possible if you have special hw and > genlocked CRTCs. In general switching a plane between two active CRTC > just won't work so well and is probably not tested at all. Just forbid > it. So, I expect msm