The function rte_zmalloc returns void * so cast is unnecessary.

Correct the indentation in that code. Not really worth backporting.

Signed-off-by: Stephen Hemminger <[email protected]>
---
 drivers/net/pcap/pcap_ethdev.c | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/drivers/net/pcap/pcap_ethdev.c b/drivers/net/pcap/pcap_ethdev.c
index d09ba5abe9..0177cbc7df 100644
--- a/drivers/net/pcap/pcap_ethdev.c
+++ b/drivers/net/pcap/pcap_ethdev.c
@@ -1221,9 +1221,8 @@ pmd_init_internals(struct rte_vdev_device *vdev,
        PMD_LOG(INFO, "Creating pcap-backed ethdev on numa socket %d",
                numa_node);
 
-       pp = (struct pmd_process_private *)
-               rte_zmalloc(NULL, sizeof(struct pmd_process_private),
-                               RTE_CACHE_LINE_SIZE);
+       pp = rte_zmalloc(NULL, sizeof(struct pmd_process_private),
+                        RTE_CACHE_LINE_SIZE);
 
        if (pp == NULL) {
                PMD_LOG(ERR,
@@ -1591,10 +1590,8 @@ pmd_pcap_probe(struct rte_vdev_device *dev)
                unsigned int i;
 
                internal = eth_dev->data->dev_private;
-                       pp = (struct pmd_process_private *)
-                               rte_zmalloc(NULL,
-                                       sizeof(struct pmd_process_private),
-                                       RTE_CACHE_LINE_SIZE);
+               pp = rte_zmalloc(NULL, sizeof(struct pmd_process_private),
+                                RTE_CACHE_LINE_SIZE);
 
                if (pp == NULL) {
                        PMD_LOG(ERR,
-- 
2.51.0

Reply via email to