From eae20b77a8b5205611fc778d97e4b903318d1ceb Mon Sep 17 00:00:00 2001 From: Jordan Woyak Date: Wed, 20 Feb 2013 02:01:54 -0600 Subject: [PATCH] Remove some unneeded sampler-related calls in BPStructs. --- Source/Core/VideoCommon/Src/BPFunctions.cpp | 1 + Source/Core/VideoCommon/Src/BPStructs.cpp | 10 ---------- 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/Source/Core/VideoCommon/Src/BPFunctions.cpp b/Source/Core/VideoCommon/Src/BPFunctions.cpp index b9a76ba7b6..8dd24f4ed1 100644 --- a/Source/Core/VideoCommon/Src/BPFunctions.cpp +++ b/Source/Core/VideoCommon/Src/BPFunctions.cpp @@ -248,6 +248,7 @@ u8 *GetPointer(const u32 &address) return Memory::GetPointer(address); } +// Never used. All backends call SetSamplerState in VertexManager::Flush void SetTextureMode(const BPCmd &bp) { g_renderer->SetSamplerState(bp.address & 3, (bp.address & 0xE0) == 0xA0); diff --git a/Source/Core/VideoCommon/Src/BPStructs.cpp b/Source/Core/VideoCommon/Src/BPStructs.cpp index af5161582c..c7159b8783 100644 --- a/Source/Core/VideoCommon/Src/BPStructs.cpp +++ b/Source/Core/VideoCommon/Src/BPStructs.cpp @@ -578,8 +578,6 @@ void BPWritten(const BPCmd& bp) // ------------------------ case BPMEM_TX_SETMODE0: // (0x90 for linear) case BPMEM_TX_SETMODE0_4: - // Shouldn't need to call this here, we call it for each active texture right before rendering - SetTextureMode(bp); break; case BPMEM_TX_SETMODE1: @@ -721,14 +719,6 @@ void BPReload() SetBlendMode(); SetColorMask(); OnPixelFormatChange(); - { - BPCmd bp = {BPMEM_TX_SETMODE0, 0xFFFFFF, static_cast(((u32*)&bpmem)[BPMEM_TX_SETMODE0])}; - SetTextureMode(bp); - } - { - BPCmd bp = {BPMEM_TX_SETMODE0_4, 0xFFFFFF, static_cast(((u32*)&bpmem)[BPMEM_TX_SETMODE0_4])}; - SetTextureMode(bp); - } { BPCmd bp = {BPMEM_FIELDMASK, 0xFFFFFF, static_cast(((u32*)&bpmem)[BPMEM_FIELDMASK])}; SetInterlacingMode(bp);