diff options
| author |   Rafael J. Wysocki <rjw@sisk.pl> | 2009-01-13 14:46:46 +0100 |
|---|---|---|
| committer |   Jesse Barnes <jbarnes@hobbes.lan> | 2009-03-19 19:29:23 -0700 |
| commit | 22106368c999246c414610dcaacd485e741605b1 (patch) | |
| tree | 2c74166b9967118e863012e6c81e03f9c7345bfd /drivers/pci/pcie/portdrv_core.c | |
| parent | PCI: PCIe portdrv: Simplily probe callback of service drivers (diff) | |
| download | linux-dev-22106368c999246c414610dcaacd485e741605b1.tar.xz linux-dev-22106368c999246c414610dcaacd485e741605b1.zip | |
PCI: PCIe portdrv: Remove struct pcie_port_service_id
The PCI Express port driver uses 'struct pcie_port_service_id' for
matching port service devices and drivers, but this structure
contains fields that duplicate information from the port device
itself (vendor, device, subvendor, subdevice) and fields that are not
used by any existing port service driver (class, class_mask,
drvier_data). Also, both existing port service drivers (AER and
PCIe HP) don't even use the vendor and device fields for device
matching. Therefore 'struct pcie_port_service_id' can be removed
altogether and the only useful members of it (port_type, service) can
be introduced directly into the port service device and port service
driver structures. That simplifies the code quite a bit and reduces
its size.
Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Diffstat (limited to 'drivers/pci/pcie/portdrv_core.c')
| -rw-r--r-- | drivers/pci/pcie/portdrv_core.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/pci/pcie/portdrv_core.c b/drivers/pci/pcie/portdrv_core.c index 682524b0c93a..843d9e30dd3b 100644 --- a/drivers/pci/pcie/portdrv_core.c +++ b/drivers/pci/pcie/portdrv_core.c @@ -140,10 +140,7 @@ static void pcie_device_init(struct pci_dev *parent, struct pcie_device *dev, dev->port = parent; dev->irq = irq; - dev->id.vendor = parent->vendor; - dev->id.device = parent->device; - dev->id.port_type = port_type; - dev->id.service_type = service_type; + dev->service = service_type; /* Initialize generic device interface */ device = &dev->device; |