linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Dell Studio 1555 eject key does not work ( small patch to fix  included )
@ 2010-06-02 21:14 Islam Amer
  2010-06-02 21:34 ` Matthew Garrett
                   ` (2 more replies)
  0 siblings, 3 replies; 21+ messages in thread
From: Islam Amer @ 2010-06-02 21:14 UTC (permalink / raw)
  To: linux-kernel; +Cc: platform-driver-x86, Matthew Garrett, akpm

Hello,

Pressing the eject key on my Dell Studio 1555 does not work and dmesg produces
this message :
dell-wmi: Unknown key 0 pressed

Adding a debugging printk in dell-wmi.c after line 222 like this :

printk(KERN_INFO "dell:wmi 0x%x , 0x%x \n", buffer_entry[1], buffer_entry[2]);

dmesg now shows :

dell:wmi 0x0 , 0xe009
dell-wmi: Unknown key 0 pressed

So for some reason buffer_entry[1] is used although it is empty.

Falling back to buffer_entry[2] in case buffer_entry[1] is 0x0 makes
the button work.

I suspect it might be better to fix the "dell_new_hk_type" logic though

I had submitted this as
https://bugzilla.kernel.org/show_bug.cgi?id=16075 but repeating the
information and patch
here as per Andrew Morton's suggestion.


Thanks.

--- linux-sidux-2.6-2.6.34/drivers/platform/x86/dell-wmi.c.orig	2010-06-03
01:02:17.418824168 +0400
+++ linux-sidux-2.6-2.6.34/drivers/platform/x86/dell-wmi.c	2010-06-03
01:01:40.641833249 +0400
@@ -221,7 +221,7 @@ static void dell_wmi_notify(u32 value, v
 			return;
 		}

-		if (dell_new_hk_type)
+		if (dell_new_hk_type || buffer_entry[1] == 0x0)
 			reported_key = (int)buffer_entry[2];
 		else
 			reported_key = (int)buffer_entry[1] & 0xffff;

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

end of thread, other threads:[~2010-09-23 19:18 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-02 21:14 Dell Studio 1555 eject key does not work ( small patch to fix included ) Islam Amer
2010-06-02 21:34 ` Matthew Garrett
2010-06-03  1:57   ` Rezwanul_Kabir
2010-06-03 20:16     ` Islam Amer
2010-06-08 10:57       ` Islam Amer
2010-06-08 16:33         ` Rezwanul_Kabir
2010-06-10 18:36         ` Rezwanul_Kabir
2010-06-10 18:40           ` Matthew Garrett
2010-06-10 23:51             ` Islam Amer
2010-06-11  0:15               ` Rezwanul_Kabir
2010-06-11 13:28                 ` Islam Amer
2010-06-11 14:08                   ` Tim Gardner
2010-06-11 14:23                     ` Islam Amer
2010-06-11 18:02                       ` Islam Amer
2010-06-11 18:35                         ` Tim Gardner
2010-06-12  1:26                           ` Islam Amer
2010-06-15 17:23                             ` Rezwanul_Kabir
2010-06-15 17:28                               ` Matthew Garrett
2010-06-24 15:09 ` Matthew Garrett
2010-09-06 19:12 ` Kyle McMartin
2010-09-23 18:43   ` [stable] " Greg KH

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).