lkml.org 
[lkml]   [2009]   [Mar]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v3 03/11] PCI: pci_scan_slot() returns newly found devices
    Date
    From: Trent Piepho <xyzzy@speakeasy.org>

    pci_scan_slot() has been rewritten to be less complex and will now
    return the number of *new* devices found.

    Existing callers need not worry because they already assume that
    they can't call pci_scan_slot() on an already-scanned slot.

    Thus, there is no semantic change for existing callers: returning
    newly found devices (this patch) is exactly equal to returning all
    found devices (before this patch).

    This patch adds some more groundwork to allow us to rescan the
    PCI bus during runtime to discover newly added devices.

    Signed-off-by: Trent Piepho <xyzzy@speakeasy.org>
    Reviewed-by: Alex Chiang <achiang@hp.com>
    Signed-off-by: Alex Chiang <achiang@hp.com>
    ---

    drivers/pci/probe.c | 35 +++++++++++++----------------------
    1 files changed, 13 insertions(+), 22 deletions(-)

    diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c
    index 66b5d1c..d392813 100644
    --- a/drivers/pci/probe.c
    +++ b/drivers/pci/probe.c
    @@ -1030,35 +1030,26 @@ EXPORT_SYMBOL(pci_scan_single_device);
    * Scan a PCI slot on the specified PCI bus for devices, adding
    * discovered devices to the @bus->devices list. New devices
    * will not have is_added set.
    + *
    + * Returns the number of new devices found.
    */
    int pci_scan_slot(struct pci_bus *bus, int devfn)
    {
    - int func, nr = 0;
    - int scan_all_fns;
    -
    - scan_all_fns = pcibios_scan_all_fns(bus, devfn);
    -
    - for (func = 0; func < 8; func++, devfn++) {
    - struct pci_dev *dev;
    + int fn, nr = 0;
    + struct pci_dev *dev;

    - dev = pci_scan_single_device(bus, devfn);
    - if (dev) {
    + if ((dev = pci_scan_single_device(bus, devfn)))
    + if (!dev->is_added) /* new device? */
    nr++;

    - /*
    - * If this is a single function device,
    - * don't scan past the first function.
    - */
    - if (!dev->multifunction) {
    - if (func > 0) {
    - dev->multifunction = 1;
    - } else {
    - break;
    - }
    + if ((dev && dev->multifunction) ||
    + (!dev && pcibios_scan_all_fns(bus, devfn))) {
    + for (fn = 1; fn < 8; fn++) {
    + if ((dev = pci_scan_single_device(bus, devfn + fn))) {
    + if (!dev->is_added)
    + nr++;
    + dev->multifunction = 1;
    }
    - } else {
    - if (func == 0 && !scan_all_fns)
    - break;
    }
    }



    \
     
     \ /
      Last update: 2009-03-09 06:53    [W:4.460 / U:0.100 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site