linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Usage of new KEY_NUMERIC_* codes in an existing driver (yealink)
@ 2008-08-22 17:01 Thomas Reitmayr
  2008-08-22 20:02 ` Dmitry Torokhov
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Reitmayr @ 2008-08-22 17:01 UTC (permalink / raw)
  To: Linux-Input

Hi,
I am in the process of extending the existing yealink driver, partly to
support various other models. The existing driver as included in the
current kernel uses the shift key to report the keys "*" and "#", but I
am hesitating to do the same bad "trick" for the other models.

As I would like to eventually submit the extended driver upstream, what
is the recommended strategy in my situation regarding usage of the new
KEY_NUMERIC_* codes? Use them only for the new models (which would
result in an ugly mix), or also update the codes for the existing
USB-P1K model (which would break userspace programs but finally fix
things for some foreign keyboard layouts), or for the existing USB-P1K
model report the old and the new codes (which might look like two key
presses)?

Thanks for your advice,
-Thomas

PS: The updated driver is available at
http://www.devbase.at/svn/view.cgi/yealink-module/trunk/?root=voip
(still including #if's reg. kernel versions, some comments to be
corrected, etc.)


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Usage of new KEY_NUMERIC_* codes in an existing driver (yealink)
  2008-08-22 17:01 Usage of new KEY_NUMERIC_* codes in an existing driver (yealink) Thomas Reitmayr
@ 2008-08-22 20:02 ` Dmitry Torokhov
  2008-08-23 12:39   ` Thomas Reitmayr
  0 siblings, 1 reply; 3+ messages in thread
From: Dmitry Torokhov @ 2008-08-22 20:02 UTC (permalink / raw)
  To: Thomas Reitmayr; +Cc: Linux-Input

Hi Thomas,

On Fri, Aug 22, 2008 at 07:01:28PM +0200, Thomas Reitmayr wrote:
> Hi,
> I am in the process of extending the existing yealink driver, partly to
> support various other models. The existing driver as included in the
> current kernel uses the shift key to report the keys "*" and "#", but I
> am hesitating to do the same bad "trick" for the other models.
> 
> As I would like to eventually submit the extended driver upstream, what
> is the recommended strategy in my situation regarding usage of the new
> KEY_NUMERIC_* codes? Use them only for the new models (which would
> result in an ugly mix), or also update the codes for the existing
> USB-P1K model (which would break userspace programs but finally fix
> things for some foreign keyboard layouts), 

Do you think that we need to add more keymaps to the kernel? What I
woudl like to see is adding setkeycode/getkeycode support to yealink
so proper keymap can be loaded from userspace (udev, hal, whatever)
when a device is plugged into a box. USB-P1K coudl also get the new
keymap loaded from userspace while keeping the current legacy keymap
for existing users.

>or for the existing USB-P1K
> model report the old and the new codes (which might look like two key
> presses)?

No, that is not a good idea.

> Thanks for your advice,
> -Thomas
> 
> PS: The updated driver is available at
> http://www.devbase.at/svn/view.cgi/yealink-module/trunk/?root=voip
> (still including #if's reg. kernel versions, some comments to be
> corrected, etc.)
> 

Would you mind posting it as a patch - it is much easier to comment on
it in e-mail... Thanks!

-- 
Dmitry

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Usage of new KEY_NUMERIC_* codes in an existing driver (yealink)
  2008-08-22 20:02 ` Dmitry Torokhov
@ 2008-08-23 12:39   ` Thomas Reitmayr
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Reitmayr @ 2008-08-23 12:39 UTC (permalink / raw)
  To: Dmitry Torokhov; +Cc: Linux-Input

Thanks Dmitry,
I will take a look at the cm109 driver to see how setkeycode/getkeycode
support is implemented and how this fits into the yealink code.
As soon as I have done that (and cleaned up a few things) I will post
the code as a patch.
Best regards,
-Thomas


Am Freitag, den 22.08.2008, 16:02 -0400 schrieb Dmitry Torokhov:
> Hi Thomas,
> 
> On Fri, Aug 22, 2008 at 07:01:28PM +0200, Thomas Reitmayr wrote:
> > Hi,
> > I am in the process of extending the existing yealink driver, partly to
> > support various other models. The existing driver as included in the
> > current kernel uses the shift key to report the keys "*" and "#", but I
> > am hesitating to do the same bad "trick" for the other models.
> > 
> > As I would like to eventually submit the extended driver upstream, what
> > is the recommended strategy in my situation regarding usage of the new
> > KEY_NUMERIC_* codes? Use them only for the new models (which would
> > result in an ugly mix), or also update the codes for the existing
> > USB-P1K model (which would break userspace programs but finally fix
> > things for some foreign keyboard layouts), 
> 
> Do you think that we need to add more keymaps to the kernel? What I
> woudl like to see is adding setkeycode/getkeycode support to yealink
> so proper keymap can be loaded from userspace (udev, hal, whatever)
> when a device is plugged into a box. USB-P1K coudl also get the new
> keymap loaded from userspace while keeping the current legacy keymap
> for existing users.
> 
> >or for the existing USB-P1K
> > model report the old and the new codes (which might look like two key
> > presses)?
> 
> No, that is not a good idea.
> 
> > Thanks for your advice,
> > -Thomas
> > 
> > PS: The updated driver is available at
> > http://www.devbase.at/svn/view.cgi/yealink-module/trunk/?root=voip
> > (still including #if's reg. kernel versions, some comments to be
> > corrected, etc.)
> > 
> 
> Would you mind posting it as a patch - it is much easier to comment on
> it in e-mail... Thanks!
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-08-23 12:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-08-22 17:01 Usage of new KEY_NUMERIC_* codes in an existing driver (yealink) Thomas Reitmayr
2008-08-22 20:02 ` Dmitry Torokhov
2008-08-23 12:39   ` Thomas Reitmayr

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).