module_adapter: Add a no_pause field for modules

ranj063 · kv2019i · commit 6e7f31c141fa · 2022-12-15T20:44:20.000+02:00
Add a new field, no_pause in struct processing_module which will be set
by modules that do not support pause, ex: mixer.

Signed-off-by: Ranjani Sridharan &lt;ranjani.sridharan@linux.intel.com&gt;
diff --git a/src/audio/module_adapter/module_adapter.c b/src/audio/module_adapter/module_adapter.c
@@ -847,6 +847,15 @@ int module_adapter_trigger(struct comp_dev *dev, int cmd)
 {
 	comp_dbg(dev, "module_adapter_trigger(): cmd %d", cmd);
 
+	/*
+	 * If the module doesn't support pause, keep it active along with the rest of the
+	 * downstream modules
+	 */
+	if (cmd == COMP_TRIGGER_PAUSE && mod->no_pause) {
+		dev->state = COMP_STATE_ACTIVE;
+		return PPL_STATUS_PATH_STOP;
+	}
+
 	return comp_set_state(dev, cmd);
 }
 
diff --git a/src/include/sof/audio/module_adapter/module/generic.h b/src/include/sof/audio/module_adapter/module/generic.h
@@ -178,6 +178,9 @@ struct processing_module {
 
 	/* module-specific flags for comp_verify_params() */
 	uint32_t verify_params_flags;
+
+	/* flag to indicate module does not pause */
+	bool no_pause;
 };
 
 /*****************************************************************************/

Original file line number	Diff line number	Diff line change
`@@ -847,6 +847,15 @@ int module_adapter_trigger(struct comp_dev *dev, int cmd)`
`847`	`847`	`{`
`848`	`848`	`comp_dbg(dev, "module_adapter_trigger(): cmd %d", cmd);`
`849`	`849`
	`850`	`+ /*`
	`851`	`+ * If the module doesn't support pause, keep it active along with the rest of the`
	`852`	`+ * downstream modules`
	`853`	`+ */`
	`854`	`+ if (cmd == COMP_TRIGGER_PAUSE && mod->no_pause) {`
	`855`	`+ dev->state = COMP_STATE_ACTIVE;`
	`856`	`+ return PPL_STATUS_PATH_STOP;`
	`857`	`+ }`
	`858`	`+`
`850`	`859`	`return comp_set_state(dev, cmd);`
`851`	`860`	`}`
`852`	`861`