From 81e0ea5cfa85ad5e772b896f329b4d736c177b7b Mon Sep 17 00:00:00 2001 From: Adam Honse Date: Sat, 5 Sep 2026 18:47:20 -0500 Subject: [PATCH] Add trace level settings packet log to MSI 185 controller --- .../MSIMotherboard185Controller.cpp | 25 ++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) diff --git a/Controllers/MSIMotherboardController/MSIMotherboard185Controller/MSIMotherboard185Controller.cpp b/Controllers/MSIMotherboardController/MSIMotherboard185Controller/MSIMotherboard185Controller.cpp index 0bf24cabc..7101be3a1 100644 --- a/Controllers/MSIMotherboardController/MSIMotherboard185Controller/MSIMotherboard185Controller.cpp +++ b/Controllers/MSIMotherboardController/MSIMotherboard185Controller/MSIMotherboard185Controller.cpp @@ -18,7 +18,9 @@ #include #include #include +#include #include "MSIMotherboard185Controller.h" +#include "LogManager.h" #include "StringUtils.h" using namespace std::chrono_literals; @@ -668,7 +670,28 @@ bool MSIMotherboard185Controller::ReadSettings() /*-----------------------------------------------------*\ | Read packet from hardware, return true if successful | \*-----------------------------------------------------*/ - return(hid_get_feature_report(dev, (unsigned char*)&data, sizeof(data)) == sizeof data); + int ret = hid_get_feature_report(dev, (unsigned char*)&data, sizeof(data)); + + /*-----------------------------------------------------*\ + | Log the entire packet data in hex at TRACE level | + \*-----------------------------------------------------*/ + if(LogManager::get()->GetLogLevel() >= LL_TRACE) + { + LOG_TRACE("[MSI 185] ReadSettings packet (%zu bytes):", sizeof(data)); + for(size_t i = 0; i < sizeof(data); i += 16) + { + std::string hex_line = ""; + for(size_t j = i; j < i + 16 && j < sizeof(data); j++) + { + char buf[4]; + snprintf(buf, sizeof(buf), "%02X ", ((unsigned char*)&data)[j]); + hex_line += buf; + } + LOG_TRACE(" %04zX: %s", i, hex_line.c_str()); + } + } + + return(ret == sizeof data); } bool MSIMotherboard185Controller::Update