qcacld-3.0: Handle get_tx_power in SAP mode

In wlan_hdd_cfg80211_get_txpower function, to avoid
frequent WMI calls to firmware to get connection tx power,
host always returns the periodically cached tx power
acquired as part of UNIFIED_LL_GET_STA stats.

The current issue is, in case of SAP mode with no
stations associated, the tx_power is not cached into the
adapter, and host always returns default value.

To fix this issue, based on timestamp when tx_power is cached
into the adapter, host will either return cached value or
value obtained from WMI command.

Change-Id: I9dfbf9ea4d8b0d9430e6ab49cdc9f26c83079676
CRs-Fixed: 3456632

core/hdd/inc/wlan_hdd_main.h

@@ -1035,6 +1035,16 @@ struct wlan_hdd_link_info {
 #endif
 };
 
+/**
+ * struct wlan_hdd_tx_power - Structure to store connection tx power info
+ * @tx_pwr: connection tx power sent by firmware
+ * @tx_pwr_cached_timestamp: timestamp when tx_pwr is cached into adapter
+ */
+struct wlan_hdd_tx_power {
+	int tx_pwr;
+	uint32_t tx_pwr_cached_timestamp;
+};
+
 /**
  * struct hdd_adapter - hdd vdev/net_device context
  * @magic: Magic cookie for adapter sanity verification.  Note that this
@@ -1159,7 +1169,7 @@ struct wlan_hdd_link_info {
  * @is_dbam_configured:
  * @deflink: Default link pointing to the 0th index of the linkinfo array
  * @link_info: Data structure to hold link specific information
- * @tx_pwr: connection tx power sent by firmware
+ * @tx_power: Structure to hold connection tx Power info
  */
 struct hdd_adapter {
 	uint32_t magic;
@@ -1353,7 +1363,7 @@ struct hdd_adapter {
 #endif
 	struct wlan_hdd_link_info *deflink;
 	struct wlan_hdd_link_info link_info[WLAN_MAX_MLD];
-	int tx_pwr;
+	struct wlan_hdd_tx_power tx_power;
 };
 
 #define WLAN_HDD_GET_STATION_CTX_PTR(adapter) \

core/hdd/src/wlan_hdd_power.c

@@ -3201,22 +3201,23 @@ int wlan_hdd_cfg80211_set_txpower(struct wiphy *wiphy,
 	return errno;
 }
 
-#ifdef FEATURE_CLUB_LL_STATS_AND_GET_STATION
-static int wlan_hdd_get_tx_power(struct hdd_adapter *adapter, int *dbm)
+#define WLAN_HDD_TX_POWER_CACHE_EXPIRY_TIME 350
+
+static QDF_STATUS
+wlan_hdd_tx_power_request_needed(struct hdd_adapter *adapter)
 {
-	if (hdd_validate_adapter(adapter)) {
-		hdd_err("Invalid adapter");
-		return -EINVAL;
-	}
-	/* TX_POWER is sent by STATION_STATS by firmware and
-	 * is copied into the adapter. So, return the cached value.
-	 */
-	*dbm = adapter->tx_pwr;
-	hdd_nofl_debug("cached tx_power: %d", *dbm);
+	uint32_t tx_pwr_cached_duration;
 
-	return 0;
+	tx_pwr_cached_duration =
+			qdf_system_ticks_to_msecs(qdf_system_ticks()) -
+			adapter->tx_power.tx_pwr_cached_timestamp;
+
+	if (tx_pwr_cached_duration <= WLAN_HDD_TX_POWER_CACHE_EXPIRY_TIME)
+		return QDF_STATUS_E_ALREADY;
+
+	return QDF_STATUS_SUCCESS;
 }
-#else
+
 static int wlan_hdd_get_tx_power(struct hdd_adapter *adapter, int *dbm)
 {
 	struct wlan_objmgr_vdev *vdev;
@@ -3233,7 +3234,6 @@ static int wlan_hdd_get_tx_power(struct hdd_adapter *adapter, int *dbm)
 	hdd_debug("power: %d", *dbm);
 	return ret;
 }
-#endif
 
 #ifdef FEATURE_ANI_LEVEL_REQUEST
 static void hdd_get_ani_level_cb(struct wmi_host_ani_level_event *ani,
@@ -3358,7 +3358,8 @@ static int __wlan_hdd_cfg80211_get_txpower(struct wiphy *wiphy,
 	struct hdd_context *hdd_ctx = (struct hdd_context *) wiphy_priv(wiphy);
 	struct net_device *ndev = wdev->netdev;
 	struct hdd_adapter *adapter = WLAN_HDD_GET_PRIV_PTR(ndev);
-	int status;
+	QDF_STATUS status;
+	int ret;
 	static bool is_rate_limited;
 	struct wlan_objmgr_vdev *vdev;
 
@@ -3371,14 +3372,14 @@ static int __wlan_hdd_cfg80211_get_txpower(struct wiphy *wiphy,
 
 	*dbm = 0;
 
-	status = wlan_hdd_validate_context(hdd_ctx);
-	if (status)
-		return status;
+	ret = wlan_hdd_validate_context(hdd_ctx);
+	if (ret)
+		return ret;
 
 	/* Validate adapter sessionId */
-	status = wlan_hdd_validate_vdev_id(adapter->deflink->vdev_id);
-	if (status)
-		return status;
+	ret = wlan_hdd_validate_vdev_id(adapter->deflink->vdev_id);
+	if (ret)
+		return ret;
 	switch (adapter->device_mode) {
 	case QDF_STA_MODE:
 	case QDF_P2P_CLIENT_MODE:
@@ -3420,6 +3421,16 @@ static int __wlan_hdd_cfg80211_get_txpower(struct wiphy *wiphy,
 		return 0;
 	}
 
+	status = wlan_hdd_tx_power_request_needed(adapter);
+	if (status == QDF_STATUS_E_ALREADY) {
+		/* TX_POWER is sent by STATION_STATS by firmware and
+		 * is copied into the adapter. So, return cached value.
+		 */
+		*dbm = adapter->tx_power.tx_pwr;
+		hdd_nofl_debug("cached tx_power: %d", *dbm);
+		return 0;
+	}
+
 	qdf_mtrace(QDF_MODULE_ID_HDD, QDF_MODULE_ID_HDD,
 		   TRACE_CODE_HDD_CFG80211_GET_TXPOWER,
 		   adapter->deflink->vdev_id, adapter->device_mode);

core/hdd/src/wlan_hdd_stats.c

@@ -270,7 +270,9 @@ static int copy_station_stats_to_adapter(struct hdd_adapter *adapter,
 			stats->peer_adv_stats->rx_bytes;
 	adapter->hdd_stats.peer_stats.fcs_count =
 			stats->peer_adv_stats->fcs_count;
-	adapter->tx_pwr = stats->pdev_stats->max_pwr;
+	adapter->tx_power.tx_pwr = stats->pdev_stats->max_pwr;
+	adapter->tx_power.tx_pwr_cached_timestamp =
+			qdf_system_ticks_to_msecs(qdf_system_ticks());
 
 	dynamic_cfg = mlme_get_dynamic_vdev_config(vdev);
 	if (!dynamic_cfg) {
@@ -7654,7 +7656,6 @@ int wlan_hdd_get_station_stats(struct hdd_adapter *adapter)
 	/* update get stats cached time stamp */
 	hdd_update_station_stats_cached_timestamp(adapter);
 	copy_station_stats_to_adapter(adapter, stats);
-
 out:
 	wlan_cfg80211_mc_cp_stats_free_stats_event(stats);
 	hdd_objmgr_put_vdev_by_user(vdev, WLAN_OSIF_STATS_ID);