On Tue, Oct 16, 2007 at 04:39:01PM +0200, Thomas Renninger wrote:
> + /*
> + * Check whether we have really a graphics device physically
> + * in the slot and registered at the system.
> + */
> + dev = acpi_get_physical_device(device->handle);
> + if (!dev) {
> + printk (KERN_DEBUG PREFIX "Video device %s.%s not physically"
> + " connected, ignoring\n", acpi_device_bid(device),
> + device->parent ? acpi_device_bid(device->parent) : "");
> + return -ENODEV;
> + }
> +
I suspect this will break other machines. Not all video extension
implementations are directly associated with the PCI ID. The Toshiba
M200 (for example) has
Device (PCI1)
{
Name (_ADR, 0x00010000)
Device (VGA)
{
Name (_ADR, 0x00)
which will result in VGA not having a physical device. You might be able
to get away with walking the parents until you find a pci ID and then
checking whether it matches the graphics adaptor, but I'm not certain of
that.
To make things more entertaining, Dell tend to implement a video
extension for both the 00:02.0 and 00:02.1 devices on Intel systems. We
need to be smarter about this, but I don't think simply looking for a
physical device is the solution.
--
Matthew Garrett | mjg59@xxxxxxxxxxxxx
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
