Re: [Mesa-dev] [PATCH 4/8] intel: prepare for dri images having more than plane

2013-04-23 Thread Pohjolainen, Topi
On Mon, Apr 22, 2013 at 10:06:19AM -0700, Eric Anholt wrote: > "Pohjolainen, Topi" writes: > > > On Tue, Apr 16, 2013 at 04:57:49PM -0700, Eric Anholt wrote: > >> Topi Pohjolainen writes: > >> > + if (image->regions[i]->bo) > >> > + intel_region_release(&image->regions[i]); > >> >

Re: [Mesa-dev] [PATCH 4/8] intel: prepare for dri images having more than plane

2013-04-22 Thread Eric Anholt
"Pohjolainen, Topi" writes: > On Tue, Apr 16, 2013 at 04:57:49PM -0700, Eric Anholt wrote: >> Topi Pohjolainen writes: >> > + if (image->regions[i]->bo) >> > + intel_region_release(&image->regions[i]); >> >> Why are you looking at the BO for deciding whether to free the region? >>

Re: [Mesa-dev] [PATCH 4/8] intel: prepare for dri images having more than plane

2013-04-19 Thread Pohjolainen, Topi
On Tue, Apr 16, 2013 at 04:57:49PM -0700, Eric Anholt wrote: > Topi Pohjolainen writes: > > > Signed-off-by: Topi Pohjolainen > > --- > > src/mesa/drivers/dri/intel/intel_screen.c | 101 > > +- > > 1 file changed, 71 insertions(+), 30 deletions(-) > > > > diff --git

Re: [Mesa-dev] [PATCH 4/8] intel: prepare for dri images having more than plane

2013-04-16 Thread Eric Anholt
Topi Pohjolainen writes: > Signed-off-by: Topi Pohjolainen > --- > src/mesa/drivers/dri/intel/intel_screen.c | 101 > +- > 1 file changed, 71 insertions(+), 30 deletions(-) > > diff --git a/src/mesa/drivers/dri/intel/intel_screen.c > b/src/mesa/drivers/dri/intel/in

[Mesa-dev] [PATCH 4/8] intel: prepare for dri images having more than plane

2013-04-16 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/intel/intel_screen.c | 101 +- 1 file changed, 71 insertions(+), 30 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_screen.c b/src/mesa/drivers/dri/intel/intel_screen.c index 3d0344e..8716688 100644