[Intel-gfx] [PATCH] drm/i915: Recognise non-VGA display devices

2011-01-21 Thread Chris Wilson
Starting with SandyBridge (though possible with earlier hacked BIOSes), the BIOS may initialise the IGFX as secondary to a discrete GPU. Prior, it would simply disable the integrated GPU. So we adjust the our PCI class mask to match any DISPLAY_CLASS device. In such a configuration, the IGFX is no

Re: [Intel-gfx] [PATCH] drm/i915: Recognise non-VGA display devices

2011-01-20 Thread Dave Airlie
On Fri, Jan 21, 2011 at 10:25 AM, Chris Wilson wrote: > On Fri, 21 Jan 2011 08:49:59 +1000, Dave Airlie wrote: >> On Fri, Jan 21, 2011 at 1:07 AM, Chris Wilson >> wrote: >> > Starting with SandyBridge (though possible with earlier hacked BIOSes), >> > the BIOS may initialise the IGFX as seconda

Re: [Intel-gfx] [PATCH] drm/i915: Recognise non-VGA display devices

2011-01-20 Thread Chris Wilson
On Fri, 21 Jan 2011 08:49:59 +1000, Dave Airlie wrote: > On Fri, Jan 21, 2011 at 1:07 AM, Chris Wilson > wrote: > > Starting with SandyBridge (though possible with earlier hacked BIOSes), > > the BIOS may initialise the IGFX as secondary to a discrete GPU. Prior, > > it would simply disable the

Re: [Intel-gfx] [PATCH] drm/i915: Recognise non-VGA display devices

2011-01-20 Thread Dave Airlie
On Fri, Jan 21, 2011 at 1:07 AM, Chris Wilson wrote: > Starting with SandyBridge (though possible with earlier hacked BIOSes), > the BIOS may initialise the IGFX as secondary to a discrete GPU. Prior, > it would simply disable the integrated GPU. In such a configuration, the > IGFX is not a primar

Re: [Intel-gfx] [PATCH] drm/i915: Recognise non-VGA display devices

2011-01-20 Thread Chris Wilson
On Thu, 20 Jan 2011 09:04:16 -0800, Keith Packard wrote: > On Thu, 20 Jan 2011 15:07:07 +, Chris Wilson > wrote: > > > + if (dev_priv->is_vga) { > > + /* If we have > 1 VGA cards with a single output, > > +* then disable the radeon VGA resources. > > +*/

Re: [Intel-gfx] [PATCH] drm/i915: Recognise non-VGA display devices

2011-01-20 Thread Keith Packard
On Thu, 20 Jan 2011 15:07:07 +, Chris Wilson wrote: > + if (dev_priv->is_vga) { > + /* If we have > 1 VGA cards with a single output, > + * then disable the radeon VGA resources. > + */ Why does this say 'radeon'? -- keith.pack...@intel.com pgp4

[Intel-gfx] [PATCH] drm/i915: Recognise non-VGA display devices

2011-01-20 Thread Chris Wilson
Starting with SandyBridge (though possible with earlier hacked BIOSes), the BIOS may initialise the IGFX as secondary to a discrete GPU. Prior, it would simply disable the integrated GPU. In such a configuration, the IGFX is not a primary VGA controller and so should not take part in VGA arbitratio