From 303dea997614a6065c2418dcb77df2023d667102 Mon Sep 17 00:00:00 2001 From: Swarn Singh Date: Tue, 24 Jan 2023 17:01:46 +0530 Subject: Wifi: Remove the default assumption of legacy feature sets Legacy feature DEBUG_RING_BUFFER_VENDOR_DATA and DEBUG_HOST_WAKE_REASON_STATS are assumed to be supported by default in all the devices as there is no mechanism to get this capability from the host driver. This assumption shall fail certain compliance tests where these legacy features are not supported. Hence removing such assumptions. Change-Id: Iee2aef2c1d77c469acf247cbb6314363b51bea8a CRs-Fixed: 3387770 --- wifi/1.6/default/hidl_struct_util.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/wifi/1.6/default/hidl_struct_util.cpp b/wifi/1.6/default/hidl_struct_util.cpp index ff3105dd77..9c4d881aad 100644 --- a/wifi/1.6/default/hidl_struct_util.cpp +++ b/wifi/1.6/default/hidl_struct_util.cpp @@ -156,10 +156,8 @@ bool convertLegacyFeaturesToHidlChipCapabilities(uint64_t legacy_feature_set, } } - // There are no flags for these 3 in the legacy feature set. Adding them to + // There is no flag for this in the legacy feature set. Adding this to // the set because all the current devices support it. - *hidl_caps |= HidlChipCaps::DEBUG_RING_BUFFER_VENDOR_DATA; - *hidl_caps |= HidlChipCaps::DEBUG_HOST_WAKE_REASON_STATS; *hidl_caps |= HidlChipCaps::DEBUG_ERROR_ALERTS; return true; } -- cgit v1.2.3