bonding: use plain memcpy in __record_default()

Also remove the pointless comment at the top.

Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 2 insertions, 7 deletions

diff --git a/drivers/net/bonding/bond_3ad.c b/drivers/net/bonding/bond_3ad.c
index b789dcb77626..0674d4149808 100644
--- a/drivers/net/bonding/bond_3ad.c
+++ b/drivers/net/bonding/bond_3ad.c
@@ -516,15 +516,10 @@ static void __record_pdu(struct lacpdu *lacpdu, struct port *port)
  */
 static void __record_default(struct port *port)
 {
-        // validate the port
         if (port) {
                 // record the partner admin parameters
-                port->partner_oper.port_number = port->partner_admin.port_number;
-                port->partner_oper.port_priority = port->partner_admin.port_priority;
-                port->partner_oper.system = port->partner_admin.system;
-                port->partner_oper.system_priority = port->partner_admin.system_priority;
-                port->partner_oper.key = port->partner_admin.key;
-                port->partner_oper.port_state = port->partner_admin.port_state;
+                memcpy(&port->partner_oper, &port->partner_admin,
+                       sizeof(struct port_params));
 
                 // set actor_oper_port_state.defaulted to true
                 port->actor_oper_port_state |= AD_STATE_DEFAULTED;