The expression *dev->dev_ops->dev_configure and
dev->dev_ops->dev_configure are equivalent.
Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
---
lib/rawdev/rte_rawdev.c | 99 ++++++++++++++++++++---------------------
1 file changed, 48 insertions(+), 51 deletions(-)
diff --git a/lib/rawdev/rte_rawdev.c b/lib/rawdev/rte_rawdev.c
index 4f8897b639..810e95a440 100644
--- a/lib/rawdev/rte_rawdev.c
+++ b/lib/rawdev/rte_rawdev.c
@@ -77,9 +77,9 @@ rte_rawdev_info_get(uint16_t dev_id, struct rte_rawdev_info
*dev_info,
rawdev = &rte_rawdevs[dev_id];
if (dev_info->dev_private != NULL) {
- if (*rawdev->dev_ops->dev_info_get == NULL)
+ if (rawdev->dev_ops->dev_info_get == NULL)
return -ENOTSUP;
- ret = (*rawdev->dev_ops->dev_info_get)(rawdev,
+ ret = rawdev->dev_ops->dev_info_get(rawdev,
dev_info->dev_private,
dev_private_size);
}
@@ -104,7 +104,7 @@ rte_rawdev_configure(uint16_t dev_id, struct
rte_rawdev_info *dev_conf,
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->dev_configure == NULL)
+ if (dev->dev_ops->dev_configure == NULL)
return -ENOTSUP;
if (dev->started) {
@@ -114,8 +114,7 @@ rte_rawdev_configure(uint16_t dev_id, struct
rte_rawdev_info *dev_conf,
}
/* Configure the device */
- diag = (*dev->dev_ops->dev_configure)(dev, dev_conf->dev_private,
- dev_private_size);
+ diag = dev->dev_ops->dev_configure(dev, dev_conf->dev_private,
dev_private_size);
if (diag != 0)
RTE_RDEV_ERR("dev%d dev_configure = %d", dev_id, diag);
else
@@ -135,10 +134,9 @@ rte_rawdev_queue_conf_get(uint16_t dev_id,
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->queue_def_conf == NULL)
+ if (dev->dev_ops->queue_def_conf == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->queue_def_conf)(dev, queue_id, queue_conf,
- queue_conf_size);
+ return dev->dev_ops->queue_def_conf(dev, queue_id, queue_conf,
queue_conf_size);
}
int
@@ -152,10 +150,9 @@ rte_rawdev_queue_setup(uint16_t dev_id,
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->queue_setup == NULL)
+ if (dev->dev_ops->queue_setup == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->queue_setup)(dev, queue_id, queue_conf,
- queue_conf_size);
+ return dev->dev_ops->queue_setup(dev, queue_id, queue_conf,
queue_conf_size);
}
int
@@ -166,9 +163,9 @@ rte_rawdev_queue_release(uint16_t dev_id, uint16_t queue_id)
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->queue_release == NULL)
+ if (dev->dev_ops->queue_release == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->queue_release)(dev, queue_id);
+ return dev->dev_ops->queue_release(dev, queue_id);
}
uint16_t
@@ -179,9 +176,9 @@ rte_rawdev_queue_count(uint16_t dev_id)
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->queue_count == NULL)
+ if (dev->dev_ops->queue_count == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->queue_count)(dev);
+ return dev->dev_ops->queue_count(dev);
}
int
@@ -194,9 +191,9 @@ rte_rawdev_get_attr(uint16_t dev_id,
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->attr_get == NULL)
+ if (dev->dev_ops->attr_get == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->attr_get)(dev, attr_name, attr_value);
+ return dev->dev_ops->attr_get(dev, attr_name, attr_value);
}
int
@@ -209,9 +206,9 @@ rte_rawdev_set_attr(uint16_t dev_id,
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->attr_set == NULL)
+ if (dev->dev_ops->attr_set == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->attr_set)(dev, attr_name, attr_value);
+ return dev->dev_ops->attr_set(dev, attr_name, attr_value);
}
int
@@ -225,9 +222,9 @@ rte_rawdev_enqueue_buffers(uint16_t dev_id,
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->enqueue_bufs == NULL)
+ if (dev->dev_ops->enqueue_bufs == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->enqueue_bufs)(dev, buffers, count, context);
+ return dev->dev_ops->enqueue_bufs(dev, buffers, count, context);
}
int
@@ -241,9 +238,9 @@ rte_rawdev_dequeue_buffers(uint16_t dev_id,
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->dequeue_bufs == NULL)
+ if (dev->dev_ops->dequeue_bufs == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->dequeue_bufs)(dev, buffers, count, context);
+ return dev->dev_ops->dequeue_bufs(dev, buffers, count, context);
}
int
@@ -254,9 +251,9 @@ rte_rawdev_dump(uint16_t dev_id, FILE *f)
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->dump == NULL)
+ if (dev->dev_ops->dump == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->dump)(dev, f);
+ return dev->dev_ops->dump(dev, f);
}
static int
@@ -264,9 +261,9 @@ xstats_get_count(uint16_t dev_id)
{
struct rte_rawdev *dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->xstats_get_names == NULL)
+ if (dev->dev_ops->xstats_get_names == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->xstats_get_names)(dev, NULL, 0);
+ return dev->dev_ops->xstats_get_names(dev, NULL, 0);
}
int
@@ -287,9 +284,9 @@ rte_rawdev_xstats_names_get(uint16_t dev_id,
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->xstats_get_names == NULL)
+ if (dev->dev_ops->xstats_get_names == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->xstats_get_names)(dev, xstats_names, size);
+ return dev->dev_ops->xstats_get_names(dev, xstats_names, size);
}
/* retrieve rawdev extended statistics */
@@ -302,9 +299,9 @@ rte_rawdev_xstats_get(uint16_t dev_id,
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -ENODEV);
const struct rte_rawdev *dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->xstats_get == NULL)
+ if (dev->dev_ops->xstats_get == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->xstats_get)(dev, ids, values, n);
+ return dev->dev_ops->xstats_get(dev, ids, values, n);
}
uint64_t
@@ -322,9 +319,9 @@ rte_rawdev_xstats_by_name_get(uint16_t dev_id,
id = &temp; /* driver never gets a NULL value */
/* implemented by driver */
- if (*dev->dev_ops->xstats_get_by_name == NULL)
+ if (dev->dev_ops->xstats_get_by_name == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->xstats_get_by_name)(dev, name, id);
+ return dev->dev_ops->xstats_get_by_name(dev, name, id);
}
int
@@ -334,9 +331,9 @@ rte_rawdev_xstats_reset(uint16_t dev_id,
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
struct rte_rawdev *dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->xstats_reset == NULL)
+ if (dev->dev_ops->xstats_reset == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->xstats_reset)(dev, ids, nb_ids);
+ return dev->dev_ops->xstats_reset(dev, ids, nb_ids);
}
int
@@ -345,9 +342,9 @@ rte_rawdev_firmware_status_get(uint16_t dev_id,
rte_rawdev_obj_t status_info)
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
struct rte_rawdev *dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->firmware_status_get == NULL)
+ if (dev->dev_ops->firmware_status_get == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->firmware_status_get)(dev, status_info);
+ return dev->dev_ops->firmware_status_get(dev, status_info);
}
int
@@ -356,9 +353,9 @@ rte_rawdev_firmware_version_get(uint16_t dev_id,
rte_rawdev_obj_t version_info)
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
struct rte_rawdev *dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->firmware_version_get == NULL)
+ if (dev->dev_ops->firmware_version_get == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->firmware_version_get)(dev, version_info);
+ return dev->dev_ops->firmware_version_get(dev, version_info);
}
int
@@ -370,9 +367,9 @@ rte_rawdev_firmware_load(uint16_t dev_id, rte_rawdev_obj_t
firmware_image)
if (!firmware_image)
return -EINVAL;
- if (*dev->dev_ops->firmware_load == NULL)
+ if (dev->dev_ops->firmware_load == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->firmware_load)(dev, firmware_image);
+ return dev->dev_ops->firmware_load(dev, firmware_image);
}
int
@@ -381,9 +378,9 @@ rte_rawdev_firmware_unload(uint16_t dev_id)
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
struct rte_rawdev *dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->firmware_load == NULL)
+ if (dev->dev_ops->firmware_load == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->firmware_unload)(dev);
+ return dev->dev_ops->firmware_unload(dev);
}
int
@@ -392,9 +389,9 @@ rte_rawdev_selftest(uint16_t dev_id)
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
struct rte_rawdev *dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->dev_selftest == NULL)
+ if (dev->dev_ops->dev_selftest == NULL)
return -ENOTSUP;
- return (*dev->dev_ops->dev_selftest)(dev_id);
+ return dev->dev_ops->dev_selftest(dev_id);
}
int
@@ -416,7 +413,7 @@ rte_rawdev_start(uint16_t dev_id)
if (dev->dev_ops->dev_start == NULL)
goto mark_started;
- diag = (*dev->dev_ops->dev_start)(dev);
+ diag = dev->dev_ops->dev_start(dev);
if (diag != 0)
return diag;
@@ -444,7 +441,7 @@ rte_rawdev_stop(uint16_t dev_id)
if (dev->dev_ops->dev_stop == NULL)
goto mark_stopped;
- (*dev->dev_ops->dev_stop)(dev);
+ dev->dev_ops->dev_stop(dev);
mark_stopped:
dev->started = 0;
@@ -458,7 +455,7 @@ rte_rawdev_close(uint16_t dev_id)
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->dev_close == NULL)
+ if (dev->dev_ops->dev_close == NULL)
return -ENOTSUP;
/* Device must be stopped before it can be closed */
if (dev->started == 1) {
@@ -467,7 +464,7 @@ rte_rawdev_close(uint16_t dev_id)
return -EBUSY;
}
- return (*dev->dev_ops->dev_close)(dev);
+ return dev->dev_ops->dev_close(dev);
}
int
@@ -478,10 +475,10 @@ rte_rawdev_reset(uint16_t dev_id)
RTE_RAWDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
dev = &rte_rawdevs[dev_id];
- if (*dev->dev_ops->dev_reset == NULL)
+ if (dev->dev_ops->dev_reset == NULL)
return -ENOTSUP;
/* Reset is not dependent on state of the device */
- return (*dev->dev_ops->dev_reset)(dev);
+ return dev->dev_ops->dev_reset(dev);
}
static inline uint8_t
--
2.47.2
