lkml.org 
[lkml]   [2020]   [Nov]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.9 187/252] HID: logitech-dj: Handle quad/bluetooth keyboards with a builtin trackpad
    Date
    From: Hans de Goede <hdegoede@redhat.com>

    [ Upstream commit ee5e58418a854755201eb4952b1230d873a457d5 ]

    Some quad/bluetooth keyboards, such as the Dinovo Edge (Y-RAY81) have a
    builtin touchpad. In this case when asking the receiver for paired devices,
    we get only 1 paired device with a device_type of REPORT_TYPE_KEYBOARD.

    This means that we do not instantiate a second dj_hiddev for the mouse
    (as we normally would) and thus there is no place for us to forward the
    mouse input reports to, causing the touchpad part of the keyboard to not
    work.

    There is no way for us to detect these keyboards, so this commit adds
    an array with device-ids for such keyboards and when a keyboard is on
    this list it adds STD_MOUSE to the reports_supported bitmap for the
    dj_hiddev created for the keyboard fixing the touchpad not working.

    Using a list of device-ids for this is not ideal, but there are only
    very few such keyboards so this should be fine. Besides the Dinovo Edge,
    other known wireless Logitech keyboards with a builtin touchpad are:

    * Dinovo Mini (TODO add its device-id to the list)
    * K400 (uses a unifying receiver so is not affected)
    * K600 (uses a unifying receiver so is not affected)

    Cc: stable@vger.kernel.org
    BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1811424
    Fixes: f2113c3020ef ("HID: logitech-dj: add support for Logitech Bluetooth Mini-Receiver")
    Signed-off-by: Hans de Goede <hdegoede@redhat.com>
    Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/hid/hid-logitech-dj.c | 19 +++++++++++++++++++
    1 file changed, 19 insertions(+)

    diff --git a/drivers/hid/hid-logitech-dj.c b/drivers/hid/hid-logitech-dj.c
    index 38ee25a813b9e..1cafb65428b04 100644
    --- a/drivers/hid/hid-logitech-dj.c
    +++ b/drivers/hid/hid-logitech-dj.c
    @@ -866,11 +866,23 @@ static void logi_dj_recv_queue_notification(struct dj_receiver_dev *djrcv_dev,
    schedule_work(&djrcv_dev->work);
    }

    +/*
    + * Some quad/bluetooth keyboards have a builtin touchpad in this case we see
    + * only 1 paired device with a device_type of REPORT_TYPE_KEYBOARD. For the
    + * touchpad to work we must also forward mouse input reports to the dj_hiddev
    + * created for the keyboard (instead of forwarding them to a second paired
    + * device with a device_type of REPORT_TYPE_MOUSE as we normally would).
    + */
    +static const u16 kbd_builtin_touchpad_ids[] = {
    + 0xb309, /* Dinovo Edge */
    +};
    +
    static void logi_hidpp_dev_conn_notif_equad(struct hid_device *hdev,
    struct hidpp_event *hidpp_report,
    struct dj_workitem *workitem)
    {
    struct dj_receiver_dev *djrcv_dev = hid_get_drvdata(hdev);
    + int i, id;

    workitem->type = WORKITEM_TYPE_PAIRED;
    workitem->device_type = hidpp_report->params[HIDPP_PARAM_DEVICE_INFO] &
    @@ -882,6 +894,13 @@ static void logi_hidpp_dev_conn_notif_equad(struct hid_device *hdev,
    workitem->reports_supported |= STD_KEYBOARD | MULTIMEDIA |
    POWER_KEYS | MEDIA_CENTER |
    HIDPP;
    + id = (workitem->quad_id_msb << 8) | workitem->quad_id_lsb;
    + for (i = 0; i < ARRAY_SIZE(kbd_builtin_touchpad_ids); i++) {
    + if (id == kbd_builtin_touchpad_ids[i]) {
    + workitem->reports_supported |= STD_MOUSE;
    + break;
    + }
    + }
    break;
    case REPORT_TYPE_MOUSE:
    workitem->reports_supported |= STD_MOUSE | HIDPP;
    --
    2.27.0


    \
     
     \ /
      Last update: 2020-11-23 14:40    [W:8.209 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site