Re: [PATCH] HID: logitech-dj: Fix check of logi_dj_recv_query_paired_devices()
From: Jiri Kosina <jikos@kernel.org>
Date: 2019-08-05 08:51:40
Also in:
lkml
From: Jiri Kosina <jikos@kernel.org>
Date: 2019-08-05 08:51:40
Also in:
lkml
On Wed, 31 Jul 2019, Hans de Goede wrote:
quoted
quoted
quoted
In delayedwork_callback(), logi_dj_recv_query_paired_devices may return positive value while success now, so check it correctly.quoted
quoted
Fixes: dbcbabf7da92 ("HID: logitech-dj: fix return value of Fixes: logi_dj_recv_query_hidpp_devices") Signed-off-by: YueHaibing <redacted>Reviewed-by: Petr Vorel <pvorel@suse.cz>OK, not only it didn't fix problems with logitech mouse (see below), but removing mouses USB dongle effectively crashes kernel, so this one probably shouldn't be applied :). [ 330.721629] logitech-djreceiver: probe of 0003:046D:C52F.0013 failed with error 7 [ 331.462335] hid 0003:046D:C52F.0013: delayedwork_callback: logi_dj_recv_query_paired_devices error: 7Please test my patch titled: "HID: logitech-dj: Really fix return value of logi_dj_recv_query_hidpp_devices" which should fix this.
Hans, have I been CCed on that patch? I don't seem to see it in in inbox. Thanks, -- Jiri Kosina SUSE Labs