From mboxrd@z Thu Jan 1 00:00:00 1970 From: Felipe Contreras Subject: Re: [Update][PATCH 0/3] Fix backlight issues on some Windows 8 systems Date: Mon, 29 Jul 2013 13:01:02 -0500 Message-ID: References: <1370818899-8595-1-git-send-email-matthew.garrett@nebula.com> <1448863.3slS729B6b@vostro.rjw.lan> <87a9l95u8m.fsf@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-lb0-f175.google.com (mail-lb0-f175.google.com [209.85.217.175]) by gabe.freedesktop.org (Postfix) with ESMTP id E1F05E6B72 for ; Mon, 29 Jul 2013 11:01:03 -0700 (PDT) Received: by mail-lb0-f175.google.com with SMTP id 13so972202lba.20 for ; Mon, 29 Jul 2013 11:01:02 -0700 (PDT) In-Reply-To: <87a9l95u8m.fsf@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Jani Nikula Cc: "Rafael J. Wysocki" , daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org Hi, On Fri, Jul 26, 2013 at 8:24 AM, Jani Nikula wrote: > On Sat, 20 Jul 2013, Felipe Contreras wrote: >> I tried this patch series and it's as I expected, it's the same as >> acpi_backlight=vendor, and the intel backlight driver doesn't work >> correctly in this machine. If you are actually serious about the >> mantra of "no user-space regressions", then for this machine at least, >> you need to use the ACPI backlight with Windows8 OSI disabled, until >> the intel backlight driver is fixed. My patch does that: >> >> http://article.gmane.org/gmane.linux.acpi.devel/60969 > > Hi Felipe, would you mind trying the following patch on top of -rc2, > without your quirk patch? Up front, I'm not sure what all your failure > modes are, I'm not claiming this fixes them, and frankly, it didn't do > anything on a machine I tried... I just have this hunch it might help > with the ACPI hotkeys on some BIOSes. Maybe you're the lucky one! :) I did try this patch and I still see the same issue: when I go to the lowest level (xbacklight -set 0), the backlight is completely off. It seems in -rc2 the intel backlight driver works better, and can be controlled from the keys just fine, but it's still not as good as the acpi one minus win8. I don't notice any difference with your patch. Cheers. -- Felipe Contreras