Merge pull request #2686 from wwylele/tex-coord-reg
pica: use correct coordinates for texture 2
This commit is contained in:
commit
2a01a03375
|
@ -93,7 +93,7 @@ ASSERT_REG_POSITION(rasterizer.viewport_corner, 0x68);
|
||||||
ASSERT_REG_POSITION(rasterizer.depthmap_enable, 0x6D);
|
ASSERT_REG_POSITION(rasterizer.depthmap_enable, 0x6D);
|
||||||
|
|
||||||
ASSERT_REG_POSITION(texturing, 0x80);
|
ASSERT_REG_POSITION(texturing, 0x80);
|
||||||
ASSERT_REG_POSITION(texturing.texture0_enable, 0x80);
|
ASSERT_REG_POSITION(texturing.main_config, 0x80);
|
||||||
ASSERT_REG_POSITION(texturing.texture0, 0x81);
|
ASSERT_REG_POSITION(texturing.texture0, 0x81);
|
||||||
ASSERT_REG_POSITION(texturing.texture0_format, 0x8e);
|
ASSERT_REG_POSITION(texturing.texture0_format, 0x8e);
|
||||||
ASSERT_REG_POSITION(texturing.fragment_lighting_enable, 0x8f);
|
ASSERT_REG_POSITION(texturing.fragment_lighting_enable, 0x8f);
|
||||||
|
|
|
@ -122,7 +122,11 @@ struct TexturingRegs {
|
||||||
BitField<0, 1, u32> texture0_enable;
|
BitField<0, 1, u32> texture0_enable;
|
||||||
BitField<1, 1, u32> texture1_enable;
|
BitField<1, 1, u32> texture1_enable;
|
||||||
BitField<2, 1, u32> texture2_enable;
|
BitField<2, 1, u32> texture2_enable;
|
||||||
};
|
BitField<8, 2, u32> texture3_coordinates; // TODO: unimplemented
|
||||||
|
BitField<10, 1, u32> texture3_enable; // TODO: unimplemented
|
||||||
|
BitField<13, 1, u32> texture2_use_coord1;
|
||||||
|
BitField<16, 1, u32> clear_texture_cache; // TODO: unimplemented
|
||||||
|
} main_config;
|
||||||
TextureConfig texture0;
|
TextureConfig texture0;
|
||||||
INSERT_PADDING_WORDS(0x8);
|
INSERT_PADDING_WORDS(0x8);
|
||||||
BitField<0, 4, TextureFormat> texture0_format;
|
BitField<0, 4, TextureFormat> texture0_format;
|
||||||
|
@ -142,9 +146,9 @@ struct TexturingRegs {
|
||||||
};
|
};
|
||||||
const std::array<FullTextureConfig, 3> GetTextures() const {
|
const std::array<FullTextureConfig, 3> GetTextures() const {
|
||||||
return {{
|
return {{
|
||||||
{texture0_enable.ToBool(), texture0, texture0_format},
|
{main_config.texture0_enable.ToBool(), texture0, texture0_format},
|
||||||
{texture1_enable.ToBool(), texture1, texture1_format},
|
{main_config.texture1_enable.ToBool(), texture1, texture1_format},
|
||||||
{texture2_enable.ToBool(), texture2, texture2_format},
|
{main_config.texture2_enable.ToBool(), texture2, texture2_format},
|
||||||
}};
|
}};
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -402,6 +402,10 @@ void RasterizerOpenGL::NotifyPicaRegisterChanged(u32 id) {
|
||||||
SyncLogicOp();
|
SyncLogicOp();
|
||||||
break;
|
break;
|
||||||
|
|
||||||
|
case PICA_REG_INDEX(texturing.main_config):
|
||||||
|
shader_dirty = true;
|
||||||
|
break;
|
||||||
|
|
||||||
// Texture 0 type
|
// Texture 0 type
|
||||||
case PICA_REG_INDEX(texturing.texture0.type):
|
case PICA_REG_INDEX(texturing.texture0.type):
|
||||||
shader_dirty = true;
|
shader_dirty = true;
|
||||||
|
|
|
@ -40,6 +40,8 @@ PicaShaderConfig PicaShaderConfig::BuildFromRegs(const Pica::Regs& regs) {
|
||||||
|
|
||||||
state.texture0_type = regs.texturing.texture0.type;
|
state.texture0_type = regs.texturing.texture0.type;
|
||||||
|
|
||||||
|
state.texture2_use_coord1 = regs.texturing.main_config.texture2_use_coord1 != 0;
|
||||||
|
|
||||||
// Copy relevant tev stages fields.
|
// Copy relevant tev stages fields.
|
||||||
// We don't sync const_color here because of the high variance, it is a
|
// We don't sync const_color here because of the high variance, it is a
|
||||||
// shader uniform instead.
|
// shader uniform instead.
|
||||||
|
@ -126,6 +128,15 @@ static bool IsPassThroughTevStage(const TevStageConfig& stage) {
|
||||||
stage.GetColorMultiplier() == 1 && stage.GetAlphaMultiplier() == 1);
|
stage.GetColorMultiplier() == 1 && stage.GetAlphaMultiplier() == 1);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static std::string TexCoord(const PicaShaderConfig& config, int texture_unit) {
|
||||||
|
if (texture_unit == 2 && config.state.texture2_use_coord1) {
|
||||||
|
return "texcoord[1]";
|
||||||
|
}
|
||||||
|
// TODO: if texture unit 3 (procedural texture) implementation also uses this function,
|
||||||
|
// config.state.texture3_coordinates should be repected here.
|
||||||
|
return "texcoord[" + std::to_string(texture_unit) + "]";
|
||||||
|
}
|
||||||
|
|
||||||
/// Writes the specified TEV stage source component(s)
|
/// Writes the specified TEV stage source component(s)
|
||||||
static void AppendSource(std::string& out, const PicaShaderConfig& config,
|
static void AppendSource(std::string& out, const PicaShaderConfig& config,
|
||||||
TevStageConfig::Source source, const std::string& index_name) {
|
TevStageConfig::Source source, const std::string& index_name) {
|
||||||
|
@ -162,7 +173,7 @@ static void AppendSource(std::string& out, const PicaShaderConfig& config,
|
||||||
out += "texture(tex[1], texcoord[1])";
|
out += "texture(tex[1], texcoord[1])";
|
||||||
break;
|
break;
|
||||||
case Source::Texture2:
|
case Source::Texture2:
|
||||||
out += "texture(tex[2], texcoord[2])";
|
out += "texture(tex[2], " + TexCoord(config, 2) + ")";
|
||||||
break;
|
break;
|
||||||
case Source::PreviousBuffer:
|
case Source::PreviousBuffer:
|
||||||
out += "combiner_buffer";
|
out += "combiner_buffer";
|
||||||
|
@ -473,8 +484,8 @@ static void WriteLighting(std::string& out, const PicaShaderConfig& config) {
|
||||||
// Bump mapping is enabled using a normal map, read perturbation vector from the selected
|
// Bump mapping is enabled using a normal map, read perturbation vector from the selected
|
||||||
// texture
|
// texture
|
||||||
std::string bump_selector = std::to_string(lighting.bump_selector);
|
std::string bump_selector = std::to_string(lighting.bump_selector);
|
||||||
out += "vec3 surface_normal = 2.0 * texture(tex[" + bump_selector + "], texcoord[" +
|
out += "vec3 surface_normal = 2.0 * texture(tex[" + bump_selector + "], " +
|
||||||
bump_selector + "]).rgb - 1.0;\n";
|
TexCoord(config, lighting.bump_selector) + ").rgb - 1.0;\n";
|
||||||
|
|
||||||
// Recompute Z-component of perturbation if 'renorm' is enabled, this provides a higher
|
// Recompute Z-component of perturbation if 'renorm' is enabled, this provides a higher
|
||||||
// precision result
|
// precision result
|
||||||
|
|
|
@ -79,6 +79,7 @@ union PicaShaderConfig {
|
||||||
Pica::FramebufferRegs::CompareFunc alpha_test_func;
|
Pica::FramebufferRegs::CompareFunc alpha_test_func;
|
||||||
Pica::RasterizerRegs::ScissorMode scissor_test_mode;
|
Pica::RasterizerRegs::ScissorMode scissor_test_mode;
|
||||||
Pica::TexturingRegs::TextureConfig::TextureType texture0_type;
|
Pica::TexturingRegs::TextureConfig::TextureType texture0_type;
|
||||||
|
bool texture2_use_coord1;
|
||||||
std::array<TevStageConfigRaw, 6> tev_stages;
|
std::array<TevStageConfigRaw, 6> tev_stages;
|
||||||
u8 combiner_buffer_input;
|
u8 combiner_buffer_input;
|
||||||
|
|
||||||
|
|
|
@ -276,8 +276,10 @@ static void ProcessTriangleInternal(const Vertex& v0, const Vertex& v1, const Ve
|
||||||
|
|
||||||
DEBUG_ASSERT(0 != texture.config.address);
|
DEBUG_ASSERT(0 != texture.config.address);
|
||||||
|
|
||||||
float24 u = uv[i].u();
|
int coordinate_i =
|
||||||
float24 v = uv[i].v();
|
(i == 2 && regs.texturing.main_config.texture2_use_coord1) ? 1 : i;
|
||||||
|
float24 u = uv[coordinate_i].u();
|
||||||
|
float24 v = uv[coordinate_i].v();
|
||||||
|
|
||||||
// Only unit 0 respects the texturing type (according to 3DBrew)
|
// Only unit 0 respects the texturing type (according to 3DBrew)
|
||||||
// TODO: Refactor so cubemaps and shadowmaps can be handled
|
// TODO: Refactor so cubemaps and shadowmaps can be handled
|
||||||
|
|
Reference in New Issue