[dpdk-dev] [PATCH v2 16/18] devargs: remove old devargs parsing function

[Gaetan Rivet]

This functionality is already covered by rte_eal_devargs_parse.

Signed-off-by: Gaetan Rivet <gaetan.ri...@6wind.com>
---
 lib/librte_eal/bsdapp/eal/rte_eal_version.map   |  1 -
 lib/librte_eal/common/eal_common_devargs.c      | 30 -------------------------
 lib/librte_eal/common/include/rte_devargs.h     | 28 -----------------------
 lib/librte_eal/linuxapp/eal/rte_eal_version.map |  1 -
 4 files changed, 60 deletions(-)

diff --git a/lib/librte_eal/bsdapp/eal/rte_eal_version.map 
b/lib/librte_eal/bsdapp/eal/rte_eal_version.map
index 13c8450..aae8d32 100644
--- a/lib/librte_eal/bsdapp/eal/rte_eal_version.map
+++ b/lib/librte_eal/bsdapp/eal/rte_eal_version.map
@@ -32,7 +32,6 @@ DPDK_2.0 {
        rte_eal_lcore_role;
        rte_eal_mp_remote_launch;
        rte_eal_mp_wait_lcore;
-       rte_eal_parse_devargs_str;
        rte_eal_process_type;
        rte_eal_remote_launch;
        rte_eal_tailq_lookup;
diff --git a/lib/librte_eal/common/eal_common_devargs.c 
b/lib/librte_eal/common/eal_common_devargs.c
index 1d87cd9..82c8573 100644
--- a/lib/librte_eal/common/eal_common_devargs.c
+++ b/lib/librte_eal/common/eal_common_devargs.c
@@ -53,36 +53,6 @@ TAILQ_HEAD(rte_devargs_list, rte_devargs);
 struct rte_devargs_list devargs_list =
        TAILQ_HEAD_INITIALIZER(devargs_list);
 
-int
-rte_eal_parse_devargs_str(const char *devargs_str,
-                       char **drvname, char **drvargs)
-{
-       char *sep;
-
-       if ((devargs_str) == NULL || (drvname) == NULL || (drvargs == NULL))
-               return -1;
-
-       *drvname = strdup(devargs_str);
-       if (*drvname == NULL)
-               return -1;
-
-       /* set the first ',' to '\0' to split name and arguments */
-       sep = strchr(*drvname, ',');
-       if (sep != NULL) {
-               sep[0] = '\0';
-               *drvargs = strdup(sep + 1);
-       } else {
-               *drvargs = strdup("");
-       }
-
-       if (*drvargs == NULL) {
-               free(*drvname);
-               *drvname = NULL;
-               return -1;
-       }
-       return 0;
-}
-
 static int
 bus_name_cmp(const struct rte_bus *bus, const void *name)
 {
diff --git a/lib/librte_eal/common/include/rte_devargs.h 
b/lib/librte_eal/common/include/rte_devargs.h
index 1fe03d6..499f7e3 100644
--- a/lib/librte_eal/common/include/rte_devargs.h
+++ b/lib/librte_eal/common/include/rte_devargs.h
@@ -74,34 +74,6 @@ struct rte_devargs {
 };
 
 /**
- * Parse a devargs string.
- *
- * For PCI devices, the format of arguments string is "PCI_ADDR" or
- * "PCI_ADDR,key=val,key2=val2,...". Examples: "08:00.1", "0000:5:00.0",
- * "04:00.0,arg=val".
- *
- * For virtual devices, the format of arguments string is "DRIVER_NAME*"
- * or "DRIVER_NAME*,key=val,key2=val2,...". Examples: "net_ring",
- * "net_ring0", "net_pmdAnything,arg=0:arg2=1".
- *
- * The function parses the arguments string to get driver name and driver
- * arguments.
- *
- * @param devargs_str
- *   The arguments as given by the user.
- * @param drvname
- *   The pointer to the string to store parsed driver name.
- * @param drvargs
- *   The pointer to the string to store parsed driver arguments.
- *
- * @return
- *   - 0 on success
- *   - A negative value on error
- */
-int rte_eal_parse_devargs_str(const char *devargs_str,
-                               char **drvname, char **drvargs);
-
-/**
  * Parse a device string.
  *
  * Verify that a bus is capable of handling the device passed
diff --git a/lib/librte_eal/linuxapp/eal/rte_eal_version.map 
b/lib/librte_eal/linuxapp/eal/rte_eal_version.map
index a6fe25c..323f799 100644
--- a/lib/librte_eal/linuxapp/eal/rte_eal_version.map
+++ b/lib/librte_eal/linuxapp/eal/rte_eal_version.map
@@ -32,7 +32,6 @@ DPDK_2.0 {
        rte_eal_lcore_role;
        rte_eal_mp_remote_launch;
        rte_eal_mp_wait_lcore;
-       rte_eal_parse_devargs_str;
        rte_eal_process_type;
        rte_eal_remote_launch;
        rte_eal_tailq_lookup;
-- 
2.1.4