Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-27 Thread Paneri, Praveen
- From: Tvrtko Ursulin [mailto:tvrtko.ursu...@linux.intel.com] Sent: Tuesday, October 25, 2016 1:36 PM To: Paneri, Praveen ; intel-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support On 24/10/2016 17:55, Praveen Paneri wrote: > This adds Y-tiling check in igt_crea

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-26 Thread Daniel Vetter
On Wed, Oct 26, 2016 at 09:09:26AM +0100, Tvrtko Ursulin wrote: > > On 26/10/2016 07:32, Daniel Vetter wrote: > > On Tue, Oct 25, 2016 at 03:02:33PM +, Paneri, Praveen wrote: > > > > So when you say that all Y tiling tests fail without this kernel hack, > > > > which tests you are referring t

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-26 Thread Ville Syrjälä
On Wed, Oct 26, 2016 at 11:20:51AM +0300, Ville Syrjälä wrote: > On Wed, Oct 26, 2016 at 09:09:26AM +0100, Tvrtko Ursulin wrote: > > > > On 26/10/2016 07:32, Daniel Vetter wrote: > > > On Tue, Oct 25, 2016 at 03:02:33PM +, Paneri, Praveen wrote: > > >>> So when you say that all Y tiling tests

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-26 Thread Ville Syrjälä
On Wed, Oct 26, 2016 at 09:09:26AM +0100, Tvrtko Ursulin wrote: > > On 26/10/2016 07:32, Daniel Vetter wrote: > > On Tue, Oct 25, 2016 at 03:02:33PM +, Paneri, Praveen wrote: > >>> So when you say that all Y tiling tests fail without this kernel hack, > >>> which tests you are referring to? >

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-26 Thread Tvrtko Ursulin
On 26/10/2016 07:32, Daniel Vetter wrote: On Tue, Oct 25, 2016 at 03:02:33PM +, Paneri, Praveen wrote: So when you say that all Y tiling tests fail without this kernel hack, which tests you are referring to? If I revert this IGT patch and do not make below kernel change, kms_draw_crc (yt

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-25 Thread Daniel Vetter
ntel.com] > Sent: Tuesday, October 25, 2016 6:07 PM > To: Paneri, Praveen ; > intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support > > > On 25/10/2016 13:06, Paneri, Praveen wrote: > > Hi Tvrtko, > > > > Along w

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-25 Thread Paneri, Praveen
rtko Ursulin [mailto:tvrtko.ursu...@linux.intel.com] Sent: Tuesday, October 25, 2016 6:07 PM To: Paneri, Praveen ; intel-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support On 25/10/2016 13:06, Paneri, Praveen wrote: > Hi Tvrtko, > > Along with this ch

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-25 Thread Tvrtko Ursulin
referring to? Regards, Tvrtko Regards, Praveen -Original Message- From: Tvrtko Ursulin [mailto:tvrtko.ursu...@linux.intel.com] Sent: Tuesday, October 25, 2016 1:36 PM To: Paneri, Praveen ; intel-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling suppor

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-25 Thread Chris Wilson
On Tue, Oct 25, 2016 at 09:20:31AM +0100, Tvrtko Ursulin wrote: > > > On 25/10/2016 09:18, Chris Wilson wrote: > >On Tue, Oct 25, 2016 at 09:06:26AM +0100, Tvrtko Ursulin wrote: > >> > >>On 24/10/2016 17:55, Praveen Paneri wrote: > >>>This adds Y-tiling check in igt_create_fb_with_bo_size as > >>

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-25 Thread Tvrtko Ursulin
On 25/10/2016 09:18, Chris Wilson wrote: On Tue, Oct 25, 2016 at 09:06:26AM +0100, Tvrtko Ursulin wrote: On 24/10/2016 17:55, Praveen Paneri wrote: This adds Y-tiling check in igt_create_fb_with_bo_size as now we should also be able to create Y-tiled FBs. Signed-off-by: Praveen Paneri ---

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-25 Thread Chris Wilson
On Tue, Oct 25, 2016 at 09:06:26AM +0100, Tvrtko Ursulin wrote: > > On 24/10/2016 17:55, Praveen Paneri wrote: > >This adds Y-tiling check in igt_create_fb_with_bo_size as > >now we should also be able to create Y-tiled FBs. > > > >Signed-off-by: Praveen Paneri > >--- > > lib/igt_fb.c | 3 ++- > >

Re: [Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-25 Thread Tvrtko Ursulin
On 24/10/2016 17:55, Praveen Paneri wrote: This adds Y-tiling check in igt_create_fb_with_bo_size as now we should also be able to create Y-tiled FBs. Signed-off-by: Praveen Paneri --- lib/igt_fb.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/igt_fb.c b/lib/igt_fb

[Intel-gfx] [PATCH 1/4] igt_fb: Add Y-tiling support

2016-10-24 Thread Praveen Paneri
This adds Y-tiling check in igt_create_fb_with_bo_size as now we should also be able to create Y-tiled FBs. Signed-off-by: Praveen Paneri --- lib/igt_fb.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/igt_fb.c b/lib/igt_fb.c index 47472f4..bf1d372 100644 --- a/lib/igt