From c59ef7d79322a099636cb6613029375cd44c6a0a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20L=C3=B3pez=20Guimaraes?= <112760654+DaniElectra@users.noreply.github.com> Date: Wed, 17 Jan 2024 07:48:42 +0100 Subject: [PATCH] cecd: Fixup GetCecInfoBuffer params order (#7361) While I was looking at the NEWS sysmodule, I noticed the params order for this command were backwards: the info type is the first param, followed by the buffer size. This is accurate to my reverse engineered code for the NEWS sysmodule. --- src/core/hle/service/cecd/cecd.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/core/hle/service/cecd/cecd.cpp b/src/core/hle/service/cecd/cecd.cpp index b40bfec92..bbd4cc0a2 100644 --- a/src/core/hle/service/cecd/cecd.cpp +++ b/src/core/hle/service/cecd/cecd.cpp @@ -682,16 +682,16 @@ void Module::Interface::Stop(Kernel::HLERequestContext& ctx) { void Module::Interface::GetCecInfoBuffer(Kernel::HLERequestContext& ctx) { IPC::RequestParser rp(ctx); - const u32 buffer_size = rp.Pop(); const u32 possible_info_type = rp.Pop(); + const u32 buffer_size = rp.Pop(); auto& buffer = rp.PopMappedBuffer(); IPC::RequestBuilder rb = rp.MakeBuilder(1, 2); rb.Push(ResultSuccess); rb.PushMappedBuffer(buffer); - LOG_DEBUG(Service_CECD, "called, buffer_size={}, possible_info_type={}", buffer_size, - possible_info_type); + LOG_DEBUG(Service_CECD, "called, possible_info_type={}, buffer_size={}", possible_info_type, + buffer_size); } void Module::Interface::GetCecdState(Kernel::HLERequestContext& ctx) {