On Thu, Sep 1, 2011 at 11:08 AM, Christoph Bumiller <e0425...@student.tuwien.ac.at> wrote: > On 01.09.2011 17:02, Younes Manton wrote: >> On Thu, Sep 1, 2011 at 10:56 AM, Michel Dänzer <mic...@daenzer.net> wrote: >>> On Don, 2011-09-01 at 15:50 +0200, Christian König wrote: >>>> Start with correctly defining IA44 and AI44 formats. >>>> >>>> Signed-off-by: Christian König <deathsim...@vodafone.de> >>>> --- >>>> src/gallium/auxiliary/util/u_format.csv | 6 +++- >>>> src/gallium/auxiliary/util/u_format_yuv.c | 38 >>>> --------------------------- >>>> src/gallium/auxiliary/util/u_format_yuv.h | 40 >>>> ----------------------------- >>>> 3 files changed, 4 insertions(+), 80 deletions(-) >>>> >>>> diff --git a/src/gallium/auxiliary/util/u_format.csv >>>> b/src/gallium/auxiliary/util/u_format.csv >>>> index a3d2aae..b1a85a5 100644 >>>> --- a/src/gallium/auxiliary/util/u_format.csv >>>> +++ b/src/gallium/auxiliary/util/u_format.csv >>>> @@ -265,5 +265,7 @@ PIPE_FORMAT_YV16 , other, 1, 1, x8 , >>>> x8 , x8 , x8 , xyzw, y >>>> PIPE_FORMAT_IYUV , other, 1, 1, x8 , x8 , x8 , x8 , >>>> xyzw, yuv >>>> PIPE_FORMAT_NV12 , other, 1, 1, x8 , x8 , x8 , x8 , >>>> xyzw, yuv >>>> PIPE_FORMAT_NV21 , other, 1, 1, x8 , x8 , x8 , x8 , >>>> xyzw, yuv >>>> -PIPE_FORMAT_IA44 , other, 1, 1, x8 , x8 , x8 , x8 , >>>> xyzw, yuv >>>> -PIPE_FORMAT_AI44 , other, 1, 1, x8 , x8 , x8 , x8 , >>>> xyzw, yuv >>>> + >>>> +# Indexed with Alpha >>>> +PIPE_FORMAT_IA44 , plain, 1, 1, un4 , un4 , , , >>>> y00x, rgb >>>> +PIPE_FORMAT_AI44 , plain, 1, 1, un4 , un4 , , , >>>> x00y, rgb >>> I think the 'I' stands for 'intensity', not 'indexed'? Same issue in >>> patch 4. >>> >>> I didn't notice any other issues in the patches. >> It's an index actually, 4 bits, into a 16-entry palette. > I'm curious, where is this palette ? I don't see it in the gallium > interface ... > > And if you have to explicitly look it up in the shader, then this is > just R4_UINT_A4_UNORM (or even just R4A4_UNORM). > > For me, "indexed" implies that the texture unit performs the lookup > implicitly.
The palette is currently an explicit lookup in the shader, but it doesn't have to be, depending on what your hardware can do (for both shader decoding and HW decoders). Anyway, we used to alias some of these video formats to existing formats where they matched, but when merging pipe-video to master there was concern among some that it wasn't the right thing to do. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev