linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PATCH] thinkpad-acpi queue for 2.6.24-rc
@ 2007-12-13 14:14 Henrique de Moraes Holschuh
       [not found] ` <1197555249-8520-1-git-send-email-hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: Henrique de Moraes Holschuh @ 2007-12-13 14:14 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, ibm-acpi-devel


Len,

Please send this patch series (which is composed of a single patch) to
Linus for merging in 2.6.24-rc.

It fixes problems on the latest Lenovo ThinkPads (the *61 series), on
most Linux distros.

Thank you.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh

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

* [PATCH 1/1] ACPI: thinkpad-acpi: fix lenovo keymap for brightness
       [not found] ` <1197555249-8520-1-git-send-email-hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
@ 2007-12-13 14:14   ` Henrique de Moraes Holschuh
  2007-12-14  3:04     ` Len Brown
  0 siblings, 1 reply; 3+ messages in thread
From: Henrique de Moraes Holschuh @ 2007-12-13 14:14 UTC (permalink / raw)
  To: Len Brown
  Cc: linux-acpi-u79uwXL29TY76Z2rM5mHXA, Henrique de Moraes Holschuh,
	ibm-acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f

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-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
---
 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) */
-- 
1.5.3.2


-------------------------------------------------------------------------
SF.Net email is sponsored by:
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services
for just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace

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

* Re: [PATCH 1/1] ACPI: thinkpad-acpi: fix lenovo keymap for brightness
  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
  0 siblings, 0 replies; 3+ messages in thread
From: Len Brown @ 2007-12-14  3:04 UTC (permalink / raw)
  To: Henrique de Moraes Holschuh; +Cc: linux-acpi, ibm-acpi-devel

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) */

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

end of thread, other threads:[~2007-12-14  3:04 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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 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).