GPU: Implemented bits 3 and 1 from the display transfer flags.

Bit 3 is used to specify a raw copy, where no processing is done to the data, seems to behave exactly as a DMA.
Bit 1 is used to specify whether to convert from a tiled format to a linear format or viceversa.

2 files changed, 64 insertions, 23 deletions

diff --git a/src/core/hw/gpu.cpp b/src/core/hw/gpu.cpp
index e6022d69..2f1a69d9 100644
--- a/src/core/hw/gpu.cpp
+++ b/src/core/hw/gpu.cpp
@@ -18,10 +18,10 @@
 #include "core/hw/gpu.h"
 
 #include "video_core/command_processor.h"
+#include "video_core/utils.h"
 #include "video_core/video_core.h"
 #include <video_core/color.h>
 
-
 namespace GPU {
 
 Regs g_regs;
@@ -116,24 +116,64 @@ inline void Write(u32 addr, const T data) {
             u8* source_pointer = Memory::GetPointer(Memory::PhysicalToVirtualAddress(config.GetPhysicalInputAddress()));
             u8* dest_pointer = Memory::GetPointer(Memory::PhysicalToVirtualAddress(config.GetPhysicalOutputAddress()));
 
-            // Cheap emulation of horizontal scaling: Just skip each second pixel of the
-            // input framebuffer. We keep track of this in the pixel_skip variable.
-            unsigned pixel_skip = (config.scale_horizontally != 0) ? 2 : 1;
-
-            u32 output_width = config.output_width / pixel_skip;
-
-            for (u32 y = 0; y < config.output_height; ++y) {
-                // TODO: Why does the register seem to hold twice the framebuffer width?
+            unsigned horizontal_scale = (config.scale_horizontally != 0) ? 2 : 1;
+            unsigned vertical_scale = (config.scale_vertically != 0) ? 2 : 1;
+
+            u32 output_width = config.output_width / horizontal_scale;
+            u32 output_height = config.output_height / vertical_scale;
+
+            if (config.raw_copy) {
+                // Raw copies do not perform color conversion nor tiled->linear / linear->tiled conversions
+                // TODO(Subv): Verify if raw copies perform scaling
+                memcpy(dest_pointer, source_pointer, config.output_width * config.output_height * 
+                        GPU::Regs::BytesPerPixel(config.output_format));
+                
+                LOG_TRACE(HW_GPU, "DisplayTriggerTransfer: 0x%08x bytes from 0x%08x(%ux%u)-> 0x%08x(%ux%u), flags 0x%08X, Raw copy",
+                    config.output_height * output_width * GPU::Regs::BytesPerPixel(config.output_format),
+                    config.GetPhysicalInputAddress(), config.input_width.Value(), config.input_height.Value(),
+                    config.GetPhysicalOutputAddress(), config.output_width.Value(), config.output_height.Value(),
+                    config.output_format.Value(), config.flags);
+
+                GSP_GPU::SignalInterrupt(GSP_GPU::InterruptId::PPF);
+                break;
+            }
 
+            // TODO(Subv): Blend the pixels when horizontal / vertical scaling is enabled, 
+            // right now we're just skipping the extra pixels.
+            for (u32 y = 0; y < output_height; ++y) {
                 for (u32 x = 0; x < output_width; ++x) {
                     struct {
                         int r, g, b, a;
                     } source_color = { 0, 0, 0, 0 };
 
+                    u32 scaled_x = x * horizontal_scale;
+                    u32 scaled_y = y * vertical_scale;
+
+                    u32 dst_bytes_per_pixel = GPU::Regs::BytesPerPixel(config.output_format);
+                    u32 src_bytes_per_pixel = GPU::Regs::BytesPerPixel(config.input_format);
+                    u32 src_offset;
+                    u32 dst_offset;
+
+                    if (config.output_tiled) {
+                        // Interpret the input as linear and the output as tiled
+                        u32 coarse_y = y & ~7;
+                        u32 stride = output_width * dst_bytes_per_pixel;
+
+                        src_offset = (scaled_x + scaled_y * config.input_width) * src_bytes_per_pixel;
+                        dst_offset = VideoCore::GetMortonOffset(x, y, dst_bytes_per_pixel) + coarse_y * stride;
+                    } else {
+                        // Interpret the input as tiled and the output as linear
+                        u32 coarse_y = scaled_y & ~7;
+                        u32 stride = config.input_width * src_bytes_per_pixel;
+
+                        src_offset = VideoCore::GetMortonOffset(scaled_x, scaled_y, src_bytes_per_pixel) + coarse_y * stride;
+                        dst_offset = (x + y * output_width) * dst_bytes_per_pixel;
+                    }
+
                     switch (config.input_format) {
                     case Regs::PixelFormat::RGBA8:
                     {
-                        u8* srcptr = source_pointer + (x * pixel_skip + y * config.input_width) * 4;
+                        u8* srcptr = source_pointer + src_offset;
                         source_color.r = srcptr[3]; // red
                         source_color.g = srcptr[2]; // green
                         source_color.b = srcptr[1]; // blue
@@ -143,7 +183,7 @@ inline void Write(u32 addr, const T data) {
 
                     case Regs::PixelFormat::RGB5A1:
                     {
-                        u16 srcval = *(u16*)(source_pointer + x * 4 * pixel_skip + y * config.input_width * 4 * pixel_skip);
+                        u16 srcval = *(u16*)(source_pointer + src_offset);
                         source_color.r = Color::Convert5To8((srcval >> 11) & 0x1F); // red
                         source_color.g = Color::Convert5To8((srcval >>  6) & 0x1F); // green
                         source_color.b = Color::Convert5To8((srcval >>  1) & 0x1F); // blue
@@ -153,7 +193,7 @@ inline void Write(u32 addr, const T data) {
 
                     case Regs::PixelFormat::RGBA4:
                     {
-                        u16 srcval = *(u16*)(source_pointer + x * 4 * pixel_skip + y * config.input_width * 4 * pixel_skip);
+                        u16 srcval = *(u16*)(source_pointer + src_offset);
                         source_color.r = Color::Convert4To8((srcval >> 12) & 0xF); // red
                         source_color.g = Color::Convert4To8((srcval >>  8) & 0xF); // green
                         source_color.b = Color::Convert4To8((srcval >>  4) & 0xF); // blue
@@ -169,7 +209,7 @@ inline void Write(u32 addr, const T data) {
                     switch (config.output_format) {
                     case Regs::PixelFormat::RGBA8:
                     {
-                        u8* dstptr = dest_pointer + (x * pixel_skip + y * config.output_width) * 4;
+                        u8* dstptr = dest_pointer + dst_offset;
                         dstptr[3] = source_color.r;
                         dstptr[2] = source_color.g;
                         dstptr[1] = source_color.b;
@@ -179,7 +219,7 @@ inline void Write(u32 addr, const T data) {
 
                     case Regs::PixelFormat::RGB8:
                     {
-                        u8* dstptr = dest_pointer + (x + y * output_width) * 3;
+                        u8* dstptr = dest_pointer + dst_offset;
                         dstptr[2] = source_color.r; // red
                         dstptr[1] = source_color.g; // green
                         dstptr[0] = source_color.b; // blue
@@ -188,7 +228,7 @@ inline void Write(u32 addr, const T data) {
 
                     case Regs::PixelFormat::RGB5A1:
                     {
-                        u16* dstptr = (u16*)(dest_pointer + x * 2 + y * config.output_width * 2);
+                        u16* dstptr = (u16*)(dest_pointer + dst_offset);
                         *dstptr = ((source_color.r >> 3) << 11) | ((source_color.g >> 3) << 6)
                                 | ((source_color.b >> 3) <<  1) | ( source_color.a >> 7);
                         break;
@@ -196,7 +236,7 @@ inline void Write(u32 addr, const T data) {
 
                     case Regs::PixelFormat::RGBA4:
                     {
-                        u16* dstptr = (u16*)(dest_pointer + x * 2 + y * config.output_width * 2);
+                        u16* dstptr = (u16*)(dest_pointer + dst_offset);
                         *dstptr = ((source_color.r >> 4) << 12) | ((source_color.g >> 4) << 8)
                                 | ((source_color.b >> 4) <<  4) | ( source_color.a >> 4);
                         break;
@@ -209,11 +249,11 @@ inline void Write(u32 addr, const T data) {
                 }
             }
 
-            LOG_TRACE(HW_GPU, "DisplayTriggerTransfer: 0x%08x bytes from 0x%08x(%ux%u)-> 0x%08x(%ux%u), dst format %x",
-                      config.output_height * output_width * 4,
-                      config.GetPhysicalInputAddress(), (u32)config.input_width, (u32)config.input_height,
-                      config.GetPhysicalOutputAddress(), (u32)output_width, (u32)config.output_height,
-                      config.output_format.Value());
+            LOG_TRACE(HW_GPU, "DisplayTriggerTransfer: 0x%08x bytes from 0x%08x(%ux%u)-> 0x%08x(%ux%u), dst format %x, flags 0x%08X",
+                      config.output_height * output_width * GPU::Regs::BytesPerPixel(config.output_format),
+                      config.GetPhysicalInputAddress(), config.input_width.Value(), config.input_height.Value(),
+                      config.GetPhysicalOutputAddress(), output_width, output_height,
+                      config.output_format.Value(), config.flags);
 
             GSP_GPU::SignalInterrupt(GSP_GPU::InterruptId::PPF);
         }
diff --git a/src/core/hw/gpu.h b/src/core/hw/gpu.h
index 75f52446..ab1dcf91 100644
--- a/src/core/hw/gpu.h
+++ b/src/core/hw/gpu.h
@@ -192,12 +192,13 @@ struct Regs {
             u32 flags;
 
             BitField< 0, 1, u32> flip_data;        // flips input data horizontally (TODO) if true
+            BitField< 1, 1, u32> output_tiled;     // Converts from linear to tiled format
+            BitField< 3, 1, u32> raw_copy;         // Copies the data without performing any processing
             BitField< 8, 3, PixelFormat> input_format;
             BitField<12, 3, PixelFormat> output_format;
-            BitField<16, 1, u32> output_tiled;     // stores output in a tiled format
 
-            // TODO: Not really sure if this actually scales, or even resizes at all.
             BitField<24, 1, u32> scale_horizontally;
+            BitField<25, 1, u32> scale_vertically;
         };
 
         INSERT_PADDING_WORDS(0x1);