Rename GPU::Regs::FramebufferFormat to PixelFormat
This name better represents what the enum does, and is less overloaded in the context. (The whole register the enum is part of is also called 'format'.)
This commit is contained in:
parent
7aae063844
commit
e7719a52cd
2 changed files with 9 additions and 9 deletions
|
@ -90,7 +90,7 @@ inline void Write(u32 addr, const T data) {
|
||||||
} source_color = { 0, 0, 0, 0 };
|
} source_color = { 0, 0, 0, 0 };
|
||||||
|
|
||||||
switch (config.input_format) {
|
switch (config.input_format) {
|
||||||
case Regs::FramebufferFormat::RGBA8:
|
case Regs::PixelFormat::RGBA8:
|
||||||
{
|
{
|
||||||
// TODO: Most likely got the component order messed up.
|
// TODO: Most likely got the component order messed up.
|
||||||
u8* srcptr = source_pointer + x * 4 + y * config.input_width * 4;
|
u8* srcptr = source_pointer + x * 4 + y * config.input_width * 4;
|
||||||
|
@ -107,7 +107,7 @@ inline void Write(u32 addr, const T data) {
|
||||||
}
|
}
|
||||||
|
|
||||||
switch (config.output_format) {
|
switch (config.output_format) {
|
||||||
/*case Regs::FramebufferFormat::RGBA8:
|
/*case Regs::PixelFormat::RGBA8:
|
||||||
{
|
{
|
||||||
// TODO: Untested
|
// TODO: Untested
|
||||||
u8* dstptr = (u32*)(dest_pointer + x * 4 + y * config.output_width * 4);
|
u8* dstptr = (u32*)(dest_pointer + x * 4 + y * config.output_width * 4);
|
||||||
|
@ -118,7 +118,7 @@ inline void Write(u32 addr, const T data) {
|
||||||
break;
|
break;
|
||||||
}*/
|
}*/
|
||||||
|
|
||||||
case Regs::FramebufferFormat::RGB8:
|
case Regs::PixelFormat::RGB8:
|
||||||
{
|
{
|
||||||
// TODO: Most likely got the component order messed up.
|
// TODO: Most likely got the component order messed up.
|
||||||
u8* dstptr = dest_pointer + x * 3 + y * config.output_width * 3;
|
u8* dstptr = dest_pointer + x * 3 + y * config.output_width * 3;
|
||||||
|
@ -220,13 +220,13 @@ void Init() {
|
||||||
framebuffer_top.width = 240;
|
framebuffer_top.width = 240;
|
||||||
framebuffer_top.height = 400;
|
framebuffer_top.height = 400;
|
||||||
framebuffer_top.stride = 3 * 240;
|
framebuffer_top.stride = 3 * 240;
|
||||||
framebuffer_top.color_format = Regs::FramebufferFormat::RGB8;
|
framebuffer_top.color_format = Regs::PixelFormat::RGB8;
|
||||||
framebuffer_top.active_fb = 0;
|
framebuffer_top.active_fb = 0;
|
||||||
|
|
||||||
framebuffer_sub.width = 240;
|
framebuffer_sub.width = 240;
|
||||||
framebuffer_sub.height = 320;
|
framebuffer_sub.height = 320;
|
||||||
framebuffer_sub.stride = 3 * 240;
|
framebuffer_sub.stride = 3 * 240;
|
||||||
framebuffer_sub.color_format = Regs::FramebufferFormat::RGB8;
|
framebuffer_sub.color_format = Regs::PixelFormat::RGB8;
|
||||||
framebuffer_sub.active_fb = 0;
|
framebuffer_sub.active_fb = 0;
|
||||||
|
|
||||||
NOTICE_LOG(GPU, "initialized OK");
|
NOTICE_LOG(GPU, "initialized OK");
|
||||||
|
|
|
@ -56,7 +56,7 @@ struct Regs {
|
||||||
"Structure size and register block length don't match")
|
"Structure size and register block length don't match")
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
enum class FramebufferFormat : u32 {
|
enum class PixelFormat : u32 {
|
||||||
RGBA8 = 0,
|
RGBA8 = 0,
|
||||||
RGB8 = 1,
|
RGB8 = 1,
|
||||||
RGB565 = 2,
|
RGB565 = 2,
|
||||||
|
@ -84,8 +84,8 @@ struct Regs {
|
||||||
|
|
||||||
INSERT_PADDING_WORDS(0x10b);
|
INSERT_PADDING_WORDS(0x10b);
|
||||||
|
|
||||||
struct {
|
struct FramebufferConfig {
|
||||||
using Format = Regs::FramebufferFormat;
|
using Format = Regs::PixelFormat;
|
||||||
|
|
||||||
union {
|
union {
|
||||||
u32 size;
|
u32 size;
|
||||||
|
@ -130,7 +130,7 @@ struct Regs {
|
||||||
INSERT_PADDING_WORDS(0x169);
|
INSERT_PADDING_WORDS(0x169);
|
||||||
|
|
||||||
struct {
|
struct {
|
||||||
using Format = Regs::FramebufferFormat;
|
using Format = Regs::PixelFormat;
|
||||||
|
|
||||||
u32 input_address;
|
u32 input_address;
|
||||||
u32 output_address;
|
u32 output_address;
|
||||||
|
|
Loading…
Reference in a new issue