Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6819fc8b authored by Amit Mehta's avatar Amit Mehta Committed by Madan Koyyalamudi
Browse files

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
parent 967ec1ce

core/hdd/src/wlan_hdd_main.c

+1 −11
Original line number Diff line number Diff line
@@ -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;

}