Prefer structure assignment over memcpy.
Found by struct-assign.cocci.

Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
---
 drivers/net/dpaa2/dpaa2_ethdev.c | 5 ++---
 drivers/net/dpaa2/dpaa2_flow.c   | 9 +++------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/drivers/net/dpaa2/dpaa2_ethdev.c b/drivers/net/dpaa2/dpaa2_ethdev.c
index a9bce854c3..452d58ae28 100644
--- a/drivers/net/dpaa2/dpaa2_ethdev.c
+++ b/drivers/net/dpaa2/dpaa2_ethdev.c
@@ -2539,8 +2539,7 @@ populate_mac_addr(struct fsl_mc_io *dpni_dev,
                                              ret);
                                goto cleanup;
                        }
-                       memcpy(&prime_mac, &phy_mac,
-                               sizeof(struct rte_ether_addr));
+                       prime_mac = phy_mac;
                }
        } else if (rte_is_zero_ether_addr(&prime_mac)) {
                /* In case phys and prime, both are zero, create random MAC */
@@ -2555,7 +2554,7 @@ populate_mac_addr(struct fsl_mc_io *dpni_dev,
        }
 
        /* prime_mac the final MAC address */
-       memcpy(mac_entry, &prime_mac, sizeof(struct rte_ether_addr));
+       *mac_entry = prime_mac;
        return 0;
 
 cleanup:
diff --git a/drivers/net/dpaa2/dpaa2_flow.c b/drivers/net/dpaa2/dpaa2_flow.c
index c94eb51ba5..299c50dcdf 100644
--- a/drivers/net/dpaa2/dpaa2_flow.c
+++ b/drivers/net/dpaa2/dpaa2_flow.c
@@ -1008,8 +1008,7 @@ dpaa2_flow_faf_add_hdr(int faf_byte,
        if (pos != dpkg->num_extracts) {
                /* Not the last pos, must have IP address extract.*/
                for (i = dpkg->num_extracts - 1; i >= pos; i--) {
-                       memcpy(&extracts[i + 1],
-                               &extracts[i], sizeof(struct dpkg_extract));
+                       extracts[i + 1] = extracts[i];
                }
        }
 
@@ -1063,8 +1062,7 @@ dpaa2_flow_pr_add_hdr(uint32_t pr_offset,
        if (pos != dpkg->num_extracts) {
                /* Not the last pos, must have IP address extract.*/
                for (i = dpkg->num_extracts - 1; i >= pos; i--) {
-                       memcpy(&extracts[i + 1],
-                               &extracts[i], sizeof(struct dpkg_extract));
+                       extracts[i + 1] = extracts[i];
                }
        }
 
@@ -1118,8 +1116,7 @@ dpaa2_flow_extract_add_hdr(enum net_prot prot,
        if (pos != dpkg->num_extracts) {
                /* Not the last pos, must have IP address extract.*/
                for (i = dpkg->num_extracts - 1; i >= pos; i--) {
-                       memcpy(&extracts[i + 1],
-                               &extracts[i], sizeof(struct dpkg_extract));
+                       extracts[i + 1] = extracts[i];
                }
        }
 
-- 
2.47.2

Reply via email to