qlcnic: Check FW capability for TSO

Driver checks TSO capability from FW before enabling it.

Signed-off-by: Anirban Chakraborty <anirban.chakraborty@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

3 files changed, 14 insertions, 6 deletions

diff --git a/drivers/net/qlcnic/qlcnic.h b/drivers/net/qlcnic/qlcnic.h
index 60ea7cb3308f..02a50e60166d 100644
--- a/drivers/net/qlcnic/qlcnic.h
+++ b/drivers/net/qlcnic/qlcnic.h
@@ -821,9 +821,10 @@ struct qlcnic_nic_intr_coalesce {
 #define QLCNIC_LRO_REQUEST_CLEANUP      4
 
 /* Capabilites received */
-#define QLCNIC_FW_CAPABILITY_BDG                (1 << 8)
-#define QLCNIC_FW_CAPABILITY_FVLANTX            (1 << 9)
-#define QLCNIC_FW_CAPABILITY_HW_LRO             (1 << 10)
+#define QLCNIC_FW_CAPABILITY_TSO                BIT_1
+#define QLCNIC_FW_CAPABILITY_BDG                BIT_8
+#define QLCNIC_FW_CAPABILITY_FVLANTX            BIT_9
+#define QLCNIC_FW_CAPABILITY_HW_LRO             BIT_10
 
 /* module types */
 #define LINKEVENT_MODULE_NOT_PRESENT                    1
diff --git a/drivers/net/qlcnic/qlcnic_ethtool.c b/drivers/net/qlcnic/qlcnic_ethtool.c
index f8e39e4cfe0f..baf5a52b2a4b 100644
--- a/drivers/net/qlcnic/qlcnic_ethtool.c
+++ b/drivers/net/qlcnic/qlcnic_ethtool.c
@@ -820,6 +820,9 @@ static u32 qlcnic_get_tso(struct net_device *dev)
 
 static int qlcnic_set_tso(struct net_device *dev, u32 data)
 {
+        struct qlcnic_adapter *adapter = netdev_priv(dev);
+        if (!(adapter->capabilities & QLCNIC_FW_CAPABILITY_TSO))
+                return -EOPNOTSUPP;
         if (data)
                 dev->features |= (NETIF_F_TSO | NETIF_F_TSO6);
         else
diff --git a/drivers/net/qlcnic/qlcnic_main.c b/drivers/net/qlcnic/qlcnic_main.c
index 18e2b2e6626d..c334f1a0a2a4 100644
--- a/drivers/net/qlcnic/qlcnic_main.c
+++ b/drivers/net/qlcnic/qlcnic_main.c
@@ -1226,10 +1226,14 @@ qlcnic_setup_netdev(struct qlcnic_adapter *adapter,
         SET_ETHTOOL_OPS(netdev, &qlcnic_ethtool_ops);
 
         netdev->features |= (NETIF_F_SG | NETIF_F_IP_CSUM |
-                NETIF_F_IPV6_CSUM | NETIF_F_GRO | NETIF_F_TSO | NETIF_F_TSO6);
-
+                NETIF_F_IPV6_CSUM | NETIF_F_GRO);
         netdev->vlan_features |= (NETIF_F_SG | NETIF_F_IP_CSUM |
-                NETIF_F_IPV6_CSUM | NETIF_F_TSO | NETIF_F_TSO6);
+                NETIF_F_IPV6_CSUM);
+
+        if (adapter->capabilities & QLCNIC_FW_CAPABILITY_TSO) {
+                netdev->features |= (NETIF_F_TSO | NETIF_F_TSO6);
+                netdev->vlan_features |= (NETIF_F_TSO | NETIF_F_TSO6);
+        }
 
         if (pci_using_dac) {
                 netdev->features |= NETIF_F_HIGHDMA;