From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH 2/2] ACPI / video / i915: Remove ACPI backlight if firmware expects Windows 8 Date: Tue, 10 Sep 2013 17:21:41 +0300 Message-ID: <87bo40ahai.fsf@intel.com> References: <522D88C3.7000808@intel.com> <1793135.3qk7NfcINN@vostro.rjw.lan> <20130909152118.GG27291@phenom.ffwll.local> <5155010.Y1gov7SKhP@vostro.rjw.lan> <87eh8wail7.fsf@intel.com> <1378821416.17982.56.camel@x230.lan> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1378821416.17982.56.camel@x230.lan> 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: Matthew Garrett Cc: Aaron Lu , Henrique de Moraes Holschuh , Seth Forshee , Daniel Vetter , "intel-gfx@lists.freedesktop.org" , Yves-Alexis Perez , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , "Rafael J. Wysocki" , ACPI Devel Mailing List , "Lee, Chun-Yi" , Igor Gnatenko , Lee Chun-Yi , Len Brown List-Id: linux-acpi@vger.kernel.org T24gVHVlLCAxMCBTZXAgMjAxMywgTWF0dGhldyBHYXJyZXR0IDxtYXR0aGV3LmdhcnJldHRAbmVi dWxhLmNvbT4gd3JvdGU6Cj4gT24gVHVlLCAyMDEzLTA5LTEwIGF0IDE2OjUzICswMzAwLCBKYW5p IE5pa3VsYSB3cm90ZToKPgo+PiBJIHRoaW5rIHRoZSBwYXJhbWV0ZXIgIkRvZXMgdGhlIEFDUEkg YmFja2xpZ2h0IGludGVyZmFjZSB3b3JrIG9yIG5vdCIKPj4gYmVsb25ncyB0byB0aGUgQUNQSSB2 aWRlbyBkcml2ZXIuCj4KPiBUaGF0IGRlcGVuZHMgb24gaG93IFdpbmRvd3MgOCB3b3Jrcy4gSWYg V2luZG93cyA4IHBvbGljeSBpcyBoYW5kbGVkIGJ5Cj4gdGhlIEdQVSBkcml2ZXJzIHRoZW4gaXQg YmVsb25ncyBpbiBpOTE1LiBJZiBpdCdzIGhhbmRsZWQgYnkgdGhlIEFDUEkKPiBjb2RlIHRoZW4g aXQgYmVsb25ncyBpbiB0aGUgQUNQSSBjb2RlLgoKSSBmYWlsIHRvIHNlZSB0aGUgbG9naWMuIFdp bmRvd3MgOCBwb2xpY3kgZGljdGF0ZXMgd2hldGhlciB3ZSBjYW4gdXNlCnRoZSBBTUwgY29kZSBv ciBub3QuIElNSE8sIEFDUEkgY29kZSBpcyBpbiB0aGUgYmVzdCBwb3NpdGlvbiB0byBmaWd1cmUK dGhpcyBvdXQgYW5kIHF1aXJrIGFzIG5lY2Vzc2FyeS4gSXQncyB0aGUgcGFydCB0aGF0IGtub3dz IGFib3V0IFdpbmRvd3MKOCwgbm90IGk5MTUuCgo+IEJ1dCBJIGhhdmUgbm8gd2F5IG9mIGRldGVy bWluaW5nIHRoYXQsIHdoZXJlYXMgeW91IHdvcmsgZm9yIGEgY29tcGFueQo+IHRoYXQgcHJvZHVj ZXMgYSBXaW5kb3dzIDggdmlkZW8gZHJpdmVy4oCmCgpIZXJlIEkgZG8gc2VlIHRoZSBsb2dpYy4g SG93ZXZlciBzZWUgc2lnbmF0dXJlOyBJJ20gbm90IGluIHRoZSBiZXN0IG9mCnBvc2l0aW9ucyB0 byBmaWd1cmUgb3V0IHRoaW5ncyBhYm91dCBXaW5kb3dzIDggdmlkZW8gZHJpdmVycy4gOykgQnV0 CkknbGwgc2VlIHdoYXQgSSBjYW4gZG8uCgpCUiwKSmFuaS4KCgotLSAKSmFuaSBOaWt1bGEsIElu dGVsIE9wZW4gU291cmNlIFRlY2hub2xvZ3kgQ2VudGVyCl9fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4 QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753482Ab3IJOTn (ORCPT ); Tue, 10 Sep 2013 10:19:43 -0400 Received: from mga09.intel.com ([134.134.136.24]:59240 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751960Ab3IJOTk convert rfc822-to-8bit (ORCPT ); Tue, 10 Sep 2013 10:19:40 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.90,878,1371106800"; d="scan'208";a="393614301" From: Jani Nikula To: Matthew Garrett Cc: "Rafael J. Wysocki" , Daniel Vetter , Aaron Lu , ACPI Devel Mailing List , Daniel Vetter , "intel-gfx\@lists.freedesktop.org" , Yves-Alexis Perez , "linux-kernel\@vger.kernel.org" , "dri-devel\@lists.freedesktop.org" , Seth Forshee , "Lee\, Chun-Yi" , Igor Gnatenko , Henrique de Moraes Holschuh , Lee Chun-Yi , Len Brown Subject: Re: [PATCH 2/2] ACPI / video / i915: Remove ACPI backlight if firmware expects Windows 8 In-Reply-To: <1378821416.17982.56.camel@x230.lan> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <522D88C3.7000808@intel.com> <1793135.3qk7NfcINN@vostro.rjw.lan> <20130909152118.GG27291@phenom.ffwll.local> <5155010.Y1gov7SKhP@vostro.rjw.lan> <87eh8wail7.fsf@intel.com> <1378821416.17982.56.camel@x230.lan> User-Agent: Notmuch/0.16+63~g12e17e8 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) Date: Tue, 10 Sep 2013 17:21:41 +0300 Message-ID: <87bo40ahai.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 On Tue, 10 Sep 2013, Matthew Garrett wrote: > On Tue, 2013-09-10 at 16:53 +0300, Jani Nikula wrote: > >> I think the parameter "Does the ACPI backlight interface work or not" >> belongs to the ACPI video driver. > > That depends on how Windows 8 works. If Windows 8 policy is handled by > the GPU drivers then it belongs in i915. If it's handled by the ACPI > code then it belongs in the ACPI code. I fail to see the logic. Windows 8 policy dictates whether we can use the AML code or not. IMHO, ACPI code is in the best position to figure this out and quirk as necessary. It's the part that knows about Windows 8, not i915. > But I have no way of determining that, whereas you work for a company > that produces a Windows 8 video driver… Here I do see the logic. However see signature; I'm not in the best of positions to figure out things about Windows 8 video drivers. ;) But I'll see what I can do. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center