Re: [Intel-gfx] [PATCH] drm/atomic-helper: Check encoder/crtc constraints

2015-11-19 Thread Jani Nikula
On Wed, 18 Nov 2015, Daniel Vetter wrote: > This was totally lost when I originally created the atomic helpers. > > We probably should also check possible_clones in the helpers, but > since the legacy ones didn't do that this is for a separate patch. > > Reported-by: Ville Syrjälä > Cc: Ville Syr

Re: [Intel-gfx] [PATCH] drm/atomic-helper: Check encoder/crtc constraints

2015-11-19 Thread Daniel Vetter
On Thu, Nov 19, 2015 at 3:24 PM, Ville Syrjälä wrote: > On Thu, Nov 19, 2015 at 03:02:09PM +0100, Daniel Vetter wrote: >> On Thu, Nov 19, 2015 at 12:12:28PM +0200, Ville Syrjälä wrote: >> > On Wed, Nov 18, 2015 at 06:46:48PM +0100, Daniel Vetter wrote: >> > > This was totally lost when I originall

Re: [Intel-gfx] [PATCH] drm/atomic-helper: Check encoder/crtc constraints

2015-11-19 Thread Ville Syrjälä
On Thu, Nov 19, 2015 at 03:02:09PM +0100, Daniel Vetter wrote: > On Thu, Nov 19, 2015 at 12:12:28PM +0200, Ville Syrjälä wrote: > > On Wed, Nov 18, 2015 at 06:46:48PM +0100, Daniel Vetter wrote: > > > This was totally lost when I originally created the atomic helpers. > > > > > > We probably shoul

Re: [Intel-gfx] [PATCH] drm/atomic-helper: Check encoder/crtc constraints

2015-11-19 Thread Daniel Vetter
On Thu, Nov 19, 2015 at 12:12:28PM +0200, Ville Syrjälä wrote: > On Wed, Nov 18, 2015 at 06:46:48PM +0100, Daniel Vetter wrote: > > This was totally lost when I originally created the atomic helpers. > > > > We probably should also check possible_clones in the helpers, but > > since the legacy one

Re: [Intel-gfx] [PATCH] drm/atomic-helper: Check encoder/crtc constraints

2015-11-19 Thread Ville Syrjälä
On Wed, Nov 18, 2015 at 06:46:48PM +0100, Daniel Vetter wrote: > This was totally lost when I originally created the atomic helpers. > > We probably should also check possible_clones in the helpers, but > since the legacy ones didn't do that this is for a separate patch. > > Reported-by: Ville Sy

Re: [Intel-gfx] [PATCH] drm/atomic-helper: Check encoder/crtc constraints

2015-11-19 Thread Daniel Stone
Hi, On 18 November 2015 at 17:46, Daniel Vetter wrote: > This was totally lost when I originally created the atomic helpers. > > We probably should also check possible_clones in the helpers, but > since the legacy ones didn't do that this is for a separate patch. Heh, before reading this chunk o

[Intel-gfx] [PATCH] drm/atomic-helper: Check encoder/crtc constraints

2015-11-18 Thread Daniel Vetter
This was totally lost when I originally created the atomic helpers. We probably should also check possible_clones in the helpers, but since the legacy ones didn't do that this is for a separate patch. Reported-by: Ville Syrjälä Cc: Ville Syrjälä Cc: Daniel Stone Signed-off-by: Daniel Vetter -