Przeglądaj źródła

qcacld-3.0: Refine lim_send_sme_tsm_ie_ind()

Function lim_send_sme_tsm_ie_ind() contains some parameters and local
variables that are not aligned with the Linux Coding Style, so update
them to be compliant. In addition, update the function documentation
to use Linux kernel-doc style, and document the interface and not the
implementation.

Change-Id: I99a13686747fa3333dc3cd4a98cc581679cbd5bf
CRs-Fixed: 2394252
Jeff Johnson 6 lat temu
rodzic
commit
4f1b915b59

+ 14 - 24
core/mac/src/pe/lim/lim_assoc_utils.c

@@ -4733,40 +4733,30 @@ void lim_send_sme_unprotected_mgmt_frame_ind(struct mac_context *mac, uint8_t fr
 #endif
 
 #ifdef FEATURE_WLAN_ESE
-/** -------------------------------------------------------------
-   \fn     lim_send_sme_tsm_ie_ind
-   \brief  Forwards the TSM IE information to SME.
-   \param  struct mac_context *   mac
-   \param  pe_session - PE session context
-   \param  tid - traffic id
-   \param  state - tsm state (enabled/disabled)
-   \param  measurementInterval - measurement interval
-   \return none
-   -------------------------------------------------------------*/
 void lim_send_sme_tsm_ie_ind(struct mac_context *mac,
 			     struct pe_session *pe_session,
-			     uint8_t tid, uint8_t state, uint16_t measInterval)
+			     uint8_t tid, uint8_t state,
+			     uint16_t measurement_interval)
 {
-	struct scheduler_msg mmhMsg = {0};
-	struct tsm_ie_ind *pSirSmeTsmIeInd = NULL;
+	struct scheduler_msg msg = {0};
+	struct tsm_ie_ind *tsm_ie_ind;
 
 	if (!mac || !pe_session)
 		return;
 
-	pSirSmeTsmIeInd = qdf_mem_malloc(sizeof(*pSirSmeTsmIeInd));
-	if (!pSirSmeTsmIeInd)
+	tsm_ie_ind = qdf_mem_malloc(sizeof(*tsm_ie_ind));
+	if (!tsm_ie_ind)
 		return;
 
-	pSirSmeTsmIeInd->sessionId = pe_session->smeSessionId;
-	pSirSmeTsmIeInd->tsm_ie.tsid = tid;
-	pSirSmeTsmIeInd->tsm_ie.state = state;
-	pSirSmeTsmIeInd->tsm_ie.msmt_interval = measInterval;
+	tsm_ie_ind->sessionId = pe_session->smeSessionId;
+	tsm_ie_ind->tsm_ie.tsid = tid;
+	tsm_ie_ind->tsm_ie.state = state;
+	tsm_ie_ind->tsm_ie.msmt_interval = measurement_interval;
 
-	mmhMsg.type = eWNI_SME_TSM_IE_IND;
-	mmhMsg.bodyptr = pSirSmeTsmIeInd;
-	mmhMsg.bodyval = 0;
+	msg.type = eWNI_SME_TSM_IE_IND;
+	msg.bodyptr = tsm_ie_ind;
+	msg.bodyval = 0;
 
-	lim_sys_process_mmh_msg_api(mac, &mmhMsg);
-	return;
+	lim_sys_process_mmh_msg_api(mac, &msg);
 }
 #endif /* FEATURE_WLAN_ESE */

+ 19 - 5
core/mac/src/pe/lim/lim_assoc_utils.h

@@ -235,13 +235,27 @@ void lim_send_sme_unprotected_mgmt_frame_ind(struct mac_context *mac, uint8_t fr
 					     struct pe_session *pe_session);
 #endif
 
+/**
+ * lim_send_sme_tsm_ie_ind() - Send TSM IE information to SME
+ * @mac: Global MAC context
+ * @pe_session: PE session context
+ * @tid: traffic id
+ * @state: tsm state (enabled/disabled)
+ * @measurement_interval: measurement interval
+ *
+ * Return: void
+ */
 #ifdef FEATURE_WLAN_ESE
-void lim_send_sme_tsm_ie_ind(struct mac_context *mac, struct pe_session *pe_session,
-			     uint8_t tid, uint8_t state, uint16_t measInterval);
+void lim_send_sme_tsm_ie_ind(struct mac_context *mac,
+			     struct pe_session *pe_session,
+			     uint8_t tid, uint8_t state,
+			     uint16_t measurement_interval);
 #else
-static inline void lim_send_sme_tsm_ie_ind(struct mac_context *mac,
-	struct pe_session *pe_session, uint8_t tid,
-	uint8_t state, uint16_t measInterval)
+static inline
+void lim_send_sme_tsm_ie_ind(struct mac_context *mac,
+			     struct pe_session *pe_session,
+			     uint8_t tid, uint8_t state,
+			     uint16_t measurement_interval)
 {}
 #endif /* FEATURE_WLAN_ESE */