Displaying 3 results from an estimated 3 matches for "pipe_format_b".
Did you mean:
pipe_format_
2015 Jan 02
2
[PATCH v2] nv50: enable texture compression
...boolean compressed)
tile_flags = 0x60 + ms;
break;
default:
+ switch (mt->base.base.format) {
+ case PIPE_FORMAT_R16G16B16A16_FLOAT:
+ case PIPE_FORMAT_R16G16B16X16_FLOAT:
+ case PIPE_FORMAT_R8G8B8A8_UNORM:
+ case PIPE_FORMAT_R8G8B8A8_SRGB:
+ case PIPE_FORMAT_B8G8R8A8_UNORM:
+ case PIPE_FORMAT_B8G8R8A8_SRGB:
+ case PIPE_FORMAT_R10G10B10A2_UNORM:
+ case PIPE_FORMAT_B10G10R10A2_UNORM:
+ case PIPE_FORMAT_R11G11B10_FLOAT:
+ case PIPE_FORMAT_B8G8R8X8_UNORM:
+ case PIPE_FORMAT_B8G8R8X8_SRGB:
+ case PIPE_FORMAT_R8G8B8X8_UNORM:...
2015 Jan 02
0
[PATCH v2] nv50: enable texture compression
...e *mt, boolean compressed)
> tile_flags = 0x60 + ms;
> break;
> default:
> + switch (mt->base.base.format) {
You're already switching on the format. You could be all clever and do
default:
compressed = false;
/* fallthrough */
case PIPE_FORMAT_A:
case PIPE_FORMAT_B:
switch (blocksize) { ... }
It's a bit unusual to have the default case in the middle, but I'm
pretty sure it works.
BTW, I assume that there was no dmesg spam or piglit regressions with
this change?
-ilia
2015 Jan 02
1
[PATCH v2] nv50: enable texture compression
...= 0x60 + ms;
>> break;
>> default:
>> + switch (mt->base.base.format) {
> You're already switching on the format. You could be all clever and do
>
> default:
> compressed = false;
> /* fallthrough */
> case PIPE_FORMAT_A:
> case PIPE_FORMAT_B:
> switch (blocksize) { ... }
>
> It's a bit unusual to have the default case in the middle, but I'm
> pretty sure it works.
We talked about broken compilers, hope we don't hit one here :D
>
> BTW, I assume that there was no dmesg spam or piglit regressions with
&g...