Hi Wade,
I retired from Plantronics in 2020. The original driver did not allow
mute button to be mapped as there were mute synchronization issues.
The headset needs to receive some type of feedback from the host when it
sends the mute event in order to synchronize with the host, ideally the
host setting or clearing the mute control in the audio control interface
but setting/clearing the mute LED would also work.
At the time Ubuntu and Chromebooks did not feedback mute state and it
was possible to mute from the headset and then unmute from the mixer or
keyboard and the headset would stay muted. The only way to unmute was
with the headset button. This was an unacceptable user experience so we
blocked mapping.
If you want to try mapping mute event then you also need to allow
mapping the mute LED for possible host feedback.
(HID_UP_TELEPHONY | 0x2f) is telephony page mute button
(HID_UP_LED | 0x09) is LED page mute LED
Then you need to test more than just the event getting to user space.
You need to check mute synchronization with the host mixer under all
mute/unmute use cases.
Regards,
Terry Junge
On 9/24/24 2:00 AM, Wang, Wade wrote:
Hi Benjamin and Greg,
May I know the review progress and anything I need to change? Thanks
Regards
Wade
-----Original Message-----
From: Wang, Wade
Sent: Monday, September 16, 2024 4:13 PM
To: Benjamin Tissoires <bentiss@xxxxxxxxxx>
Cc: jikos@xxxxxxxxxx; linux-input@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; stable@xxxxxxxxxxxxxxx
Subject: RE: [PATCH] HID: plantronics: Update to map micmute controls
Hi Benjamin,
This patch is for all Poly HS devices, and it does not depends on other patches, it can apply directly by " git am -3".
With this patch, MicMute button key event will be send to user space, I have tested on the below Poly devices:
Plantronics EncorePro 500 Series
Plantronics Blackwire_3325 Series
Poly Voyager 4320 HS + BT700 Dongle
Regards
Wade
-----Original Message-----
From: Benjamin Tissoires <bentiss@xxxxxxxxxx>
Sent: Friday, September 13, 2024 10:04 PM
To: Wang, Wade <wade.wang@xxxxxx>
Cc: jikos@xxxxxxxxxx; linux-input@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; stable@xxxxxxxxxxxxxxx
Subject: Re: [PATCH] HID: plantronics: Update to map micmute controls
CAUTION: External Email
On Sep 13 2024, Wade Wang wrote:
telephony page of Plantronics headset is ignored currently, it caused
micmute button no function, Now follow native HID key mapping for
telephony page map, telephony micmute key is enabled by default
For which devices this patch is required? Is it related to the other patch you sent today? If so please make a mention of the concerned devices and make sure both patches are sent in a single v3 series.
Also, have you tested this change with other Plantronics headsets? Where there any changes in behavior from them?
Cheers,
Benjamin
Cc: stable@xxxxxxxxxxxxxxx
Signed-off-by: Wade Wang <wade.wang@xxxxxx>
---
drivers/hid/hid-plantronics.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/hid/hid-plantronics.c
b/drivers/hid/hid-plantronics.c index 2a19f3646ecb..2d17534fce61
100644
--- a/drivers/hid/hid-plantronics.c
+++ b/drivers/hid/hid-plantronics.c
@@ -77,10 +77,10 @@ static int plantronics_input_mapping(struct hid_device *hdev,
}
}
/* handle standard types - plt_type is 0xffa0uuuu or 0xffa2uuuu */
- /* 'basic telephony compliant' - allow default consumer page map */
+ /* 'basic telephony compliant' - allow default consumer &
+ telephony page map */
else if ((plt_type & HID_USAGE) >= PLT_BASIC_TELEPHONY &&
(plt_type & HID_USAGE) != PLT_BASIC_EXCEPTION) {
- if (PLT_ALLOW_CONSUMER)
+ if (PLT_ALLOW_CONSUMER || (usage->hid & HID_USAGE_PAGE)
+ == HID_UP_TELEPHONY)
goto defaulted;
}
/* not 'basic telephony' - apply legacy mapping */
--
2.34.1
