qcacld-3.0: remove pktlog feature for lithium based target

For lithium based target, rx pktlog depends on monitor status ring
and this ring is reaped by timer in host. if host try to disable
pktlog and high DL T-put traffic is ongoing in parallel, there is
no guarantee that HW stop to update PPDU status to monitor status
ring after host stopped reaping timer, unless host add some delay
between HTT ring reset msg and timer stopping, then FW will have
enough time to reset ring configuration.
Since FW side has disabled pktlog and looks pktlog is not necessary
for lithium based target, remove pktlog support by macro
FEATURE_PKTLOG, macro REMOVE_PKT_LOG is not preferred as it will
remove WDI_EVENT support that impact CFR feature.

Change-Id: I723f65ba3d2ffa58469b886e6795956f69d4e891
CRs-Fixed: 2763095
This commit is contained in:
Jinwei Chen 2020-09-09 19:13:40 +08:00 committed by snandini
parent a34969e63d
commit a11d408201
2 changed files with 7 additions and 6 deletions

View File

@ -552,14 +552,15 @@ endif
# builds. Other OEMs are also protected using the TARGET_BUILD_VARIANT
# config.
ifneq ($(TARGET_BUILD_VARIANT),user)
ifeq ($(CONFIG_LITHIUM), y)
CONFIG_FEATURE_PKTLOG := n
else
CONFIG_FEATURE_PKTLOG := y
endif
CONFIG_FEATURE_PKTLOG := y
CONFIG_WLAN_DEBUG_CRASH_INJECT := y
endif
#Disable pktlog feature for lithium based target
ifeq ($(CONFIG_LITHIUM), y)
CONFIG_FEATURE_PKTLOG := n
endif
#Enable WLAN/Power debugfs feature only if debug_fs is enabled
ifeq ($(CONFIG_DEBUG_FS), y)
# Flag to enable debugfs. Depends on CONFIG_DEBUG_FS in kernel

View File

@ -7180,7 +7180,7 @@ static void wma_enable_specific_fw_logs(tp_wma_handle wma_handle,
* Return: None
*
*/
#ifdef REMOVE_PKT_LOG
#if !defined(FEATURE_PKTLOG) || defined(REMOVE_PKT_LOG)
static void wma_set_wifi_start_packet_stats(void *wma_handle,
struct sir_wifi_start_log *start_log)
{