Indent previous commit, rename functions (internal and local functions do
not need rte_eal_ prefix).

Signed-off-by: David Marchand <david.marchand at 6wind.com>
---
 lib/librte_eal/common/eal_common_pci.c | 106 ++++++++++++++++-----------------
 1 file changed, 52 insertions(+), 54 deletions(-)

diff --git a/lib/librte_eal/common/eal_common_pci.c 
b/lib/librte_eal/common/eal_common_pci.c
index a6791c1..fd88305 100644
--- a/lib/librte_eal/common/eal_common_pci.c
+++ b/lib/librte_eal/common/eal_common_pci.c
@@ -206,50 +206,49 @@ pci_driver_supports_device(const struct rte_pci_driver 
*dr,
  * driver.
  */
 static int
-rte_eal_pci_probe_one_driver(struct rte_pci_driver *dr,
-                            struct rte_pci_device *dev)
+pci_probe_device(struct rte_pci_driver *dr, struct rte_pci_device *dev)
 {
        int ret;
-               struct rte_pci_addr *loc = &dev->addr;
+       struct rte_pci_addr *loc = &dev->addr;

-               RTE_LOG(DEBUG, EAL, "PCI device "PCI_PRI_FMT" on NUMA socket 
%i\n",
-                               loc->domain, loc->bus, loc->devid, 
loc->function,
-                               dev->numa_node);
+       RTE_LOG(DEBUG, EAL, "PCI device "PCI_PRI_FMT" on NUMA socket %i\n",
+               loc->domain, loc->bus, loc->devid, loc->function,
+               dev->numa_node);

-               RTE_LOG(DEBUG, EAL, "  probe driver: %x:%x %s\n", 
dev->id.vendor_id,
-                               dev->id.device_id, dr->name);
+       RTE_LOG(DEBUG, EAL, "  probe driver: %x:%x %s\n", dev->id.vendor_id,
+               dev->id.device_id, dr->name);

-               /* no initialization when blacklisted, return without error */
-               if (dev->devargs != NULL &&
-                       dev->devargs->type == RTE_DEVTYPE_BLACKLISTED_PCI) {
-                       RTE_LOG(DEBUG, EAL, "  Device is blacklisted, not 
initializing\n");
-                       return 1;
-               }
+       /* no initialization when blacklisted, return without error */
+       if (dev->devargs != NULL &&
+               dev->devargs->type == RTE_DEVTYPE_BLACKLISTED_PCI) {
+               RTE_LOG(DEBUG, EAL, "  Device is blacklisted, not 
initializing\n");
+               return 1;
+       }

-               if (dr->drv_flags & RTE_PCI_DRV_NEED_MAPPING) {
+       if (dr->drv_flags & RTE_PCI_DRV_NEED_MAPPING) {
 #ifdef RTE_PCI_CONFIG
-                       /*
-                        * Set PCIe config space for high performance.
-                        * Return value can be ignored.
-                        */
-                       pci_config_space_set(dev);
+               /*
+                * Set PCIe config space for high performance.
+                * Return value can be ignored.
+                */
+               pci_config_space_set(dev);
 #endif
-                       /* map resources for devices that use igb_uio */
-                       ret = pci_map_device(dev);
-                       if (ret != 0)
-                               return ret;
-               } else if (dr->drv_flags & RTE_PCI_DRV_FORCE_UNBIND &&
-                               rte_eal_process_type() == RTE_PROC_PRIMARY) {
-                       /* unbind current driver */
-                       if (pci_unbind_kernel_driver(dev) < 0)
-                               return -1;
-               }
-
-               /* reference driver structure */
-               dev->driver = dr;
-
-               /* call the driver devinit() function */
-               return dr->devinit(dr, dev);
+               /* map resources for devices that use igb_uio */
+               ret = pci_map_device(dev);
+               if (ret != 0)
+                       return ret;
+       } else if (dr->drv_flags & RTE_PCI_DRV_FORCE_UNBIND &&
+                  rte_eal_process_type() == RTE_PROC_PRIMARY) {
+               /* unbind current driver */
+               if (pci_unbind_kernel_driver(dev) < 0)
+                       return -1;
+       }
+
+       /* reference driver structure */
+       dev->driver = dr;
+
+       /* call the driver devinit() function */
+       return dr->devinit(dr, dev);
 }

 /*
@@ -257,29 +256,28 @@ rte_eal_pci_probe_one_driver(struct rte_pci_driver *dr,
  * driver.
  */
 static int
-rte_eal_pci_detach_dev(struct rte_pci_driver *dr,
-               struct rte_pci_device *dev)
+pci_detach_device(struct rte_pci_driver *dr, struct rte_pci_device *dev)
 {
-               struct rte_pci_addr *loc = &dev->addr;
+       struct rte_pci_addr *loc = &dev->addr;

-               RTE_LOG(DEBUG, EAL, "PCI device "PCI_PRI_FMT" on NUMA socket 
%i\n",
-                               loc->domain, loc->bus, loc->devid,
-                               loc->function, dev->numa_node);
+       RTE_LOG(DEBUG, EAL, "PCI device "PCI_PRI_FMT" on NUMA socket %i\n",
+               loc->domain, loc->bus, loc->devid,
+               loc->function, dev->numa_node);

-               RTE_LOG(DEBUG, EAL, "  remove driver: %x:%x %s\n", 
dev->id.vendor_id,
-                               dev->id.device_id, dr->name);
+       RTE_LOG(DEBUG, EAL, "  remove driver: %x:%x %s\n", dev->id.vendor_id,
+               dev->id.device_id, dr->name);

-               if (dr->devuninit && (dr->devuninit(dev) < 0))
-                       return -1;      /* negative value is an error */
+       if (dr->devuninit && (dr->devuninit(dev) < 0))
+               return -1;      /* negative value is an error */

-               /* clear driver structure */
-               dev->driver = NULL;
+       /* clear driver structure */
+       dev->driver = NULL;

-               if (dr->drv_flags & RTE_PCI_DRV_NEED_MAPPING)
-                       /* unmap resources for devices that use igb_uio */
-                       pci_unmap_device(dev);
+       if (dr->drv_flags & RTE_PCI_DRV_NEED_MAPPING)
+               /* unmap resources for devices that use igb_uio */
+               pci_unmap_device(dev);

-               return 0;
+       return 0;
 }

 /*
@@ -297,7 +295,7 @@ pci_probe_all_drivers(struct rte_pci_device *dev)
                if (!pci_driver_supports_device(dr, dev))
                        continue;

-               rc = rte_eal_pci_probe_one_driver(dr, dev);
+               rc = pci_probe_device(dr, dev);
                if (rc < 0)
                        /* negative value is an error */
                        return -1;
@@ -324,7 +322,7 @@ pci_detach_all_drivers(struct rte_pci_device *dev)
                if (!pci_driver_supports_device(dr, dev))
                        continue;

-               rc = rte_eal_pci_detach_dev(dr, dev);
+               rc = pci_detach_device(dr, dev);
                if (rc < 0)
                        /* negative value is an error */
                        return -1;
-- 
1.9.1

Reply via email to