From: Alan Cox
Stop it poking at random registers on the i740 cards that may be out there
still.
As per Matthew's feedback remove the conditional checks and never enable the
opregion handling unless an appropriate driver has been loaded.
Signed-off-by: Alan Cox
---
drivers/acpi/video.c | 2
On Wed, 25 Apr 2012 13:40:18 +0100
Matthew Garrett wrote:
> On Wed, Apr 25, 2012 at 01:24:38PM +0100, Alan Cox wrote:
>
> > Because as I said before there are two drivers involved and you may only
> > have one compiled into your kernel. If you have
> >
> > CONFIG_DRM_GMA500=y
> > CONFIG_DRM_I91
On Wed, Apr 25, 2012 at 01:49:40PM +0100, Alan Cox wrote:
> On Wed, 25 Apr 2012 13:40:18 +0100
> Matthew Garrett wrote:
> > No you don't - there's various platforms that will hang if you do that.
> > It's necessary to make sure that the DIDL fields are set up before
> > calling any ACPI video fu
On Wed, Apr 25, 2012 at 01:24:38PM +0100, Alan Cox wrote:
> Because as I said before there are two drivers involved and you may only
> have one compiled into your kernel. If you have
>
> CONFIG_DRM_GMA500=y
> CONFIG_DRM_I915=n
>
> you want the acpi stuff to autoregister even if an i915 is found,
On Wed, 25 Apr 2012 12:31:52 +0100
Matthew Garrett wrote:
> On Wed, Apr 25, 2012 at 11:27:11AM +0100, Alan Cox wrote:
> > On Tue, 24 Apr 2012 23:30:22 +0100
> > Matthew Garrett wrote:
> > > Right now you seem to set opregion unconditionally on PVR, which seems
> > > to be equivalent to the 0xfc
On Wed, Apr 25, 2012 at 11:27:11AM +0100, Alan Cox wrote:
> On Tue, 24 Apr 2012 23:30:22 +0100
> Matthew Garrett 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, b
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 longe
From: Alan Cox
Stop it poking at random registers on the i740 cards that may be out there
still.
As per Matthew's feedback remove the conditional checks and never enable the
opregion handling unless an appropriate driver has been loaded.
Signed-off-by: Alan Cox
---
drivers/acpi/video.c | 2
On Wed, Apr 25, 2012 at 01:49:40PM +0100, Alan Cox wrote:
> On Wed, 25 Apr 2012 13:40:18 +0100
> Matthew Garrett wrote:
> > No you don't - there's various platforms that will hang if you do that.
> > It's necessary to make sure that the DIDL fields are set up before
> > calling any ACPI video fu
On Wed, 25 Apr 2012 13:40:18 +0100
Matthew Garrett wrote:
> On Wed, Apr 25, 2012 at 01:24:38PM +0100, Alan Cox wrote:
>
> > Because as I said before there are two drivers involved and you may only
> > have one compiled into your kernel. If you have
> >
> > CONFIG_DRM_GMA500=y
> > CONFIG_DRM_I91
On Wed, Apr 25, 2012 at 01:24:38PM +0100, Alan Cox wrote:
> Because as I said before there are two drivers involved and you may only
> have one compiled into your kernel. If you have
>
> CONFIG_DRM_GMA500=y
> CONFIG_DRM_I915=n
>
> you want the acpi stuff to autoregister even if an i915 is found,
On Wed, 25 Apr 2012 12:31:52 +0100
Matthew Garrett wrote:
> On Wed, Apr 25, 2012 at 11:27:11AM +0100, Alan Cox wrote:
> > On Tue, 24 Apr 2012 23:30:22 +0100
> > Matthew Garrett wrote:
> > > Right now you seem to set opregion unconditionally on PVR, which seems
> > > to be equivalent to the 0xfc
On Wed, Apr 25, 2012 at 11:27:11AM +0100, Alan Cox wrote:
> On Tue, 24 Apr 2012 23:30:22 +0100
> Matthew Garrett 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, b
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 longe
On Tue, 24 Apr 2012 22:02:18 +0100
Matthew Garrett wrote:
> On Tue, Apr 24, 2012 at 04:45:01PM +0100, Alan Cox wrote:
> > From: Alan Cox
> >
> > Handle the GMA500/600/36x0 cases. Also stop it poking at random registers on
> > the i740 cards that may be out there still.
>
> The PowerVR Intels I
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 Pow
On Tue, Apr 24, 2012 at 04:45:01PM +0100, Alan Cox wrote:
> From: Alan Cox
>
> Handle the GMA500/600/36x0 cases. Also stop it poking at random registers on
> the i740 cards that may be out there still.
The PowerVR Intels I'd seen had the opregion address in the 0xfc
register as well. Is this no
From: Alan Cox
Handle the GMA500/600/36x0 cases. Also stop it poking at random registers on
the i740 cards that may be out there still.
This should also allow the legacy gma500 stub driver to go away as the ACPI
video layer will now do the right thing rather than assume all the world is
Gen grap
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 Pow
On Tue, 24 Apr 2012 22:02:18 +0100
Matthew Garrett wrote:
> On Tue, Apr 24, 2012 at 04:45:01PM +0100, Alan Cox wrote:
> > From: Alan Cox
> >
> > Handle the GMA500/600/36x0 cases. Also stop it poking at random registers on
> > the i740 cards that may be out there still.
>
> The PowerVR Intels I
On Tue, Apr 24, 2012 at 04:45:01PM +0100, Alan Cox wrote:
> From: Alan Cox
>
> Handle the GMA500/600/36x0 cases. Also stop it poking at random registers on
> the i740 cards that may be out there still.
The PowerVR Intels I'd seen had the opregion address in the 0xfc
register as well. Is this no
From: Alan Cox
Handle the GMA500/600/36x0 cases. Also stop it poking at random registers on
the i740 cards that may be out there still.
This should also allow the legacy gma500 stub driver to go away as the ACPI
video layer will now do the right thing rather than assume all the world is
Gen grap
22 matches
Mail list logo
