nouveau/acpi: improve detection of what is IGD and what is DIS.
authorDave Airlie <airlied@redhat.com>
Mon, 6 Dec 2010 02:56:44 +0000 (12:56 +1000)
committerDave Airlie <airlied@redhat.com>
Wed, 5 Jan 2011 03:43:32 +0000 (13:43 +1000)
This improves the IGD/DIS picking using firstly if Intel, then
if the bus is bus 0. There may be a correct way to do this, but
I've no idea what it is.

Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/nouveau/nouveau_acpi.c

index 1191526..a542380 100644 (file)
@@ -130,10 +130,15 @@ static int nouveau_dsm_init(void)
 
 static int nouveau_dsm_get_client_id(struct pci_dev *pdev)
 {
-       if (nouveau_dsm_priv.dhandle == DEVICE_ACPI_HANDLE(&pdev->dev))
+       /* easy option one - intel vendor ID means Integrated */
+       if (pdev->vendor == PCI_VENDOR_ID_INTEL)
                return VGA_SWITCHEROO_IGD;
-       else
-               return VGA_SWITCHEROO_DIS;
+
+       /* is this device on Bus 0? - this may need improving */
+       if (pdev->bus->number == 0)
+               return VGA_SWITCHEROO_IGD;
+
+       return VGA_SWITCHEROO_DIS;
 }
 
 static struct vga_switcheroo_handler nouveau_dsm_handler = {