tree 9552ecfbdb7610ec697c9fd27def236db49f6feb
parent f9dc6b875ec0a6a6d4091cd9603d193ec98c75a2
author Richard Fitzgerald <rf@opensource.cirrus.com> 1681226723 +0100
committer Mark Brown <broonie@kernel.org> 1681317275 +0100
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmQ23ZsACgkQJNaLcl1U
 h9ALOQf9EopmEThL+0ckPY5iwx7Y4Dv5DqdSL0AwISpCqp4f+buckjHw3MaLuIPS
 IcLTbcoX+NkARv+jwIb1RvwITTFJG1v+dIEMUENyxiO83ecVQJGi8Zn+JWPDjij6
 dB/1MtcjCRL695bHlGFsStoG1LjPrGNx+OgCdEteJVZGojkhqnyj6WY6qkzLKC1K
 JZM0Y2JsDYJkzTHRak0jEZqSWOOodjIropljdtBIEneIBF1Bkx1ZAVFYzhelCeqs
 sAoWbe557TFA6cNIrbzWYWKi+KGpUDPiI2lXgNd0gZI4XqHogwcr8er2/Ga4OEEa
 4WRfEyJhP7GtVjCNAwN+0gxe2VMHkQ==
 =FzsH
 -----END PGP SIGNATURE-----

ASoC: cs35l56: Remove quick-cancelling of dsp_work()

Delete the 'removing' flag and don't kick init_completion to make a
quick cancel of dsp_work(). Just let it timeout on the wait for the
completion.

Simplify the code to standard cancelling or flushing of the work.
This avoids introducing corner cases from a layer of custom signalling.
It also avoids potential race conditions when system-suspend handling
is added.

Unless the hardware is broken, the dsp_work() will already have started
and passed the completion before the driver would want to cancel it.

Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Link: https://lore.kernel.org/r/168122674746.26.16881587647873355224@mailman-core.alsa-project.org
Signed-off-by: Mark Brown <broonie@kernel.org>