ASoC: rt5640: Only cancel jack-detect work on suspend if active
authorHans de Goede <hdegoede@redhat.com>
Tue, 12 Sep 2023 11:32:45 +0000 (13:32 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 12 Sep 2023 13:37:51 +0000 (14:37 +0100)
commit8fc7cc507d61fc655172836c74fb7fcc8b7a978b
treefe6ad441e73bf68d7859678102bd08870ebc602b
parent8c8bf3df6b7c0ed1c4dd373b23eb0ce13a63f452
ASoC: rt5640: Only cancel jack-detect work on suspend if active

If jack-detection is not used; or has already been disabled then
there is no need to call rt5640_cancel_work().

Move the rt5640_cancel_work() inside the "if (rt5640->jack) {}" block,
grouping it together with the disabling of the IRQ which queues the work
in the first place.

This also makes suspend() symetrical with resume() which re-queues the work
in an "if (rt5640->jack) {}" block.

Cc: Oder Chiou <oder_chiou@realtek.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20230912113245.320159-7-hdegoede@redhat.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/rt5640.c