Re: [Mesa-dev] [PATCH 02/27] i965/miptree: Remove redundant check for null texture

2017-01-17 Thread Jason Ekstrand
On Tue, Jan 17, 2017 at 12:21 AM, Pohjolainen, Topi < topi.pohjolai...@gmail.com> wrote: > On Mon, Jan 16, 2017 at 08:33:09AM -0800, Jason Ekstrand wrote: > >On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen > ><[1]topi.pohjolai...@gmail.com> wrote: > > > > There exact same check earl

Re: [Mesa-dev] [PATCH 02/27] i965/miptree: Remove redundant check for null texture

2017-01-17 Thread Pohjolainen, Topi
On Mon, Jan 16, 2017 at 08:33:09AM -0800, Jason Ekstrand wrote: >On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen ><[1]topi.pohjolai...@gmail.com> wrote: > > There exact same check earlier in brw_miptree_layout() which > intel_miptree_create_layout() in turn calls unconditionall

Re: [Mesa-dev] [PATCH 02/27] i965/miptree: Remove redundant check for null texture

2017-01-16 Thread Jason Ekstrand
On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen < topi.pohjolai...@gmail.com> wrote: > There exact same check earlier in brw_miptree_layout() which > intel_miptree_create_layout() in turn calls unconditionally. > > Signed-off-by: Topi Pohjolainen > --- > src/mesa/drivers/dri/i965/intel_mipmap_

Re: [Mesa-dev] [PATCH 02/27] i965/miptree: Remove redundant check for null texture

2017-01-16 Thread Samuel Iglesias Gonsálvez
Reviewed-by: Samuel Iglesias Gonsálvez On Mon, 2017-01-16 at 11:13 +0200, Topi Pohjolainen wrote: > There exact same check earlier in brw_miptree_layout() which > intel_miptree_create_layout() in turn calls unconditionally. > > Signed-off-by: Topi Pohjolainen > --- >  src/mesa/drivers/dri/i965/

[Mesa-dev] [PATCH 02/27] i965/miptree: Remove redundant check for null texture

2017-01-16 Thread Topi Pohjolainen
There exact same check earlier in brw_miptree_layout() which intel_miptree_create_layout() in turn calls unconditionally. Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/mesa/drivers/d