public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andreas Dilger <adilger@turbolabs.com>
To: Marcelo Tosatti <marcelo@conectiva.com.br>
Cc: lkml <linux-kernel@vger.kernel.org>
Subject: Re: 2.4.17: Dell Laptop extra buttons patch (fwd)
Date: Wed, 26 Dec 2001 13:53:38 -0700	[thread overview]
Message-ID: <20011226135338.B782@lynx.no> (raw)
In-Reply-To: <Pine.LNX.4.21.0112261711200.9973-200000@freak.distro.conectiva>
In-Reply-To: <Pine.LNX.4.21.0112261711200.9973-200000@freak.distro.conectiva>; from marcelo@conectiva.com.br on Wed, Dec 26, 2001 at 05:11:53PM -0200

On Dec 26, 2001  17:11 -0200, Marcelo Tosatti wrote:
> Could someone with Dell laptops test this for me ?

I don't have a Dell laptop, but AFAIK in the past patches like this have
been rejected by Andries Brouwer (I think) because it is possible to do
this from user space with key mapping tools like setkeycodes, xkeycaps,
or xmodmap.

Cheers, Andreas
> ---------- Forwarded message ----------
> From: Alan Ford <alan@whirlnet.co.uk>
> 
> Would you consider adding this patch to the next kernel release? I've been
> using it for a while, and since there is now the Dell laptop module for 
> extra BIOS features available in the kernel it seems this patch could be 
> appropriate and useful to include. It adds keycodes for the four shortcut 
> buttons that are provided on Dell Inspiron laptops.
> 
> Regards,
> Alan
> -- 
> Alan Ford * alan@whirlnet.co.uk 

> --- linux/drivers/char/pc_keyb.c.old	Wed Dec 26 19:06:21 2001
> +++ linux/drivers/char/pc_keyb.c	Wed Dec 26 19:08:13 2001
> @@ -228,9 +228,17 @@
>  #define E0_MSLW	125
>  #define E0_MSRW	126
>  #define E0_MSTM	127
> +/*
> + * Dell Inspiron Laptop Keyboard has four shortcut buttons
> + * e0 01 - e0 04  [alan@whirlnet.co.uk]
> + */
> +#define E0_DELL1 121
> +#define E0_DELL2 122
> +#define E0_DELL3 123
> +#define E0_DELL4 124
>  
>  static unsigned char e0_keys[128] = {
> -  0, 0, 0, 0, 0, 0, 0, 0,			      /* 0x00-0x07 */
> +  0, E0_DELL1, E0_DELL2, E0_DELL3, E0_DELL4, 0, 0, 0, /* 0x00-0x07 */
>    0, 0, 0, 0, 0, 0, 0, 0,			      /* 0x08-0x0f */
>    0, 0, 0, 0, 0, 0, 0, 0,			      /* 0x10-0x17 */
>    0, 0, 0, 0, E0_KPENTER, E0_RCTRL, 0, 0,	      /* 0x18-0x1f */
--
Andreas Dilger
http://sourceforge.net/projects/ext2resize/
http://www-mddsp.enel.ucalgary.ca/People/adilger/


  reply	other threads:[~2001-12-26 20:54 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-12-26 19:11 2.4.17: Dell Laptop extra buttons patch (fwd) Marcelo Tosatti
2001-12-26 20:53 ` Andreas Dilger [this message]
2001-12-26 19:40   ` Marcelo Tosatti
  -- strict thread matches above, loose matches on Subject: below --
2001-12-28  6:27 Zwane Mwaikambo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20011226135338.B782@lynx.no \
    --to=adilger@turbolabs.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcelo@conectiva.com.br \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox