Re: [Mesa-dev] [PATCH] clover: fix image resource depth and array_size

2015-08-03 Thread Zoltán Gilián
Fixed it. Committed as aa46fba. On Mon, Aug 3, 2015 at 1:36 PM, Zoltán Gilián wrote: >> Any reason you didn't fix this other branch to do the same? Or maybe >> just init it to one after the if? > > I've only observed problems with images if that line is missing (the > surface is rejected because

Re: [Mesa-dev] [PATCH] clover: fix image resource depth and array_size

2015-08-03 Thread Zoltán Gilián
> Any reason you didn't fix this other branch to do the same? Or maybe > just init it to one after the if? I've only observed problems with images if that line is missing (the surface is rejected because of 0 buffer size), but it makes sense to set the array_size outside the if. On Mon, Aug 3,

Re: [Mesa-dev] [PATCH] clover: fix image resource depth and array_size

2015-08-03 Thread Francisco Jerez
Zoltan Gilian writes: > --- > src/gallium/state_trackers/clover/core/memory.cpp | 2 +- > src/gallium/state_trackers/clover/core/resource.cpp | 1 + > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/gallium/state_trackers/clover/core/memory.cpp > b/src/gallium/state_track

[Mesa-dev] [PATCH] clover: fix image resource depth and array_size

2015-08-03 Thread Zoltan Gilian
--- src/gallium/state_trackers/clover/core/memory.cpp | 2 +- src/gallium/state_trackers/clover/core/resource.cpp | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/core/memory.cpp b/src/gallium/state_trackers/clover/core/memory.cpp index 0553