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

2015-08-30 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 7266 -Summary- Platform Delta drm-intel-nightly Series Applied ILK -1

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

2015-08-30 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 7264 -Summary- Platform Delta drm-intel-nightly Series Applied ILK -2

Re: [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 pro

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

2015-08-26 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 sin

Re: [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 t

[Intel-gfx] [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 user

Re: [Intel-gfx] [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 wrote:

Re: [Intel-gfx] [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 > >

Re: [Intel-gfx] [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 >> > wrote: >> >> Very strictly speaking this is possible if you have sp

Re: [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 well a

Re: [Intel-gfx] [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 plan

Re: [Intel-gfx] [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

Re: [Intel-gfx] [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 shoul

Re: [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 th

[Intel-gfx] [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 don't