Skip to content

Commit

Permalink
qed*: remove version dependency
Browse files Browse the repository at this point in the history
Inbox drivers don't need versioning scheme in order to guarantee
compatibility, as both qed and qede are compiled from same codebase.

Signed-off-by: Rahul Verma <rahul.verma@qlogic.com>
Signed-off-by: Yuval Mintz <Yuval.Mintz@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Rahul Verma authored and David S. Miller committed Apr 14, 2016
1 parent 6c61403 commit 9511434
Show file tree
Hide file tree
Showing 7 changed files with 3 additions and 42 deletions.
2 changes: 0 additions & 2 deletions drivers/net/ethernet/qlogic/qed/qed.h
Original file line number Diff line number Diff line change
Expand Up @@ -507,6 +507,4 @@ u32 qed_unzip_data(struct qed_hwfn *p_hwfn,

int qed_slowpath_irq_req(struct qed_hwfn *hwfn);

#define QED_ETH_INTERFACE_VERSION 300

#endif /* _QED_H */
8 changes: 1 addition & 7 deletions drivers/net/ethernet/qlogic/qed/qed_l2.c
Original file line number Diff line number Diff line change
Expand Up @@ -2043,14 +2043,8 @@ static const struct qed_eth_ops qed_eth_ops_pass = {
.get_vport_stats = &qed_get_vport_stats,
};

const struct qed_eth_ops *qed_get_eth_ops(u32 version)
const struct qed_eth_ops *qed_get_eth_ops(void)
{
if (version != QED_ETH_INTERFACE_VERSION) {
pr_notice("Cannot supply ethtool operations [%08x != %08x]\n",
version, QED_ETH_INTERFACE_VERSION);
return NULL;
}

return &qed_eth_ops_pass;
}
EXPORT_SYMBOL(qed_get_eth_ops);
Expand Down
11 changes: 0 additions & 11 deletions drivers/net/ethernet/qlogic/qed/qed_main.c
Original file line number Diff line number Diff line change
Expand Up @@ -1172,14 +1172,3 @@ const struct qed_common_ops qed_common_ops_pass = {
.chain_free = &qed_chain_free,
.set_led = &qed_set_led,
};

u32 qed_get_protocol_version(enum qed_protocol protocol)
{
switch (protocol) {
case QED_PROTOCOL_ETH:
return QED_ETH_INTERFACE_VERSION;
default:
return 0;
}
}
EXPORT_SYMBOL(qed_get_protocol_version);
2 changes: 0 additions & 2 deletions drivers/net/ethernet/qlogic/qede/qede.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,6 @@
__stringify(QEDE_REVISION_VERSION) "." \
__stringify(QEDE_ENGINEERING_VERSION)

#define QEDE_ETH_INTERFACE_VERSION 300

#define DRV_MODULE_SYM qede

struct qede_stats {
Expand Down
11 changes: 1 addition & 10 deletions drivers/net/ethernet/qlogic/qede/qede_main.c
Original file line number Diff line number Diff line change
Expand Up @@ -141,19 +141,10 @@ static
int __init qede_init(void)
{
int ret;
u32 qed_ver;

pr_notice("qede_init: %s\n", version);

qed_ver = qed_get_protocol_version(QED_PROTOCOL_ETH);
if (qed_ver != QEDE_ETH_INTERFACE_VERSION) {
pr_notice("Version mismatch [%08x != %08x]\n",
qed_ver,
QEDE_ETH_INTERFACE_VERSION);
return -EINVAL;
}

qed_ops = qed_get_eth_ops(QEDE_ETH_INTERFACE_VERSION);
qed_ops = qed_get_eth_ops();
if (!qed_ops) {
pr_notice("Failed to get qed ethtool operations\n");
return -EINVAL;
Expand Down
2 changes: 1 addition & 1 deletion include/linux/qed/qed_eth_if.h
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ struct qed_eth_ops {
struct qed_eth_stats *stats);
};

const struct qed_eth_ops *qed_get_eth_ops(u32 version);
const struct qed_eth_ops *qed_get_eth_ops(void);
void qed_put_eth_ops(void);

#endif
9 changes: 0 additions & 9 deletions include/linux/qed/qed_if.h
Original file line number Diff line number Diff line change
Expand Up @@ -271,15 +271,6 @@ struct qed_common_ops {
enum qed_led_mode mode);
};

/**
* @brief qed_get_protocol_version
*
* @param protocol
*
* @return version supported by qed for given protocol driver
*/
u32 qed_get_protocol_version(enum qed_protocol protocol);

#define MASK_FIELD(_name, _value) \
((_value) &= (_name ## _MASK))

Expand Down

0 comments on commit 9511434

Please sign in to comment.