Yaya48/dolphin
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

AMMediaboard: fix TestHardware string endianness in legacy Execute path

Browse Source 
The legacy Execute1 path (offset=0, length=0) was using Write_U32
(big-endian) to write the "TEST OK" status string. However, the PPC
display code in segaboot reads this buffer with lwz followed by manual
bswap32, so the data must be stored in little-endian. Use Write_U32_Swap
to match the other two TestHardware paths.
This commit is contained in:
naari3 committed 2026-02-21 14:38:38 +09:00
1 parent a297bb8068
commit adbecd00ec
1 file changed
+2 -2
Download Patch File Download Diff File Expand all files Collapse all files
Source/Core/Core/HW/DVD/AMMediaboard.cpp
+2 -2
View File
@@ -2112,8 +2112,8 @@ u32 ExecuteCommand(std::array<u32, 3>& dicmd_buf, u32* diimm_buf, u32 address, u
// On real systems it shows the status about the DIMM/GD-ROM here // On real systems it shows the status about the DIMM/GD-ROM here
// We just show "TEST OK" // We just show "TEST OK"
memory.Write_U32(0x54455354, s_media_buffer_32[12]); memory.Write_U32_Swap(0x54455354, s_media_buffer_32[12]);
memory.Write_U32(0x204F4B00, s_media_buffer_32[12] + 4); memory.Write_U32_Swap(0x204F4B00, s_media_buffer_32[12] + 4);
s_media_buffer_32[1] = s_media_buffer_32[9]; s_media_buffer_32[1] = s_media_buffer_32[9];
break; break;