From 6819fc8b59361b7ff35b73faec801e11c9ed5f51 Mon Sep 17 00:00:00 2001 From: Amit Mehta Date: Wed, 10 Nov 2021 15:07:08 +0530 Subject: [PATCH] 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 --- core/hdd/src/wlan_hdd_main.c | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/core/hdd/src/wlan_hdd_main.c b/core/hdd/src/wlan_hdd_main.c index 2128c5918a80..e172f8fdcc5a 100644 --- a/core/hdd/src/wlan_hdd_main.c +++ b/core/hdd/src/wlan_hdd_main.c @@ -5127,7 +5127,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 * @@ -5137,7 +5137,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) { @@ -5154,15 +5153,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; }