From 30690d0ebdb97c122d0c98eee0bcf388d076b199 Mon Sep 17 00:00:00 2001 From: Andy Herbert Date: Thu, 4 Jun 2026 16:54:51 +0000 Subject: [PATCH] New device- MSI MAG272CQR monitor --- .../MSIMonitorController.cpp | 121 +++++++++++++ .../MSIMonitorController.h | 53 ++++++ .../MSIMonitorControllerDetect.cpp | 38 ++++ .../RGBController_MSIMonitor.cpp | 171 ++++++++++++++++++ .../RGBController_MSIMonitor.h | 38 ++++ 5 files changed, 421 insertions(+) create mode 100644 Controllers/MSIMonitorController/MSIMonitorController.cpp create mode 100644 Controllers/MSIMonitorController/MSIMonitorController.h create mode 100644 Controllers/MSIMonitorController/MSIMonitorControllerDetect.cpp create mode 100644 Controllers/MSIMonitorController/RGBController_MSIMonitor.cpp create mode 100644 Controllers/MSIMonitorController/RGBController_MSIMonitor.h diff --git a/Controllers/MSIMonitorController/MSIMonitorController.cpp b/Controllers/MSIMonitorController/MSIMonitorController.cpp new file mode 100644 index 000000000..9c9c4ed2c --- /dev/null +++ b/Controllers/MSIMonitorController/MSIMonitorController.cpp @@ -0,0 +1,121 @@ +/*---------------------------------------------------------*\ +| RGBController_MSIMonitor.cpp | +| | +| RGBController for MSI monitor (gaming controller) | +| | +| Andy Herbert 2026 June 4 | +| | +| This file is part of the OpenRGB project | +| SPDX-License-Identifier: GPL-2.0-or-later | +\*---------------------------------------------------------*/ + +#include +#include "MSIMonitorController.h" +#include "StringUtils.h" + +MSIMonitorController::MSIMonitorController(hid_device* dev_handle, const hid_device_info& info, std::string dev_name) +{ + dev = dev_handle; + location = info.path; + name = dev_name; +} + +MSIMonitorController::~MSIMonitorController() +{ + hid_close(dev); +} + +std::string MSIMonitorController::GetDeviceLocation() +{ + return("HID: " + location); +} + +std::string MSIMonitorController::GetNameString() +{ + return(name); +} + +std::string MSIMonitorController::GetSerialString() +{ + wchar_t serial_string[128]; + int ret = hid_get_serial_number_string(dev, serial_string, 128); + + if(ret != 0) + { + return(""); + } + + return(StringUtils::wstring_to_string(serial_string)); +} + +void MSIMonitorController::Set(uint8_t mode_value, const std::vector colors, uint8_t last_bit) +{ + /*---------------------------------------------------------*\ + | Prepare color data | + \*---------------------------------------------------------*/ + uint8_t data[MSI_MONITOR_PACKET_SIZE]; + memset(data, 0x00, MSI_MONITOR_PACKET_SIZE); + + unsigned int offset = 0; + + data[offset++] = 0x71; + data[offset++] = 0x01; + for(int i = 0; i < 3; i++) + { + data[offset++] = 0x00; + } + + data[offset++] = 0x01; + data[offset++] = 0x64; + for(int i = 0; i < 5; i++) + { + data[offset++] = 0x00; + } + + /*---------------------------------------------------------*\ + | put mode_value | + \*---------------------------------------------------------*/ + data[offset++] = mode_value; + + for(int i = 0; i < 3; i++) + { + data[offset++] = 0x00; + } + data[offset++] = 0x01; + data[offset++] = 0x64; + for(int i = 0; i < 5; i++) + { + data[offset++] = 0x00; + } + + /*-----------------------------------------------------------------------*\ + | this data looks like placeholder for additional LEDs in other monitors | + \*-----------------------------------------------------------------------*/ + for(int i = 0; i < 9; i++) + { + data[offset++] = 0xff; + data[offset++] = 0x00; + data[offset++] = 0x00; + } + + //RGB values begin + for(const RGBColor color: colors) + { + data[offset++] = RGBGetRValue(color); + data[offset++] = RGBGetGValue(color); + data[offset++] = RGBGetBValue(color); + } + + /*-------------------------------------------------------------------------------*\ + | last bit is probably a write to device bit- 0x01 saves to device, 0x00 doesn't. | + | For direct mode, bit is set to 0x00, otherwise lights will flicker | + \*-------------------------------------------------------------------------------*/ + + data[offset++] = last_bit; + + /*---------------------------------------------------------*\ + | Send the data (1 packet) | + \*---------------------------------------------------------*/ + + hid_send_feature_report(dev, data, MSI_MONITOR_PACKET_SIZE); +} diff --git a/Controllers/MSIMonitorController/MSIMonitorController.h b/Controllers/MSIMonitorController/MSIMonitorController.h new file mode 100644 index 000000000..6df4437b0 --- /dev/null +++ b/Controllers/MSIMonitorController/MSIMonitorController.h @@ -0,0 +1,53 @@ +/*---------------------------------------------------------*\ +| RGBController_MSIMonitor.cpp | +| | +| RGBController for MSI monitor (gaming controller) | +| | +| Andy Herbert 2026 June 1 | +| | +| This file is part of the OpenRGB project | +| SPDX-License-Identifier: GPL-2.0-or-later | +\*---------------------------------------------------------*/ + +#pragma once + +#include +#include +#include "RGBController.h" + +#define MSI_MONITOR_LEDS 9 +#define MSI_MONITOR_PACKET_SIZE 78 + +enum +{ + MSI_MONITOR_OFF_MODE_VALUE = 0x00, + MSI_MONITOR_STATIC_MODE_VALUE = 0x01, + MSI_MONITOR_BREATHING_MODE_VALUE = 0x02, + MSI_MONITOR_FLASHING_MODE_VALUE = 0x03, + MSI_MONITOR_LIGHTNING_MODE_VALUE = 0x05, + MSI_MONITOR_MARQUEE_MODE_VALUE = 0x06, + MSI_MONITOR_METEOR_MODE_VALUE = 0x08, + MSI_MONITOR_RAINBOW_MODE_VALUE = 0x1A, + MSI_MONITOR_RANDOM_MODE_VALUE = 0x1F +}; + +class MSIMonitorController +{ +public: + MSIMonitorController(hid_device *dev_handle, const hid_device_info &info, std::string dev_name); + ~MSIMonitorController(); + + std::string GetDeviceLocation(); + std::string GetFirmwareVersion(); + std::string GetNameString(); + std::string GetSerialString(); + + void Set(uint8_t mode_value, const std::vector colors, uint8_t last_bit); + +private: + hid_device *dev; + std::string description; + std::string location; + std::string name; + std::string version; +}; diff --git a/Controllers/MSIMonitorController/MSIMonitorControllerDetect.cpp b/Controllers/MSIMonitorController/MSIMonitorControllerDetect.cpp new file mode 100644 index 000000000..04595bdee --- /dev/null +++ b/Controllers/MSIMonitorController/MSIMonitorControllerDetect.cpp @@ -0,0 +1,38 @@ +/*---------------------------------------------------------*\ +| MSIKeyboardControllerDetect.cpp | +| | +| Detector for MSI monitor (MSI Gaming Controller) | +| | +| This file is part of the OpenRGB project | +| SPDX-License-Identifier: GPL-2.0-or-later | +\*---------------------------------------------------------*/ + +#include "Detector.h" +#include "MSIMonitorController.h" +#include "RGBController_MSIMonitor.h" + +#define MSI_USB_VID 0x1462 +#define MSI_USB_PID 0x3FA4 + +/*----------------------------------------------------------*\ +| | +| DetectMSIMonitorController | +| | +| Detect MSI monitor and maybe others | +| | +\*----------------------------------------------------------*/ + +static void DetectMSIMonitorController(hid_device_info* info, const std::string& name) +{ + hid_device* dev = hid_open_path(info->path); + + if(dev) + { + MSIMonitorController* controller = new MSIMonitorController(dev, *info, name); + RGBController_MSIMonitor* rgb_controller = new RGBController_MSIMonitor(controller); + + ResourceManager::get()->RegisterRGBController(rgb_controller); + } +} + +REGISTER_HID_DETECTOR_IPU("MSI Monitor (Gaming Controller)", DetectMSIMonitorController, MSI_USB_VID, MSI_USB_PID, 0, 0x01, 0); diff --git a/Controllers/MSIMonitorController/RGBController_MSIMonitor.cpp b/Controllers/MSIMonitorController/RGBController_MSIMonitor.cpp new file mode 100644 index 000000000..b361b7f2e --- /dev/null +++ b/Controllers/MSIMonitorController/RGBController_MSIMonitor.cpp @@ -0,0 +1,171 @@ +/*---------------------------------------------------------*\ +| RGBController_MSIMonitor.cpp | +| | +| RGBController for MSI monitor (gaming controller) | +| | +| Andy Herbert 2026 June 1 | +| | +| This file is part of the OpenRGB project | +| SPDX-License-Identifier: GPL-2.0-or-later | +\*---------------------------------------------------------*/ + +#include +#include +#include "RGBController_MSIMonitor.h" + +using namespace std::chrono_literals; + +/**------------------------------------------------------------------*\ + @name MSIMonitor + @category Accessory + @type USB + @save :robot: + @direct :white_check_mark: + @effects :white_check_mark: + @detectors DetectMSIMonitorController + @comment Developed with MSI MAG272CQR +\*-------------------------------------------------------------------*/ +RGBController_MSIMonitor::RGBController_MSIMonitor(MSIMonitorController* controller_ptr) +{ + controller = controller_ptr; + + name = controller->GetNameString(); + vendor = "MSI"; + type = DEVICE_TYPE_MONITOR; + description = "MSI Monitor (Gaming Controller)"; + location = controller->GetDeviceLocation(); + serial = controller->GetSerialString(); + + mode Direct; + Direct.name = "Direct"; + Direct.value = MSI_MONITOR_STATIC_MODE_VALUE; + Direct.flags = MODE_FLAG_HAS_PER_LED_COLOR; + Direct.color_mode = MODE_COLORS_PER_LED; + modes.push_back(Direct); + + mode Static; + Static.name = "Static"; + Static.value = MSI_MONITOR_STATIC_MODE_VALUE; + Static.flags = MODE_FLAG_HAS_PER_LED_COLOR; + Static.color_mode = MODE_COLORS_PER_LED; + modes.push_back(Static); + + + mode Breathing; + Breathing.name = "Breathing"; + Breathing.value = MSI_MONITOR_BREATHING_MODE_VALUE; + Breathing.flags = MODE_FLAG_HAS_PER_LED_COLOR; + Breathing.color_mode = MODE_COLORS_PER_LED; + modes.push_back(Breathing); + + mode Flashing; + Flashing.name = "Flashing"; + Flashing.value = MSI_MONITOR_FLASHING_MODE_VALUE; + Flashing.flags = MODE_FLAG_HAS_PER_LED_COLOR; + Flashing.color_mode = MODE_COLORS_PER_LED; + modes.push_back(Flashing); + + mode Lightning; + Lightning.name = "Lightning"; + Lightning.value = MSI_MONITOR_LIGHTNING_MODE_VALUE; + Lightning.flags = MODE_FLAG_HAS_PER_LED_COLOR; + Lightning.color_mode = MODE_COLORS_PER_LED; + modes.push_back(Lightning); + + mode Marquee; + Marquee.name = "Marquee"; + Marquee.value = MSI_MONITOR_MARQUEE_MODE_VALUE; + Marquee.flags = MODE_FLAG_AUTOMATIC_SAVE; + Marquee.color_mode = MODE_COLORS_NONE; + modes.push_back(Marquee); + + mode Meteor; + Meteor.name = "Meteor"; + Meteor.value = MSI_MONITOR_METEOR_MODE_VALUE; + Meteor.flags = MODE_FLAG_AUTOMATIC_SAVE; + Meteor.color_mode = MODE_COLORS_NONE; + modes.push_back(Meteor); + + mode Rainbow; + Rainbow.name = "Rainbow"; + Rainbow.value = MSI_MONITOR_RAINBOW_MODE_VALUE; + Rainbow.flags = MODE_FLAG_AUTOMATIC_SAVE; + Rainbow.color_mode = MODE_COLORS_NONE; + modes.push_back(Rainbow); + + mode Random; + Random.name = "Random"; + Random.value = MSI_MONITOR_RANDOM_MODE_VALUE; + Random.flags = MODE_FLAG_AUTOMATIC_SAVE; + Random.color_mode = MODE_COLORS_RANDOM; + modes.push_back(Random); + + mode Off; + Off.name = "Off"; + Off.value = MSI_MONITOR_OFF_MODE_VALUE; + Off.flags = MODE_FLAG_AUTOMATIC_SAVE; + Off.color_mode = MODE_COLORS_NONE; + modes.push_back(Off); + + SetupZones(); +} + +RGBController_MSIMonitor::~RGBController_MSIMonitor() +{ + keepalive_thread_run = 0; + keepalive_thread->join(); + delete keepalive_thread; + + delete controller; +} + +void RGBController_MSIMonitor::SetupZones() +{ + zone new_zone; + + new_zone.name = "Rear"; + new_zone.type = ZONE_TYPE_LINEAR; + new_zone.leds_min = 9; + new_zone.leds_max = 9; + new_zone.leds_count = 9; + new_zone.matrix_map = nullptr; + + zones.emplace_back(new_zone); + + for(unsigned int i = 0 ; i < 9; i ++) + { + led new_led; + new_led.name = "LED " + std::to_string(i + 1); + leds.push_back(new_led); + } + + SetupColors(); +} + +void RGBController_MSIMonitor::ResizeZone(int /*zone*/, int /*new_size*/) +{ + /*---------------------------------------------------------*\ + | This device does not support resizing zones | + \*---------------------------------------------------------*/ +} + +void RGBController_MSIMonitor::DeviceUpdateLEDs() +{ + last_update_time = std::chrono::steady_clock::now(); + controller->Set(modes[active_mode].value, colors, active_mode == 0 ? 0x00 : 0x01); +} + +void RGBController_MSIMonitor::UpdateZoneLEDs(int /*zone*/) +{ + DeviceUpdateLEDs(); +} + +void RGBController_MSIMonitor::UpdateSingleLED(int /*led*/) +{ + DeviceUpdateLEDs(); +} + +void RGBController_MSIMonitor::DeviceUpdateMode() +{ + controller->Set(modes[active_mode].value, colors, 0x01); +} diff --git a/Controllers/MSIMonitorController/RGBController_MSIMonitor.h b/Controllers/MSIMonitorController/RGBController_MSIMonitor.h new file mode 100644 index 000000000..92fcbfff6 --- /dev/null +++ b/Controllers/MSIMonitorController/RGBController_MSIMonitor.h @@ -0,0 +1,38 @@ +/*---------------------------------------------------------*\ +| RGBController_MSIMonitor.cpp | +| | +| RGBController for MSI monitor (gaming controller) | +| | +| Andy Herbert 2026 May 16 | +| | +| This file is part of the OpenRGB project | +| SPDX-License-Identifier: GPL-2.0-or-later | +\*---------------------------------------------------------*/ + +#pragma once + +#include +#include "RGBController.h" +#include "MSIMonitorController.h" + +class RGBController_MSIMonitor : public RGBController +{ +public: + RGBController_MSIMonitor(MSIMonitorController* controller_ptr); + ~RGBController_MSIMonitor(); + + void SetupZones(); + void ResizeZone(int zone, int new_size); + void DeviceUpdateLEDs(); + void UpdateZoneLEDs(int zone); + void UpdateSingleLED(int led); + void DeviceUpdateMode(); + +private: + MSIMonitorController* controller; + std::thread* keepalive_thread; + std::atomic keepalive_thread_run; + std::chrono::time_point last_update_time; + + void KeepaliveThread(); +};