Hi,
On 18 July 2017 at 21:30, Kenneth Graunke <kenn...@whitecape.org> wrote:
> On Tuesday, July 18, 2017 1:46:13 AM PDT Topi Pohjolainen wrote:
> [snip]
>> diff --git a/src/mesa/drivers/dri/i965/intel_blit.h
>> b/src/mesa/drivers/dri/i965/intel_blit.h
>> index 5e4d1f5eb4..90514dc893 100644
>> --- a/src/mesa/drivers/dri/i965/intel_blit.h
>> +++ b/src/mesa/drivers/dri/i965/intel_blit.h
>> @@ -41,17 +41,32 @@ isl_tiling_to_bufmgr_tiling(enum isl_tiling tiling)
>> return I915_TILING_NONE;
>> }
>>
>> +static inline enum isl_tiling
>> +bufmgr_tiling_to_isl_tiling(unsigned tiling)
>> +{
>> + switch (tiling) {
>> + case I915_TILING_NONE:
>> + return ISL_TILING_LINEAR;
>> + case I915_TILING_X:
>> + return ISL_TILING_X;
>> + case I915_TILING_Y:
>> + return ISL_TILING_Y0;
>> + }
>> +
>> + unreachable("Invalid tiling mode");
>> +}
>> +
>
> This doesn't appear to be used at this point. If it's used later, great,
> but I'm not sure why you'd need a converter in this direction?
It also belongs aside isl_tiling_to_i915_tiling() inside isl itself;
that function can itself replace isl_tiling_to_bufmgr_tiling().
Cheers,
Daniel
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
