Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754935Ab3IYQUB (ORCPT ); Wed, 25 Sep 2013 12:20:01 -0400 Received: from mga03.intel.com ([143.182.124.21]:17315 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754840Ab3IYQT5 convert rfc822-to-8bit (ORCPT ); Wed, 25 Sep 2013 12:19:57 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.90,978,1371106800"; d="scan'208";a="400936639" From: Jani Nikula To: =?utf-8?Q?J=C3=B6rg?= Otte Cc: Aaron Lu , Igor Gnatenko , ACPI Devel Maling List , intel-gfx , dri-devel@lists.freedesktop.org, Linux Kernel Mailing List , Daniel Vetter , "Rafael J. Wysocki" , Matthew Garrett , Seth Forshee , Lee Chun-Yi , Richard Purdie , Yves-Alexis Perez , Felipe Contreras , Henrique de Moraes Holschuh , Ben Jencks , Steven Newbury , James Hogan , Kamal Mostafa , Joerg Platte , Kalle Valo , Martin Steigerwald , Mike Galbraith Subject: Re: [PATCH v3 0/4] Fix Win8 backlight issue In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <1380016052-15315-1-git-send-email-aaron.lu@intel.com> <1380026575.3371.1.camel@ThinkPad-X230.localdomain> <20130925091720.GA1921@aaronlu.sh.intel.com> <87vc1p4207.fsf@intel.com> User-Agent: Notmuch/0.16+82~gc83e2d2 (http://notmuchmail.org) Emacs/24.3.1 (x86_64-pc-linux-gnu) Date: Wed, 25 Sep 2013 19:17:04 +0300 Message-ID: <87eh8clvun.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2294 Lines: 67 On Wed, 25 Sep 2013, Jörg Otte wrote: > 2013/9/25 Jani Nikula : >> On Wed, 25 Sep 2013, Aaron Lu wrote: >>> On Wed, Sep 25, 2013 at 10:29:37AM +0200, Jörg Otte wrote: >>>> Backlight can't be modified with this patch set - neither with >>>> function keys nor with the gui. It is a step backward to v3.11-rc1 >> >> So both hotkeys and GUI work in v3.11-rc1? And v3.12-rc2? > > In v3.11-rc1 it didn't work. > Since a later rc (I don't remember exactly which) it did work. > In v3.12-rc1/2 it does work. > In v3.12-rc2 + Aaron's patch set it again doesn't work. > >> >>> Thanks for the test. >>> >>> Please check /sys/class/backlight, is there only one link file >>> intel_backlight? >> >> Indeed, are there others? fujitsu-laptop perhaps? If yes, try >> CONFIG_FUJITSU_LAPTOP=n or an appropriate module blacklist. >> >> Just checking, you do have CONFIG_BACKLIGHT_CLASS_DEVICE=y? > > There is only one intel_backlight link. > Yes, I have CONFIG_BACKLIGHT_CLASS_DEVICE=y > >> Embarrassingly there was a bug in i915 fixed just recently where the >> backlight device wasn't registered for >> CONFIG_BACKLIGHT_CLASS_DEVICE=m... >> >>> If so, please cd inside and try modify the brightness file using echo >>> with some values in the range of 0 - max_brightness, does the >>> backlight level change when you echo a new value? I guess it doesn't, >>> or you wouldn't notice problem. If indeed so, perhaps file a bug at >>> http://bugzilla.kernel.org, Drivers/Video(DRI-Intel)? I suppose Jani >>> and Daniel can help fix your problem. >> >> Yup, please check the above, and report back. > > - echo 0..max_brightness > brightness does not work. Thanks for the info. How about v3.12-rc2 without Aaron's patches? Does intel_backlight also not work there? How about the acpi_video0 (which I presume is present) sysfs interface? BR, Jani. > >>>> >>>> Video driver: i915 >>>> FUJITSU LIFEBOOK AH532/FJNBB1C, BIOS Version 1.09 05/22/2012 >>>> >>>> acpi_backlight=video works. > > Jörg -- 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/