All of lore.kernel.org
 help / color / mirror / Atom feed
From: Len Brown <lenb@kernel.org>
To: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: linux-acpi@vger.kernel.org, ibm-acpi-devel@lists.sourceforge.net
Subject: Re: [PATCH 1/1] ACPI: thinkpad-acpi: fix lenovo keymap for brightness
Date: Thu, 13 Dec 2007 22:04:17 -0500	[thread overview]
Message-ID: <200712132204.17820.lenb@kernel.org> (raw)
In-Reply-To: <1197555249-8520-2-git-send-email-hmh@hmh.eng.br>

Applied.

thanks,
-Len

On Thursday 13 December 2007 09:14, Henrique de Moraes Holschuh wrote:
> Several reports from X60 users complained that the default Lenovo keymap
> issuing EV_KEY KEY_BRIGHTNESS_UP/DOWN input events caused major issues when
> the proper brightness support through ACPI video.c was loaded.
> 
> Therefore, remove the generation of these events by default, which is the
> right thing for T60, X60, R60, T61, X61 and R61 with their latest BIOSes.
> 
> Distros that want to misuse these events into OSD reporting (which requires
> an ugly hack from hell in HAL) are welcome to set up the key map they need
> through HAL.  That way, we don't break everyone else's systems.
> 
> Signed-off-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
> ---
>  drivers/misc/thinkpad_acpi.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/misc/thinkpad_acpi.c b/drivers/misc/thinkpad_acpi.c
> index ab23a32..cf56647 100644
> --- a/drivers/misc/thinkpad_acpi.c
> +++ b/drivers/misc/thinkpad_acpi.c
> @@ -987,9 +987,9 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
>  		KEY_UNKNOWN,	/* 0x0C: FN+BACKSPACE */
>  		KEY_UNKNOWN,	/* 0x0D: FN+INSERT */
>  		KEY_UNKNOWN,	/* 0x0E: FN+DELETE */
> -		KEY_BRIGHTNESSUP,	/* 0x0F: FN+HOME (brightness up) */
> +		KEY_RESERVED,	/* 0x0F: FN+HOME (brightness up) */
>  		/* Scan codes 0x10 to 0x1F: Extended ACPI HKEY hot keys */
> -		KEY_BRIGHTNESSDOWN,	/* 0x10: FN+END (brightness down) */
> +		KEY_RESERVED,	/* 0x10: FN+END (brightness down) */
>  		KEY_RESERVED,	/* 0x11: FN+PGUP (thinklight toggle) */
>  		KEY_UNKNOWN,	/* 0x12: FN+PGDOWN */
>  		KEY_ZOOM,	/* 0x13: FN+SPACE (zoom) */

      reply	other threads:[~2007-12-14  3:04 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-13 14:14 [GIT PATCH] thinkpad-acpi queue for 2.6.24-rc Henrique de Moraes Holschuh
     [not found] ` <1197555249-8520-1-git-send-email-hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
2007-12-13 14:14   ` [PATCH 1/1] ACPI: thinkpad-acpi: fix lenovo keymap for brightness Henrique de Moraes Holschuh
2007-12-14  3:04     ` Len Brown [this message]

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=200712132204.17820.lenb@kernel.org \
    --to=lenb@kernel.org \
    --cc=hmh@hmh.eng.br \
    --cc=ibm-acpi-devel@lists.sourceforge.net \
    --cc=linux-acpi@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.