On Thu, 2018-07-19 at 18:56 +0530, Balaji Pothunoori wrote: > > > - if (test_bit(WMI_SERVICE_TX_DATA_ACK_RSSI, ar->wmi.svc_map)) > + if (test_bit(WMI_SERVICE_TX_DATA_ACK_RSSI, ar->wmi.svc_map) || > + test_bit(WMI_SERVICE_HTT_MGMT_TX_COMP_VALID_FLAGS, ar->wmi.svc_map)) > wiphy_ext_feature_set(ar->hw->wiphy, > - NL80211_EXT_FEATURE_DATA_ACK_SIGNAL_SUPPORT); > + NL80211_EXT_FEATURE_DATA_ACK_SIGNAL_SUPPORT); You should use the new name, and fix indentation. johannes