Re: [Intel-gfx] [PATCH i-g-t 1/2] kms_pipe_crc_basic: Only test one pipe in the basic set

2016-04-26 Thread Daniel Vetter
On Mon, Apr 25, 2016 at 08:12:41PM +, Zanoni, Paulo R wrote: > One of the things about our driver is that it tries to be very very > conservative regarding the eDP timings, leading to waits longer than > necessary for pretty much everyone. We tried to optimize this in the > past, but due to

Re: [Intel-gfx] [PATCH i-g-t 1/2] kms_pipe_crc_basic: Only test one pipe in the basic set

2016-04-25 Thread Zanoni, Paulo R
Em Seg, 2016-04-25 às 10:45 +0100, Tvrtko Ursulin escreveu: > On 25/04/16 09:51, Daniel Vetter wrote: > > > > On Fri, Apr 22, 2016 at 04:11:11PM +0100, Chris Wilson wrote: > > > > > > On Fri, Apr 22, 2016 at 03:41:55PM +0100, Tvrtko Ursulin wrote: > > > > > > > > From: Tvrtko Ursulin

Re: [Intel-gfx] [PATCH i-g-t 1/2] kms_pipe_crc_basic: Only test one pipe in the basic set

2016-04-25 Thread Tvrtko Ursulin
On 25/04/16 09:51, Daniel Vetter wrote: On Fri, Apr 22, 2016 at 04:11:11PM +0100, Chris Wilson wrote: On Fri, Apr 22, 2016 at 03:41:55PM +0100, Tvrtko Ursulin wrote: From: Tvrtko Ursulin I would argue it is enough to test one pipe in the BAT set. Signed-off-by:

Re: [Intel-gfx] [PATCH i-g-t 1/2] kms_pipe_crc_basic: Only test one pipe in the basic set

2016-04-25 Thread Daniel Vetter
On Fri, Apr 22, 2016 at 04:11:11PM +0100, Chris Wilson wrote: > On Fri, Apr 22, 2016 at 03:41:55PM +0100, Tvrtko Ursulin wrote: > > From: Tvrtko Ursulin > > > > I would argue it is enough to test one pipe in the BAT set. > > > > Signed-off-by: Tvrtko Ursulin

Re: [Intel-gfx] [PATCH i-g-t 1/2] kms_pipe_crc_basic: Only test one pipe in the basic set

2016-04-25 Thread Jani Nikula
On Fri, 22 Apr 2016, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > I would argue it is enough to test one pipe in the BAT set. FWIW BYT/CHV DSI may be fixed to port C which is fixed to pipe B, skipping the test for pipe A. BR, Jani.

Re: [Intel-gfx] [PATCH i-g-t 1/2] kms_pipe_crc_basic: Only test one pipe in the basic set

2016-04-22 Thread Chris Wilson
On Fri, Apr 22, 2016 at 03:41:55PM +0100, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > I would argue it is enough to test one pipe in the BAT set. > > Signed-off-by: Tvrtko Ursulin > --- > tests/kms_pipe_crc_basic.c | 23

[Intel-gfx] [PATCH i-g-t 1/2] kms_pipe_crc_basic: Only test one pipe in the basic set

2016-04-22 Thread Tvrtko Ursulin
From: Tvrtko Ursulin I would argue it is enough to test one pipe in the BAT set. Signed-off-by: Tvrtko Ursulin --- tests/kms_pipe_crc_basic.c | 23 +-- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git