qed*: remove version dependency
authorRahul Verma <rahul.verma@qlogic.com>
Sun, 10 Apr 2016 09:42:59 +0000 (12:42 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 14 Apr 2016 04:43:20 +0000 (00:43 -0400)
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>
drivers/net/ethernet/qlogic/qed/qed.h
drivers/net/ethernet/qlogic/qed/qed_l2.c
drivers/net/ethernet/qlogic/qed/qed_main.c
drivers/net/ethernet/qlogic/qede/qede.h
drivers/net/ethernet/qlogic/qede/qede_main.c
include/linux/qed/qed_eth_if.h
include/linux/qed/qed_if.h

index fcb8e9ba51d9b6ad85ecf6bd2361e71896c6b592..a3ee9df16dfef927e8d783df8cdf19647ce46033 100644 (file)
@@ -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 */
index 3f35c6ca92528d3cc1951df3d09f8ef19c3dd8ba..e848d5a1f7f6900c9107baa53edc8c95cbf0b1fd 100644 (file)
@@ -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);
index 26d40db07ddd12a13ae07f7494e3244e03f70af4..c31d485f72d68dd4fc5d861a15b848b27718fa9b 100644 (file)
@@ -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);
index d023251544d9e7f8df8a1e6c897293385bd45fc1..e0a696a57d4d08828c68bdf601d42756ce50e63e 100644 (file)
@@ -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 {
index 518af329502ddff20c8fc322ebed3303b7b0b9a9..a55d93eb41fa3593019ab269c7175432dbfc01e3 100644 (file)
@@ -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;
index e1d69834a11fb38c45fe8e6fe0b3d0ad53af7ee4..e00c8dbfc3247ad431f7140cea7ad197a7385a93 100644 (file)
@@ -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
index 1f7599c77cd4185de5579d6c5801655706a982f3..b007011e1c82fb19384fd5d1b5b8eb6ae77e5111 100644 (file)
@@ -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))