diff options
author | Thomas Richter <tmricht@linux.vnet.ibm.com> | 2015-12-11 06:27:54 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-12-14 16:16:45 -0500 |
commit | b475e3161ce74a7e2ee72322585ed5f39985c06a (patch) | |
tree | 9f7ba449d1d217e09cd0755f02a86166860a10ef /drivers/s390 | |
parent | e043046a5a29bebe7e9a68b92c4580e2888f8ed3 (diff) |
qeth use common function qeth_get_setassparms_cmd
There have been 2 identical versions of function
qeth_get_setassparms_cmd() for layer 2 and layer 3.
Remove the layer 3 function qeth_l3_get_setassparms_cmd()
and call the common one named qeth_get_setassparms_cmd()
located in qeth_core_main.c
Signed-off-by: Thomas Richter <tmricht@linux.vnet.ibm.com>
Signed-off-by: Ursula Braun <ubraun@linux.vnet.ibm.com>
Reviewed-by: Eugene Crosser <Eugene.Crosser@ru.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/s390')
-rw-r--r-- | drivers/s390/net/qeth_core.h | 4 | ||||
-rw-r--r-- | drivers/s390/net/qeth_core_main.c | 9 | ||||
-rw-r--r-- | drivers/s390/net/qeth_l3_main.c | 37 |
3 files changed, 17 insertions, 33 deletions
diff --git a/drivers/s390/net/qeth_core.h b/drivers/s390/net/qeth_core.h index 1766a20ebcb1..ec2e014e885c 100644 --- a/drivers/s390/net/qeth_core.h +++ b/drivers/s390/net/qeth_core.h | |||
@@ -981,6 +981,10 @@ int qeth_send_setassparms(struct qeth_card *, struct qeth_cmd_buffer *, __u16, | |||
981 | int (*reply_cb)(struct qeth_card *, | 981 | int (*reply_cb)(struct qeth_card *, |
982 | struct qeth_reply *, unsigned long), | 982 | struct qeth_reply *, unsigned long), |
983 | void *); | 983 | void *); |
984 | struct qeth_cmd_buffer *qeth_get_setassparms_cmd(struct qeth_card *, | ||
985 | enum qeth_ipa_funcs, | ||
986 | __u16, __u16, | ||
987 | enum qeth_prot_versions); | ||
984 | int qeth_start_ipa_tx_checksum(struct qeth_card *); | 988 | int qeth_start_ipa_tx_checksum(struct qeth_card *); |
985 | int qeth_set_rx_csum(struct qeth_card *, int); | 989 | int qeth_set_rx_csum(struct qeth_card *, int); |
986 | 990 | ||
diff --git a/drivers/s390/net/qeth_core_main.c b/drivers/s390/net/qeth_core_main.c index 31ac53fa5cee..fe1845acf560 100644 --- a/drivers/s390/net/qeth_core_main.c +++ b/drivers/s390/net/qeth_core_main.c | |||
@@ -5297,10 +5297,10 @@ static int qeth_setassparms_cb(struct qeth_card *card, | |||
5297 | return 0; | 5297 | return 0; |
5298 | } | 5298 | } |
5299 | 5299 | ||
5300 | static struct qeth_cmd_buffer *qeth_get_setassparms_cmd(struct qeth_card *card, | 5300 | struct qeth_cmd_buffer *qeth_get_setassparms_cmd(struct qeth_card *card, |
5301 | enum qeth_ipa_funcs ipa_func, | 5301 | enum qeth_ipa_funcs ipa_func, |
5302 | __u16 cmd_code, __u16 len, | 5302 | __u16 cmd_code, __u16 len, |
5303 | enum qeth_prot_versions prot) | 5303 | enum qeth_prot_versions prot) |
5304 | { | 5304 | { |
5305 | struct qeth_cmd_buffer *iob; | 5305 | struct qeth_cmd_buffer *iob; |
5306 | struct qeth_ipa_cmd *cmd; | 5306 | struct qeth_ipa_cmd *cmd; |
@@ -5319,6 +5319,7 @@ static struct qeth_cmd_buffer *qeth_get_setassparms_cmd(struct qeth_card *card, | |||
5319 | 5319 | ||
5320 | return iob; | 5320 | return iob; |
5321 | } | 5321 | } |
5322 | EXPORT_SYMBOL_GPL(qeth_get_setassparms_cmd); | ||
5322 | 5323 | ||
5323 | int qeth_send_setassparms(struct qeth_card *card, | 5324 | int qeth_send_setassparms(struct qeth_card *card, |
5324 | struct qeth_cmd_buffer *iob, __u16 len, long data, | 5325 | struct qeth_cmd_buffer *iob, __u16 len, long data, |
diff --git a/drivers/s390/net/qeth_l3_main.c b/drivers/s390/net/qeth_l3_main.c index 543960e96b42..e2eb88c0d41d 100644 --- a/drivers/s390/net/qeth_l3_main.c +++ b/drivers/s390/net/qeth_l3_main.c | |||
@@ -1043,28 +1043,6 @@ static int qeth_l3_default_setassparms_cb(struct qeth_card *card, | |||
1043 | return 0; | 1043 | return 0; |
1044 | } | 1044 | } |
1045 | 1045 | ||
1046 | static struct qeth_cmd_buffer *qeth_l3_get_setassparms_cmd( | ||
1047 | struct qeth_card *card, enum qeth_ipa_funcs ipa_func, __u16 cmd_code, | ||
1048 | __u16 len, enum qeth_prot_versions prot) | ||
1049 | { | ||
1050 | struct qeth_cmd_buffer *iob; | ||
1051 | struct qeth_ipa_cmd *cmd; | ||
1052 | |||
1053 | QETH_CARD_TEXT(card, 4, "getasscm"); | ||
1054 | iob = qeth_get_ipacmd_buffer(card, IPA_CMD_SETASSPARMS, prot); | ||
1055 | |||
1056 | if (iob) { | ||
1057 | cmd = (struct qeth_ipa_cmd *)(iob->data+IPA_PDU_HEADER_SIZE); | ||
1058 | cmd->data.setassparms.hdr.assist_no = ipa_func; | ||
1059 | cmd->data.setassparms.hdr.length = 8 + len; | ||
1060 | cmd->data.setassparms.hdr.command_code = cmd_code; | ||
1061 | cmd->data.setassparms.hdr.return_code = 0; | ||
1062 | cmd->data.setassparms.hdr.seq_no = 0; | ||
1063 | } | ||
1064 | |||
1065 | return iob; | ||
1066 | } | ||
1067 | |||
1068 | #ifdef CONFIG_QETH_IPV6 | 1046 | #ifdef CONFIG_QETH_IPV6 |
1069 | static int qeth_l3_send_simple_setassparms_ipv6(struct qeth_card *card, | 1047 | static int qeth_l3_send_simple_setassparms_ipv6(struct qeth_card *card, |
1070 | enum qeth_ipa_funcs ipa_func, __u16 cmd_code) | 1048 | enum qeth_ipa_funcs ipa_func, __u16 cmd_code) |
@@ -1073,7 +1051,7 @@ static int qeth_l3_send_simple_setassparms_ipv6(struct qeth_card *card, | |||
1073 | struct qeth_cmd_buffer *iob; | 1051 | struct qeth_cmd_buffer *iob; |
1074 | 1052 | ||
1075 | QETH_CARD_TEXT(card, 4, "simassp6"); | 1053 | QETH_CARD_TEXT(card, 4, "simassp6"); |
1076 | iob = qeth_l3_get_setassparms_cmd(card, ipa_func, cmd_code, | 1054 | iob = qeth_get_setassparms_cmd(card, ipa_func, cmd_code, |
1077 | 0, QETH_PROT_IPV6); | 1055 | 0, QETH_PROT_IPV6); |
1078 | if (!iob) | 1056 | if (!iob) |
1079 | return -ENOMEM; | 1057 | return -ENOMEM; |
@@ -2344,10 +2322,11 @@ static int qeth_l3_query_arp_cache_info(struct qeth_card *card, | |||
2344 | 2322 | ||
2345 | QETH_CARD_TEXT_(card, 3, "qarpipv%i", prot); | 2323 | QETH_CARD_TEXT_(card, 3, "qarpipv%i", prot); |
2346 | 2324 | ||
2347 | iob = qeth_l3_get_setassparms_cmd(card, IPA_ARP_PROCESSING, | 2325 | iob = qeth_get_setassparms_cmd(card, IPA_ARP_PROCESSING, |
2348 | IPA_CMD_ASS_ARP_QUERY_INFO, | 2326 | IPA_CMD_ASS_ARP_QUERY_INFO, |
2349 | sizeof(struct qeth_arp_query_data) - sizeof(char), | 2327 | sizeof(struct qeth_arp_query_data) |
2350 | prot); | 2328 | - sizeof(char), |
2329 | prot); | ||
2351 | if (!iob) | 2330 | if (!iob) |
2352 | return -ENOMEM; | 2331 | return -ENOMEM; |
2353 | cmd = (struct qeth_ipa_cmd *)(iob->data+IPA_PDU_HEADER_SIZE); | 2332 | cmd = (struct qeth_ipa_cmd *)(iob->data+IPA_PDU_HEADER_SIZE); |
@@ -2439,7 +2418,7 @@ static int qeth_l3_arp_add_entry(struct qeth_card *card, | |||
2439 | return -EOPNOTSUPP; | 2418 | return -EOPNOTSUPP; |
2440 | } | 2419 | } |
2441 | 2420 | ||
2442 | iob = qeth_l3_get_setassparms_cmd(card, IPA_ARP_PROCESSING, | 2421 | iob = qeth_get_setassparms_cmd(card, IPA_ARP_PROCESSING, |
2443 | IPA_CMD_ASS_ARP_ADD_ENTRY, | 2422 | IPA_CMD_ASS_ARP_ADD_ENTRY, |
2444 | sizeof(struct qeth_arp_cache_entry), | 2423 | sizeof(struct qeth_arp_cache_entry), |
2445 | QETH_PROT_IPV4); | 2424 | QETH_PROT_IPV4); |
@@ -2480,7 +2459,7 @@ static int qeth_l3_arp_remove_entry(struct qeth_card *card, | |||
2480 | return -EOPNOTSUPP; | 2459 | return -EOPNOTSUPP; |
2481 | } | 2460 | } |
2482 | memcpy(buf, entry, 12); | 2461 | memcpy(buf, entry, 12); |
2483 | iob = qeth_l3_get_setassparms_cmd(card, IPA_ARP_PROCESSING, | 2462 | iob = qeth_get_setassparms_cmd(card, IPA_ARP_PROCESSING, |
2484 | IPA_CMD_ASS_ARP_REMOVE_ENTRY, | 2463 | IPA_CMD_ASS_ARP_REMOVE_ENTRY, |
2485 | 12, | 2464 | 12, |
2486 | QETH_PROT_IPV4); | 2465 | QETH_PROT_IPV4); |