PCI: portdrv: check capabilities first
authorKenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
Wed, 25 Nov 2009 12:03:27 +0000 (21:03 +0900)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Fri, 4 Dec 2009 23:55:44 +0000 (15:55 -0800)
Move capability check capability to the beginning of
pcie_port_device_register() prevents redundant execution path.

Signed-off-by: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/pcie/portdrv_core.c

index 079bbc3..52c4e6f 100644 (file)
@@ -297,6 +297,10 @@ int pcie_port_device_register(struct pci_dev *dev)
        int status, capabilities, irq_mode, i, nr_serv;
        int vectors[PCIE_PORT_DEVICE_MAXSERVICES];
 
+       capabilities = get_port_device_capability(dev);
+       if (!capabilities)
+               return -ENODEV;
+
        port_data = kzalloc(sizeof(*port_data), GFP_KERNEL);
        if (!port_data)
                return -ENOMEM;
@@ -304,8 +308,6 @@ int pcie_port_device_register(struct pci_dev *dev)
 
        port_data->port_type = dev->pcie_type;
 
-       capabilities = get_port_device_capability(dev);
-
        irq_mode = assign_interrupt_mode(dev, vectors, capabilities);
        if (irq_mode == PCIE_PORT_NO_IRQ) {
                /*