schedule: remove schedule_task_complete()

schedule_task_complete() is never used, remove it.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>

Author: lyakh

src/include/sof/schedule/schedule.h

@@ -99,17 +99,6 @@ struct scheduler_ops {
 	 */
 	int (*schedule_task_running)(void *data, struct task *task);
 
-	/**
-	 * Sets task into completed state along with executing additional
-	 * scheduler specific operations.
-	 * @param data Private data of selected scheduler.
-	 * @param task Task to be set into completed state.
-	 * @return 0 if succeeded, error code otherwise.
-	 *
-	 * This operation is optional.
-	 */
-	int (*schedule_task_complete)(void *data, struct task *task);
-
 	/**
 	 * Reschedules already scheduled task with new start time.
 	 * @param data Private data of selected scheduler.
@@ -221,28 +210,6 @@ static inline int schedule_task_running(struct task *task)
 	return -ENODEV;
 }
 
-/** See scheduler_ops::schedule_task_complete */
-static inline int schedule_task_complete(struct task *task)
-{
-	struct schedulers *schedulers = *arch_schedulers_get();
-	struct schedule_data *sch;
-	struct list_item *slist;
-
-	list_for_item(slist, &schedulers->list) {
-		sch = container_of(slist, struct schedule_data, list);
-		if (task->type == sch->type) {
-			/* optional operation */
-			if (!sch->ops->schedule_task_complete)
-				return 0;
-
-			return sch->ops->schedule_task_complete(sch->data,
-								task);
-		}
-	}
-
-	return -ENODEV;
-}
-
 /** See scheduler_ops::schedule_task */
 static inline int schedule_task(struct task *task, uint64_t start,
 				uint64_t period)

src/platform/library/schedule/edf_schedule.c

@@ -82,7 +82,6 @@ static int schedule_edf_task_free(void *data, struct task *task)
 static struct scheduler_ops schedule_edf_ops = {
 	.schedule_task		= schedule_edf_task,
 	.schedule_task_running	= NULL,
-	.schedule_task_complete = NULL,
 	.reschedule_task	= NULL,
 	.schedule_task_cancel	= schedule_edf_task_cancel,
 	.schedule_task_free	= schedule_edf_task_free,

src/platform/library/schedule/ll_schedule.c

@@ -153,19 +153,6 @@ static void *ll_thread(void *data)
 	return NULL;
 }
 
-static int schedule_ll_task_complete(void *data, struct task *task)
-{
-	struct ll_vcore *vc = data;
-
-	/* task is complete so remove it from list */
-	pthread_mutex_lock(&vc->list_mutex);
-	list_item_del(&task->list);
-	task->state = SOF_TASK_STATE_COMPLETED;
-	pthread_mutex_unlock(&vc->list_mutex);
-
-	return 0;
-}
-
 /* schedule new LL task */
 static int schedule_ll_task(void *data, struct task *task, uint64_t start,
 			    uint64_t period)
@@ -285,7 +272,6 @@ static int schedule_ll_task_free(void *data, struct task *task)
 static struct scheduler_ops schedule_ll_ops = {
 	.schedule_task		= schedule_ll_task,
 	.schedule_task_running	= NULL,
-	.schedule_task_complete = schedule_ll_task_complete,
 	.reschedule_task	= NULL,
 	.schedule_task_cancel	= schedule_ll_task_cancel,
 	.schedule_task_free	= schedule_ll_task_free,

src/schedule/edf_schedule.c

@@ -329,7 +329,6 @@ static void schedule_edf(struct edf_schedule_data *edf_sch)
 static const struct scheduler_ops schedule_edf_ops = {
 	.schedule_task		= schedule_edf_task,
 	.schedule_task_running	= schedule_edf_task_running,
-	.schedule_task_complete = schedule_edf_task_complete,
 	.reschedule_task	= NULL,
 	.schedule_task_cancel	= schedule_edf_task_cancel,
 	.schedule_task_free	= schedule_edf_task_free,

src/schedule/ll_schedule.c

@@ -802,5 +802,4 @@ static const struct scheduler_ops schedule_ll_ops = {
 	.scheduler_free		= scheduler_free_ll,
 	.scheduler_restore	= NULL,
 	.schedule_task_running	= NULL,
-	.schedule_task_complete	= NULL,
 };

zephyr/edf_schedule.c

@@ -75,12 +75,6 @@ static int schedule_edf_task_cancel(void *data, struct task *task)
 	return 0;
 }
 
-static int schedule_edf_task_complete(void *data, struct task *task)
-{
-	task_complete(task);
-	return 0;
-}
-
 static int schedule_edf_task_running(void *data, struct task *task)
 {
 	task->state = SOF_TASK_STATE_RUNNING;
@@ -99,7 +93,6 @@ static int schedule_edf_task_free(void *data, struct task *task)
 static struct scheduler_ops schedule_edf_ops = {
 	.schedule_task		= schedule_edf_task,
 	.schedule_task_running	= schedule_edf_task_running,
-	.schedule_task_complete = schedule_edf_task_complete,
 	.schedule_task_cancel	= schedule_edf_task_cancel,
 	.schedule_task_free	= schedule_edf_task_free,
 };