From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alan Cox Subject: Re: [PATCH 2/3] acpi_video: Intel video is not always i915 Date: Wed, 25 Apr 2012 11:27:11 +0100 Message-ID: <20120425112711.613a3cf9@pyramind.ukuu.org.uk> References: <20120424154402.1575.90373.stgit@bluebook> <20120424154434.1575.91463.stgit@bluebook> <20120424210218.GA25961@srcf.ucam.org> <20120424233117.6de0da6f@pyramind.ukuu.org.uk> <20120424223022.GA27765@srcf.ucam.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from lxorguk.ukuu.org.uk ([81.2.110.251]:45630 "EHLO lxorguk.ukuu.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754975Ab2DYKYc (ORCPT ); Wed, 25 Apr 2012 06:24:32 -0400 In-Reply-To: <20120424223022.GA27765@srcf.ucam.org> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Matthew Garrett Cc: airlied@linux.ie, dri-devel@lists.freedesktop.org, linux-acpi@vger.kernel.org On Tue, 24 Apr 2012 23:30:22 +0100 Matthew Garrett wrote: > On Tue, Apr 24, 2012 at 11:31:17PM +0100, Alan Cox wrote: > > On Tue, 24 Apr 2012 22:02:18 +0100 > > Matthew Garrett wrote: > > > The PowerVR Intels I'd seen had the opregion address in the 0xfc > > > register as well. Is this no longer true on the latest? > > > > PowerVR does - i740 never did. > > > > The PowerVR 0xfc poking also doesn't currently work once the driver takes > > over because it isn't yet implementing the driver end of the weird ACPI > > messaging/event stuff. > > > > Once it does the GMA500 will be able to do an ACPI video register, but we > > will still need the check to get the ifdeffery right for what drivers are > > compiled for the kernel. > > Right now you seem to set opregion unconditionally on PVR, which seems > to be equivalent to the 0xfc check that was there before - I can > understand excluding i740, but the PVR check could be left with the gen > hardware one? Not really - they are two drivers. If you build with i915 and not GMA500 you need the opregion for one and the acpi fallback for the other, and vice versa. So we have to check both CONFIG_xxx macro sets. Right now GMA500 needs the ACPI video stuff never to be enabled on some machines. Until we've got full opregion support in the driver that won't change. Alan