qcacld-3.0: Remove cds_context param to cds_sched_close()

Currently cds_sched_close() takes a cds_context param. However CDS
already maintains its own context, hence this param introduces an
unnecessary coupling, so remove it.

Change-Id: Ic7ceb85653acd2acdb700ae9d2ef15f4e09b81c7
CRs-Fixed: 2109435
This commit is contained in:
Jeff Johnson 2017-09-13 15:12:54 -07:00
commit 7d56227ba8
5 changed files with 6 additions and 10 deletions

View file

@ -424,8 +424,6 @@ QDF_STATUS cds_sched_open(void *p_cds_context,
- The Tx thread is closed
\param p_cds_context - pointer to the global QDF Context
\return QDF_STATUS_SUCCESS - Scheduler was successfully initialized and
is ready to be used.
@ -437,7 +435,7 @@ QDF_STATUS cds_sched_open(void *p_cds_context,
\sa cds_sched_close()
---------------------------------------------------------------------------*/
QDF_STATUS cds_sched_close(void *p_cds_context);
QDF_STATUS cds_sched_close(void);
p_cds_sched_context get_cds_sched_ctxt(void);

View file

@ -665,7 +665,7 @@ err_bmi_close:
err_sched_close:
if (hdd_ctx->driver_status == DRIVER_MODULES_UNINITIALIZED ||
cds_is_driver_recovering()) {
qdf_status = cds_sched_close(gp_cds_context);
qdf_status = cds_sched_close();
if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
hdd_err("Failed to close CDS Scheduler");
QDF_ASSERT(false);

View file

@ -842,7 +842,6 @@ static int cds_ol_rx_thread(void *arg)
/**
* cds_sched_close() - close the cds scheduler
* @p_cds_context: Pointer to the global CDS Context
*
* This api closes the CDS Scheduler upon successful closing:
* - All the message queues are flushed
@ -852,7 +851,7 @@ static int cds_ol_rx_thread(void *arg)
*
* Return: qdf status
*/
QDF_STATUS cds_sched_close(void *p_cds_context)
QDF_STATUS cds_sched_close(void)
{
QDF_TRACE(QDF_MODULE_ID_QDF, QDF_TRACE_LEVEL_INFO_HIGH,
"%s: invoked", __func__);

View file

@ -5866,7 +5866,6 @@ static void wlan_destroy_bug_report_lock(void)
*/
static void hdd_wlan_exit(struct hdd_context *hdd_ctx)
{
v_CONTEXT_t p_cds_context = hdd_ctx->pcds_context;
QDF_STATUS qdf_status;
struct wiphy *wiphy = hdd_ctx->wiphy;
int driver_status;
@ -5949,7 +5948,7 @@ static void hdd_wlan_exit(struct hdd_context *hdd_ctx)
* no thread is scheduled after the each module close is
* is called i.e after all the data structures are freed.
*/
qdf_status = cds_sched_close(p_cds_context);
qdf_status = cds_sched_close();
if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
hdd_alert("Failed to close CDS Scheduler");
QDF_ASSERT(QDF_IS_STATUS_SUCCESS(qdf_status));
@ -9794,7 +9793,7 @@ err_stop_modules:
err_hdd_free_psoc:
if (DRIVER_MODULES_CLOSED == hdd_ctx->driver_status) {
status = cds_sched_close(hdd_ctx->pcds_context);
status = cds_sched_close();
if (!QDF_IS_STATUS_SUCCESS(status)) {
hdd_err("Failed to close CDS Scheduler");
QDF_ASSERT(QDF_IS_STATUS_SUCCESS(status));

View file

@ -1284,7 +1284,7 @@ QDF_STATUS hdd_wlan_shutdown(void)
}
#endif
qdf_status = cds_sched_close(p_cds_context);
qdf_status = cds_sched_close();
if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
hdd_err("Failed to close CDS Scheduler");
QDF_ASSERT(false);