Add helpers to create IPC command buffer headers and descriptors

2 files changed, 9 insertions, 7 deletions

diff --git a/src/core/hle/service/apt/apt.cpp b/src/core/hle/service/apt/apt.cpp
index 5d14f393..b454a270 100644
--- a/src/core/hle/service/apt/apt.cpp
+++ b/src/core/hle/service/apt/apt.cpp
@@ -45,7 +45,7 @@ void Initialize(Service::Interface* self) {
     u32 app_id = cmd_buff[1];
     u32 flags  = cmd_buff[2];
 
-    cmd_buff[2] = 0x04000000; // According to 3dbrew, this value should be 0x04000000
+    cmd_buff[2] = IPC::MoveHandleDesc(2);
     cmd_buff[3] = Kernel::g_handle_table.Create(notification_event).MoveFrom();
     cmd_buff[4] = Kernel::g_handle_table.Create(start_event).MoveFrom();
 
@@ -70,11 +70,13 @@ void GetSharedFont(Service::Interface* self) {
         // an easy way to do this, but the copy should be sufficient for now.
         memcpy(Memory::GetPointer(SHARED_FONT_VADDR), shared_font.data(), shared_font.size());
 
-        cmd_buff[0] = 0x00440082;
+        cmd_buff[0] = IPC::MakeHeader(0x44, 2, 2);
         cmd_buff[1] = RESULT_SUCCESS.raw; // No error
         cmd_buff[2] = SHARED_FONT_VADDR;
+        cmd_buff[3] = IPC::MoveHandleDesc();
         cmd_buff[4] = Kernel::g_handle_table.Create(shared_font_mem).MoveFrom();
     } else {
+        cmd_buff[0] = IPC::MakeHeader(0x44, 1, 0);
         cmd_buff[1] = -1; // Generic error (not really possible to verify this on hardware)
         LOG_ERROR(Kernel_SVC, "called, but %s has not been loaded!", SHARED_FONT);
     }
diff --git a/src/core/hle/service/y2r_u.cpp b/src/core/hle/service/y2r_u.cpp
index 17cb4f0f..ac1967da 100644
--- a/src/core/hle/service/y2r_u.cpp
+++ b/src/core/hle/service/y2r_u.cpp
@@ -121,7 +121,7 @@ static void SetBlockAlignment(Service::Interface* self) {
 static void SetTransferEndInterrupt(Service::Interface* self) {
     u32* cmd_buff = Kernel::GetCommandBuffer();
 
-    cmd_buff[0] = 0x000D0040;
+    cmd_buff[0] = IPC::MakeHeader(0xD, 1, 0);
     cmd_buff[1] = RESULT_SUCCESS.raw;
     LOG_DEBUG(Service_Y2R, "(STUBBED) called");
 }
@@ -279,7 +279,7 @@ static void StartConversion(Service::Interface* self) {
 static void StopConversion(Service::Interface* self) {
     u32* cmd_buff = Kernel::GetCommandBuffer();
 
-    cmd_buff[0] = 0x00270040;
+    cmd_buff[0] = IPC::MakeHeader(0x27, 1, 0);
     cmd_buff[1] = RESULT_SUCCESS.raw;
     LOG_DEBUG(Service_Y2R, "called");
 }
@@ -328,7 +328,7 @@ static void SetConversionParams(Service::Interface* self) {
     conversion.alpha = params->alpha;
 
 cleanup:
-    cmd_buff[0] = 0x00290040; // TODO verify
+    cmd_buff[0] = IPC::MakeHeader(0x29, 1, 0);
     cmd_buff[1] = result.raw;
 }
 
@@ -360,7 +360,7 @@ static void DriverInitialize(Service::Interface* self) {
 
     completion_event->Clear();
 
-    cmd_buff[0] = 0x002B0040;
+    cmd_buff[0] = IPC::MakeHeader(0x2B, 1, 0);
     cmd_buff[1] = RESULT_SUCCESS.raw;
     LOG_DEBUG(Service_Y2R, "called");
 }
@@ -368,7 +368,7 @@ static void DriverInitialize(Service::Interface* self) {
 static void DriverFinalize(Service::Interface* self) {
     u32* cmd_buff = Kernel::GetCommandBuffer();
 
-    cmd_buff[0] = 0x002C0040;
+    cmd_buff[0] = IPC::MakeHeader(0x2C, 1, 0);
     cmd_buff[1] = RESULT_SUCCESS.raw;
     LOG_DEBUG(Service_Y2R, "called");
 }