From 0f8b6684b2849f99e2cd2026dea77e912a08b539 Mon Sep 17 00:00:00 2001 From: Amit Mehta Date: Wed, 10 Nov 2021 14:55:01 +0530 Subject: [PATCH] qcacmn: Remove csum_enabled flag and related changes As part of code change we are removing csum_enabled flag check. In that case, csum_enabled becomes redundant since we are not using it in the code base, So remove redundant code as part of the cleanup. Change-Id: Iac411b20f06436053b1969a1af9e3b3ee418c34c CRs-Fixed: 3070858 --- dp/inc/cdp_txrx_cmn_struct.h | 7 ++----- dp/wifi3.0/dp_main.c | 5 ----- dp/wifi3.0/dp_tx.c | 26 -------------------------- dp/wifi3.0/dp_types.h | 3 --- 4 files changed, 2 insertions(+), 39 deletions(-) diff --git a/dp/inc/cdp_txrx_cmn_struct.h b/dp/inc/cdp_txrx_cmn_struct.h index e8dd3a280abc..3313ee325a95 100644 --- a/dp/inc/cdp_txrx_cmn_struct.h +++ b/dp/inc/cdp_txrx_cmn_struct.h @@ -1,5 +1,6 @@ /* - * Copyright (c) 2011-2020 The Linux Foundation. All rights reserved. + * Copyright (c) 2011-2021 The Linux Foundation. All rights reserved. + * Copyright (c) 2021 Qualcomm Innovation Center, Inc. All rights reserved. * * Permission to use, copy, modify, and/or distribute this software for * any purpose with or without fee is hereby granted, provided that the @@ -1138,8 +1139,6 @@ enum cdp_pdev_param_type { * * @cdp_psoc_param_en_rate_stats: set rate stats enable/disable * @cdp_psoc_param_en_nss_cfg: set nss cfg - * - * @cdp_enable_tx_checksum: Flag to specify if HW Tx checksum enabled */ typedef union cdp_config_param_t { /* peer params */ @@ -1209,7 +1208,6 @@ typedef union cdp_config_param_t { int cdp_psoc_param_preferred_hw_mode; bool cdp_psoc_param_pext_stats; - bool cdp_enable_tx_checksum; } cdp_config_param_type; /** @@ -1307,7 +1305,6 @@ enum cdp_vdev_param_type { #endif CDP_SAFEMODE, CDP_DROP_UNENC, - CDP_ENABLE_CSUM, CDP_ENABLE_IGMP_MCAST_EN, CDP_ENABLE_HLOS_TID_OVERRIDE, #ifdef QCA_SUPPORT_WDS_EXTENDED diff --git a/dp/wifi3.0/dp_main.c b/dp/wifi3.0/dp_main.c index 6067336cb479..2ec38f9eb9ae 100644 --- a/dp/wifi3.0/dp_main.c +++ b/dp/wifi3.0/dp_main.c @@ -9324,11 +9324,6 @@ dp_set_vdev_param(struct cdp_soc_t *cdp_soc, uint8_t vdev_id, val.cdp_vdev_param_mesh_mode); break; #endif - case CDP_ENABLE_CSUM: - dp_info("vdev_id %d enable Checksum %d", vdev_id, - val.cdp_enable_tx_checksum); - vdev->csum_enabled = val.cdp_enable_tx_checksum; - break; case CDP_ENABLE_HLOS_TID_OVERRIDE: dp_info("vdev_id %d enable hlod tid override %d", vdev_id, val.cdp_vdev_param_hlos_tid_override); diff --git a/dp/wifi3.0/dp_tx.c b/dp/wifi3.0/dp_tx.c index ce555cea9100..a499384d3dd4 100644 --- a/dp/wifi3.0/dp_tx.c +++ b/dp/wifi3.0/dp_tx.c @@ -5017,30 +5017,6 @@ qdf_nbuf_t dp_tx_non_std(struct cdp_soc_t *soc_hdl, uint8_t vdev_id, } #endif -static void dp_tx_vdev_update_feature_flags(struct dp_vdev *vdev) -{ - struct wlan_cfg_dp_soc_ctxt *cfg; - - struct dp_soc *soc; - - soc = vdev->pdev->soc; - if (!soc) - return; - - cfg = soc->wlan_cfg_ctx; - if (!cfg) - return; - - if (vdev->opmode == wlan_op_mode_ndi) - vdev->csum_enabled = wlan_cfg_get_nan_checksum_offload(cfg); - else if ((vdev->subtype == wlan_op_subtype_p2p_device) || - (vdev->subtype == wlan_op_subtype_p2p_cli) || - (vdev->subtype == wlan_op_subtype_p2p_go)) - vdev->csum_enabled = wlan_cfg_get_p2p_checksum_offload(cfg); - else - vdev->csum_enabled = wlan_cfg_get_checksum_offload(cfg); -} - /** * dp_tx_vdev_attach() - attach vdev to dp tx * @vdev: virtual device instance @@ -5072,8 +5048,6 @@ QDF_STATUS dp_tx_vdev_attach(struct dp_vdev *vdev) dp_tx_vdev_update_search_flags(vdev); - dp_tx_vdev_update_feature_flags(vdev); - return QDF_STATUS_SUCCESS; } diff --git a/dp/wifi3.0/dp_types.h b/dp/wifi3.0/dp_types.h index 87587b070bfc..248e22d4dcb7 100644 --- a/dp/wifi3.0/dp_types.h +++ b/dp/wifi3.0/dp_types.h @@ -2579,9 +2579,6 @@ struct dp_vdev { /* IGMP multicast enhancement enabled */ uint8_t igmp_mcast_enhanc_en; - /* HW TX Checksum Enabled Flag */ - uint8_t csum_enabled; - /* vdev_id - ID used to specify a particular vdev to the target */ uint8_t vdev_id;