[PATCH 2/3] acpi_video: Intel video is not always i915
mjg59 at srcf.ucam.org
Wed Apr 25 04:31:52 PDT 2012
On Wed, Apr 25, 2012 at 11:27:11AM +0100, Alan Cox wrote:
> On Tue, 24 Apr 2012 23:30:22 +0100
> Matthew Garrett <mjg59 at srcf.ucam.org> wrote:
> > 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
opregion always seems to be set to 1 if is_gma_pvr() is true, which
means we'll now never bind the acpi video driver on PVR hardware. If
that's what you want, why distinguish between PVR and GEN?
Matthew Garrett | mjg59 at srcf.ucam.org
More information about the dri-devel