Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030349AbXBZRWj (ORCPT ); Mon, 26 Feb 2007 12:22:39 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1030356AbXBZRWj (ORCPT ); Mon, 26 Feb 2007 12:22:39 -0500 Received: from twin.jikos.cz ([213.151.79.26]:48365 "EHLO twin.jikos.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030349AbXBZRWi (ORCPT ); Mon, 26 Feb 2007 12:22:38 -0500 Date: Mon, 26 Feb 2007 18:21:59 +0100 (CET) From: Jiri Kosina To: Henrique de Moraes Holschuh cc: Richard Purdie , linux-kernel@vger.kernel.org Subject: Re: [PATCH] ACPI: ibm-acpi: improve backlight power handling In-Reply-To: <20070226161203.GD2909@khazad-dum.debian.net> Message-ID: References: <1172490091.5824.30.camel@localhost.localdomain> <20070226142157.GA2909@khazad-dum.debian.net> <1172501357.5824.91.camel@localhost.localdomain> <20070226152035.GB2909@khazad-dum.debian.net> <20070226161203.GD2909@khazad-dum.debian.net> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1875 Lines: 60 On Mon, 26 Feb 2007, Henrique de Moraes Holschuh wrote: > Improve the backlight code to emulate as much as possible the power > management events, as we are unable to really power on or power off the > backlight. This still easily leads to confusing behavior, doesn't it? As there are power-related calls from backlight driver, which won't get handled properly by your code, in result confusing the brightness status. I would suggest applying something like the patch below instead, if you find it OK. From: Jiri Kosina [PATCH] ibm-acpi: handle power calls from backlight class Don't ignore the power-related calls from backlight class driver and always adjust the brightness accordingly. Signed-off-by: Jiri Kosina --- drivers/acpi/ibm_acpi.c | 8 +++++++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/drivers/acpi/ibm_acpi.c b/drivers/acpi/ibm_acpi.c index 7c1b418..4cfa5f8 100644 --- a/drivers/acpi/ibm_acpi.c +++ b/drivers/acpi/ibm_acpi.c @@ -87,6 +87,7 @@ #include #include #include +#include #include #include @@ -1701,7 +1702,12 @@ static int brightness_write(char *buf) static int brightness_update_status(struct backlight_device *bd) { - return brightness_set(bd->props.brightness); + int brightness = 0; + + if (bd->props.fb_blank == FB_BLANK_UNBLANK || bd->props.power == FB_BLANK_UNBLANK) + brightness = bd->props.brightness; + return brightness_set(brightness); + } static struct backlight_ops ibm_backlight_data = { - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/