Commit 4b764d1d authored Jan 12, 2017 by Julian Wiedmann Committed by David S. Miller Jan 12, 2017

s390/qeth: don't convert return code twice



qeth_l2_send_groupmac() already translates the return code, so
calling qeth_setdel_makerc() a second time only produces garbage.

Signed-off-by: Julian Wiedmann <jwi@linux.vnet.ibm.com>
Reviewed-by: Thomas Richter <tmricht@linux.vnet.ibm.com>
Reviewed-by: Ursula Braun <ubraun@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent c07cbf2e

drivers/s390/net/qeth_l2_main.c

+1 −2

Original line number	Diff line number	Diff line
		@@ -210,8 +210,7 @@ static int qeth_l2_write_mac(struct qeth_card card, struct qeth_mac mac)
		qeth_l2_send_setdelmac(card, mac->mac_addr,
		IPA_CMD_SETVMAC));
		} else {
		rc = qeth_setdel_makerc(card,
		qeth_l2_send_setgroupmac(card, mac->mac_addr));
		rc = qeth_l2_send_setgroupmac(card, mac->mac_addr);
		}
		return rc;
		}