Browse Source

qcacld-3.0: Remove redundant code

As part of code change we are removing the csum_enabled flag.
In that case, the call to set vdev params becomes redundant
since we are calling it to update the csum_enabled flag value,
So remove redundant code as part of the cleanup.

Change-Id: I41e5ffba0a375bc20146ece9dabca391016d5b67
CRs-Fixed: 3071428
Amit Mehta 3 years ago
parent
commit
a844eaded0
1 changed files with 1 additions and 11 deletions
  1. 1 11
      core/hdd/src/wlan_hdd_main.c

+ 1 - 11
core/hdd/src/wlan_hdd_main.c

@@ -5624,7 +5624,7 @@ static netdev_features_t hdd_fix_features(struct net_device *net_dev,
 	return changed_features;
 }
 /**
- * __hdd_set_features - Update device config for resultant change in feature
+ * __hdd_set_features - Notify device about change in features
  * @net_dev: Handle to net_device
  * @features: Existing + requested feature after resolving the dependency
  *
@@ -5634,7 +5634,6 @@ static int __hdd_set_features(struct net_device *net_dev,
 			      netdev_features_t features)
 {
 	struct hdd_adapter *adapter = netdev_priv(net_dev);
-	cdp_config_param_type vdev_param;
 	ol_txrx_soc_handle soc = cds_get_context(QDF_MODULE_ID_SOC);
 
 	if (!adapter->handle_feature_update) {
@@ -5649,15 +5648,6 @@ static int __hdd_set_features(struct net_device *net_dev,
 		  adapter->device_mode, adapter->vdev_id, net_dev->features,
 		  features);
 
-	if (features & (NETIF_F_IP_CSUM | NETIF_F_IPV6_CSUM))
-		vdev_param.cdp_enable_tx_checksum = true;
-	else
-		vdev_param.cdp_enable_tx_checksum = false;
-
-	if (cdp_txrx_set_vdev_param(soc, adapter->vdev_id, CDP_ENABLE_CSUM,
-				    vdev_param))
-		hdd_debug("Failed to set DP vdev params");
-
 	return 0;
 }