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/
next prev parent 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