Bug 14796
Summary: | yealink kernel module put errors tens times a second to sys messages but works fine | ||
---|---|---|---|
Product: | Drivers | Reporter: | Serg Krasnikov (su) |
Component: | Input Devices | Assignee: | drivers_input-devices |
Status: | RESOLVED OBSOLETE | ||
Severity: | normal | CC: | alan, cebbert, dmitry.torokhov, pahan, treitmayr |
Priority: | P1 | ||
Hardware: | All | ||
OS: | Linux | ||
Kernel Version: | 2.6.31.5 | Subsystem: | |
Regression: | No | Bisected commit-id: |
Description
Serg Krasnikov
2009-12-13 12:25:01 UTC
On Sun, Dec 13, 2009 at 12:25:02PM +0000, bugzilla-daemon@bugzilla.kernel.org wrote: > In source /usr/src/linux/drivers/input/misc/yealink.c line 434 found: Care to send a patch to the input subsystem maintainer for this? > Care to send a patch to the input subsystem maintainer for this?
I'd with pleasure, but to who exactly?
* drivers/usb/input/yealink.c
*
* Copyright (c) 2005 Henk Vergonet <Henk.Vergonet(c)gmail.com>
To that man?
Googled that people tried send him mail and received no answer from him (maybe that e-mail has changed). Driver hasn't changed since 2005 year..
I'll just change in from err to dbg. (In reply to comment #3) > I'll just change in from err to dbg. Apparently not done yet? No, not done. I looked more into the issue and realized that simply changing from err() to dbg() is wrong. The device apparently requires that we throttle sending URB requests at 10ms intervals. I have similar problem with this driver - https://bugzilla.redhat.com/show_bug.cgi?id=625751 but phone also does not work at all. @Dmitry Torokhov / @Serg Krasnikov My extended version of the driver performs scan requests at 100 ms intervals, which is perfectly sufficient for properly detecting key presses. This avoids the observed response 0xfd and significantly reduces the IRQ load from USB traffic. You could try it, however the W1DL might report a model number which is not known to the driver, as it does not blindly assume that any connected Yealink handset talks the P1K's protocol. So you might have to adjust the macro YLD_IS_P1K(v) in yealink.h Link: http://devbase.homelinux.org:5071/viewvc/voip/yealink-module (sorry for the slow server...) Still you will miss support for the dot matrix display. The protocol might be the same as implemented here: http://lekernel.net/blog/?p=91 |