From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756613Ab1AaWB7 (ORCPT ); Mon, 31 Jan 2011 17:01:59 -0500 Received: from ogre.sisk.pl ([217.79.144.158]:55231 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754069Ab1AaWB5 (ORCPT ); Mon, 31 Jan 2011 17:01:57 -0500 From: "Rafael J. Wysocki" To: Thomas Meyer Subject: Re: 2.6.38-rc2: acpi backlight control missing Date: Mon, 31 Jan 2011 23:01:29 +0100 User-Agent: KMail/1.13.5 (Linux/2.6.38-rc2+; KDE/4.4.4; x86_64; ; ) Cc: "rui.zhang@intel.com" , Linux Kernel Mailing List , ACPI Devel Mailing List , Len Brown References: <1295693349.2120.3.camel@localhost.localdomain> <201101312139.03634.rjw@sisk.pl> In-Reply-To: MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201101312301.29434.rjw@sisk.pl> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Monday, January 31, 2011, Thomas Meyer wrote: > Am 31.01.2011 um 21:39 schrieb "Rafael J. Wysocki" : > > > On Friday, January 28, 2011, Thomas Meyer wrote: > >> Am Samstag, den 22.01.2011, 20:43 +0100 schrieb Rafael J. Wysocki: > >>> On Saturday, January 22, 2011, Thomas Meyer wrote: > >>>> with 2.6.38-rc2 the acpi backlight is gone: > >>>> > >>>> any ideas? > >>> > >>> Not really, but Matthew may have some (CCed). > >> > >> Okay, found the problem. > >> > >> This commit broke my backlight: > >> > >> commit 677bd810eedce61edf15452491781ff046b92edc > >> Author: Zhang Rui > >> Date: Mon Dec 6 15:04:21 2010 +0800 > >> > >> ACPI video: remove output switching control > >> > >> This is because in function > >> "acpi_is_video_device()" (drivers/acpi/video_detect.c) the backlight > >> device is only checked when some video capabilities were found, but by > >> removing the check for the VIDEO_OUTPUT_SWITCHING (that seems to be the > >> only video capability on my system) the check for the backlight device > >> isn't done anymore. > >> > >> by reverting above commit, the backlight is back again. > > > > Is it the same issue as https://bugzilla.kernel.org/show_bug.cgi?id=27702 ? > > Yes, it is. > > > In case it is, can you test the patch in there, please? > > > > Didn't test the patch. But it should do the trick. Great, thanks! Rafael