GPU/DisplayTransfer: Made the scaling bits a single 2bit value
Rephrased some comments.
This commit is contained in:
parent
ae0dfcae1d
commit
23b401c3ac
|
@ -116,8 +116,14 @@ inline void Write(u32 addr, const T data) {
|
||||||
u8* src_pointer = Memory::GetPointer(Memory::PhysicalToVirtualAddress(config.GetPhysicalInputAddress()));
|
u8* src_pointer = Memory::GetPointer(Memory::PhysicalToVirtualAddress(config.GetPhysicalInputAddress()));
|
||||||
u8* dst_pointer = Memory::GetPointer(Memory::PhysicalToVirtualAddress(config.GetPhysicalOutputAddress()));
|
u8* dst_pointer = Memory::GetPointer(Memory::PhysicalToVirtualAddress(config.GetPhysicalOutputAddress()));
|
||||||
|
|
||||||
unsigned horizontal_scale = (config.scale_x != 0 || config.scale_xy != 0) ? 2 : 1;
|
if (config.scaling > config.ScaleXY) {
|
||||||
unsigned vertical_scale = (config.scale_xy != 0) ? 2 : 1;
|
LOG_CRITICAL(HW_GPU, "Unimplemented display transfer scaling mode %u", config.scaling.Value());
|
||||||
|
UNIMPLEMENTED();
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
|
unsigned horizontal_scale = (config.scaling != config.NoScale) ? 2 : 1;
|
||||||
|
unsigned vertical_scale = (config.scaling == config.ScaleXY) ? 2 : 1;
|
||||||
|
|
||||||
u32 output_width = config.output_width / horizontal_scale;
|
u32 output_width = config.output_width / horizontal_scale;
|
||||||
u32 output_height = config.output_height / vertical_scale;
|
u32 output_height = config.output_height / vertical_scale;
|
||||||
|
@ -138,7 +144,7 @@ inline void Write(u32 addr, const T data) {
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO(Subv): Blend the pixels when horizontal / vertical scaling is enabled,
|
// TODO(Subv): Implement the box filter when scaling is enabled
|
||||||
// right now we're just skipping the extra pixels.
|
// right now we're just skipping the extra pixels.
|
||||||
for (u32 y = 0; y < output_height; ++y) {
|
for (u32 y = 0; y < output_height; ++y) {
|
||||||
for (u32 x = 0; x < output_width; ++x) {
|
for (u32 x = 0; x < output_width; ++x) {
|
||||||
|
|
|
@ -188,6 +188,12 @@ struct Regs {
|
||||||
BitField<16, 16, u32> input_height;
|
BitField<16, 16, u32> input_height;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
enum ScalingMode : u32 {
|
||||||
|
NoScale = 0, // Doesn't scale the image
|
||||||
|
ScaleX = 1, // Downscales the image in half in the X axis and applies a box filter
|
||||||
|
ScaleXY = 2, // Downscales the image in half in both the X and Y axes and applies a box filter
|
||||||
|
};
|
||||||
|
|
||||||
union {
|
union {
|
||||||
u32 flags;
|
u32 flags;
|
||||||
|
|
||||||
|
@ -197,8 +203,7 @@ struct Regs {
|
||||||
BitField< 8, 3, PixelFormat> input_format;
|
BitField< 8, 3, PixelFormat> input_format;
|
||||||
BitField<12, 3, PixelFormat> output_format;
|
BitField<12, 3, PixelFormat> output_format;
|
||||||
|
|
||||||
BitField<24, 1, u32> scale_x; // Shrinks the image in half horizontally, blending the extra pixels
|
BitField<24, 2, ScalingMode> scaling; // Determines the scaling mode of the transfer
|
||||||
BitField<25, 1, u32> scale_xy; // Shrinks the image horizontally and vertically, blending the extra pixels
|
|
||||||
};
|
};
|
||||||
|
|
||||||
INSERT_PADDING_WORDS(0x1);
|
INSERT_PADDING_WORDS(0x1);
|
||||||
|
|
Reference in New Issue