Unverified Commit c792c369 authored by Pierre-Louis Bossart's avatar Pierre-Louis Bossart Committed by Mark Brown
Browse files

ASoC: rt5682-sdw: cancel_work_sync() in .remove and .suspend 



Follow pattern from other drivers and use cancel_work_sync() for both
.remove() and .suspend().

Fixes: 03f6fc6d ('ASoC: rt5682: Add the soundwire support')
Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: default avatarBard Liao <yung-chuan.liao@linux.intel.com>
Reviewed-by: default avatarRander Wang <rander.wang@linux.intel.com>
Link: https://lore.kernel.org/r/20210204201739.25206-4-pierre-louis.bossart@linux.intel.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 121871a7

sound/soc/codecs/rt5682-sdw.c

+3 −1
Original line number Diff line number Diff line
@@ -703,7 +703,7 @@ static int rt5682_sdw_remove(struct sdw_slave *slave) 
	struct rt5682_priv *rt5682 = dev_get_drvdata(&slave->dev);



	if (rt5682 && rt5682->hw_init)

		cancel_delayed_work(&rt5682->jack_detect_work);

		cancel_delayed_work_sync(&rt5682->jack_detect_work);



	return 0;

}
@@ -721,6 +721,8 @@ static int __maybe_unused rt5682_dev_suspend(struct device *dev) 
	if (!rt5682->hw_init)

		return 0;



	cancel_delayed_work_sync(&rt5682->jack_detect_work);



	regcache_cache_only(rt5682->regmap, true);

	regcache_mark_dirty(rt5682->regmap);