On Tue, Nov 7, 2017 at 5:43 PM, Wladimir J. van der Laan <laa...@gmail.com> wrote: > Sampler TS introduces yet another format enumeration for renderable > formats. Introduce it into etnaviv_format as unobtrusively as possible. > > Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com> > --- > src/gallium/drivers/etnaviv/etnaviv_format.c | 19 +++++++++++++++++++ > src/gallium/drivers/etnaviv/etnaviv_format.h | 3 +++ > 2 files changed, 22 insertions(+)
I think I found a potential problem with this approach of mapping name-to-name. Though most of the RS formats map immediately to TS formats, there are some exceptions, like the depth formats: _T(Z16_UNORM, D16, SWIZ(X, Y, Z, W), A4R4G4B4), _T(X8Z24_UNORM, D24X8, SWIZ(X, Y, Z, W), A8R8G8B8), _T(S8_UINT_Z24_UNORM, D24X8, SWIZ(X, Y, Z, W), A8R8G8B8), For practical reasons these have been assigned RS formats that happen to have the same size, which is enough for RS blit. Sampler TS on the other hand has dedicated formats for depth: TS_SAMPLER_FORMAT_D24X8 TS_SAMPLER_FORMAT_D16 Not sure how much difference this makes, though... Depth texturing (as in glmark shadow benchmark) seems to just work. Wladimir _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev