diff options
author | Simon Trimmer <simont@opensource.cirrus.com> | 2023-04-14 14:37:52 +0100 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2023-04-17 12:55:50 +0100 |
commit | 4be476830bf96c0095e4e1acdfc12081aa0fb165 (patch) | |
tree | 97b751603e6c2e65b5b408a0e14dd0df831da38a /sound/soc/codecs/cs35l56.h | |
parent | 440c2d38950f738d7a35d3d29533728e74586d54 (diff) |
ASoC: cs35l56: Remove redundant dsp_ready_completion
dsp_ready_completion is redundant and can be replaced by a call
flush_work() to wait for cs35l56_dsp_work() to complete.
As the dsp_work is queued by component_probe() it must run before other
ASoC component callbacks and therefore there is no risk of calling
flush_work() before the dsp_work() has been queued.
Signed-off-by: Simon Trimmer <simont@opensource.cirrus.com>
Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20230414133753.653139-5-rf@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/codecs/cs35l56.h')
-rw-r--r-- | sound/soc/codecs/cs35l56.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sound/soc/codecs/cs35l56.h b/sound/soc/codecs/cs35l56.h index ac2e9237c27d..09762e70ce81 100644 --- a/sound/soc/codecs/cs35l56.h +++ b/sound/soc/codecs/cs35l56.h @@ -34,7 +34,6 @@ struct cs35l56_private { struct wm_adsp dsp; /* must be first member */ struct work_struct dsp_work; struct workqueue_struct *dsp_wq; - struct completion dsp_ready_completion; struct mutex irq_lock; struct snd_soc_component *component; struct device *dev; |