[ibm-acpi-devel] [ltp] thinkpad-acpi release 0.17-20071002 uploaded to ibm-acpi.sf.net

Henrique de Moraes Holschuh linux-thinkpad@linux-thinkpad.org
Wed, 3 Oct 2007 15:25:40 -0300


--Kj7319i9nmIyA2yE
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

Please try this patch, and send me the debug output from thinkpad-acpi...

-- 
  "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

--Kj7319i9nmIyA2yE
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename=patch

diff --git a/drivers/misc/thinkpad_acpi.c b/drivers/misc/thinkpad_acpi.c
index a4884aa..517c79a 100644
--- a/drivers/misc/thinkpad_acpi.c
+++ b/drivers/misc/thinkpad_acpi.c
@@ -3584,6 +3584,8 @@ static int __init brightness_check_levels(void)
 	struct acpi_buffer buffer = { ACPI_ALLOCATE_BUFFER, NULL };
 	union acpi_object *obj;
 
+	printk(IBM_ERR "debug: start check for 16 brightness levels\n");
+
 	if (!vid_handle) {
 		IBM_ACPIHANDLE_INIT(vid);
 	}
@@ -3591,6 +3593,8 @@ static int __init brightness_check_levels(void)
 	if (!vid_handle)
 		return 0;
 
+	printk(IBM_ERR "debug: will call _BCL\n");
+
 	/* Using BCLL would be better, but that is not a method, just
 	 * a package.  The safest way would be to get BCLL data, and
 	 * check the highest value in it, as it will be the max_level
@@ -3603,6 +3607,11 @@ static int __init brightness_check_levels(void)
 	status = (obj && obj->type == ACPI_TYPE_PACKAGE &&
 		  obj->package.count == 18);
 
+	if (obj) {
+		printk(IBM_ERR "debug: _BCL call sucessfull, obj type is %d, obj count is %d, status %d\n",
+			obj->type, obj->package.count, status);
+	}
+
 	kfree(buffer.pointer);
 	return status;
 }

--Kj7319i9nmIyA2yE--