linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Len Brown <lenb@kernel.org>
To: linux-acpi@vger.kernel.org
Cc: Henrique de Moraes Holschuh <hmh@hmh.eng.br>,
	Len Brown <len.brown@intel.com>
Subject: [PATCH 28/32] ACPI: thinkpad-acpi: check version of hot key firmware
Date: Fri, 21 Sep 2007 23:46:34 -0400	[thread overview]
Message-ID: <1190432821532-git-send-email-lenb@kernel.org> (raw)
Message-ID: <5a45e2370d4cb99f8afc48274e4bba0978a089df.1190432418.git.len.brown@intel.com> (raw)
In-Reply-To: <1190432820190-git-send-email-lenb@kernel.org>
In-Reply-To: <fd1caaed466de2ee100e250b6c755376eda7ba3b.1190432417.git.len.brown@intel.com>

From: Henrique de Moraes Holschuh <hmh@hmh.eng.br>

Check the HKEY firmware version (HKEY.MHKV handler), and refuse to load if
it is unknown.  Use this instead of the presence of HKEY.DHKV to detect hot
key mask capability.

Signed-off-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Signed-off-by: Len Brown <len.brown@intel.com>
---
 drivers/misc/thinkpad_acpi.c |   28 +++++++++++++++++++++++-----
 1 files changed, 23 insertions(+), 5 deletions(-)

diff --git a/drivers/misc/thinkpad_acpi.c b/drivers/misc/thinkpad_acpi.c
index 2155139..9a61140 100644
--- a/drivers/misc/thinkpad_acpi.c
+++ b/drivers/misc/thinkpad_acpi.c
@@ -999,6 +999,7 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
 
 	int res, i;
 	int status;
+	int hkeyv;
 
 	vdbg_printk(TPACPI_DBG_INIT, "initializing hotkey subdriver\n");
 
@@ -1024,18 +1025,35 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
 			return res;
 
 		/* mask not supported on 570, 600e/x, 770e, 770x, A21e, A2xm/p,
-		   A30, R30, R31, T20-22, X20-21, X22-24 */
-		tp_features.hotkey_mask =
-			acpi_evalf(hkey_handle, NULL, "DHKN", "qv");
+		   A30, R30, R31, T20-22, X20-21, X22-24.  Detected by checking
+		   for HKEY interface version 0x100 */
+		if (acpi_evalf(hkey_handle, &hkeyv, "MHKV", "qd")) {
+			if ((hkeyv >> 8) != 1) {
+				printk(IBM_ERR "unknown version of the "
+				       "HKEY interface: 0x%x\n", hkeyv);
+				printk(IBM_ERR "please report this to %s\n",
+				       IBM_MAIL);
+			} else {
+				/*
+				 * MHKV 0x100 in A31, R40, R40e,
+				 * T4x, X31, and later
+				 * */
+				tp_features.hotkey_mask = 1;
+			}
+		}
 
 		vdbg_printk(TPACPI_DBG_INIT, "hotkey masks are %s\n",
 			str_supported(tp_features.hotkey_mask));
 
 		if (tp_features.hotkey_mask) {
-			/* MHKA available in A31, R40, R40e, T4x, X31, and later */
 			if (!acpi_evalf(hkey_handle, &hotkey_all_mask,
-					"MHKA", "qd"))
+					"MHKA", "qd")) {
+				printk(IBM_ERR
+				       "missing MHKA handler, "
+				       "please report this to %s\n",
+				       IBM_MAIL);
 				hotkey_all_mask = 0x080cU; /* FN+F12, FN+F4, FN+F3 */
+			}
 		}
 
 		res = hotkey_get(&hotkey_orig_status, &hotkey_orig_mask);
-- 
1.5.3.1.27.g57543

  parent reply	other threads:[~2007-09-22  3:47 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-22  3:46 ACPI patches i test branch queued for 2.6.24 Len Brown
     [not found] ` <fd1caaed466de2ee100e250b6c755376eda7ba3b.1190432417.git.len.brown@intel.com>
2007-09-22  3:46   ` [PATCH 01/32] sony-laptop: old Vaio models contain 2 IO port entries Len Brown
     [not found]   ` <d0482533c73a8685f7ce0951a10280cfd58b8825.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 02/32] fujitsu-laptop: create Fujitsu laptop platform specific driver Len Brown
     [not found]   ` <21bc42ab852549f4a547d18d77e0e4d1b24ffd96.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 03/32] ACPI: thermal: use round_jiffies when thermal zone polling is enabled Len Brown
     [not found]   ` <481ad1a30bb2f3fba90f004055ad266f27e19199.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 04/32] ACPI: AC: Add sysfs interface Len Brown
     [not found]   ` <0c1304d9e1295dd19568a937f106379f8e3d7629.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 05/32] ACPI: Battery: don't use acpi_extract_package() Len Brown
     [not found]   ` <f4e23e2f3ce393bf95038f7eb24352592ec9d91a.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 06/32] ACPI: Battery: simplify update scheme Len Brown
     [not found]   ` <89543b032d6a389f7d75e239b1d45b0ce3604a5f.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 07/32] ACPI: Battery: Misc clean-ups, no functional changes Len Brown
     [not found]   ` <cb7ce92d4707dcac07c934f794195e39bd08221f.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 08/32] ACPI: Add acpi_bus_generate_event4() function Len Brown
     [not found]   ` <e59812330152303aff434d2d3a8a2d9340fbb66e.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 09/32] ACPI: EC: Add new query handler to list head Len Brown
     [not found]   ` <f0ebb0c2228ebb95b1dc400bcc05d6fe873e2b3f.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 10/32] ACPI: SBS: Split host controller (ACPI0001) from SBS driver (ACPI0002) Len Brown
     [not found]   ` <86b80c709d723d55844097e589f2e58d39857d21.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 11/32] ACPI: SBS: Simplify data structures in SBS Len Brown
     [not found]   ` <011c6997c0b54067d89fde31b4cb5b01792afcd3.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 12/32] ACPI: SBS: Make SBS reads table-driven Len Brown
     [not found]   ` <fa5e2338bbbe17700781e6fdabf6cc158667c987.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 13/32] ACPI: SBS: Add support for power_supply class (and sysfs) Len Brown
     [not found]   ` <041a0232f1e89dfa7cffed74a6cbb121032c36f8.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 14/32] ACPI: SBS: Add ACPI_PROCFS around procfs handling code Len Brown
     [not found]   ` <f127e3766686c2d525d930fb2a707cc90df9e2f1.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 15/32] ACPI: SBS: Add sysfs alarm Len Brown
     [not found]   ` <6e2157858ac94530fddbf19dc59ab6b392baf1f3.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 16/32] ACPI: Thermal: Drop concurrent thermal checks Len Brown
     [not found]   ` <4500ca8e221e72cc38e7c239b9b1a041031ee450.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 17/32] ACPI: video: Don't call absent methods Len Brown
     [not found]   ` <63f0edfc0b7f8058f9d3f9b572615ec97ae011ba.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 18/32] ACPI: VIDEO: Adjust current level to closest available one Len Brown
     [not found]   ` <4c611060660f0de3e9b8f02df207312bc6f5c331.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 19/32] ACPI: EC: Drop ECDT-based boot_ec as soon as we find DSDT-based one Len Brown
     [not found]   ` <8a66074c379cacc79168681bf7dea37ad278f5d1.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 20/32] sony-laptop/thinkpad-acpi: fix INPUT=n build Len Brown
     [not found]   ` <03e2bf261e832abf48d40113ce08a70338dd18c9.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 21/32] acpi_video: kernel build error if !INPUT Len Brown
     [not found]   ` <063a87f8cab5e3fd3ddbd979da7a15746947add2.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 22/32] ACPI: Battery: Add sysfs support Len Brown
     [not found]   ` <0b5466f638ab34c816c862802833ea2e465ee58a.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 23/32] ACPI: Battery: add sysfs alarm Len Brown
     [not found]   ` <3c309933a8a7b80d83d0c5e1b1ea0ef1c18aa533.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 24/32] ACPI: thinkpad-acpi: make room for more features in tp_features bitfield Len Brown
     [not found]   ` <bb7babb9ad37bb7b915a9bbb0f46ebc4f8b7300f.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 25/32] ACPI: thinkpad-acpi: issue EV_SYNC after EV_SWITCH Len Brown
     [not found]   ` <92746a28e48c4a850f574f3dea1c073cada35a22.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 26/32] ACPI: thinkpad-acpi: add mutex-based locking to input device event send path Len Brown
     [not found]   ` <bb9267cf59a5419b512a9d100155050e17b037d7.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 27/32] ACPI: thinkpad-acpi: keep track of module state Len Brown
     [not found]   ` <5a45e2370d4cb99f8afc48274e4bba0978a089df.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` Len Brown [this message]
     [not found]   ` <245cd52bc6364cd274901cd1fee1ecb36cec8c4f.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 29/32] ACPI: thinkpad-acpi: dequeue all pending hot key events at once Len Brown
     [not found]   ` <f46d7bb7d0f22efe5de65cce4c5fbd6fe01239c2.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 30/32] ACPI: thinkpad-acpi: fix regression on HKEY LID event handling Len Brown
     [not found]   ` <8a4b57f787502d9fef5ac7e02f657f801ea15567.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 31/32] ACPI: hpet: ACPI Error (utglobal-0126): Unknown exception code: 0xFFFFFFF0 Len Brown
     [not found]   ` <be23f43fd65a639b190a2f5b2e4979d1268b0f52.1190432418.git.len.brown@intel.com>
2007-09-22  3:46     ` [PATCH 32/32] ACPI: Hibernate erroneously disabled Suspend wakeup devices Len Brown

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=1190432821532-git-send-email-lenb@kernel.org \
    --to=lenb@kernel.org \
    --cc=hmh@hmh.eng.br \
    --cc=len.brown@intel.com \
    --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 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).