From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Garrett Subject: Re: [Intel-gfx] [PATCH] ACPI/Intel: Rework Opregion support Date: Tue, 15 Mar 2011 13:43:11 +0000 Message-ID: <20110315134311.GA5892@srcf.ucam.org> References: <1298404994-2907-1-git-send-email-mjg@redhat.com> <1300125566-29648-1-git-send-email-chris@chris-wilson.co.uk> <20110315015226.GA26454@srcf.ucam.org> <20110315133055.GA33193@dspnet.fr> <20110315133240.GA5471@srcf.ucam.org> <20110315134059.GB33193@dspnet.fr> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20110315134059.GB33193@dspnet.fr> Sender: linux-kernel-owner@vger.kernel.org To: Olivier Galibert Cc: Indan Zupancic , Dave Airlie , intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Len Brown , Alan Cox List-Id: intel-gfx@lists.freedesktop.org On Tue, Mar 15, 2011 at 02:40:59PM +0100, Olivier Galibert wrote: > On Tue, Mar 15, 2011 at 01:32:40PM +0000, Matthew Garrett wrote: > > Opregion is one mechanism to provide VBT - it doesn't define it. > > Then let me repeat that I haven't seen anything in the VBT tables of > the gma500-using netbook I have that didn't seem to be parsed > correctly by the current gpu/drm/i915/intel_bios.c code and its > friends. Have a look at it if you want. gma600 *certainly* has a bunch of additional functionality. It may be that this is code that can be split out in a reasonable manner, but there's been sufficient fragility here that I think doing it at this stage for the benefit of a driver in staging doesn't buy us a great deal. -- Matthew Garrett | mjg59@srcf.ucam.org