Displaying 3 results from an estimated 3 matches for "pipe_format_a".
Did you mean:
pipe_format_
2015 Jan 02
2
[PATCH v2] nv50: enable texture compression
We enable compression only for some supported formats
Suggested-by: Ilia Mirkin <imirkin at alum.mit.edu>
Signed-off-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>
---
src/gallium/drivers/nouveau/nv50/nv50_miptree.c | 23 +++++++++++++++++++++--
src/gallium/drivers/nouveau/nv50/nv50_screen.c | 7 +++++++
2 files changed, 28 insertions(+), 2 deletions(-)
diff --git
2015 Jan 02
0
[PATCH v2] nv50: enable texture compression
...e(struct nv50_miptree *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
...t;> 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.
We talked about broken compilers, hope we don't hit one here :D
>
> BTW, I assume that there was no dmesg spam or p...