feat: expose DS5 player-indicator and mic-mute LED outputs

Add player_led and mic_led to GamepadOutputKind and GamepadOutput, and
parse them from the DualSense output report in append_dualsense_outputs
(player indicator gated by valid_flag1 bit 0x10 at offset+43; mic-mute LED
gated by bit 0x01 at offset+8), mirroring the existing rgb_led handling.
The DS5 profile advertises both via new supports_player_led /
supports_mic_led capability flags.

Signed-off-by: Joel Coelho <joelfernandocscoelho@gmail.com>

Author: joelfcscoelho

src/core/gamepad_adapter.cpp

@@ -83,6 +83,8 @@ namespace lvh {
     support.supports_rumble = profile.capabilities.supports_rumble;
     support.supports_rgb_led = profile.capabilities.supports_rgb_led;
     support.supports_adaptive_triggers = profile.capabilities.supports_adaptive_triggers;
+    support.supports_player_led = profile.capabilities.supports_player_led;
+    support.supports_mic_led = profile.capabilities.supports_mic_led;
     support.supports_motion = profile.capabilities.supports_motion;
     support.supports_touchpad = profile.capabilities.supports_touchpad;
     support.supports_battery = profile.capabilities.supports_battery;
@@ -144,6 +146,10 @@ namespace lvh {
         return support.supports_adaptive_triggers;
       case raw_report:
         return profile.output_report_size > 0U;
+      case player_led:
+        return support.supports_player_led;
+      case mic_led:
+        return support.supports_mic_led;
     }
 
     return false;

src/core/profiles.cpp

@@ -2012,6 +2012,8 @@ namespace lvh::profiles {
         .supports_rgb_led = true,
         .supports_battery = true,
         .supports_adaptive_triggers = true,
+        .supports_player_led = true,
+        .supports_mic_led = true,
       };
       profile.report_descriptor =
         bus_type == BusType::bluetooth ? make_dualsense_bluetooth_report_descriptor() : make_dualsense_usb_report_descriptor();

src/core/report.cpp

@@ -61,7 +61,9 @@ namespace lvh::reports {
 
     constexpr auto dualsense_flag0_left_trigger = std::byte {0x08};
 
+    constexpr auto dualsense_flag1_mic_mute_led = std::byte {0x01};
     constexpr auto dualsense_flag1_lightbar = std::byte {0x04};
+    constexpr auto dualsense_flag1_player_indicator = std::byte {0x10};
 
     constexpr auto dualsense_flag2_compatible_vibration = std::byte {0x04};
 
@@ -936,6 +938,22 @@ namespace lvh::reports {
         outputs.push_back(std::move(output));
       }
 
+      if (has_flag(valid_flag1, dualsense_flag1_player_indicator)) {
+        GamepadOutput output;
+        output.kind = GamepadOutputKind::player_led;
+        output.player_led = raw_report[offset + 43U];
+        output.raw_report = raw_report;
+        outputs.push_back(std::move(output));
+      }
+
+      if (has_flag(valid_flag1, dualsense_flag1_mic_mute_led)) {
+        GamepadOutput output;
+        output.kind = GamepadOutputKind::mic_led;
+        output.mic_led = raw_report[offset + 8U];
+        output.raw_report = raw_report;
+        outputs.push_back(std::move(output));
+      }
+
       const auto trigger_flags = valid_flag0 & (dualsense_flag0_left_trigger | dualsense_flag0_right_trigger);
       if (trigger_flags != zero_byte) {
         GamepadOutput output;

src/include/libvirtualhid/gamepad_adapter.hpp

@@ -40,6 +40,16 @@ namespace lvh {
      */
     bool supports_adaptive_triggers = false;
 
+    /**
+     * @brief Whether the profile supports player-indicator LED output.
+     */
+    bool supports_player_led = false;
+
+    /**
+     * @brief Whether the profile supports mic-mute LED output.
+     */
+    bool supports_mic_led = false;
+
     /**
      * @brief Whether the profile exposes motion sensor input.
      */

src/include/libvirtualhid/types.hpp

@@ -270,6 +270,16 @@ namespace lvh {
      * @brief Whether the profile supports adaptive trigger output.
      */
     bool supports_adaptive_triggers = false;
+
+    /**
+     * @brief Whether the profile supports player-indicator LED output.
+     */
+    bool supports_player_led = false;
+
+    /**
+     * @brief Whether the profile supports mic-mute LED output.
+     */
+    bool supports_mic_led = false;
   };
 
   /**
@@ -998,6 +1008,8 @@ namespace lvh {
     adaptive_triggers,  ///< Adaptive trigger output.
     raw_report,  ///< Raw output report bytes.
     trigger_rumble,  ///< Independent trigger rumble output.
+    player_led,  ///< Player-indicator LED output.
+    mic_led,  ///< Mic-mute LED output.
   };
 
   /**
@@ -1044,6 +1056,16 @@ namespace lvh {
      */
     std::uint8_t blue = 0;
 
+    /**
+     * @brief Player-indicator LED bitmask.
+     */
+    std::uint8_t player_led = 0;
+
+    /**
+     * @brief Mic-mute LED state (0 = off, 1 = on, 2 = pulse).
+     */
+    std::uint8_t mic_led = 0;
+
     /**
      * @brief Adaptive trigger event flags from a profile-specific output report.
      */

tests/unit/test_gamepad_adapter.cpp

@@ -44,6 +44,8 @@ TEST(GamepadAdapterTest, ReportsProfileSupport) {
   EXPECT_TRUE(dualsense_support.supports_rumble);
   EXPECT_TRUE(dualsense_support.supports_rgb_led);
   EXPECT_TRUE(dualsense_support.supports_adaptive_triggers);
+  EXPECT_TRUE(dualsense_support.supports_player_led);
+  EXPECT_TRUE(dualsense_support.supports_mic_led);
   EXPECT_TRUE(dualsense_support.supports_motion);
   EXPECT_TRUE(dualsense_support.supports_touchpad);
   EXPECT_TRUE(dualsense_support.supports_battery);
@@ -96,6 +98,8 @@ TEST(GamepadAdapterTest, ChecksButtonsAndOutputsByProfile) {
   EXPECT_FALSE(lvh::supports_gamepad_output(dualshock4, lvh::GamepadOutputKind::trigger_rumble));
   EXPECT_TRUE(lvh::supports_gamepad_output(dualshock4, lvh::GamepadOutputKind::raw_report));
   EXPECT_TRUE(lvh::supports_gamepad_output(dualsense, lvh::GamepadOutputKind::adaptive_triggers));
+  EXPECT_TRUE(lvh::supports_gamepad_output(dualsense, lvh::GamepadOutputKind::player_led));
+  EXPECT_TRUE(lvh::supports_gamepad_output(dualsense, lvh::GamepadOutputKind::mic_led));
   EXPECT_TRUE(lvh::supports_gamepad_output(switch_pro, lvh::GamepadOutputKind::rumble));
   EXPECT_TRUE(lvh::supports_gamepad_output(switch_pro, lvh::GamepadOutputKind::raw_report));
   EXPECT_TRUE(lvh::supports_gamepad_output(generic, lvh::GamepadOutputKind::raw_report));

tests/unit/test_profiles.cpp

@@ -213,6 +213,8 @@ TEST(ProfileTest, StreamingControllerProfilesArePresent) {
   EXPECT_TRUE(dualsense.capabilities.supports_touchpad);
   EXPECT_TRUE(dualsense.capabilities.supports_rgb_led);
   EXPECT_TRUE(dualsense.capabilities.supports_adaptive_triggers);
+  EXPECT_TRUE(dualsense.capabilities.supports_player_led);
+  EXPECT_TRUE(dualsense.capabilities.supports_mic_led);
   EXPECT_GT(dualsense.input_report_size, 14U);
   EXPECT_GT(dualsense.output_report_size, 5U);
   EXPECT_EQ(dualsense.manufacturer, "Sony Interactive Entertainment");

tests/unit/test_report.cpp

@@ -525,6 +525,45 @@ TEST(ReportTest, ParsesDualSenseBluetoothOutputReportEvents) {
   EXPECT_EQ(outputs[2].kind, lvh::GamepadOutputKind::adaptive_triggers);
 }
 
+TEST(ReportTest, ParsesDualSensePlayerAndMicLedOutputReportEvents) {
+  const auto profile = lvh::profiles::dualsense_usb();
+  std::vector<std::uint8_t> report(profile.output_report_size, 0);
+  report[0] = 0x02;
+  report[2] = 0x11;  // valid_flag1: player-indicator (0x10) | mic-mute LED (0x01)
+  report[9] = 0x02;  // mic-mute LED state (offset + 8): pulse
+  report[44] = 0x1F;  // player-indicator LEDs (offset + 43): all five lit
+
+  const auto outputs = lvh::reports::parse_output_reports(profile, report);
+
+  ASSERT_EQ(outputs.size(), 2U);
+  EXPECT_EQ(outputs[0].kind, lvh::GamepadOutputKind::player_led);
+  EXPECT_EQ(outputs[0].player_led, 0x1F);
+  EXPECT_EQ(outputs[1].kind, lvh::GamepadOutputKind::mic_led);
+  EXPECT_EQ(outputs[1].mic_led, 0x02);
+}
+
+TEST(ReportTest, OmitsDualSensePlayerAndMicLedWhenFlagsClear) {
+  const auto profile = lvh::profiles::dualsense_usb();
+  std::vector<std::uint8_t> report(profile.output_report_size, 0);
+  report[0] = 0x02;
+  report[1] = 0x01;  // valid_flag0: rumble only
+  report[2] = 0x04;  // valid_flag1: lightbar only (player/mic flags clear)
+  report[3] = 0x80;
+  report[4] = 0x40;
+  report[9] = 0x02;  // mic-mute LED byte populated, but its valid flag is clear
+  report[44] = 0x1F;  // player-indicator byte populated, but its valid flag is clear
+  report[45] = 0x11;
+  report[46] = 0x22;
+  report[47] = 0x33;
+
+  const auto outputs = lvh::reports::parse_output_reports(profile, report);
+
+  for (const auto &output : outputs) {
+    EXPECT_NE(output.kind, lvh::GamepadOutputKind::player_led);
+    EXPECT_NE(output.kind, lvh::GamepadOutputKind::mic_led);
+  }
+}
+
 TEST(ReportTest, ParsesDualShock4OutputReportEvents) {
   const auto profile = lvh::profiles::dualshock4_usb();
   std::vector<std::uint8_t> report(profile.output_report_size, 0);

tests/unit/test_virtualhid_control_model.cpp

@@ -58,6 +58,8 @@ TEST(VirtualHidControlModelTest, NamesKnownAndFallbackEnumValues) {
   EXPECT_EQ(control::output_kind_name(lvh::GamepadOutputKind::adaptive_triggers), L"adaptive triggers");
   EXPECT_EQ(control::output_kind_name(lvh::GamepadOutputKind::raw_report), L"raw report");
   EXPECT_EQ(control::output_kind_name(lvh::GamepadOutputKind::trigger_rumble), L"trigger rumble");
+  EXPECT_EQ(control::output_kind_name(lvh::GamepadOutputKind::player_led), L"player led");
+  EXPECT_EQ(control::output_kind_name(lvh::GamepadOutputKind::mic_led), L"mic led");
   EXPECT_EQ(control::output_kind_name(static_cast<lvh::GamepadOutputKind>(255)), L"raw report");
 
   EXPECT_EQ(control::battery_state_name(lvh::GamepadBatteryState::unknown), L"unknown");

tools/virtualhid_control_model.cpp

@@ -77,6 +77,10 @@ namespace lvh::tools::virtualhid_control {
         return L"raw report";
       case trigger_rumble:
         return L"trigger rumble";
+      case player_led:
+        return L"player led";
+      case mic_led:
+        return L"mic led";
     }
     return L"raw report";
   }