From 2985056340b04c81df6afcf841dd541c94204817 Mon Sep 17 00:00:00 2001 From: Subv Date: Wed, 18 Apr 2018 18:11:14 -0500 Subject: GPU: Implemented the B5G6R5 format. --- src/video_core/renderer_opengl/gl_rasterizer_cache.cpp | 15 ++++++++++----- src/video_core/renderer_opengl/gl_rasterizer_cache.h | 16 +++++++++++++--- src/video_core/textures/decoders.cpp | 4 ++++ src/video_core/textures/texture.h | 1 + 4 files changed, 28 insertions(+), 8 deletions(-) diff --git a/src/video_core/renderer_opengl/gl_rasterizer_cache.cpp b/src/video_core/renderer_opengl/gl_rasterizer_cache.cpp index 9ccc63090..2a0858eac 100644 --- a/src/video_core/renderer_opengl/gl_rasterizer_cache.cpp +++ b/src/video_core/renderer_opengl/gl_rasterizer_cache.cpp @@ -48,8 +48,9 @@ struct FormatTuple { u32 compression_factor; }; -static constexpr std::array tex_format_tuples = {{ +static constexpr std::array tex_format_tuples = {{ {GL_RGBA8, GL_RGBA, GL_UNSIGNED_INT_8_8_8_8_REV, false, 1}, // ABGR8 + {GL_RGB, GL_RGB, GL_UNSIGNED_SHORT_5_6_5_REV, false, 1}, // B5G6R5 {GL_COMPRESSED_RGB_S3TC_DXT1_EXT, GL_RGB, GL_UNSIGNED_INT_8_8_8_8, true, 16}, // DXT1 }}; @@ -117,15 +118,19 @@ void MortonCopy(u32 stride, u32 block_height, u32 height, u8* gl_buffer, VAddr b } } -static constexpr std::array morton_to_gl_fns = - { +static constexpr std::array + morton_to_gl_fns = { MortonCopy, + MortonCopy, MortonCopy, }; -static constexpr std::array gl_to_morton_fns = - { +static constexpr std::array + gl_to_morton_fns = { MortonCopy, + MortonCopy, // TODO(Subv): Swizzling the DXT1 format is not yet supported nullptr, }; diff --git a/src/video_core/renderer_opengl/gl_rasterizer_cache.h b/src/video_core/renderer_opengl/gl_rasterizer_cache.h index 0ff0ce90f..23e4d02d8 100644 --- a/src/video_core/renderer_opengl/gl_rasterizer_cache.h +++ b/src/video_core/renderer_opengl/gl_rasterizer_cache.h @@ -53,10 +53,15 @@ enum class ScaleMatch { struct SurfaceParams { enum class PixelFormat { ABGR8 = 0, - DXT1 = 1, + B5G6R5 = 1, + DXT1 = 2, + + Max, Invalid = 255, }; + static constexpr size_t MaxPixelFormat = static_cast(PixelFormat::Max); + enum class ComponentType { Invalid = 0, SNorm = 1, @@ -78,8 +83,9 @@ struct SurfaceParams { if (format == PixelFormat::Invalid) return 0; - constexpr std::array bpp_table = { + constexpr std::array bpp_table = { 32, // ABGR8 + 16, // B5G6R5 64, // DXT1 }; @@ -115,6 +121,8 @@ struct SurfaceParams { switch (format) { case Tegra::Texture::TextureFormat::A8R8G8B8: return PixelFormat::ABGR8; + case Tegra::Texture::TextureFormat::B5G6R5: + return PixelFormat::B5G6R5; case Tegra::Texture::TextureFormat::DXT1: return PixelFormat::DXT1; default: @@ -128,6 +136,8 @@ struct SurfaceParams { switch (format) { case PixelFormat::ABGR8: return Tegra::Texture::TextureFormat::A8R8G8B8; + case PixelFormat::B5G6R5: + return Tegra::Texture::TextureFormat::B5G6R5; case PixelFormat::DXT1: return Tegra::Texture::TextureFormat::DXT1; default: @@ -189,7 +199,7 @@ struct SurfaceParams { } static SurfaceType GetFormatType(PixelFormat pixel_format) { - if ((unsigned int)pixel_format <= static_cast(PixelFormat::DXT1)) { + if (static_cast(pixel_format) < MaxPixelFormat) { return SurfaceType::ColorTexture; } diff --git a/src/video_core/textures/decoders.cpp b/src/video_core/textures/decoders.cpp index 9c2a10d2e..f4c7e40df 100644 --- a/src/video_core/textures/decoders.cpp +++ b/src/video_core/textures/decoders.cpp @@ -50,6 +50,8 @@ u32 BytesPerPixel(TextureFormat format) { return 8; case TextureFormat::A8R8G8B8: return 4; + case TextureFormat::B5G6R5: + return 2; default: UNIMPLEMENTED_MSG("Format not implemented"); break; @@ -70,6 +72,7 @@ std::vector UnswizzleTexture(VAddr address, TextureFormat format, u32 width, unswizzled_data.data(), true, block_height); break; case TextureFormat::A8R8G8B8: + case TextureFormat::B5G6R5: CopySwizzledData(width, height, bytes_per_pixel, bytes_per_pixel, data, unswizzled_data.data(), true, block_height); break; @@ -89,6 +92,7 @@ std::vector DecodeTexture(const std::vector& texture_data, TextureFormat switch (format) { case TextureFormat::DXT1: case TextureFormat::A8R8G8B8: + case TextureFormat::B5G6R5: // TODO(Subv): For the time being just forward the same data without any decoding. rgba_data = texture_data; break; diff --git a/src/video_core/textures/texture.h b/src/video_core/textures/texture.h index 09d2317e0..86e45aa88 100644 --- a/src/video_core/textures/texture.h +++ b/src/video_core/textures/texture.h @@ -15,6 +15,7 @@ namespace Texture { enum class TextureFormat : u32 { A8R8G8B8 = 0x8, + B5G6R5 = 0x15, DXT1 = 0x24, DXT23 = 0x25, DXT45 = 0x26, -- cgit v1.2.3