From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aaron Lu Subject: Re: [PATCH v3] ACPI / video: Add systems that should favor native backlight interface Date: Wed, 22 Jan 2014 09:58:38 +0800 Message-ID: <52DF25CE.7090402@intel.com> References: <1390053249-18919-1-git-send-email-i.gnatenko.brain@gmail.com> <52DCDA75.2000108@intel.com> <1390224857.10605.0.camel@x230> <52DDDA64.90407@intel.com> <1390274270.5320.2.camel@x230> <52DE067A.60400@intel.com> <1390306271.5320.4.camel@x230> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1390306271.5320.4.camel@x230> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Matthew Garrett , "daniel@ffwll.ch" , "Rafael J. Wysocki" Cc: "james@albanarts.com" , "bitbucket@online.de" , "felipe.contreras@gmail.com" , "dri-devel@lists.freedesktop.org" , "platform-driver-x86@vger.kernel.org" , "hmh@hmh.eng.br" , "kvalo@adurom.com" , "corsac@debian.org" , "linux-acpi@vger.kernel.org" , "joeyli.kernel@gmail.com" , "ben@bjencks.net" , "jrg.otte@gmail.com" , "intel-gfx@lists.freedesktop.org" , "Martin@lichtvoll.de" , "bugs@ttyhoney.com" , "jplatte@naasa.net" , "seth.forshee@canonical.com" , lennart@poettering.ne List-Id: linux-acpi@vger.kernel.org On 01/21/2014 08:11 PM, Matthew Garrett wrote: > On Tue, 2014-01-21 at 13:32 +0800, Aaron Lu wrote: >> On 01/21/2014 11:17 AM, Matthew Garrett wrote: >>> We know that Windows 8 graphics drivers don't use the ACPI interface, >>> and that systems change their behaviour as a result, in some cases with >>> absolutely no way for the ACPI interface could possibly work. I haven't >>> seen any cases where that's obviously true for any non-Windows 8 >> >> Perhaps I'm not clear, I didn't mean non-Windows 8 systems will all favor >> GPU's interface, I just meant for one specific win7 laptop I could re-use >> the existing code to make the GPU's interface as the only one left. And to >> achieve this, the Win8 OSI check in acpi_video_verify_backlight_support >> has to be gone. > > We could do that, but why do we think that's the correct fix? The plan > is to remove the native list entirely and do this for all Windows 8 What do you mean by native list, systems that are Win8 based but don't work with the GPU's backlight interface? If so, it doesn't seem we have such a native list now and if we don't make use_native_backlight=1 by default, we couldn't generate such a list to start with... BTW, it doesn't seem everyone agrees with this plan. May I ask, Rafael and Daniel, what's your opinion please? We need to agree with something to move things forward.